私のプログラムで3つのwhileループのうち2つがスキップされているのはなぜですか？

Question

なぜ私の最初のwhileループはユーザーからの入力の有効性をチェックしますが、他の2つのwhileループは月と年の間にスキップされます（無視されます）。ユーザーに日付を正しく入力させ、不可能な値を入力させないようにしようとしています。あなたの最初のループ条件以来

// A read() method to read in the account details from the user 
boolean success = false; 
public void read() 
{ 
    Scanner keyboardIn = new Scanner(System.in);   
    System.out.println("ENTER ACCOUNT DETAILS: "); 
    System.out.print("User Title: "); 
    String title = keyboardIn.nextLine(); 
    name.setTitle(title); 
    System.out.print("User First name: "); 
    String firstname = keyboardIn.nextLine(); 
    name.setFirstName(firstname); 
    System.out.print("User Second name: "); 
    String secondname = keyboardIn.nextLine(); 
    name.setSurname(secondname); 
    System.out.print("Account Address: "); 
    address = keyboardIn.nextLine(); 
    // To make sure day is entered correctly (1 - 31) 
    while(!success) 
    { 
     try 
     { 
      System.out.print("Enter the day the account opened: "); 
      int d = keyboardIn.nextInt(); 
      dateOpened.setDay(d); 
      success = true; 
     }catch(Exception e) 
     { 
      System.out.println(e.getMessage()); 
     } 
    } 

    // To make sure month is entered correctly (1 - 12) 
    while(!success) 
    { 
     try 
     { 
      System.out.print("Enter the month the account opened: "); 
      int m = keyboardIn.nextInt(); 
      dateOpened.setMonth(m); 
      success = true; 
     }catch(Exception e) 
     { 
      System.out.println(e.getMessage()); 
     } 
    } 

    // To make sure year is entered correctly (< 1900 not permitted) 
    while(!success) 
    { 
     try 
     { 
      System.out.print("Enter the year the account opened: "); 
      int y = keyboardIn.nextInt(); 
      dateOpened.setYear(y); 
      success = true; 
     }catch(Exception e) 
     { 
      System.out.println(e.getMessage()); 
     } 
    } 

    System.out.print("Enter the initial balance: "); 
    balance = keyboardIn.nextDouble(); 
    System.out.print("Enter the overdraft amount: "); 
    overdraftAmount = keyboardIn.nextDouble(); 
    System.out.println("Account number: " + accountNo); 
    System.out.println(); 
} 

Answer 1

ループが唯一の成功がtrueの場合、成功はループしながら、他の二つのためにtrueになり、彼らが実行されることはありません終了している間。

"成功"をループ条件として使用することはお勧めしません。あなたのような何かを行うことができます

System.out.print("Enter the day the account opened: "); 
int d = keyboardIn.nextInt(); 

while(d < 1 || d>31){ 

System.out.print("Enter the day the account opened: "); 
d = keyboardIn.nextInt(); 

} 

//set d as day in object 

Answer 2

それは可能性が2つの理由のために、最初のものは、ユーザが入力したデータが、その可能性、InputMismatchException例外またはdateOpened.setDay(d)が失敗したとのラインでのスロー整数、ではないということですdateOpenedがヌルあるかどうなる方法setDayが失敗した場合にどのしません真から成功変数セット。

Answer 3

最初のwhileループの後、ループ条件 "success"をリセットしないでください。これは単に一方小さい方法でそれを実現し、その方法にローカル変数の条件があまりにもそれを解決する製造上の各ループ

後にそれをリセットすることで解決する

一つの方法。

public void read() 
{ 
    Scanner keyboardIn = new Scanner(System.in);   
    System.out.println("ENTER ACCOUNT DETAILS: "); 
    System.out.print("User Title: "); 
    String title = keyboardIn.nextLine(); 
    name.setTitle(title); 
    System.out.print("User First name: "); 
    String firstname = keyboardIn.nextLine(); 
    name.setFirstName(firstname); 
    System.out.print("User Second name: "); 
    String secondname = keyboardIn.nextLine(); 
    name.setSurname(secondname); 
    System.out.print("Account Address: "); 
    address = keyboardIn.nextLine(); 
    // To make sure day is entered correctly (1 - 31) 
    readDay(keyboardIn); 

    // To make sure month is entered correctly (1 - 12) 
    readMonth(keyboardIn); 

    // To make sure year is entered correctly (< 1900 not permitted) 
    readYear(keyboardIn); 

    System.out.print("Enter the initial balance: "); 
    balance = keyboardIn.nextDouble(); 
    System.out.print("Enter the overdraft amount: "); 
    overdraftAmount = keyboardIn.nextDouble(); 
    System.out.println("Account number: " + accountNo); 
    System.out.println(); 

} 

private void readDay(Scanner keyboardIn){ 
    boolean success = false; 
    while(!success) 
    { 
     try 
     { 
      System.out.print("Enter the day the account opened: "); 
      int d = keyboardIn.nextInt(); 
      dateOpened.setDay(d); 
      success = true; 
     }catch(Exception e) 
     { 
      System.out.println(e.getMessage()); 
      keyboardIn.next(); 
     } 
    } 
} 
private void readMonth(Scanner keyboardIn){ 
    boolean success = false; 
    while(!success) 
    { 
     try 
     { 
      System.out.print("Enter the month the account opened: "); 
      int m = keyboardIn.nextInt(); 
      dateOpened.setMonth(m); 
      success = true; 
     }catch(Exception e) 
     { 
      System.out.println(e.getMessage()); 
      keyboardIn.next(); 
     } 
    } 
} 
private void readMonth(Scanner keyboardIn){ 
    boolean success = false; 
    while(!success) 
    { 
     try 
     { 
      System.out.print("Enter the year the account opened: "); 
      int y = keyboardIn.nextInt(); 
      dateOpened.setYear(y); 
      success = true; 
     }catch(Exception e) 
     { 
      System.out.println(e.getMessage()); 
      keyboardIn.next(); 
     } 
    } 
}