[woahxx]

Oh please stop the discussion / debate, please only post if it's something to do with my program... Thanks.

--
Cheers, xBoom.

[Dialog]

Oh please stop the discussion / debate, please only post if it's something to do with my program... Thanks.

--
Cheers, xBoom.

Alright then, I'll give you some feedback:

• Rename your controls! Right now, you have FlatButton1, FlatButton2, and so on...
  This gets really complicated when you have a lot of controls. If you simply call them btnStart, txtSubject etc. it will be a lot easier to recognize what they do for yourself and other people reading your code.
  
  


• You should use String.Format().
  Just to quickly grab a part of your code:
  

  "&url=" + FlatTextbox3.Text + "&user="

  
  
  Instead, you should use
  
  Code

  String.Format("&url={0}&user=", FlatTextbox3.Text)
  
  Refer to this page for documentation regarding the method.
  
  


• Use MessageBox.Show() instead of MsgBox(). This is because MsgBox() is an old left-over from VB6. The only reason why it's still here, is for programmers who are used to using it.
  As a new programmer, you should be using the modern MessageBox.Show() method.
  
  


• Don't use webbrowser components for things like these.
  Use a WebClient or HttpWebRequest instead.
  
  


• Right now, you are declaring new objects for MailMessage, MailAddress, SmtpClient and NetworkCredential every time your timer's tick event gets fired. This is really unecessary and unefficient.
  Simply keep "SMTPServer.Send(Mymailssage)" in the tick event handler, and throw all other code in the button's click event.
  Declare SMTPServer and MyMailssage globally in your class, so you can access it from Timer1_Tick.
  
  


• It may be preferred to set Timer.Enabled to true or false ove
  calling it's start/stop methods. Internally it does exactly the same, so this is really just personal preference.
  

[woahxx]

Alright then, I'll give you some feedback:

• Rename your controls! Right now, you have FlatButton1, FlatButton2, and so on...
  This gets really complicated when you have a lot of controls. If you simply call them btnStart, txtSubject etc. it will be a lot easier to recognize what they do for yourself and other people reading your code.
  
  


• You should use String.Format().
  Just to quickly grab a part of your code:
  
  Instead, you should use
  
  Code

  String.Format("&url={0}&user=", FlatTextbox3.Text)
  
  Refer to this page for documentation regarding the method.
  
  


• Use MessageBox.Show() instead of MsgBox(). This is because MsgBox() is an old left-over from VB6. The only reason why it's still here, is for programmers who are used to using it.
  As a new programmer, you should be using the modern MessageBox.Show() method.
  
  


• Don't use webbrowser components for things like these.
  Use a WebClient or HttpWebRequest instead.
  
  


• Right now, you are declaring new objects for MailMessage, MailAddress, SmtpClient and NetworkCredential every time your timer's tick event gets fired. This is really unecessary and unefficient.
  Simply keep "SMTPServer.Send(Mymailssage)" in the tick event handler, and throw all other code in the button's click event.
  Declare SMTPServer and MyMailssage globally in your class, so you can access it from Timer1_Tick.
  
  


• It may be preferred to set Timer.Enabled to true or false ove
  
  calling it's start/stop methods. Internally it does exactly the same, so this is really just personal preference.
  

Sorry I dont really understand what you talk about.

---

Alright then, I'll give you some feedback:

• Rename your controls! Right now, you have FlatButton1, FlatButton2, and so on...
  This gets really complicated when you have a lot of controls. If you simply call them btnStart, txtSubject etc. it will be a lot easier to recognize what they do for yourself and other people reading your code.
  
  


• You should use String.Format().
  Just to quickly grab a part of your code:
  
  Instead, you should use
  
  Code

  String.Format("&url={0}&user=", FlatTextbox3.Text)
  
  Refer to this page for documentation regarding the method.
  
  


• Use MessageBox.Show() instead of MsgBox(). This is because MsgBox() is an old left-over from VB6. The only reason why it's still here, is for programmers who are used to using it.
  As a new programmer, you should be using the modern MessageBox.Show() method.
  
  


• Don't use webbrowser components for things like these.
  Use a WebClient or HttpWebRequest instead.
  
  


• Right now, you are declaring new objects for MailMessage, MailAddress, SmtpClient and NetworkCredential every time your timer's tick event gets fired. This is really unecessary and unefficient.
  Simply keep "SMTPServer.Send(Mymailssage)" in the tick event handler, and throw all other code in the button's click event.
  Declare SMTPServer and MyMailssage globally in your class, so you can access it from Timer1_Tick.
  
  


• It may be preferred to set Timer.Enabled to true or false ove
  
  calling it's start/stop methods. Internally it does exactly the same, so this is really just personal preference.
  

Sorry I dont really understand what you talk about.

[shebang]

Sorry I dont really understand what you talk about.

---

Sorry I dont really understand what you talk about.

What don't you understand? Be clear so we can assist you.

[Нyреrion]

Sorry I dont really understand what you talk about.

---

Sorry I dont really understand what you talk about.

This is exactly why you shouldn't try and sell your code, when you can't comprehend even the most basic of things in .net.

[woahxx]

I read it again, I understand it now - thanks I will add a amount box!

[Mаestro]

Hey, I'm happy to introduce Windy Tools.



This included the full sourcecode and permissions to do what ever you want with it just give some credits.

The email bomber is coded in VB.net and Snapchat & Email spoofer uses apis!

CLICK HERE TO DOWNLOAD

Screenshots:

Instead of webbrowsers, look into webrequests should make it a lot better.

[woahxx]

Instead of webbrowsers, look into webrequests should make it a lot better.

The web requests wont work, untill I found out about HTTP Web Requests.

[Mаestro]

The web requests wont work, untill I found out about HTTP Web Requests.

I meant either one, I've never tried so I didn't know which one you would need.

[iMythik]

Looks good, I like the theme.
As Indebt said, you should work on not using webbrowsers.

[hakenz]

So does this work? - Why are people complaining about you using "webbrowser"? is that bad or good? - And is this infected?

[Dispersia]

The web requests wont work, untill I found out about HTTP Web Requests.

... what, I don't even know what you mean by that

---

So does this work? - Why are people complaining about you using "webbrowser"? is that bad or good? - And is this infected?

webbrowsers are slow and create much more than is needed, when you don't need visuals OF the page, and just the data on it, aka it's kinda useless and takes a lot longer to do each request.

[hakenz]

... what, I don't even know what you mean by that

---

webbrowsers are slow and create much more than is needed, when you don't need visuals OF the page, and just the data on it, aka it's kinda useless and takes a lot longer to do each request.

Sooo.. this program is bad? not worth the download?

[Dispersia]

Sooo.. this program is bad? not worth the download?

I mean if it works, it works. However it could be MUCH better.

[woahxx]

So I'm new into VB and my solutions is kinda weird. I know.