[iKQLY]

Saw this being sold in the Premium Sellers Section. I really don't believe people should have to pay for something that takes only an hour or two, so I'll release my twitch scraper.
It is multi-threaded so it shouldn't lag when scraping

All you need to do is enter in a channel name. The listbox will then populate all viewers watching. You can then export to a text file.

If anyone wants to comment on the code and possible suggest any way it could be optimized, feel free to post here.

Source Code:Spoiler (Click to View)
Mediafire link


Application Only:Spoiler (Click to View)
Mediafire link


Image: Spoiler (Click to View)



Credit to Tidas for theme: https://hackforums.net/showthread.php?ti...itch+Theme

[+Haze]

What theme is that, looks nice IMHO.

[iKQLY]

What theme is that, looks nice IMHO.

Thanks :) It's Twitch Theme, I think Tidas released this.

Here's link for theme
https://hackforums.net/showthread.php?ti...itch+Theme

[Charles Carmichael]

Whats the purpose of scraping mass twitch usernames :o

[DBots]

Okay I'll bite.

On your form load event, you have this:


Don't do that. That is not proper threading which is what I suppose you were trying to accomplish here. Once you remove that, you'll be on your way to properly threading this since you won't be suppressing any errors and you can more easily debug your application.

--

You instantiate a WebClient but then it is possible you may not use it and didn't even dispose of it.



Rather you check if there may be something stopping you from proceeding first.
--



Here, you acknowledge there may be an error and when there is, you still try to process the result, which there isn't one.
I would rather take the approach of making sure the string was downloaded before moving on to trying to process it.

--


You call this at the end but there is no need to.

[+Haze]

I agree with what dbots said. I see you're trying to use Multi threading. In that case use Webclient download string async method. It's so much better and faster when dealing with large amounts of data or doing things quickly.

[iKQLY]

Okay I'll bite.

On your form load event, you have this:


Don't do that. That is not proper threading which is what I suppose you were trying to accomplish here. Once you remove that, you'll be on your way to properly threading this since you won't be suppressing any errors and you can more easily debug your application.

--

You instantiate a WebClient but then it is possible you may not use it and didn't even dispose of it.



Rather you check if there may be something stopping you from proceeding first.
--



Here, you acknowledge there may be an error and when there is, you still try to process the result, which there isn't one.
I would rather take the approach of making sure the string was downloaded before moving on to trying to process it.

--


You call this at the end but there is no need to.

Awesome, thanks! Just added in what you pointed out

---

I agree with what dbots said. I see you're trying to use Multi threading. In that case use Webclient download string async method. It's so much better and faster when dealing with large amounts of data or doing things quickly.

Thanks, never really bothered to look at async method. Just added it as well