Sollten geschweifte Klammern in einer eigenen Zeile erscheinen? [geschlossen]

Sollten geschweifte Klammern in einer eigenen Linie stehen oder nicht? Was denkst du darüber?

if (you.hasAnswer()) {
    you.postAnswer();
} else {
    you.doSomething();
}

oder sollte es sein

if (you.hasAnswer())
{
    you.postAnswer();
}
else
{
    you.doSomething();
}

oder auch

if (you.hasAnswer())
    you.postAnswer();
else
    you.doSomething();

Bitte seien Sie konstruktiv! Erklären Sie, warum, tauschen Sie Erfahrungen aus und belegen Sie sie mit Fakten und Referenzen.

Als Student habe ich geschweifte Klammern in dieselbe Zeile gesetzt, damit weniger Zeilen vorhanden sind und der Code auf weniger Seiten gedruckt wird. Es ist ärgerlich, sich ein einzelnes Klammerzeichen anzusehen, das als einziges in einer Zeile gedruckt wird. (Umwelt, Papierverschwendung)

Wenn Sie jedoch große Anwendungen codieren, ist es angesichts des Gruppierungsgefühls erschwinglich, einige Zeilen nur mit geschweiften Klammern zuzulassen.

Unabhängig davon, welchen Stil Sie auswählen, sollten Sie konsistent sein, damit es für Ihr eigenes Gehirn nicht zu einem Aufwand wird, mehrere Stile in verwandten Codeteilen zu verarbeiten . In verschiedenen Szenarien (wie oben) würde ich sagen, dass es in Ordnung ist, verschiedene Stile zu verwenden. Es ist einfacher, den Kontext auf hoher Ebene zu wechseln.

Sie sollten niemals die 3. Methode durchführen.

Wenn Sie beim ersten Mal auf geschweifte Klammern verzichten, sparen Sie möglicherweise ein paar Tastatureingaben, aber der nächste Programmierer, der mitkommt, fügt Ihrer else-Klausel etwas hinzu, ohne zu bemerken, dass der Block fehlt.

Schreiben Sie Ihren Code für andere Personen.

Lange Zeit argumentierte ich, dass sie gleichwertig oder so nahe beieinander lagen, dass der mögliche Gewinn durch die Auswahl der richtigen Option weit unter den Kosten für Streitigkeiten lag .

Es ist jedoch wichtig , konsequent zu sein . Also sagte ich, lasst uns eine Münze werfen und mit dem Schreiben von Code beginnen.

Ich habe schon einmal gesehen, wie Programmierer sich solchen Veränderungen widersetzten. Komm darüber hinweg! Ich habe in meiner Karriere oft gewechselt. Ich verwende in meinem C # sogar andere Stile als in meiner PowerShell.

Vor ein paar Jahren arbeitete ich in einem Team (ca. 20 Entwickler), das entschied, Eingaben anzufordern, eine Entscheidung zu treffen und diese dann in der gesamten Codebasis durchzusetzen. Wir hätten 1 Woche Zeit, um uns zu entscheiden.

Viel Stöhnen und Augenzwinkern. Viele "Ich mag meinen Weg, weil es besser ist", aber keine Substanz.

Während wir uns mit den Feinheiten der Frage befassten, fragte jemand, wie man mit diesem Problem auf der gleichen Linie umgeht:

void MyFunction(
    int parameterOne,
    int parameterTwo) {
    int localOne,
    int localTwo
}

Beachten Sie, dass nicht sofort ersichtlich ist, wo die Parameterliste endet und der Body beginnt. Vergleichen mit:

void MyFunction(
    int parameterOne,
    int parameterTwo) 
{
    int localOne,
    int localTwo
}

Wir haben gelesen, wie Menschen auf der ganzen Welt mit diesem Problem umgegangen sind, und dabei das Muster gefunden, nach der offenen Klammer eine Leerzeile einzufügen:

void MyFunction(
    int parameterOne,
    int parameterTwo) {

    int localOne,
    int localTwo
}

Wenn Sie eine visuelle Pause einlegen möchten, können Sie dies auch mit einer Zahnspange tun. Dann werden auch Ihre visuellen Pausen konsistent.

Edit : Zwei Alternativen zur 'Extra Blank Line'-Lösung bei Verwendung von K & R:

1 / Die Funktionsargumente werden anders als im Funktionskörper eingerückt

2 / Setzen Sie das erste Argument in dieselbe Zeile wie den Funktionsnamen und richten Sie weitere Argumente in neuen Zeilen an diesem ersten Argument aus

Beispiele:

1 /

void MyFunction(
        int parameterOne,
        int parameterTwo) {
    int localOne,
    int localTwo
}

2 /

void MyFunction(int parameterOne,
                int parameterTwo) {
    int localOne,
    int localTwo
}

/Bearbeiten

Ich behaupte immer noch, dass Konsistenz wichtiger ist als andere Überlegungen, aber wenn wir keinen etablierten Präzedenzfall haben , ist Klammer-an-Nächste-Linie der richtige Weg.

Die Grundregeln sind:

1. Befolgen Sie den bestehenden Kodierungsstandard des Projekts.
2. Wenn es keinen Codierungsstandard gibt und Sie eine vorhandene Codebasis bearbeiten, deren Eigentümer eine andere Person ist, sollten Sie mit dem Stil des vorhandenen Codes übereinstimmen, unabhängig davon, wie sehr Sie ihn mögen / nicht mögen.
3. Wenn Sie an einem Projekt auf der grünen Wiese arbeiten, besprechen Sie dies mit anderen Teammitgliedern und einigen Sie sich auf einen formellen oder informellen Kodierungsstandard.
4. Wenn Sie als einziger Entwickler an einem Projekt auf der grünen Wiese arbeiten, entscheiden Sie sich selbst und seien Sie dann rücksichtslos konsequent.

