Code-Optimierungen: allgemeine Fragen

[DieDolly]

Ich habe mal meinen Code durchgeguckt und gesehen, dass ich sehr viele Funktionen und Prozeduren habe, die doch sehr lang sind.

- wie lang sollte eine Funktion oder Prozedur maximal sein?
- wie weit sollte man Code in Units aufteilen? Ich habe schon viel aufgeteilt und mittlerweile 58 Units mit thematisch aufgeteiltem Code. Was ist mit einer Funktion/Prozedur pro Unit, wenn ich nur eine für dieses Themengebiet habe?
- was kann man sonst noch verbessern außer möglichst const in den Köpfen zu verwenden?

[Redeemer]

Bei Stack-Overflow würden sie das Thema schließen weil zu allgemein und zu subjektiv.

[Andreas13]

Pascal Analyzer https://www.peganza.com/ liefert dafür viele nützliche Hinweise.
Gruß, Andreas

[Hobbycoder]

Also ich halte das so, dass nach Möglichkeit keine Prozedur ode Funktion mehr als ca. 20 Zeilen hat. Es gibt natürlich immer mal wieder den Fall wo sowas schlecht geht oder suboptimal ist. In dem Fall wird der Grund mit dokumentiert.

Units, die am Ende nur eine Prozedur enthalten werden in eine Funktionssammlung „common“ zusammengefasst. Manchmal auch in mehreren commons („graphiccommon“, „guicommon“, etc)

Ansonsten halt gut dokumentieren, damit man (oder andere) auch nach Jahren noch nachlesen kann, welchen Blödsinn man seinerzeit verzapft hatte

[DieDolly]

20 Zeilen? Das klingt nach einem Ding der Unmöglichkeit.

[Uwe Raabe]

Ich bevorzuge die Faustregel, dass der Code einer Methode (alles zwischen begin und end) komplett auf eine Bildschirmseite passt. Zu Turbo Pascal Zeiten waren das anfangs gerade mal 24 Zeilen, heute können das auch mal 50 oder mehr sein - kommt halt auf die Größe des Bildschirms und der Schrift an.

Bei der Zeilenlänge habe ich mich aktuell auf 130 Zeichen eingestellt. Das ist noch gut überschaubar und vermeidet so manchen störenden Umbruch.

Hintergrund ist einfach, daß man den Code mit einem Blick erfassen kann ohne scrollen zu müssen.

[freimatz]

Zitat von DieDolly:

20 Zeilen? Das klingt nach einem Ding der Unmöglichkeit.

In dem Beitrag davor steht ganz unten:
Alle sagten: "Das geht nicht.". Dann kam einer, der wusste das nicht, und hat's einfach gemacht.

Es geht. Man kann es lernen wenn man will.

Ich finde es super, dass Du schon so viel gemacht hast.

Zitat von TiGü:

Ich möchte dir folgende ... Seiten/Links ans Herz legen:
https://clean-code-developer.de/

metoo

[Hobbycoder]

Gelöscht

[jfheins]

Wir machen in letzter Zeit mehr typescript, aber da gelten eigentlich letztlich die gleichen Sachen wie überall.

Wir versuchen, Funktionen klein zu halten und Logik zu kapseln, damit es übersichtlich bleibt. Es gibt dennoch harte Limits: Wir haben den linter aktiv, sodass code Änderungen pauschal abgelehnt werden, falls die cyclomatische Komplexität einer Funktion zu groß ist oder die Datei länger ist als 300 Zeilen.

[Harry Stahl]

Zitat von Andreas13:

Pascal Analyzer https://www.peganza.com/ liefert dafür viele nützliche Hinweise.
Gruß, Andreas

Das kann ich unterstützen, das hat mir auch geholfen und ich habe einige echt blöde Sachen gefunden.

Das mit der Zeilenanzahl, die so oder so lang sein soll, ist echt recht bedeutunglos. Die Funktion hat soviel Zeilen wie nötig. Wenn es Sinn macht und es geht, kann man einen Teil der Logik in mehrfach verwendbare Code-Teile aufteilen, wo es nicht geht, ist die Funktion eben auch mal 200 oder 300 Zeilen lang...