
Zitat von
Sascha_OW:
ich woltle heute mal meine neue Kreation vorstellen und wissen was ihr dazu sagt. Ich habe im Forum schön öfters gesagt ich bin Anfänger und möchte bitte ehrliche konstrotive kritik.
1.) seitlich wird als
horizontal bezeichnet
2.) hoch und runter wird als
vertikal bezeichnet
3.) für horizontal & vertikal kann man doch auch schöne Bildchen malen und den Text ganz weglassen
4.) mehr als 2 Nachkommastellen machen in der Anzeige keinen Sinn
5.) das Hauptfenster Unit1 braucht viel zu viel Platz
6.) Unit1, Unit2, Unit3 ist nicht aussagekräftig; sprechende Namen verwenden z.B. UHistoryForm
7.) das History Formular sollte die Daten besser mit einem ListView mit ViewStyle auf vsReport anzeigen. das sieht wesentlich besser aus
8.) die Funktion SQRT((cm*cm) + (cm1*cm1)) wird öfters benötigt -> in eigene Funktion auslagern
Delphi-Quellcode:
function Distance(a, b:Extended):Extended;
begin
Result := SQRT(a*a+b*b);
end;
9.) wozu brauchst du 3 Timer; das geht auch mit einem
10.) statt nur 2 Einheiten (cm und m) würde ich eine Combobox mit Meter, cm, Fuss, km nehmen und so die Umschaltung vornehmen. (informier dich mal, welche Längeneinheiten es bei den Amis noch so gibt)
11.) du solltest alle Distanzen grundsätzlich nur in Meter ausrechnen. Erst bei der Anzeige erfolgt dann die Umrechnung in andere Einheiten.
Das wars vorerst mal... fröhliches