C++ Unwanted infinite while loop

Question

Je reçois une boucle infinie lorsque j'utilise le code suivant en C++ et je ne comprends pas pourquoi. Je suspecte que le problème est dans la fonction input_words(). Voici le code:

#include<iostream> 
using namespace std; 

string input_words(int maxWords) { 
    int nWord = 0; 
    string words[maxWords]; 
    string aWord = ""; 
    while (aWord != "Quit" && nWord < maxWords) { 
     cout << "Enter a number ('Quit' to stop): "; 
     getline (cin, aWord); 
     words[nWord] = aWord; 
     nWord++; 
    } 
    return *words; 
} 

int num_words (string words[], int maxWords) { 
    int numWords = 0; 
    for (int i=0; i<maxWords; i++) { 
     if (words[i] == "Quit") { 
      break; 
     } 
     numWords++; 
    } 
    return numWords; 
} 

int main() { 

    const int MAX_WORDS = 100; 
    string words[MAX_WORDS] = input_words(MAX_WORDS); 

    int lenWords = num_words(words, MAX_WORDS); 
    cout << "\nThere are " << lenWords << " words:\n"; 

    for (int i=0; i<MAX_WORDS; i++) { 
     if (words[i] == "Quit") { 
      break; 
     } 
     cout << words[i] << "\n"; 
    } 
    return 0; 
} 

Plus précisément, je ne peux pas sortir, même quand je tape « Quitter » lorsque vous êtes invité pour un mot. Comment pourrais-je résoudre ceci? Je sais que ce code est Noob :) Je viens juste de commencer sur

Answer 1

J'ai modifié la fonction de telle manière:

string input_words(int maxWords) { 
    cout << "started" << endl; 
    int nWord = 0; 
    string words[maxWords]; 
    string aWord = ""; 
    while (aWord != "Quit" && nWord < maxWords) { 
     cout << "Enter a number ('Quit' to stop): "; 
     getline (cin, aWord); 
     words[nWord] = aWord; 
     nWord++; 
    } 
    cout << "finished" << endl; 
    return *words; 
} 

Après avoir entré Quit, il imprime "fini", puis "a commencé" à nouveau. Votre code appelle la fonction plusieurs fois.

Le problème est que la fonction retourne une seule chaîne. de sorte que la ligne

string words[MAX_WORDS] = input_words(MAX_WORDS); 

semble appeler la fonction input_words MAX_WORDS fois.

Une bonne façon serait de passer à vector<string>:

vector<string> input_words(int maxWords) { 
    vector<string> words; 
    string aWord; 
    while (aWord != "Quit" && nWord < maxWords) { 
     cout << "Enter a number ('Quit' to stop): "; 
     getline (cin, aWord); 
     words.push_back(aWord); 
    } 
    return words; 
} 

... 
vector<string> words = input_words(MAX_WORDS); 

Answer 2

C++ J'ai essayé le programme de test suivant, cela fonctionne:

{0,506}$> cat testcin.cpp && make testcin && ./testcin.exe 
#include <iostream> 
using namespace std; 

int main() 
{ 
    const int maxWords = 5; 
    int nWord = 0; 
    string words[maxWords]; 
    string aWord = ""; 
    while (aWord != "Quit" && nWord < maxWords) { 
     cout << "Enter a number ('Quit' to stop): "; 
     getline (cin, aWord); 
     words[nWord] = aWord; 
     nWord++; 
    }  
} 

make: `testcin' is up to date. 
Enter a number ('Quit' to stop): test 
Enter a number ('Quit' to stop): Quit 

[Vlad@rabbit] [20:53:53] [~/c++] 
{0,507}$> 

Answer 3

Le problème, je pense que dans votre principale où vous revenez le résultat de input_words() qui est un string pour initialiser les mots dans main() qui est de type string[]. C'est définitivement ce problème.

réécrite pour utiliser vector:

#include<iostream> 
#include<vector> 
#include<string> 
using namespace std; 

vector<string> input_words(int maxWords) { 
    int nWord = 0; 
    vector<string> words; 
    string aWord = ""; 
    while (aWord != "Quit" && nWord < maxWords) { 
     cout << "Enter a number ('Quit' to stop): "; 
     getline (cin, aWord); 
     words.push_back(aWord); 
     nWord++; 
    } 
    return words; 
} 

int num_words (vector<string> words) { 
    // return words.size(); 

    int numWords = 0; 
    vector<string>::iterator it = words.begin(); 
    for (; it != words.end(); it++) { 
     if (*it == "Quit") { 
      break; 
     } 
     numWords++; 
    } 
    return numWords; 
} 

int main() { 

    const int MAX_WORDS = 100; 
    vector<string> words = input_words(MAX_WORDS); 

    int lenWords = num_words(words); 
    cout << "\nThere are " << lenWords << " words:\n"; 

    vector<string>::iterator it = words.begin(); 
    for (; it != words.end(); it++) { 
     if (*it == "Quit") { 
      break; 
     } 
     cout << *it << endl; 
    } 
    return 0; 
} 

Oubliez ce qui suit, le getline() C++ supprime le '\ n' automatiquement.

Avez-vous vérifié si vos mots getline() ont des nouvelles lignes à la fin d'eux? C'est,

"Quit" != "Quit\n".