Bevorzugen Sie eine präzise oder lesbare Darstellung in Ihrem Code? [geschlossen]

Sprachverknüpfungen können häufig verwendet werden, um den Code übersichtlicher zu gestalten.

Ternäre und Null-Koaleszenz-Operatoren können beispielsweise die Codemenge reduzieren, was jedoch die Lesbarkeit beeinträchtigt:

In C #:

Person newGuy = new Person();
if (boss == null) {
    newGuy.Boss = GetDefaultBoss();
} else {
    newGuy.Boss = boss;
}

ist funktional äquivalent zu:

Person newGuy = new Person();
newGuy.Boss = boss ?? GetDefaultBoss();

aber offensichtlich viel ausführlicher.

Wo ziehen Sie die Grenze zwischen Prägnanz und Lesbarkeit?

Beide.

Ihr erstes Beispiel ist sicherlich ausführlicher und wohl auch expliziter ... aber es erfordert auch, dass ich statt einer fünf Zeilen scanne. Schlimmer noch, es hebt seinen Zweck hervor - einen Wert zuzuweisen newGuy.Boss.

Ihr zweites Beispiel kann mich eine Sekunde kosten, wenn ich mit dem Null-Koaleszenz-Operator nicht vertraut bin, aber es besteht kein Zweifel an seinem Zweck, und wenn ich eine größere Routine durchsuche, um nach der Quelle eines Werts zu suchen, wird dies der Fall sein Es ist viel einfacher für mich, dies herauszufinden.

Nun kontrastiere dies:

if (boss == null) {
    newGuy.Boss = GetDefaultBoss();
    newGuy.IsTemp = true;
    newGuy.AddTask("orientation");
} else {
    newGuy.Boss = boss;
    newGuy.IsTemp = false;
}

...mit:

newGuy.Boss = boss ?? GetDefaultBoss();
newGuy.IsTemp = boss == null;
if ( boss == null ) newGuy.AddTask("orientation");

Das letztgenannte Beispiel ist wieder viel kürzer, aber jetzt wird sein Zweck dadurch verschleiert, dass Aufgaben, die durch denselben Test ausgelöst wurden, als unterschiedlich erscheinen. Hier halte ich die Ausführlichkeit der ersteren für gerechtfertigt.

Obwohl beide gute Ziele sind, stehe ich immer auf der Seite der Lesbarkeit, wenn ich gezwungen bin , eines zu wählen.

Ich würde argumentieren, dass Ihr Beispiel sowohl die Lesbarkeit als auch die Kürze verbessert . Beachten Sie jedoch Folgendes:

if( a > b )
{
    foo = bar
}
else
{
    if( c.isThing() ){
        foo = somethingElse;
    }
    else{
        foo = someFurtherThing.GetFoo();
    }
}

im Gegensatz zu

foo = a > b ? bar ?? whatever.something : c.isThing() ? somethingElse : someFurtherThing.GetFoo();

Letzteres ist prägnant, aber schwer durchzulesen. Ersteres ist ausführlich, aber der Fluss der Logik ist klar.

Letztendlich dient die Kürze nur dem Zweck, mehr auf den Bildschirm zu passen. Die Lesbarkeit erleichtert das Debuggen und sollte daher generell bevorzugt werden.

Ich würde generell sagen, dass die Lesbarkeit niemals aus Gründen der Übersichtlichkeit geopfert wird, aber die Lesbarkeit niemals aufgrund mangelnder Kenntnisse eines anderen Programmierers zu diesem Thema beurteilt wird.

Prägnanz und Lesbarkeit sind keine Gegensätze. Wie diese Antwort ist manchmal kürzer besser lesbar.

Ich würde sagen, ich bevorzuge Lesbarkeit, obwohl das manchmal die Verwendung von prägnantem Code bedeutet. (Dh ternär für relativ einfache Bedingungen innerhalb eines größeren Bedingungsblocks.)

Grundsätzlich sollten Sie es nicht tun, wenn es unnötig schwer zu verstehen ist.

Die Lesbarkeit steht an erster Stelle, wenn sie im Widerspruch zur Prägnanz steht, da Code häufiger geändert wird als ursprünglich geschrieben. Auf der anderen Seite:

1. Syntaktisches Rauschen und Boilerplate-Code verschleiern häufig die Absichten und beeinträchtigen somit die Lesbarkeit. Manchmal ist der prägnantere Code auch besser lesbar. Denken Sie beispielsweise an Lambda-Funktionen oder Delegaten / First-Class-Funktionen im Vergleich zu Klassen mit nur einer Methode, die eine Schnittstelle mit nur einer Methode implementieren.

2. Die Lesbarkeit sollte danach beurteilt werden, wie einfach der Code zu lesen ist, und zwar für einen einigermaßen erfahrenen Programmierer, der die Sprache und ihre einzigartigen / erweiterten Funktionen ziemlich gut kennt, und nicht für einen kaum kompetenten Code-Affen, der nur den kleinsten gemeinsamen Nenner kennt.

Ein Aspekt, von dem ich glaube, dass er noch nicht erwähnt wurde: Was sind Ihre Ziele?

Wenn Ihnen nur die Arbeitsplatzsicherheit am Herzen liegt, sollten Sie bei allen anderen Aspekten auf Prägnanz und Kompaktheit achten. Überspringen Sie auch das Kommentieren Ihres Codes.

Wenn Sie in der Lage sein möchten, Ihren Code einfach an andere weiterzugeben, während Sie an einem coolen neuen Projekt arbeiten, sollten Sie auf Lesbarkeit, Klarheit und viele solide Kommentare achten.

_{^{Hinweis: Bei den obigen Angaben handelt es sich nicht um Sie persönlich, @Damovisa; Es ist für jeden, der zwischen den beiden Positionen wählt.}}

Es gibt eine Sache, die die ausführliche Version als Vorteil hat.

Es hat mehr Zeilen und die meisten Debugger sind zeilenorientiert ! Es ist sehr schwierig, einen Haltepunkt in der Mitte eines Ausdrucks zu setzen, aber normalerweise ist es trivial einfach, ihn in einer Blockanweisung zu setzen.

Mit anderen Worten, welche möchten Sie in Ihrem Editor sehen, wenn Ihr Debugger wann aktiv werden soll boss == null?

(Das heißt ich mag den ?? - Operator)

Die Lesbarkeit sollte an erster Stelle stehen. Langfristig verbringen die meisten Menschen die meiste Zeit damit, vorhandenen Code zu ändern oder zu erweitern. Die Lesbarkeit ist ein großer Teil der Wartbarkeit.

Das heißt, Prägnanz ist etwas, das zur Lesbarkeit beitragen kann. In Ihrer Frage ist beispielsweise das zweite Snippet lesbarer und übersichtlicher.