*Uiuiui*, Du glaubst nicht im Ernst, dass jemand jetzt die Hunderte von Zeilen Quellcode Zeile für Zeile durchgeht, oder? Trotzdem ein paar Anmerkungen:
- Du deklarierst Deine Records und Klassen in beiden Units, das ist nicht so doll, schreib lieber eine eigen
Unit dafür und binde diese dann in beiden Formularen ein.
- Der Destruktor darf nicht Free, sondern muss Destroy heißen.
- Trenne Logik und Darstellung, d.h. die Daten werden in Deinen Klassen/Records abgelegt und in den
VCL-Komponenten lediglich dargestellt.
- Statt mit statischen Arrays solltest Du mit dynamischen Arrays oder mit Listen arbeiten.
Das sind die Punkte, die mir gleich ins Auge gefallen sind, vermutlich gibt es noch mehr, aber das soll erstmal reichen.