Übergebenes nil erkennen?

[Stevie]

Zitat von Furtbichler:

Zitat von Stevie:

Für mich persönlich ist es sehr viel verwirrender, wenn ich sowas im Sourcecode sehe. Dann frag ich mich, ob hier entweder ein ahnungsloser am Werke war, oder ob es irgendein Problem gab, weswegen das gemacht wurde.

Beide Gedanken könntest Du dir abgewöhnen (und solltet Du vielleicht auch), denn nur weil der Code nicht deinem Geschmack entspricht, heißt das nicht, das ein Ahnungloser ab Werk war oder es ein Problem gab. Das ist so, als ob ein impressionistischer Künstler einem Expressionisten jegliche (handwerkliche) Fähigkeit abspricht. Etwas eng gefasst, die Sichtweise, wenn Du mich fragst.

Wir reden hier eigentlich nicht von künstlerischer Freiheit.

In allen objektorientierten Sprachen, mit denen ich bisher gearbeite habe (eventuell ist es aber auch zu lang her, zuletzt nur mit Delphi und C#, also korrigier mich, wenn ich falsch liege) war es so, dass Felder von Objekten immer den Default Wert hatten und man nicht Explizit auf false, Leerstring, nil/null oder 0 setzen musste. Das ist auch eins der ersten Dinge, die ich einem Neuling sage, wenn er mit OOP anfängt. Daher ist überflüssiger Code keine Dokumentation für mich, sondern Code Bloat.

Zitat von Furtbichler:

Zitat:

Genauso initialisier ich eine Variable nur dann, wenn deren Wert irgendwo benutzt wird, bevor ihr ein Ergebnis einer Operation zugewiesen wird. (Stichwort H2077 Value assigned to <variable> never used)

Wer würde das nicht tun?

Ich hab nicht gezählt, wie viele Warnings und Hints (unter anderem der besagte) wir in unserem Source im letzten Jahr beseitigt haben, weil sich da scheinbar einige nicht dran gehalten haben in der Vergangenheit.

[Furtbichler]

Zitat von Stevie:

Wir reden hier eigentlich nicht von künstlerischer Freiheit.

In gewisser Weise schon, denn erstens gibt nicht nur einen richtigen Code und zweitens drücke ich mit den Code etwas aus. Jeder Programmierer hat eine andere Sprache, um dies zu tun. Hauptsache, es ist verständlich und nachvollziehbar.

Deinen weiteren Ausführungen entnehme ich, das es 'sonnenklar' ist, das Felder mit 0 vorbelegt werden, aber lokale Variablen nicht. War mir entfallen.

Zitat:

Daher ist überflüssiger Code keine Dokumentation für mich, sondern Code Bloat.

Ja, das kann man so sehen. Das 'für mich' ist entscheidend (=künstlerische Freiheit, siehe oben). Bei Visual Studio habe ich mir sehr mächtige Refaktorisierungstools zugelegt, die mir die von dir bemängelte Redundanz austreiben, sodaß ich tendentiell doch zu dem von Dir favorisierten Code übergehen werde.