Die beste Methode zum Formatieren der bedingten Prüfungen in der "if" -Anweisung

8

Dieser Code sieht schmutzig aus und ich kann nicht herausfinden, wie er formatiert wird, damit ich ihn lesen, verstehen und gleichzeitig sauber aussehen kann.

%Vor%

Irgendwelche Vorschläge?

    
Lightyear Buzz 30.08.2011, 05:58
quelle

8 Antworten

17

Ich würde die booleschen Ausdrücke in Variablen zerlegen, die nach Lesbarkeit benannt sind. Etwas wie:

%Vor%     
cdmckay 30.08.2011, 06:05
quelle
6

Erwägen Sie ein Refactoring. Sie könnten Sub-Ausdrücke in ihre eigenen Funktionen einfügen und so ihren Zweck benennen.

Zum Beispiel:

%Vor%

Beachten Sie, dass im obigen Beispiel die Funktionsnamen möglicherweise falsch sind (ich habe nicht wirklich versucht zu verstehen, was der Zweck des Codes ist), aber Sie sollten das Bild bekommen.

    
Christian.K 30.08.2011 06:06
quelle
2

Für etwas Kompliziertes würde ich es wahrscheinlich in etwas zerlegen, was jede Bedingung (gruppiert nach geteilten & amp; & amp; & amp;;) versucht zu signalisieren und es einem vernünftigen Variablennamen zuzuordnen.

    
Shaun Wilde 30.08.2011 06:06
quelle
2

Sie können höchstens zusätzliche Klammern entfernen, Leerzeichen hinzufügen und die logischen Partitionen in verschiedene Zeilen einfügen,

%Vor%

Bearbeiten : Sie haben eine redundante Bedingung (x + width/2) == i , die ich hier optimiert habe.

    
iammilind 30.08.2011 06:03
quelle
1

Fast alle Klammern sind überflüssig ... und fügt einige Leerzeichen hinzu:

%Vor%     
ybungalobill 30.08.2011 06:05
quelle
1

Dies ist der gleiche wie der von Ihnen gepostete Code:

%Vor%     
Exa 30.08.2011 06:13
quelle
1

Wenn Sie es neu anordnen, erhalten Sie etwa Folgendes:

%Vor%     
Sander De Dycker 30.08.2011 06:13
quelle
1

Ich würde es so machen

%Vor%     
Ed Heal 30.08.2011 06:06
quelle

Tags und Links

Django: Verwenden von Annotate, Count und Distinct in einem Queryset zeigt die Linien der x-Achse und y-Achse ohne die Gitterlinien unter Verwendung von flot an