将三元运算符与 4 个表达式一起使用

Question

这是可接受的编码实践吗？

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
    }
}

或者我最好选择经典的......

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

Answer 1

应限制 ?: 运算符的使用，以使代码更具可读性。一个经典的例子：

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

在这种情况下，如果将代码分成大约 5 个 if/else 行，代码的可读性就会降低。

我通常将整个运算符放在括号中，以便在阅读它时我在心里将其解析为单个值。

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

另一种变体是

messageColor = color || messageColor;

，在某些语言中，它会计算为“颜色”，除非颜色计算为“假”，在这种情况下，值为 messageColor。在我看来，应该避免这种情况，因为它可能会让人们感到困惑。

最重要的是保持一致，以便下一个人阅读你的代码（即使是你）的认知开销最小。

Answer 2

在这种情况下，可读性、易于理解性等都是相同的（我的意思是，来吧......）。我不喜欢第一个示例中的重复和明显的自我分配；它会翻译成这样：

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

这有点愚蠢。

我通常会把第二个写在一行中，但这是个人喜好的问题。编码风格指南：

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

编辑（我现在比 8 年前更加固执己见）

因为您正在寻找最佳实践：

// 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);
    }
}

Answer 3

三元运算符的使用以及其他编码标准通常是一个敏感问题。它的使用可能最好由您站点的编码标准决定。

然而，在这种具体情况下，我肯定会推荐第二种选择；不仅更加清晰，而且这里完全不需要使用三元运算符。不需要将 messageColor 重新分配给自身，因此在这种特殊情况下三元运算符的唯一功能是代码混淆。

Answer 4

三元运算符在 C 程序员中更为常见。在 C 语言中，如果避免控制结构，通常可以获得更好的流水线，因为没有分支预测会出错。我怀疑您会看到 Java 中的任何性能差异，并且 if-null-then-assign 模式比三元模式更常见。但是，如果您正在维护现有的代码库，通常最好与现有代码保持一致。

如果您发现自己经常这样做，则可以编写 defaultIfNull、firstNonNull 或 coalesce 函数，这可能会使代码更加简洁。 Apache Commons Lang 包含一个 defaultIfNull 函数。

某些语言包含一个 ||= 运算符，这是这些语言中默认值的常用习惯用法。

Answer 5

我更喜欢第二个，因为它更清楚地表达了您的意思：您只想在颜色不为空时更改颜色。第一种方法并没有把这一点说得那么清楚。

Answer 6

就您而言，我更喜欢“经典”实现，因为对我来说，理解起来更快，如果用户有首选颜色，您只想使用新颜色。

如果我想避免 NPE，有时我会在方法调用中使用它，但我通常会在接下来的重构之一中消除那些丑陋的代码片段；）

Answer 7

三元运算符经常被滥用，因为它们生成的代码看起来聪明而紧凑。

事实上，它们使代码的可读性降低并且更容易出错。
只要有可能，建议使用较长版本的

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

而不是三元语法。

Answer 8

对我来说似乎很好（我经常使用Python的三元运算符），但这种风格问题通常是非常主观的。如果项目有编码风格文档，您可能需要检查它。