Das Programm an sich ist ok, aber zum Code:
1. Benutz das Entity Framework für die Verbindung
2. Fang an Konventionen zu beachten:
(Sind jetzt so meine Konventionen die können aber auch abweichen ist halt von der eigenen Gewöhnung abhängig)
-Variablen für Getter/Setter immer mit _ anfangen
-Parameter mit p kennzeichen (bsp pArg)
-lokale Variablen mit l kennzeichen (bsp lLoc)
-Methoden wenn private erstes Wort klein dann anfangsbuchstaben groß, wenn public dann Anfangsbuchstabe immer groß
-keine Umlaute (statt ü z.B. ue)
-Controls ordentlich benennen
-OO programmieren (nicht alles in Form1 stopfen)
-kein Denglish (entweder Englisch oder Deutsch)
-Alles an Code was nicht auf dem ersten Blick selbstverständlich ist kommentieren
und 3. vermeide am besten Try catch, und wenns nicht geht dann am besten nur einzelne Exceptions catchen, auch wenn das bei so einem kleinen Projekt jetzt nicht viel ausmacht