Prinzipiell kann man das schon so machen, aber ein paar kleine Dinge fallen mir auf:
- Wieso greifst Du ständig auf die globale Variable Form2 zu?
Weil ich das aus der Form1 tue. Sonst kann ich die ComboBox nicht erreichen.
- SELECT * sollte man nicht verwenden, sondern lieber die abzufragenden Felder einzeln benennen.
Hm... wieso eigentlich? Ich brauche alle Felder, deswegen habe ich den * genommen. Sonst frage ich die Felder natürlich einzeln ab.
- Du solltest nach Möglichkeit
SQL-Parameter verwenden, allein schon um die Gefahr der
SQL-Injection zu vermeiden oder zumindest zu minimieren.
Wie meinst du das?
- Statt auf eine ComboBo direkt zuzugreifen, würde ich persönlich mir eine Prozedur schreiben, die ein TStrings-Objekt befüllt, damit ist man flexibler, man kann ihr dann z.B. ComboBox.Items, ListBox.Items oder auch eine TStringlist übergeben.
Stimmt, du hast recht, habe irgendwie daran nicht gedacht... muss mal überlegen und es so machen. Danke Dir!