Verwenden eines ternären Operators mit 4 Ausdrücken

Question

Ist dies eine akzeptable Codierungspraxis?

public class MessageFormat { 
    private static final Color DEFAULT_COLOR = Color.RED; 

    private Color messageColor = DEFAULT_COLOR; 

    public MessageFormat(Person person) { 
     Color color = person.getPreferredColor(); 
     messageColor = (color != null) ? color : messageColor; // this line 
    } 
} 

oder bin ich besser dran mit dem klassischen gehen ...

if (color != null) { 
    messageColor = color; 
} 

Answer 1

Die Verwendung des Operators?: Sollte darauf beschränkt sein, den Code lesbarer zu machen. Ein klassisches Beispiel:

a = sprintf("There are %i green bottle%s on the wall.", i, (i==1?"":"s")); 

In diesem Fall wäre der Code weniger lesbar, wenn Sie es in etwa auflösen 5 if/else Linien.

Ich lege normalerweise Klammern um den gesamten Operator, so dass ich es beim Lesen mental als einen einzigen Wert analysiere.

messageColor = (color != null ? color : messageColor); 

Eine andere Variante ist

messageColor = color || messageColor; 

die in einigen Sprachen auswerten wird „Farbe, es sei denn, Farbe wertet auf‚false‘, in welchem ​​Fall Wert von messageColor. Meiner Meinung nach sollte dies als vermieden werden kann es die Menschen verwirren.

das wichtigste im Einklang sein, so dass die nächste Person, Ihren Code zu lesen (auch wenn du es ist) Mindest kognitiven Aufwand hat.

Answer 2

Scheint gut zu mir (ich benutze Python ternären Operator viel), aber diese Art von Stil Thema ist in der Regel sehr subjektiv . Wenn das Projekt über ein Codierungsstildokument verfügt, sollten Sie dies überprüfen.

Answer 3

Ich bevorzuge die zweite, weil es deutlicher ausdrückt, was Sie meinen: Sie wollen nur die Farbe ändern, wenn sie nicht null ist. Die erste Methode macht das nicht so klar.

Answer 4

Die Verwendung des ternären Operators ist oft ein heikles Thema, zusammen mit anderen Codierungsstandards. Die Verwendung wird wahrscheinlich am besten durch Kodierungsstandards auf Ihrer Website bestimmt.

Allerdings würde ich in dieser speziellen Situation definitiv die zweite Option empfehlen; es ist nicht nur klarer, aber die Verwendung des ternären Operators ist hier völlig überflüssig. Es ist nicht notwendig, messageColor für sich selbst neu zuzuordnen, daher ist die einzige Funktion des ternären Operators in dieser speziellen Situation die Codeverschleierung.

Answer 5

Ternäre Operatoren werden oft missbraucht, weil der Code, den sie erzeugen, clever und kompakt erscheint.

In der Tat machen sie den Code weniger lesbar und fehleranfälliger. Wann immer möglich, ist es ratsam, die längere Version

if (<condition>) { 
    <action> ; 
} 

Statt einer ternären Syntax zu verwenden.

Answer 6

Der ternäre Operator ist häufiger unter C-Programmierern. In C, wenn Sie Kontrollstrukturen vermeiden, können Sie häufig bessere Pipelining erhalten, da keine Verzweigungsvorhersage falsch ist. Ich bezweifle, dass Sie einen Leistungsunterschied in Java sehen würden, und das If-Null-Take-Assign-Muster ist viel häufiger als das Ternär. Wenn Sie jedoch eine vorhandene Codebasis beibehalten, ist es normalerweise am besten, mit dem vorhandenen Code zu konsistent zu bleiben.

Wenn Sie sich dabei häufig befinden, können Sie eine defaultIfNull, oder coalesce Funktion schreiben, die den Code noch prägnanter machen kann. Apache Commons Lang includes a defaultIfNull function.

Einige Sprachen enthalten einen Operator ||=, der das übliche Idiom für Standardwerte in diesen Sprachen ist.

Answer 7

In Ihrem Fall würde ich die "klassische" Implementierung bevorzugen, denn für mich ist es schneller zu verstehen, dass Sie nur eine neue Farbe verwenden möchten, wenn die Person eine bevorzugte Farbe hat.

ich es manchmal in Methode aufruft, wenn ich NPE vermeiden wollen, aber ich in der Regel diese hässlichen Teile des Codes in einer der nächsten Refactorings elimate;)

Answer 8

Ablesbarkeit, ein leichteres Verständnis usw. sind die gleichen in dieser Fall (ich meine, komm schon ...). Ich mag die Duplizierung und scheinbare Selbstzuweisung im ersten Beispiel nicht; es würde zu etwas wie übersetzen:

if (colour != null) {messageColour = colour;} 
    else {messageColour = messageColour;}; 

das ist ein bisschen dumm.

Ich würde normalerweise die zweite in einer Zeile schreiben, aber das ist eine Frage der individuellen Phantasie resp. Codierstil Richtlinien:

if (colour != null) {messageColour = colour;}; 

EDIT (Ich bin jetzt mehr opinionated als 8 Jahren)

Da Sie für Best Practices suchen:

// Use default visibility by default, especially in examples. 
// Public needs a reason. 
class MessageFormat { 
    static final Color DEFAULT_COLOR = Color.RED; 

    // Strongly prefer final fields. 
    private final Color messageColor; 

    // Protect parameters and variables against abuse by other Java developers 
    MessageFormat (final Person person) { 
     // Use Optionals; null is a code smell 
     final Optional<Color> preferredColor = person.getPreferredColor(); 
     // Bask in the clarity of the message 
     this.messageColor = preferredColor.orElse(DEFAULT_COLOR); 
    } 
}