PMDの警告：UseCollectionIsEmptyを修正するにはどうすればよいですか？

Question

私はPMD警告 - UseCollectionIsEmptyを取得しています。正確なメッセージは以下の通りです：

"size（）== 0（またはsize（）！= 0、size（）> 0、size ）< 1）最後のelse ifステートメントでisEmpty（）を呼び出します。

私の問題は、私のステートメントがArrayListが空でないかどうかをチェックしていることです。しかし、PMDの警告はisEmpty（）に変更するように指示していますが、これは私が行う必要のないものです。これを修正するための回避策は何ですか？代わりに、あなたが使用する必要があります

} else if (myPossDir.size() > 0) { 
     myDirection = myPossDir.get(random.nextInt(myPossDir.size())); 
    } else { 
     myDirection = getDirection().reverse(); 
    } 

の

 if (theNeighbors.get(getDirection()) == Terrain.CROSSWALK) { 
      myDirection = getDirection(); 
     } else if (theNeighbors.get(getDirection().left()) == Terrain.CROSSWALK) { 
      myDirection = getDirection().left(); 
     } else if (theNeighbors.get(getDirection().right()) == Terrain.CROSSWALK) { 
      myDirection = getDirection().right(); 
     } else if (myPossDir.size() > 0) { 
      myDirection = myPossDir.get(random.nextInt(myPossDir.size())); 
     } else { 
      myDirection = getDirection().reverse(); 
     } 

Answer 1

：

} else if (myPossDir.isEmpty()) { 
     myDirection = getDirection().reverse(); 
    } else { 
     myDirection = myPossDir.get(random.nextInt(myPossDir.size())); 
    } 

これは、上記のPMDの警告が離れて行くようになります、そしてそれがより明確に自分の意図を述べて、あなたがいることを確認したいですコレクションは空ではありません。 size()を使用するのは、それをゼロと比較しない場合です。

EDIT：IFSの変更オーダーなので、今は代わりに!isEmpty()の（別のPMDの警告を、そしてif Sを否定として読みやすい否定ではないものとではありません）isEmpty()を使用