Auch wenn Sie keine externen Einschränkungen haben, ist es (IMO) am besten, nach einem vorhandenen (weit verbreiteten) Codierungsstandard oder einer Stilrichtlinie zu suchen und diese zu befolgen. Wenn Sie Ihren eigenen Stil rollen, besteht eine gute Chance, dass Sie es in ein paar Jahren bereuen werden.

Schließlich ist ein Stil, der mit vorhandenen Stilprüfern und Code-Formatierern implementiert / implementiert werden kann, besser als einer, der manuell "erzwungen" werden muss.

Der Vorteil der ersten Methode besteht darin, dass sie vertikal kompakter ist, sodass Sie mehr Code auf Ihrem Bildschirm anzeigen können. Deshalb bevorzuge ich sie. Das einzige Argument, das ich für die zweite Methode gehört habe, ist, dass es einfacher ist, öffnende und schließende Klammern zu koppeln, aber die meisten IDEs haben eine Tastenkombination dafür und es ist tatsächlich eine falsche Aussage - anstatt eine öffnende Klammer mit einem schließenden zu koppeln eckige Klammer Sie können eine schließende eckige Klammer mit dem Ausdruck "Blockanfang" (wenn, sonst für, während) auf derselben Einrückungsstufe koppeln, damit Sie genauso leicht feststellen können, wo der Blockanfang liegt.

Ich sehe keinen Grund, eine ganze Zeile nur für eine Klammer zu verschwenden, wenn das vorhergehende for / while / if-Konstrukt bereits den Beginn eines Blocks optisch anzeigt.

Trotzdem glaube ich, dass die schließende Klammer in einer eigenen Zeile stehen sollte, da wir etwas brauchen, um das Ende eines Blocks und seine Einrückungsstruktur auf sichtbare Weise anzuzeigen.

ich bevorzuge

if (you.hasAnswer())
{
    you.postAnswer();
}
else
{
    you.doSomething();
}

Über

if (you.hasAnswer()) {
    you.postAnswer();
} else {
    you.doSomething();
}

weil die Zeile you.postAnswer();auf den ersten Blick viel leichter zu lesen und zu finden ist. Zweitens verschmilzt es mit der darüber liegenden Linie ( you.hasAnswer()), sodass meine Augen mehr fokussieren müssen, um es zu lesen.

Ich bevorzuge die erste Methode. Zahnspangen sind keine separate Linie wert.

Die Sache ist, dass Zahnspangen nicht wichtig sind. Sie sind nur syntaktischer Müll , der absolut unnötig ist, um zu verstehen, wozu Code dient, wozu er dient und wie er implementiert wird. Sie sind nur eine Hommage an alte C-ähnliche Sprachen, bei denen eine visuelle Gruppierung der Operatoren aufgrund des geringen verfügbaren Bildschirmplatzes nicht möglich war.

Es gibt Sprachen (Python, Haskell, Ruby), die ohne geschweifte Klammern überhaupt in Ordnung sind. Dies bestätigt nur, dass es sich bei geschweiften Klammern um Papierkorb handelt und dass sie nach Möglichkeit keine Zeile verdienen sollten:

if (you.hasAnswer()){
    you.postAnswer();
}else{
    you.doSomething();
}

Verwenden Sie Python und umgehen Sie das Argument vollständig.

Die Position der geschweiften Klammern sollte sein

Metadaten

Vom Programmierer in der IDE konfigurierbar. Auf diese Weise sehen diese lästigen Klammern im gesamten Code unabhängig vom Autor gleich aus.

Es hängt davon ab, ob.

Wenn ich in Javascript oder jQuery codiere, verwende ich das erste Formular:

jQuery(function($) { 
    if ($ instanceOf jQuery) { 
        alert("$ is the jQuery object!"); 
    } 
}); 

Wenn ich jedoch in C # codiere, verwende ich die zweite Form, da dies die kanonische Methode ist, um dies in C # zu tun.

public int CalculateAge(DateTime birthDate, DateTime now) 
{ 
    int age = now.Year - birthDate.Year; 
    if (now.Month < birthDate.Month 
        || (now.Month == birthDate.Month && now.Day < birthDate.Day)) 
        age--; 
    return age; 
} 

Beachten Sie, dass Ihr Beispiel geschrieben werden kann

if (you.hasAnswer())
    you.postAnswer();
else
    you.doSomething();

in C #.

Ich bevorzuge die erste, weil es mir schwerer fällt, den Fehler in diesem Beispiel zu erkennen.

if (value > maximum);
{
    dosomething();
}

als es in diesem Beispiel ist

if (value > maximum); {
    dosomething();
}

Das ; {sieht für mich eher falsch aus als eine Zeile, die mit endet, ;sodass ich es eher bemerke.

Ich bevorzuge eine leichte Variante von 1)

if (you.hasAnswer()) {
    you.postAnswer();
} // note the break here
else {
    you.doSomething();
}

Warum?

• Ich denke, immer geschweifte Klammern in die eigene Zeile zu setzen, verringert die Lesbarkeit. Ich kann nur eine bestimmte Menge Quellcode auf meinem Bildschirm anzeigen. Bracket-Stil 2) macht Hove-Algorithmen mit vielen verschachtelten Schleifen und Bedingungen schmerzhaft lang.

• Ich möchte elsejedoch auf einer neuen Linie beginnen, weil ifund zusammen elsegehören, visuell. Wenn sich eine Klammer vor der befindet else, ist es viel schwieriger zu erkennen, was zu was gehört.

• 3) disqualifiziert sich. Wir alle wissen, was schlimme Dinge passieren können, wenn Sie die Klammern weglassen und es vergessen.