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?
- SELECT * sollte man nicht verwenden, sondern lieber die abzufragenden Felder einzeln benennen.
- Du solltest nach Möglichkeit
SQL-Parameter verwenden, allein schon um die Gefahr der
SQL-Injection zu vermeiden oder zumindest zu minimieren.
- 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.