NullPointerExceptionを回避する方法は？

Question

私は、eclipseとgridworldを使って、ゲームのオセロで動きが合法であるかどうかをチェックしようとしています。私がその場所に行う最初のことは、それが有効であるかどうかをチェックすることですが、有効性の場所を確認するためにはnullにする必要はありません。問題は、それが法的な動きであるという要件の1つは、それがヌル/空/非占有であるということです。これを避けるにはどうすればいいですか？私はエラーがどこにあるのかを指摘しました。 （これは誰を混同している場合申し訳ありません。）

public boolean isLegal(Location loc1) 
{ 
    boolean isLegal = false; 
    String currentColor = currentPlayer.getColor(); 
    int row = loc1.getRow(); 
    int col = loc1.getCol(); 
    if(board.isValid(loc1)) 
    { 
     if(board.get(loc1) == null) 
     { 
      for(Location tempLoc : board.getValidAdjacentLocations(loc1)) 
      { 
       **if(!board.get(tempLoc).equals(currentColor))** 
       { 
        if((row != tempLoc.getRow()) && (col == tempLoc.getCol())) 
        { 
         //count up column 
         if(tempLoc.getRow() < row) 
         { 
          for(int i = row; i > 1;) 
          { 
           Location tempLoc2 = new Location(i-2, col); 
           if(!board.get(tempLoc2).equals(currentColor)) 
           { 
            i--; 
           } 
           else 
           { 
            i=-1; 
            isLegal = true; 
           } 
          } 
         } 
         //count down column 
         else 
         { 
          for(int i = row; i < 6;) 
          { 
           Location tempLoc2 = new Location(i+2, col); 
           if(!board.get(tempLoc2).equals(currentColor)) 
           { 
            i++; 
           } 
           else 
           { 
            i=9; 
            isLegal = true; 
           } 
          } 
         } 
        } 
        else if(col != tempLoc.getCol() && row == tempLoc.getRow()) 
        { 
         //count right row 
         if(col > tempLoc.getCol()) 
         { 
          for(int i = col; i > 1;) 
          { 
           Location tempLoc2 = new Location(row, i-2); 
           if(!board.get(tempLoc2).equals(currentColor)) 
           { 
            i--; 
           } 
           else 
           { 
            i=-1; 
            isLegal = true; 
           } 
          } 
         } 
         //count left row 
         else 
         { 
          for(int i = col; i < 6;) 
          { 
           Location tempLoc2 = new Location(row, i+2); 
           if(!board.get(tempLoc2).equals(currentColor)) 
           { 
            i++; 
           } 
           else 
           { 
            i=9; 
            isLegal = true; 
           } 
          } 
         } 
        } 
        else 
        { //count up/right diag 
         if(row-1 == tempLoc.getRow() && col+1 == tempLoc.getCol()) 
         { 
          int j = col; 
          for(int i = row; i > 1;) 
          { 
           Location tempLoc2 = new Location(i-1, j+1); 
           if(!board.get(tempLoc2).equals(currentColor)) 
           { 
            i--; 
            j++; 
           } 
           else 
           { 
            i=-1; 
            isLegal = true; 
           } 
          } 
         } 
         //count down/left diag 
         else if(row+1 == tempLoc.getRow() && col-1 == tempLoc.getCol()) 
         { 
          int i = row; 
          for(int j = col; j > 1;) 
          { 
           Location tempLoc2 = new Location(i+1, j-1); 
           if(!board.get(tempLoc2).equals(currentColor)) 
           { 
            i++; 
            j--; 
           } 
           else 
           { 
            i=9; 
            isLegal = true; 
           } 
          } 
         } 
         //count up/left diag 
         else if(row-1 == tempLoc.getRow() && col-1 == tempLoc.getCol()) 
         { 
          int j = col; 
          for(int i = row; i > 1;) 
          { 
           Location tempLoc2 = new Location(i-1, j-1); 
           if(!board.get(tempLoc2).equals(currentColor)) 
           { 
            i--; 
            j--; 
           } 
           else 
           { 
            i=-1; 
            isLegal = true; 
           } 
          } 
         } 
         //count down/right diag 
         else 
         { 
          int j = col; 
          for(int i = row; i > 6;) 
          { 
           Location tempLoc2 = new Location(i+1, j+1); 
           if(!board.get(tempLoc2).equals(currentColor)) 
           { 
            i++; 
            j++; 
           } 
           else 
           { 
            i=-1; 
            isLegal = true; 
           } 
          } 
         } 
        } 
       } 
      } 
     } 
    } 
    return isLegal; 
} 

Answer 1

一つの解決策は何の場所が今までnullしないように、あなたのデザインを変更することです。

nullは「空」または「空」とみなされているようです。代わりにすべてのポジションを最初に作成してください（オセロボードには多くのポジションはありません）。boolean occupied = falseまたは同等のメンバー変数ですべてのポジションを初期化してください。それから、あなたはヌルチェックではなく

if (!board.get(loc1).isOccupied()) { /*stuff*/ } 

の代わりになります。

これは空の場所がまだ場所であり、操作可能であるため、オブジェクト指向のデザインが優れています。

Answer 2

ロジックの一部としてnullを使用しないでください。
nullこれは状態ではありません、の状態であることを示す記号です。
nullをロジックの外に残す必要があります。参照番号がnullの場合は、モデルに関係なく実際に何かが起こっているかどうかを確認する必要があります。 Locationの中には、たとえばisEmpty()などのメソッドを作成することができますので、nullと簡単に比較することはできません。

Answer 3

使用する値の代わりにStringのBLACK、WHITEとVACANTとenumは、各位置にある何色トークン格納します。 PlayerクラスのgetColor()から同じenumを返します。