無限循環在Java中

Question

嘿！我正在嘗試做一些數據輸入驗證，但我一直無法弄清楚。當我嘗試驗證輸入的第一個字符是否是字母時，我得到一個無限循環。 。 。 。

感謝您的幫助！

public class methods 
{ 
    public static void main(String args[]) throws IOException 
    { 
     String input =""; 
     int qoh=0; 
     boolean error=true; 

     Scanner keyboard = new Scanner (System.in); 

     //while (error) 
     //{ 
      //error=true; 

     while (error==true) 
     { 
      System.out.print("\nEnter Quantity on Hand: "); 
      input = keyboard.nextLine(); 

      if (input.length() <1) 
      { 
       System.out.println("\n**ERROR06** - Quantity on hand must be between 0 and 500"); 
       error=true; 
       System.out.println(qoh); 
       System.out.println(input); 
      } 
      else 
      { 
       error=false; 
      } 
     } 

     error = true; 

     while (error==true) 
     { 
      if (Character.isLetter(input.charAt(0))) 
      { 
       System.out.println("\n**ERROR06** - Quantity on hand must be between 0 and 500"); 
       error=true; 
       System.out.println(qoh); 
       System.out.println(input); 
      } 
      else 
      { 
       qoh = Integer.parseInt(input); 
       error=false; 
       } 
      } 
     } 
    } 

Answer 1

您的第二個while循環中沒有input = keyboard.nextLine();。

您可以重構您的代碼，以便在發生錯誤時僅詢問新輸入。所以，在'錯誤...'的系統出現之後，我會實際做到這個不同。開頭的'error = true'有點令人困惑，因爲可能沒有錯誤。

例如，您可以編寫一個名爲tryProcessLine方法，它讀取輸入，如果有錯誤，則返回true，如果確定與假，並不僅僅是做類似下面while(!tryProcessLine()){ }

工作例如：

import java.io.IOException; 
import java.util.Scanner; 

public class Methods { 

    private static int qoh; 

    public static void main(String args[]) throws IOException { 

    while (!tryProcessLine()) { 
     System.out.println("error... Trying again"); 
    } 

    System.out.println("succeeded! Result: " + qoh); 

    } 

    public static boolean tryProcessLine() { 

    String input = ""; 

    Scanner keyboard = new Scanner(System.in); 

    System.out.print("\nEnter Quantity on Hand: "); 

    input = keyboard.nextLine(); 

    try { 
     qoh = Integer.valueOf(input); 

     if (qoh < 0 || qoh > 500) { 
      System.out.println("\n**ERROR06** - Quantity on hand must be between 0 and 500"); 
      return false; 
     } else { 
      return true; 
     } 
    } catch (NumberFormatException e) { 
     System.out.println("\n**ERROR06** - Quantity on hand must be numeric"); 
     return false; 
    } 
    } 
} 

Answer 2

發生死循環是因爲第二個while循環是反覆檢查字符串中的第一個字符（input.charAt(0)）是否是字母。假設這個檢查的結果是真的，循環將永遠不會終止。

您的代碼可以簡化爲這樣的：

Integer qty = null; 

while (scanner.hasNext() && qty == null) { 
    String line = scanner.next(); 
    try { 
    qty = Integer.parseInt(line); 
    } catch(NumberFormatException ex) { 
    System.err.println("Warning: Ignored non-integer value: " + line); 
    } 
} 

if (qty == null) { 
    System.err.println("Warning: No quantity specified."); 
} 

Answer 3

如果它是一個字符，你允許誤差仍然=真，這是造成該循環永遠繼續下去，你永遠不回到開始並閱讀另一行。

下面是一些代碼，它可以實現你想要的，並且結構更好一些。

public class ScanInfo { 

    Scanner keyboard = new Scanner(System.in); 

    public ScanInfo(){ 
    String line = getLineFromConsole(); 
    while(null != line && !"quit".equals(line)){ 
     if(isValidInput(line)){ 
     int validNumber = Integer.parseInt(line); 
     System.out.println("I recieved valid input: "+validNumber); 
     }else{ 
     System.out.println("\n**ERROR06** - Quantity on hand must be between 0 and 500"); 
     } 
     line = getLineFromConsole(); 
    } 

    } 

    private boolean isValidInput(String line){ 
    //basic sanity 
    if(null == line || line.length() < 1){ 
     return false; 
    } 


    try { 
     int number = Integer.parseInt(line); 

     return (number >= 0 && number <= 500); 

    } catch (NumberFormatException e) { 
     return false; 
    } 

    } 


    public static void main(String[] args) { 
    new ScanInfo(); 

    } 

    public String getLineFromConsole(){ 
    System.out.print("\nEnter Quantity on Hand: "); 
    return keyboard.nextLine(); 

    } 

} 

Answer 4

問題是，在本節：

     while (error==true) 
         { 
          if (Character.isLetter(input.charAt(0))) 
          { 
           System.out.println("\n**ERROR06** - Quantity on hand must be between 0 and 500"); 
           error=true; 
           System.out.println(qoh); 
           System.out.println(input); 
          } 
          else 
          { 
           qoh = Integer.parseInt(input); 
           error=false; 
          } 
         } 

一旦你在第一位置的字母，這個循環可以永遠終止。它檢查一封信是否在第一個位置（它是），打印它並重復。嘗試更改爲：

      while (error==true) 
          { 
           if (Character.isLetter(input.charAt(0))) 
           { 
            System.out.println("\n**ERROR06** - Quantity on hand must be between 0 and 500"); 
            error=false; 

            ... 

此外，一對夫婦的其他東西：

while (error == true)可縮短至while(error)。

另外，Integer.parseInt將拋出NumberFormatException如果輸入不是一個整數 - 你需要捕捉並處理這個。

另外，爲什麼你需要第二個循環呢？它似乎只是爲了驗證輸入 - 如果是的話，你可以將這個邏輯移入第一個循環，並消除第二個循環。只對需要重複發生的事情使用循環（如用戶輸入輸入數據）。不需要重複檢查相同的輸入。