無限循環，不會以while循環結束

Question

好吧，所以我寫了這個程序，它將計算某些字母和空格，我希望它執行的操作是讓用戶不斷輸入短語，並繼續循環直到用戶輸入quit終止。我無法看到放置while循環的位置。我知道我應該在while循環下嵌套所有的循環，當我這樣做時，程序進入無限循環。

import java.util.Scanner; 

public class Count 
{ 
    public static void main (String[] args) 
    { 
     String phrase; // a string of characters 
     int countBlank; // the number of blanks (spaces) in the phrase 
     int length;  // the length of the phrase 
     char ch;   // an individual character in the string 
     int countA=0,countE=0,countS=0,countT=0; 


    Scanner scan = new Scanner(System.in); 
     // Print a program header 
     System.out.println(); 
     System.out.println ("Character Counter"); 
     System.out.println(); 

     // Read in a string and find its length 
     System.out.print ("Enter a sentence or phrase or enter (Quit) to quit: "); 
     phrase = scan.nextLine(); 
while(!phrase.equalsIgnoreCase ("Quit")) 
{ 
     length = phrase.length(); 

      // Initialize counts 
      countBlank = 0; 

     // a for loop to go through the string character by character 

      for (int i = 0; i < phrase.length(); i++) 
     { 
      if(phrase.charAt(i) == ' ') countBlank++; 

      switch(ch=phrase.charAt(i)) 
      { 
      case 'a': 
      case 'A': countA++; 
      break; 

      case 'e': 
      case 'E': countE++; 
      break; 

      case 's': 
      case 'S': countS++; 
      break; 

      case 't': 
      case 'T': countT++; 
      break; 

     } 
    } 


     // Print the results 
     System.out.println(); 
     System.out.println ("Number of blank spaces: " + countBlank); 
     System.out.println ("Number of a: " + countA); 
     System.out.println ("Number of e: " + countE); 
     System.out.println ("Number of s: " + countS); 
     System.out.println ("Number of t: " + countT); 
     System.out.println(); 

    } 
} 
} 

Answer 1

在while循環中，你永遠不會讀下一行。您需要添加

phrase = scan.nextLine(); 

'for'循環後，但仍在'while'循環中。否則，短語將始終保持您第一次讀入的內容。

Answer 2

您應該在while循環結束時掃描一行新行。現在你的循環的構造方式，你不斷迭代相同的輸入。

phrase = scan.nextLine(); 
while(!phrase.equalsIgnoreCase ("Quit")){ 
    // do stuff 
    // ... 
    phrase = scan.nextLine(); 
} 

Answer 3

// Read in a string and find its length 
System.out.print ("Enter a sentence or phrase or enter (Quit) to quit: "); 

while (true) { 
    phrase = scan.nextLine(); 
    if (phrase.equalsIgnoreCase("Quit")) break; 

    // Initialize your counter variables here 

    length = phrase.length(); 
    // and so on... 
} 

Answer 4

複製實在是太差了，並試圖避免duplicaiton可能是你所遇到（不想把scan.nextLine兩次，有很好的本能）的問題。我認爲Philippe的答案是正確的。

讓我作弊，並重新寫菲利普的回答在一對夫婦不同的方式

do { 
    phrase = scan.nextLine(); 
    // do stuff 
    // ... 

} while(!phrase.equalsIgnoreCase ("Quit")); 

這將消除重複，但會導致「東西」被「做」，即使這句話是「退出」，那也不好 - 而且增加休息會毀了它。

while(true) { 
    phrase = scan.nextLine(); 
    if(!phrase.equalsIgnoreCase ("Quit")) 
     break; 
    // do stuff 
} 

這完美的作品，但同時（真）使得一些人真的很不舒服 - 這是一個宗教，有些人在年輕的時候學習，不能克服，所以你可能不希望推動這一 - 他們如何確信，這比其他解決方案更可能導致「無限循環」，因爲它們在功能上是正確的，所以完成垃圾，但它也隱藏了可能令人討厭的循環退出標準。

另外一個是有效的，但讓某些人不舒服：

while((phrase = scan.nextLine()).equalsIgnoreCase ("Quit")) { 
    // do stuff 
} 

其實很少有人在Java中使用此 - 我從來沒有使用它，我不能完全確定，因爲我不想它是有效的」不要用它，但我認爲=仍然返回要操作的值。無論如何，如果我不得不測試它是非常罕見的，那麼讓其他人花費更多的時間來看待它，這是非常罕見的，這是不好的。

那麼最好的解決方案是什麼？也許把它分解成方法調用：

private string phrase; 

boolean readPhrase() { 
    phrase=scan.nextLine(); 
    return !phrase.equalsIgnoreCase("Quit"); 
} 

while(readPhrase()) { 
    // do stuff 
} 

我知道這好像更多的工作，但如果你打破東西出來成可用塊這樣的習慣，讓你會從長遠來看，更快樂。您現在擁有更多的模塊化，可理解的代碼 - 並且沒有任何成本（只要它們是簡單，簡短，易於理解的方法，而且您不重複邏輯，從不認爲更多的方法是成本）。老實說，如果你打算繼續使用它來做任何真實的事情，我甚至可能會創建一個包含字符串變量和.countCharacter（「」）的.read（）方法的「phrase」類。方法。這將使你的代碼看起來像這樣：

while(phrase.read()) { 
    System.out.println("Number of a="+phrase.countCharactr("a")); // assume that countCharacter does a toLower() 
    System.out.println("Number of a="+phrase.countCharactr("e")); 
    ... 
} 

這突然是非常可讀和緊湊。

另請注意，當你像這樣減少代碼時，其他模式變得容易識別。在這種情況下，你甚至可能會發現，你可以分解出重複線的上方，因爲他們只有在每種情況下的單個字符改變：

char[] chars="aest".toCharArray(); 

while(phrase.read()) 
    for(char c : chars) 
     System.out.println("Number of "+c+"="+phrase.countCharactr(c)); // assume that countCharacter does a toLower() 
// this is untested and some of the conversions/methods/etc may need tweaking.  

你的整個「主」減少到幾行簡單的輔助類。是不是更好？

對不起，過分分析。沒有工作，開始錯過編碼。