Ich hab mal in den Sourcecode reingeschaut...
Für ein Schulprojekt ist das erstaunlich gute Qualität.
Auch die Benutzeroberfläche ist sehr ansprechend und auch für Touchscreens geeignet.
Gut, mir gefiel bis zum Schluss die
GUI nicht im geringsten
. Aber nun bin ich damit auch zufrieden
Aber ich habe auch noch ein bisschen was zu merkern:
Insgesamt ist der Code zu wenig kommentiert und falls Kommentare vorhanden sind, dann beschreiben sie nur das Offensichtliche. Beispiel:
Bt_Correct.Visible := False; //Button ausblenden
Lieber diese Null-Info-Kommentare weglassen und eine Kurz-Beschreibung vor jeder Klasse.
Beim Kommentieren (mit dem ich auch noch nicht ganz fertig war) stelle ich mich immer komisch an .. ich weiß nie was ich kommentieren soll und was offensichtlich ist. Aber ich werde es mal probieren, vor der Methode eine erklärung zu schreiben
Und man sollte eine eigene Klasse auch nicht TList nennen weil es
TList
schon in der
VCL gibt.
Ahh, da hat NRW wohl mist gebaut
die haben in den Abitur Vorgaben die Klasse so genannt, und unser Kurs hat es so übernommen.
Danke für dein feedback
