[Classified]

Hey everyone. The beauty of VB.NET is the simplicity of the syntax which makes it easy to learn! I certainly am no pro but I've learned a lot over the year, in which I began to self teach myself. Here is a simple Account Generator which uses a text file hosted on a webhost along with a php file that extracts one random account from the text file. 
Setting up the web portion of this project:
*You'll need an text file containing all the accounts you'd like to be generated by the program. Please keep it to 1 account per line. 
*You'll need to make a php file 

PHP Code:

<?php $f_contents = file("NAME OF THE TXT FILE YOU UPLOADED.txt"); $line = $f_contents[array_rand($f_contents)]; $data = $line; Echo $data; ?> 'This is the file you'll link into the program

To make the program you'll need a textbox,progress bar(optional),button


In a bit I'll post a more comprehensive account generator.
To download my version of the program to here is the link
http://greatergood.zz.mu/BasicAccountGenerator.zip

Regards,
Classified

[Pirate]

Looks awesome. Keep up the good work!

[Kotomi]

It's easy to grab that file this is a bad way of doing it.

Nice tutorial though I guess, don't try to sell account generator with this because your lists will get dumped.

[Classified]

It's easy to grab that file this is a bad way of doing it.

Nice tutorial though I guess, don't try to sell account generator with this because your lists will get dumped.

You're correct this is not the best way to do it. In fact just decompiling the program will give you the link. However it is a great starting point for those who wish to learn the language.

[Killpot]

Ok, so first off, rename your buttons to represent what their function is, it's very tedious to look through someones work and have all of their buttons be unlabeled.

Second, once again rename your elements, MetroProgressBar1 is unnamed, and why are you incrementing it in 100? why not just use increments of 1 and iterate through them?

Third, MsgBox is a depreciated function, this is VB.Net not VB6 you should be using Messagebox.Show

Fourth, you're not properly disposing of the webclient, use the "Using" syntax instead of declaring it.

Fith, you should add error handling for the downloadstring function, what happens if that person doesn't have an internet connection, or they connection is blocked for whatever reason? You should check that they even receive any form of data from DownloadString.

[Classified]

Ok, so first off, rename your buttons to represent what their function is, it's very tedious to look through someones work and have all of their buttons be unlabeled.

Second, once again rename your elements, MetroProgressBar1 is unnamed, and why are you incrementing it in 100? why not just use increments of 1 and iterate through them?

Third, MsgBox is a depreciated function, this is VB.Net not VB6 you should be using Messagebox.Show

Fourth, you're not properly disposing of the webclient, use the "Using" syntax instead of declaring it.

Fith, you should add error handling for the downloadstring function, what happens if that person doesn't have an internet connection, or they connection is blocked for whatever reason? You should check that they even receive any form of data from DownloadString.

Thanks for the criticism and remarks. The progress bar was just added for appealment only. As for the MsgBox I need to start typing out Messagebox.Show.  In the next update I'll definitely reciprocate the advice and possibly include more options.