Code-Optimierungen: allgemeine Fragen

[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...

[venice2]

Zitat:

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...

Dem kann ich nur beipflichten.

Welchen Sinn macht es eine Funktion zu erstellen die mir eine HDC zurückgeben soll, anschließend schleife ich dieses über 50 weitere Funktionen hindurch
nur um eine maximale Anzahl von 20 Zeilen pro Funktion zu erreichen.
Mit solchen Code möchte ich nicht arbeiten.

Zitat:

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

Ja, man kann hier schön prüfen wo eventuelle Const in Funktionen\Proceduren fehlen.
Aber leider sind hier auch ein paar Unstimmigkeiten die Ausgewertet werden. Aber das ist ein anderes Thema.

[Klaus01]

.. wenn man in Richtung CleanCode schaut - eine Methode sollte eine Aufgabe haben.
Hat sie mehere -> Refacto

Grüße
Klaus

[TigerLilly]

Pascal Analyzer, MMX und Delphi selbst liefern Komplexitätsmetriken, die hilfreich sind. Für mich gilt auch: Muss auf den Bildschirm passen + Zeilen werden bei 130 Zeichen umgebrochen. Wobei es manchmal schon den Spagat zu machen gibt zwischen: relevanten Code an einer Stelle haben + Methoden nicht zu lange werden zu lassen.

[venice2]

Zitat:

Muss auf den Bildschirm passen

Na dann viel Spaß wenn du mal richtig mit Winprocs arbeitest zeig mir mal wie du diese in einzelne Häppchen verteilen willst.
Es sei denn du erstellst für jede Message eine eigene Funktion.
Seltsame Programmoptimierung die was bringt? Nichts!

[TigerLilly]

Zitat von venice2:

Seltsame Programmoptimierung die was bringt? Nichts!

Wie so oft: für alles gibt es Ausnahmen, aber für unseren Code passen diese Regeln gut + haben sich bewährt.

[Sherlock]

Es ist schlicht so, daß Übersichtlichkeit auch in der SW-Entwicklung das A und O ist. Wenn eine Methode auf eine Bildschirmseite paßt, ist sie für jeden Menschen leichter zu erfassen als wenn es nicht so ist. Natürlich kann es Ausnahmen geben, wo es nicht anders geht. Aber man sollte es zumindest versuchen, den Code so zu restrukturieren, daß es paßt.

Sherlock

[Rollo62]

Zitat von Harry Stahl:

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...

Solche festen Regeln können viel Sinn machen, das ist Richtig.
Es sollte aber auch immer mal Ausnahmen geben dürfen, so sehe ich das auch.

Was ich aber Wichtiger oder besser als Ergänzung zu solchen numerischen Beschränkungen sehe,
ist eine generelle, gut lesbare Formatierungsstruktur der Texte.
Zum Beispiel formatiere ich Texte meist so das alle ":" untereinander stehen, um Variable/Typ auch optisch sauber zu trennen.
Auch lege ich mittlerweile immer Alles mit Indent 4 tatt 2 an, weil dadurch der Code aus meiner (und Anderer) viel lesbarer wird.

Mit guter Formatierung erkennt man auf einem Blick Zusammenhänge, und auch (Tipp)fehler viel schneller und direkter.
Zusätzlich ist die "Formatierungsarbeit" eine gute Gelegenheit den Code nochmal Revue passieren zu lassen, und eventuell zu überdenken und zu überarbeiten.

Was nützt mir ein gequetschter Spaggetticode ohne Punkt und Komme auf einer Seite, wenn ich diesen dann nur schwer wieder entschlüsseln kann ?
Alternativ würde ich diesen durch saubere Formatierung viel lesbarer bekommen, auch wenn er womöglich etwas länger wird als geplant.

Es gibt ja nicht Umsonst den Beruf der Designer, welche sich mit Schriften, Layout, Darstellung beschäftigen, weil es eben gewisse physiologische/psychologische Gegebenheiten gibt die man auch im Code beachten sollte.