Great work, some points, if it can help you.
In Dynamic1 it is better to use select case unstead of if elseif it better cause your code dont have to check all if to go on the good thing. It point directly to good case
Code: Select all
Select Case Cint(ComboBox1.Text)
Case 2
TextBox1.Enabled = True
TextBox2.Enabled = True
TextBox3.Enabled = False
TextBox4.Enabled = False
TextBox5.Enabled = False
TextBox6.Enabled = False
TextBox7.Enabled = False
TextBox8.Enabled = False
TextBox9.Enabled = False
TextBox10.Enabled = False
SpawnBox1.Enabled = True
SpawnBox2.Enabled = True
SpawnBox3.Enabled = False
SpawnBox4.Enabled = False
SpawnBox5.Enabled = False
SpawnBox6.Enabled = False
SpawnBox7.Enabled = False
SpawnBox8.Enabled = False
SpawnBox9.Enabled = False
SpawnBox10.Enabled = False
Case 3
TextBox1.Enabled = True
TextBox2.Enabled = True
TextBox3.Enabled = True
TextBox4.Enabled = False
TextBox5.Enabled = False
TextBox6.Enabled = False
TextBox7.Enabled = False
TextBox8.Enabled = False
TextBox9.Enabled = False
TextBox10.Enabled = False
SpawnBox1.Enabled = True
SpawnBox2.Enabled = True
SpawnBox3.Enabled = True
SpawnBox4.Enabled = False
SpawnBox5.Enabled = False
SpawnBox6.Enabled = False
SpawnBox7.Enabled = False
SpawnBox8.Enabled = False
SpawnBox9.Enabled = False
SpawnBox10.Enabled = False
Case 4
TextBox1.Enabled = True
TextBox2.Enabled = True
TextBox3.Enabled = True
TextBox4.Enabled = True
TextBox5.Enabled = False
TextBox6.Enabled = False
TextBox7.Enabled = False
TextBox8.Enabled = False
TextBox9.Enabled = False
TextBox10.Enabled = False
SpawnBox1.Enabled = True
SpawnBox2.Enabled = True
SpawnBox3.Enabled = True
SpawnBox4.Enabled = True
SpawnBox5.Enabled = False
SpawnBox6.Enabled = False
SpawnBox7.Enabled = False
SpawnBox8.Enabled = False
SpawnBox9.Enabled = False
SpawnBox10.Enabled = False
Case 5
TextBox1.Enabled = True
TextBox2.Enabled = True
TextBox3.Enabled = True
TextBox4.Enabled = True
TextBox5.Enabled = True
TextBox6.Enabled = False
TextBox7.Enabled = False
TextBox8.Enabled = False
TextBox9.Enabled = False
TextBox10.Enabled = False
SpawnBox1.Enabled = True
SpawnBox2.Enabled = True
SpawnBox3.Enabled = True
SpawnBox4.Enabled = True
SpawnBox5.Enabled = True
SpawnBox6.Enabled = False
SpawnBox7.Enabled = False
SpawnBox8.Enabled = False
SpawnBox9.Enabled = False
SpawnBox10.Enabled = False
Case 6
TextBox1.Enabled = True
TextBox2.Enabled = True
TextBox3.Enabled = True
TextBox4.Enabled = True
TextBox5.Enabled = True
TextBox6.Enabled = True
TextBox7.Enabled = False
TextBox8.Enabled = False
TextBox9.Enabled = False
TextBox10.Enabled = False
SpawnBox1.Enabled = True
SpawnBox2.Enabled = True
SpawnBox3.Enabled = True
SpawnBox4.Enabled = True
SpawnBox5.Enabled = True
SpawnBox6.Enabled = True
SpawnBox7.Enabled = False
SpawnBox8.Enabled = False
SpawnBox9.Enabled = False
SpawnBox10.Enabled = False
Case 7
TextBox1.Enabled = True
TextBox2.Enabled = True
TextBox3.Enabled = True
TextBox4.Enabled = True
TextBox5.Enabled = True
TextBox6.Enabled = True
TextBox7.Enabled = True
TextBox8.Enabled = False
TextBox9.Enabled = False
TextBox10.Enabled = False
SpawnBox1.Enabled = True
SpawnBox2.Enabled = True
SpawnBox3.Enabled = True
SpawnBox4.Enabled = True
SpawnBox5.Enabled = True
SpawnBox6.Enabled = True
SpawnBox7.Enabled = True
SpawnBox8.Enabled = False
SpawnBox9.Enabled = False
SpawnBox10.Enabled = False
Case 8
TextBox1.Enabled = True
TextBox2.Enabled = True
TextBox3.Enabled = True
TextBox4.Enabled = True
TextBox5.Enabled = True
TextBox6.Enabled = True
TextBox7.Enabled = True
TextBox8.Enabled = True
TextBox9.Enabled = False
TextBox10.Enabled = False
SpawnBox1.Enabled = True
SpawnBox2.Enabled = True
SpawnBox3.Enabled = True
SpawnBox4.Enabled = True
SpawnBox5.Enabled = True
SpawnBox6.Enabled = True
SpawnBox7.Enabled = True
SpawnBox8.Enabled = True
SpawnBox9.Enabled = False
SpawnBox10.Enabled = False
Case 9
TextBox1.Enabled = True
TextBox2.Enabled = True
TextBox3.Enabled = True
TextBox4.Enabled = True
TextBox5.Enabled = True
TextBox6.Enabled = True
TextBox7.Enabled = True
TextBox8.Enabled = True
TextBox9.Enabled = True
TextBox10.Enabled = False
SpawnBox1.Enabled = True
SpawnBox2.Enabled = True
SpawnBox3.Enabled = True
SpawnBox4.Enabled = True
SpawnBox5.Enabled = True
SpawnBox6.Enabled = True
SpawnBox7.Enabled = True
SpawnBox8.Enabled = True
SpawnBox9.Enabled = True
SpawnBox10.Enabled = False
Case 10
TextBox1.Enabled = True
TextBox2.Enabled = True
TextBox3.Enabled = True
TextBox4.Enabled = True
TextBox5.Enabled = True
TextBox6.Enabled = True
TextBox7.Enabled = True
TextBox8.Enabled = True
TextBox9.Enabled = True
TextBox10.Enabled = True
SpawnBox1.Enabled = True
SpawnBox2.Enabled = True
SpawnBox3.Enabled = True
SpawnBox4.Enabled = True
SpawnBox5.Enabled = True
SpawnBox6.Enabled = True
SpawnBox7.Enabled = True
SpawnBox8.Enabled = True
SpawnBox9.Enabled = True
SpawnBox10.Enabled = True
Case Else
TextBox1.Enabled = False
TextBox2.Enabled = False
TextBox3.Enabled = False
TextBox4.Enabled = False
TextBox5.Enabled = False
TextBox6.Enabled = False
TextBox7.Enabled = False
TextBox8.Enabled = False
TextBox9.Enabled = False
TextBox10.Enabled = False
SpawnBox1.Enabled = False
SpawnBox2.Enabled = False
SpawnBox3.Enabled = False
SpawnBox4.Enabled = False
SpawnBox5.Enabled = False
SpawnBox6.Enabled = False
SpawnBox7.Enabled = False
SpawnBox8.Enabled = False
SpawnBox9.Enabled = False
SpawnBox10.Enabled = False
End Select
For your MySQL connexion use ConnectionStringBuilder (if some server are different MySQL Connector fix it)
Code: Select all
Dim conncStringBuild As New MySql.Data.MySqlClient.MySqlConnectionStringBuilder
Dim conncString As String
conncStringBuild.Server = TextBox3.Text 'Define Server Host
conncStringBuild.UserID = TextBox4.Text 'Define User
conncStringBuild.Password = TextBox2.Text 'Define Pass
conncStringBuild.Database = TextBox1.Text 'Define Database
conncString = conncStringBuild.GetConnectionString(True)
Use parameters for your query it automaticly fix some cool things like decimal separator, date, string ' char...
Code: Select all
spwn.Connection = Conn
spwn.Parameters.AddWithValue("@group_id", TextBox1.Text)
spwn.Parameters.AddWithValue("@percentage", SpawnBox1.Text)
spwn.CommandText = "INSERT INTO spawn_location_group_chances (group_id,percentage) VALUES (@group_id, @percentage);"
And last, prevent user to type char in your textboxs (add this event on TextBox KeyPress)
Code: Select all
Private Sub TextBoxKeyPress(ByVal sender As Object, ByVal e As System.Windows.Forms.KeyPressEventArgs)
'If not digit Handled event
If Char.IsDigit(e.KeyChar) = False And Asc(e.KeyChar) <> System.Windows.Forms.Keys.Back Then e.Handled = True
End Sub
You can also do it with decimal
Code: Select all
Private Sub TextBoxKeyPress(ByVal sender As Object, ByVal e As System.Windows.Forms.KeyPressEventArgs)
If Asc(e.KeyChar) = 46 Then e.KeyChar = "," 'Check we have good separator
If Asc(e.KeyChar) = 44 And (CType(sender, TextBox).Text.IndexOf(",") > 0 Or CType(sender, TextBox).Text = "") Then e.Handled = True
If Char.IsDigit(e.KeyChar) = False And Asc(e.KeyChar) <> System.Windows.Forms.Keys.Back And Asc(e.KeyChar) <> 44 Then e.Handled = True
End Sub
Hope it can help you

Fight with me... Or die, like the rest.
J.A. say: "I think Xinux tried to tell me this, but I ignore most things he suggests."