當用戶輸入錯誤時循環，輸入正確時遞增

Question

我的作業的一部分是創建一個程序，要求用戶輸入範圍從1到1000的項目價格。如果用戶輸入的價格不在該範圍內，則程序循環回詢問相同料號的價格（料品號範圍從1-10）。如果輸入價格在該範圍內，則程序增加項目編號。並要求用戶輸入該項目的價格。這是我做到的。但是隻有第1項行爲正確。請advise.Thanks

boolean priceinrange = false;  

do { 
     System.out.println("Please enter the price of item " + x + ":"); 
     price = kb.nextDouble(); 
     if (price>1 && price<1000) 
      priceinrange= true; 

    } 
    while (!priceinrange); 

    do { 

     System.out.println("Please enter the price of item:"+ (x+=1) + ":"); 
     price = kb.nextDouble(); 
     if (price>1 && price<1000) 
      priceinrange= true; 

    } 

    while (!priceinrange); 

    do { 
     System.out.println("Please enter the price of item"+ (x+=1) + ":"); 
     price = kb.nextDouble(); 
     if (price>1 && price<1000) 
      priceinrange= true; 
     else 
      System.out.println ("Please enter the price of item" + x +  ":"); 

    } 
    while (!priceinrange); 

    do { 
     System.out.println("Please enter the price of item " + (x+=1) + ":"); 
     price = kb.nextDouble(); 
     if (price>1 && price<1000) 
      priceinrange= true; 
     else 
      System.out.println ("Please enter the price of item" + x + ":"); 
    } 
    while (!priceinrange); 

    do { 
     System.out.println("Please enter the price of item" + (x+=1)+":"); 
     price = kb.nextDouble(); 
     if (price>1 && price<1000) 
      priceinrange= true; 
     else 
      System.out.println ("Please enter the price of item" + x + ":"); 
    } 
    while (!priceinrange); 

    do { 
     System.out.println("Please enter the price of item" + (x+=1)+ ":"); 
     price = kb.nextDouble(); 
     if (price>1 && price<1000) 
      priceinrange= true; 
     else 
      System.out.println ("Please enter the price of item :"); 
    } 
    while (!priceinrange); 

    do { 
     System.out.println("Please enter the price of item" + (x+=1) + ":"); 
     price = kb.nextDouble(); 
     if (price>1 && price<1000) 
      priceinrange= true; 
     else 
      System.out.println ("Please enter the price of item :"); 
    } 
    while (!priceinrange); 

    do { 
     System.out.println("Please enter the price of item" + (x+=1)+ ":"); 
     price = kb.nextDouble(); 
     if (price>1 && price<1000) 
      priceinrange= true; 
     else 
      System.out.println ("Please enter the price of item :"); 
    } 
    while (!priceinrange); 

    do { 
     System.out.println("Please enter the price of item" + (x+=1) +":"); 
     price = kb.nextDouble(); 
     if (price>1 && price<1000) 
      priceinrange= true; 
     else 
      System.out.println ("Please enter the price of item :"); 
    } 
    while (!priceinrange); 

    do { 
     System.out.println("Please enter the price of item" +(x+=1)+ ":"); 
     price = kb.nextDouble(); 
     if (price>1 && price<1000) 
      priceinrange= true; 
     else 
      System.out.println ("Please enter the price of item :"); 
    } 

    while (!priceinrange); 

Answer 1

您需要刪除所有的複製和粘貼代碼。額外的代碼只會讓這個更難。

真正的竅門是當輸入錯誤和正確時循環。您只有在閱讀了10個有效價格後纔會停止循環。你不關心它循環結束了多少次。你關心你有10個價格。

如果你這樣想，那麼只需要一個循環。 A do while (x <= 10)和if (price > 1 && price < 1000) { x += 1;}將爲您帶來一個美好的世界。

你應該能夠用少於25行的代碼解決這個問題。我解決了它，根本不使用priceinrange布爾值。

如果這還不足以讓我知道。

Answer 2

您需要設置priceinrange回到虛假每個do { } while()之前。否則，即使您輸入了無效的價格，它仍然是true（來自「之前」）。

另一種方法是始終將其設置爲：priceinrange = (price > 1 && price < 1000)而不是僅當滿足條件時將priceinrange設置爲true。