String.isEmpty（）からの永続的なjava.lang.NullPointerExceptionエラー

Question

私はバグを検索しようとしていましたが、見つけられませんでした。間違ったことを解決しようと既に1時間を費やしました。エラーは、コードがisPlayerSetメソッドwhile (!player.isPlayerSet()) {に入ると開始されます。私は既に使用されているプロパティを ""設定しましたが、このnullpointerexeptionエラーが発生しています。私はプログラミング、特にJavaでかなり新しいことを理解してください。

ここでここでの主なクラスが

public class Main { 
    public static void main(String[] args) { 
     Scanner scanner = new Scanner(System.in); 

     String playerName = ""; 
     int chosenPokemon = 0; 
     boolean isANumber = false;; 
     Player player; 

     /* 
     * Initialize Players 
     */ 
     Player[] players = new Player[2]; 

     for (int counter = 0; counter < players.length; counter++) { 
      player = new Player(); 
     } 

     /* 
     * Get details of trainers 
     */ 
     for (int counter = 0; counter <= players.length-1; counter++) { 
      player = players[counter]; 
      while (!player.isPlayerSet()) { 
       /* 
       * Input player name 
       */ 
       if(player.getPlayerName() == "") { 
        System.out.println("Enter a valid name for Player " + (counter+1) + ":"); 
        player.setPlayerName(playerName); 
       } 
       /* 
       * Choose Pokemon 
       */ 
       if(player.getChosenPokemon() == ""){ 
        System.out.println("Choose a starting pokemon for Player " + (counter+1) + ":"); 
        System.out.println("[1] Charmander"); 
        System.out.println("[2] Bulbasaur"); 
        System.out.println("[3] Squirtle"); 

        do { 
         if(!scanner.hasNextInt()) 
         { 
          System.out.println("Input must be a valid integer. Try Again."); 
          scanner.next(); 
         } 
         else if(!(chosenPokemon >= 1) && !(chosenPokemon <= 3)) 
         { 
          System.out.println("Input must be a number from 1-3. Try Again."); 
          scanner.next(); 
         } 
         else { 
          chosenPokemon = scanner.nextInt(); 
          isANumber = true; 
         } 
        } while(!isANumber); 
        player.setChosenPokemon(chosenPokemon); 
       } 
      } // End of while loop 
     } // End of for loop 
    } 
} 

だとプレイヤークラスは

public class Player { 
    Scanner scanner = new Scanner(System.in); 

    private String playerName = ""; 
    private String chosenPokemon = ""; 

    public String getPlayerName() { 
     return this.playerName; 
    } 

    public void setPlayerName(String playerName) { 
     do { 
      playerName = scanner.nextLine(); 
      if(!isAlpha(playerName)) { 
       System.out.println("Invalid input. Try again"); 
      } 

      if(playerName.isEmpty()) { 
       System.out.println("Player name cannot be blank! Try again"); 
      } 
     } while(!isAlpha(playerName) || playerName.isEmpty()); 

     this.playerName = playerName; 
     System.out.println("Welcome " + this.playerName); 
    } 

    public String getChosenPokemon() { 
     return chosenPokemon; 
    } 

    public void setChosenPokemon(int chosenPokemon) { 
     if(chosenPokemon == 1) { 
      this.chosenPokemon = "Charmander"; 
     } else if(chosenPokemon == 2) { 
      this.chosenPokemon = "Bulbasaur"; 
     } else { 
      this.chosenPokemon = "Squirtle"; 
     } 
    } 

    public boolean isPlayerSet() { 
     if (this.playerName.isEmpty() && this.chosenPokemon.isEmpty()) { 
      return false; 
     } 

     return true; 
    } 

    public static boolean isAlpha(String name) { 
     char[] chars = name.toCharArray(); 

     for (char c : chars) { 
      if (!Character.isLetter(c)) { 
       return false; 
      } 
     } 

     return true; 
    } 
} 

だ私はまた別の質問があり、それがPlayer playerでplayers[counter]を交換することをお勧めしますか？

Answer 1

あなたは新しいプレーヤーを作成しているが、ここでオブジェクト：

for (int counter = 0; counter < players.length; counter++) { 
    player = new Player(); 
} 

しかし：あなたははあなたが上記で定義された配列にそれらのプレーヤーを格納していませんが。したがって、配列要素は初期値のままです。つまり、配列内のすべてのプレイヤーがnullであることを意味します。

だからあなたのループが

players[counter] = new Player(); 

を言うべきそしてもちろん、あなたが本当にこのhereを読むことをお勧めします。

Answer 2

このループ内で同じ変数をクローバーしています。

for (int counter = 0; counter < players.length; counter++) { 
    player = new Player(); 
} 

オプション1：

for (int counter = 0; counter < players.length; counter++) { 
     players[counter] = new Player(); 
    } 

オプション2（わずかにより簡潔且つエレガント）：ループで

for (Player p: players) p = new Player(); 

Answer 3

for (int counter = 0; counter < players.length; counter++) { 
    player = new Player(); 
} 

ローカル変数playerを初期化します、したがってwhile (!player.isPlayerSet())playerは、nullです。あなたはこの

for (int counter = 0; counter < players.length; counter++) { 
    player = new Player(); 
} 

は、このする必要がありますplayers配列

for (int counter = 0; counter < players.length; counter++) { 
    players[counter] = new Player(); 
} 

Answer 4

でインスタンスを初期化する必要があります。

for (int counter = 0; counter < players.length; counter++) { 
    players[counter] = new Player(); 
}