Es ist unsauber, weil das
TForm1.FormCreate(Sender: TObject) ein
Eventhandler ist. Das heißt, er wird von dem Formular aufgerufen.
Ideeal wäre es jetzt, wenn du den eigentlichen Code und diee Events nicht miteinander vermischst, sodass du auch mal andere Events benutzen kannst, ohne dabei Code umkopieren zu müssen.
Aber als Anfänger ist das durchaus okay
P.S. Nein, ich habe keine Lust deinen Code umzuschreiben