Das Problem an deinem Code ist, dass da ja aber überhaupt nicht hervorgeht, wie du auf die
DB verbindest... Deshalb kann an dieser Stelle nix gesagt werden, ausser kritisiert werden, dass deine
Unit- und Objektnamensgebung nicht gerade optimal gewählt ist. Ausserdem könnte deine Button1Click Prozedur wesentlich übersichtlicher gestaltet werden... (und nicht nur die).. Aber das ist ja hier primär nicht das Problem.
EDIT: Und um an der Aussage von Omate noch anzuknüpfen: Bei einem Wechsel auf ein anderes DBMS könnte dann gleich in einem Zuge auch mal der Code gescheit aufgeräumt werden.