我已经写了一些代码;以下是相关片段:
@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()))