Angepinnt Beispiele für guten und schlechten Code (Stil)

roddy schrieb:

@milaim:

Wenn schon Beispiele bringen, dann syntaktisch richtige (beidesmal fehlt das "Then").

Tja, das kann passieren, wenn eine Entwicklungsumgebung solche Sachen automatisch macht, wenn man Enter drückt.. Kann man leicht vergessen, das selber einzutippen.

Ich wollte aber nicht das If - Then - Else - End If erklären das ist dir schon klar oder? Meine Hauptaussage ist angekommen..

ein "Hey hast das 'Then' vergessen" hätte mir auch gereicht.. tzz

da muss ich kevin89, mal ausnahmsweise zustimmen.

Danke für das ausnahmsweise

Sorry, milaim.

Du hast Recht. Ich hätte das auch freundlicher sagen können. Ich hoffe, du kannst mir nochmal verzeihen.

Dass es Umgebungen gibt, bei denen das "Then" automatisch gesetzt wird, wusste ich nicht. Bei mir (VBA) kommt ohne "Then" die Meldung, dass Then oder GoTo erwartet wird.

dusti schrieb:

Es wird aber sogar im MSDN als schlechter Stil bezeichnet. Ob das jetzt aussagekräftig ist, muss aber jeder für sich entscheiden...

Nicht nur in der MSDN, auch an vielen FHs in Deutschland wird den Studenten dies als schlechter Stil beschrieben.

Mit welcher Begründung? Ist viel klarer wenn man über den Code guckt finde ich.


finde ich nicht so klar wie

man schreibt ja auch

Also warum nicht True?

Weil du das ja 2x abfragst. Wenn du Auto fährst und an eine rote Ampel kommst denkst du dir ja auch nur "Aha, rot, also anhalten" und nicht "Ist die rote Ampel rot? Ja? dann anhalten" Eine rote Ampel mit einer roten Ampel zu vergleichen macht einfach keinen Sinn.

Hm ja macht keinen Sinn (geht ja auch ohne) aber es hat nichts mit Stil zu tun, da stimmst du mir hoffentlich zu oder

Also ich mach das auch immer ohne "= True" oder "= False".

Statt



schreibe ich



Mir geht es da wie dusti. Wozu eine Boolean-Variable mit einem Boolean-Wert vergleichen, wenn man auch direkt die Boolean-Variable verwenden kann. Und wie dusti angerissen hat, werden bei "move = True" zwei Vorgänge ausgelöst (Laden des Werts von move und Vergleich mit True), während bei "move" nur der erste Vorgang ausgelöst wird, sodass die Ausführung ohne "= True" prinzipiell schneller geht. Inwieweit das dann tatsächlich spürbar ist und sich auf die Performance auswirkt, weiß ich jedoch nicht.

Jo if move = false ist genauso ein schrott
du schreibt doch auch nicht folgendes:


...

Nein, dass nicht, aber an bestimmten Stellen schon. Ich schreibe manchmal auch folgendes:

Statt:


das hier:



... macht doch Sinn, oder?

Dafür wäre wesentlich besser geeignet..

~ Chris

Nein, macht es nicht.

Das ist ja so mit das grausigste was ich hier gesehen habe. Wenn du Klammern einfügen würdest, würde es wieder Sinn machen aber so... Ein zweifacher Vergleich ohne Klammern wird doch von links nach rechts ausgewertet, oder? Wenn du aber auf sowas bauen musst, ist das schlechter Stil.
Anstatt

Schreibt man

oder eventuell noch etwas mit Not, aber doch nicht so einen komplizierten und unschönen Zweifachvergleich

Hab ich das nicht gerade geschrieben? ôO

~ Chris

Schau mal die zeit an. Ich habe für meinen Post länger als 3 Min gebraucht, habe also zeitgleich, wenn nicht sogar vor dir angefangen.

Ah stimmt, nicht gesehen *duck*

~ Chris

np

Ok das sehe ich ein, wieder was dazugelernt

Dafür schreibe ich öfters etwas in diesem Schema:



Ich finde halt cases viel übersichtlicher, da auch Zeilen nicht so lang dabei werden.
Dies soll ein ein Beispiel verdeutlichen:



Mag aber auch alles Ansichtssache sein, ich persönlich finde es jedenfalls überschaubarer.

Noch netter finde ich übrigens folgende kaum bekannte Möglichkeit:



Das bastelt einmal mit if, einfach nur schrecklich ...

vanitas-mundi schrieb:

VB.NET-Quellcode

1. select case true
2. case i = 5
3. case s = "hans"
4. case obj is nothing
5. case else
6. end select

Das bastelt einmal mit if, einfach nur schrecklich ...

Das ist allerdings alles schlechter Stil. Auch wenn es "einfacher" ist.
In C# gibt es dafür jedoch eine Möglichkeit, die es in VB nicht gibt. Man kann dort Kurzformen benutzen.

Und schon hat man einen Einzeiler, der folgendem Konstrukt entspräche (in VB):

Oder es gibt die Möglichkeit, folgendes zu tun:

(Sollte funktionieren, ungetestet :D)
das entspräche:

Die Schlussfolgerung aus diesem Ganzen: VB macht es nicht gerade einfach, mit gutem Stil zu programmieren.

In VB geht übrigens auch das:



und das:



Das sind auch jeweils Einzeiler.

Ich persönlich benutze die mehrzeilige Variante nur, wenn ich mehrere Then- oder Else-Befehle habe oder wenn die Zeile extrem lang werden würde.

Noch was zu Select Case:

Bei folgenden beiden Code-Beispielen passiert im Prinzip das gleiche:





jedoch mit dem Unterschied, dass bei "Select Case" die Boolean-Werte mit True verglichen werden. Damit wären wir wieder beim Thema "If move = True Then".

Wenn man den Vorgang von "Select Case" mit If-Anweisungen genau nachbauen will, müsste man schreiben:



Demnach ist auch meiner Meinung nach die "Select Case True"-Methode schlechter Stil.