null != null - 关于如何重新安排代码或防止这种情况的任何想法？

Question

目前正在调试，发现一个 if 语句，没有（我认为......）原因给了我一个 NPE，显然是有原因的。该语句似乎是 if(false && (null != null || null != Color))。

if(destination != null && (destination.getPiece() != null || destination.getPiece().getColour() !=pieceColour)) - if 语句

两个目标 < em>可以为空，而碎片可以为。 getColour() 方法从piece 返回一个Color 类型的属性，如果该piece 为null，则该属性必须为null。目标处的片段具有与 if 语句中的片段颜色不同的属性。

具体来说，如何重新排列 (destination.getPiece() != null) ？

Answer 1

该声明第二部分的逻辑非常混乱。

如果 getPiece() 为 null，则 getPiece().getColour() 无效，反之亦然。

换句话说，destination.getPiece() == null => destination.getPiece().getColour() 是一个 NPE。
同样，destination.getPiece().getColour() 是任何东西 => destination.getPiece() != null

对我来说唯一有意义的是：

if( destination != null && (destination.getPiece() == null || destination.getPiece().getColour() != pieceColour))

也就是说，目的地不为空，并且该块要么为空，要么为不同的颜色。

Answer 2

其他答案也有效，但为了清楚起见，我建议将 destination.getPiece() 的结果存储在局部变量中：

if (destination != null)
{
    Piece piece = destination.getPiece();

    if (piece == null || piece.getColour() != pieceColour))
    {
        // Do stuff.
    }
}

Answer 3

Object obj = destination.getPiece();

如果 obj 为 null 则 (obj != null) 返回 false，但在检查 obj.getColor() 时将抛出 NullPointerException。这就是为什么，它应该是这样的：

if(destination != null && (destination.getPiece() != null && destination.getPiece().getColour() != pieceColour))

Answer 4

最简单的答案是将其分成两个 if 语句：

if(destination != NULL)
{
  if(destination.getPiece() != null || destination.getPiece().getColour() != pieceColour)
  {
    //Do stuff
  }
}

Answer 5

destination.getPiece() != null || destination.getPiece().getColour() != pieceColour

应该是：

destination.getPiece() != null && destination.getPiece().getColour() != pieceColour