Code-Style: wie verschönert/verbessert ihr die Lesbarkeit großer Dateien?

[Luckie]

Das Problem bei so einen Copy and waste Vorgehen ist, dass man unter Garantie eine Stelle vergisst, wenn man etwas ändern muss. Und dann geht die Fehlersuche los.

[mm1256]

Yupp. Und darum glaube ich auch, dass die Regel "wenn du einen Code 2x benötigst, dann lagere ihn in eine procedure/function aus" seine Berechtigung hat. Für die Lesbarkeit mag das zwar dann in manchen Fällen etwas schlechter sein, aber für die Wartbarkeit ist es ein Gewinn.

[ConnorMcLeod]

Zitat von mm1256:

Yupp. Und darum glaube ich auch, dass die Regel "wenn du einen Code 2x benötigst, dann lagere ihn in eine procedure/function aus" seine Berechtigung hat. Für die Lesbarkeit mag das zwar dann in manchen Fällen etwas schlechter sein, aber für die Wartbarkeit ist es ein Gewinn.

Zustimmung - die Auslagerung ist sinnvoll.
Einspruch - wenn der Funktionsname gut gewählt ist, leidet die Lesbarkeit nicht.

[Sherlock]

Die Lesbarkeit leidet eben nicht darunter, weil man ja den Code durch einen beschreibenden Methodennamen ersetzt. Es geht dann so weit, daß ehemals 500 Zeilen Spaghetti in einer Methode dann zu 30 Zeilen werden, die man bestens lesen und verstehen kann (untendrunter gehts halt wieder ans blutige):

Delphi-Quellcode:

begin
  ErrechneBildGrenzen;
  SetzeMinimalWerte(NeueKoordinaten);
  Muss := VergleicheIstMitSoll(Ist, Soll);
  ZeichneWert(Muss);
{etc...}
end;

Was ist daran auszusetzen? Selbst wenn jeder ersetzte Block nur einmal benutzt wird, so ist die Lesbarkeit für das grobe Überfliegen des Codes und vor allem zum Neueinstieg nicht zu übertreffen.

Edit: What Connor said...

Sherlock