forループ後のコードが実行されない（Java）

Question

入力されたコードネームとパスワードをテキストファイルに保存されているものと照合するコードスニペットがあります。一致するものがあればゲームを開始し、すべてが正常です。しかし、一致するものがなければ、再びログインしようとするかどうかを尋ねるダイアログがポップアップされるはずです。

int input=0; //yes 

do { 
    codename=JOptionPane.showInputDialog(null,"Enter Codename: "); 
    String password=JOptionPane.showInputDialog(null, "Enter Password: "); 

    for(int i=0;i<users.length;i++){ 
     if((codename.equals(users[i].getCodeName())) && (password.equals(users[i].getPassword()))){ 
      System.out.println("\n\nCorrect"); 
      new Game(); 
     } else { 
      System.out.println("\n\nIncorrect"); 
     } 
    } 
    input = JOptionPane.showConfirmDialog(null, "Incorrect User name/Password\nWould you like to try again?");   
} while(input==0); //run while input is yes 

問題：ループが実行されないために後のコード。私はユーザーに対して[私は]のための変数をチェックすると、forループの後のコードは実行されませんが、私は[]などのユーザーに対してチェックし、それはうまく動作します。

この問題であればわからないが、私はいつもこのエラーが出る：

Exception in thread "main" java.lang.NullPointerException 
    at com.ramtin.Game.logOn(Game.java:505) 
    at com.ramtin.Game.main(Game.java:397) 

は私がパスワードとコードネームが一致し、プログラムを完全に実行しても、それを得ます。

上記のコードのための

FULL CODE：UserDataのため

public static void logOn(){ 
     //ASK FOR CODENAME & PASSWORD FROM TEXTFILE BEFORE GAME BEGINS 
     //read from text file 

     UserData[]users=new UserData[20]; 
     int countU=0; 

     try{ 
      BufferedReader readU = new BufferedReader(new FileReader("userdata.txt")); 

      String line; 

      while((line=readU.readLine())!=null){ 
       String []parts=line.split("#"); 

       String codeName = parts[0]; 
       String password=parts[1]; 

       //    System.out.println(parts[0]); 
       //    System.out.println(parts[1]); 
       users[countU]=new UserData(codeName, password); 
       countU++; 
      } 
      readU.close(); 
     } 
     catch(FileNotFoundException e){ 

     } 
     catch(IOException e){ 

     } 

     //PASSWORD & CODENAME 

     int input=0; //yes 

     do{ 
      codename=JOptionPane.showInputDialog(null,"Enter Codename: "); 
      String password=JOptionPane.showInputDialog(null, "Enter Password: "); 

      for(int i=0;i<users.length;i++){ 
       if((codename.equals(users[i].getCodeName()))&&(password.equals(users[i].getPassword()))){ 
        System.out.println("\n\nCorrect"); 
        new Game(); 
       } 
       else{ 
        System.out.println("\n\nIncorrect"); 
       } 
      } 

      input = JOptionPane.showConfirmDialog(null, "Incorrect Username/Password\nWould you like to try again?");   
     } 
     while(input==0); //run while input is yes 
    } 
} 

FULL CODE：

public class UserData { 

    private String codename; 
    private String password; 

    UserData (String codeName, String password) 
    { 
     this.codename = codeName; 
     this.password= password; 
    } 


    String getCodeName() 
    { 
     return codename; 
    } 


    String getPassword() 
    { 
     return password; 
    } 


    public String toString() 
    { 
     String temp = "\nCode name: "+codename+"\nPassword: " + password; 
     return temp; 
    } 
    } 

Answer 1

for(int i=0;i<users.length;i++){

をしないでくださいは、後for(int i=0;i<countU;i++){

すべての配列要素をしますかcountUを呼び出すとNPEが発生します。代わりに、長さが20で配列を使用してのusers[i].getCodeName()

Answer 2

などのヌル要素の方法、それは動的に拡大するとして、あなたはArrayListのを使用することができます。

List<UserData> users = new ArrayList<>(); 

を次にリスト

users.add(new UserData(codeName, password)); 

に、各ユーザデータを追加

と

for(int i=0 ;i<users.size(); i++) { 

を繰り返す。これは防ぐことができますNullPointerは、ユーザー数と同じ数のエントリしか持たないため（ユーザーの数に応じて動的に拡大/縮小されます）