Home Upgrade Search Memberlist Extras Hacker Tools Award Goals Help Wiki Contact

.NET 

Twitter Account & Tweet Scraper

Here's my note: 4/10. (This used to be a 6, when I saw point 4)
And now, here's why:

1. WebRequest in the UI thread. Bad thing is bad. Multi-threading dude !
2. "https://twitter.com/search/realtime?q=" + bb + "&src=typd")" No. When you want to concat strings, you should use "&"
3. This code below V. First, it's alway better to use explicit variable name. Second, why aren't they declared as string ? (You can also reduct this in one line)
Code
Dim lol = Environment.GetFolderPath(Environment.SpecialFolder.Desktop)
Dim la As String = "\twitter.txt"
Dim aaa = lol + la
4. Just saw it when writing this line. Just NO. You don't use threading for webrequest, so what is the point of this ? VERY bad habit, and for nothing.
Code
Private Sub Form1_Load(sender As Object, e As EventArgs) Handles MyBase.Load
  CheckForIllegalCrossThreadCalls = False
    End Sub
5.
Code
Private Sub SWAG()
... Not a bad point in itself, just ... Explicit name, please, and not that !_!
[6.] Yeah, just one remark, no big deal:
Code
If CheckBox2.Checked = True Then
You can just write "if CheckBox2.Checked Then"

That's all. Be sure to correct these points in the future ;)
Didn't really looking in the RegEx and request in themself, I just saw some loose of time.
This post is not offensive. It's only meant to help.
(Feb 25, 2013 07:59 PM)Infinit3 Wrote:  Thanks for the feedback will have a look back on everything

No problem dude.
Oh thanks, someone that don't get mad when you correct them :)
Welcome to the .Net section dude !


Possibly Related Threads...
Thread / Author Replies Views Last Post
[Python] by littleblueguy24Aug 20, 2023 01:46 AM 8 309 Aug 20, 2023 01:46 AM
Last Post: DashPHP
[C#] by flAmingw0rmDec 31, 2017 06:15 PM 11 703 Dec 31, 2017 06:15 PM
Last Post: Ninja
[Python] by gongas19Nov 15, 2016 06:08 AM 10 2,043 Nov 15, 2016 06:08 AM
Last Post: bytez