java-三元运算符在不满足条件的情况下评估条件语句

我已经写了一些代码；以下是相关片段：

@NonNullByDefault
public class Score<NUMERAL, LITERAL>
{
    protected NUMERAL value;
    @Nullable
    protected LITERAL literal;
    [...]

我已经如下覆盖了equals()方法：

@Override
public boolean equals(@Nullable Object object)
{
    if(object == null) return false;
    if(object == this) return true;

    if( object instanceof Score)
    {
        return ((Score<NUMERAL, LITERAL>) object).getValue().equals(value) &&
                literal == null ? ((Score<NUMERAL, LITERAL>) object).getLiteral() == null : literal.equals(((Score<NUMERAL, LITERAL>) object).getLiteral());
    }
    return false;
}

基本上,想法是分数可能仅具有数字值,在这种情况下文字为null.我已经编写了一些单元测试,并使用以下代码获取了空指针异常：

[....]
Score<Float, String> score = new Score<>(0.0f);
Score<Float, String> anotherScore = new Score<>(1.0f, "One");
[....]

assertFalse(score.equals(anotherScore));

如果我没记错的话,在&&之后,不应简捷地等于从被执行为第一个表达式已经是假的？此外,为什么例外？由于条件为真,因此我希望对三元表达式进行求值,并跳过条件表达式.根据我在规范中阅读的内容,这应该是行为.此外,我发现了一个问题：Java ternary (immediate if) evaluation,应该为我的思考过程提供更多的杠杆作用.

也许我忽略了一些显而易见的事情,但是我没有主意.也许你可以帮忙吗？

解决方法:

它会短路,但是与您想要的方式完全不同. &&优先级高于三元？：-因此(缩进,换行和添加注释以澄清)

((Score<NUMERAL, LITERAL>) object).getValue().equals(value) &&
literal == null
    ? ((Score<NUMERAL, LITERAL>) object).getLiteral() == null
    : literal.equals(((Score<NUMERAL, LITERAL>) object).getLiteral())

实际上意味着：

//the first line as a whole is the condition for ?:
((Score<NUMERAL, LITERAL>) object).getValue().equals(value) && literal == null
    ? ((Score<NUMERAL, LITERAL>) object).getLiteral() == null
    : literal.equals(((Score<NUMERAL, LITERAL>) object).getLiteral())

实际上,这意味着,如果条件的第一部分为false但文字为null,则您将自动输入表达式的：部分,在其中调用literal.equals,从而导致NullPointerException.

修复很简单：添加括号以告诉Java您希望对事物进行哪种评估：

((Score<NUMERAL, LITERAL>) object).getValue().equals(value) && 
(literal == null 
    ? ((Score<NUMERAL, LITERAL>) object).getLiteral() == null
    : literal.equals(((Score<NUMERAL, LITERAL>) object).getLiteral()))

标签：nullpointerexception,ternary-operator,java
来源： https://codeday.me/bug/20191121/2054413.html