Почему это чтение cin заклинило?

Question

Я выделил ошибку в моей программе, которая не позволяет мне присвоить значение переменной addAntonymAnswer1. Я попытался запустить cin.clear() перед оператором, чтобы получить вещь, прочитав мой yes/no ответ, но код просто не отвечает.

Программный бит, который не работает, находится внутри void dictionaryMenu(vector <WordInfo> &wordInfoVector) и читает

         cin.clear();

         cout<<">";
         cin>>addAntonymAnswer1;

         // cin reading STUCK, why!?  

чтобы добраться до этой точки программы, пользователь должен добавить слово, а затем добавить синоним.

Вход для запуска программы:

dictionary.txt
1 cute
2 hello
3 ugly
4 easy
5 difficult
6 tired
7 beautiful
synonyms
1 7
7 1
3 2
antonyms
1 3
3 1 7
4 5
5 4
7 3

---

#include <iostream>
#include <fstream>
#include <string>

#include <sstream>
#include <vector>


using namespace std;



class WordInfo{

      public:

             WordInfo(){}

             WordInfo(string newWord){
                           word=newWord;                
             }

             ~WordInfo() { }

             int id() const {return myId;}

             void readWords(istream &in)
             {
               in>>myId>>word;     
             }

             vector <int> & getSynonyms () {

                    return mySynonyms;
             }

            vector <int> & getAntonyms() {

                     return myAntonyms;
            }

            string getWord() {
                     return word;
                   }


          void dictionaryMenu (vector <WordInfo> &wordInfoVector){

          cout<<endl<<"Would you like to add a word?"<<endl;   
          cout<<"(yes/no)"<<endl;
          cout<<">";
          string addWordAnswer;
          cin>>addWordAnswer;

          if (addWordAnswer=="yes")
          // case if the guy wants to add a word
          {
          cout<<endl;                  
          cout<<"Please, write the word "<<endl;                  

          string newWord;
          cout<<">";
          cin>>newWord;
          cout<<endl;         

          WordInfo newWordInfo (newWord);

          int newWordId = wordInfoVector.size() +1;

          newWordInfo.myId=newWordId;

          cout<<"The id of "<<newWordInfo.word<<" is "<<newWordInfo.myId<<endl<<endl; 

          wordInfoVector.push_back(newWordInfo);


          cout<<"Would you like to define which words on the existing dictionary are" <<endl 
          <<"synonyms of "<<newWordInfo.word<<"?"<<endl;


          cout<<"(yes/no)"<<endl;

          string addSynonymAnswer, addAntonymAnswer1, addAntonymAnswer2;
          cout<<">";
          cin>>addSynonymAnswer;  




          if (addSynonymAnswer=="yes")
          {

            cout<<endl;
            cout<<"Please write on a single line the ids for the synonyms of "
            <<newWordInfo.word<<endl<<"starting with its id, which is "<<newWordInfo.myId<<endl<<endl;

            cout<<"For example, to define that the synonym of the word 'cute', which has an id 1, is" 
            <<"'beautiful', which has an id 7, you should write: 1 7"<<endl<<endl;

            cout<<"In the case of "<<newWordInfo.word<<" you should start with "<<newWordInfo.myId<<endl;

            cin.clear();
            string lineOfSyns;
            cout<<">";

            cin>>lineOfSyns;

            newWordInfo.pushSynonyms(lineOfSyns, wordInfoVector); 

            cin.clear();     


                 cout<<"Would you like to define which words on the existing dictionary are" <<endl 
                 <<"antonyms of "<<newWordInfo.word<<"?"<<endl; 

                  //##HERE THE CIN READING OF addAntonymAnswer1 FAILS, WHY?

                 cin.clear();
                 cout<<">";
                 cin>>addAntonymAnswer1;

                 // cin reading STUCK, why!?   


                 if (addAntonymAnswer1=="yes"){ }                        

                    else if (addAntonymAnswer1=="no"){
                         // END DICTIONARY MENU
                         }                  
          }
             else if (addSynonymAnswer=="no"){

                cout<<"Would you like to define which words on the existing dictionary are" <<endl 
                 <<"antonyms of "<<newWordInfo.word<<"?"<<endl; 


                 cout<<">";
                 cin>>addAntonymAnswer2;

                 if (addAntonymAnswer2=="yes"){ }                        

                    else if (addAntonymAnswer2=="no"){
                         // END DICTIONARY MENU
                         }  

             }


          } // if addWordAnswer == "no"

          else if (addWordAnswer=="no"){

               // ######RETURN TO MAIN MENU############
               }        






            }

             void pushSynonyms (string synline, vector<WordInfo> &wordInfoVector){


             stringstream synstream(synline);

             vector<int> synsAux;

             // synsAux tiene la línea de sinónimos

             int num;

             while (synstream >> num) {synsAux.push_back(num);}

             int wordInfoVectorIndex;

             int synsAuxCopyIndex;



             if (synsAux.size()>=2){ // takes away the runtime Error



             for (wordInfoVectorIndex=0; wordInfoVectorIndex <wordInfoVector.size(); wordInfoVectorIndex++)
             {


                 if (synsAux[0]==wordInfoVector[wordInfoVectorIndex].id()){


                    // this is the line that's generating a Runtime Error, Why?                                                       
                    for (synsAuxCopyIndex=1; synsAuxCopyIndex<synsAux.size(); synsAuxCopyIndex++){

                    // won't run yet    
                    wordInfoVector[wordInfoVectorIndex].mySynonyms.push_back(synsAux[synsAuxCopyIndex]);      
                        }                                                          
                 }     
             } 

             }// end if size()>=2


            } // end pushSynonyms








             void pushAntonyms (string antline, vector <WordInfo> &wordInfoVector)
             {



             stringstream antstream(antline);

             vector<int> antsAux;

             int num;

             while (antstream >> num) antsAux.push_back(num);


             int wordInfoVectorIndex;

             int antsAuxCopyIndex;   



             if (antsAux.size()>=2){ // takes away the runtime Error             

             for (wordInfoVectorIndex=0; wordInfoVectorIndex <wordInfoVector.size(); wordInfoVectorIndex++)
             {


                 if (antsAux[0]==wordInfoVector[wordInfoVectorIndex].id()){


                    // this is the line that's generating a Runtime Error, Why?                                                       
                    for (antsAuxCopyIndex=1; antsAuxCopyIndex<antsAux.size(); antsAuxCopyIndex++){

                    // won't run yet    
                    wordInfoVector[wordInfoVectorIndex].myAntonyms.push_back(antsAux[antsAuxCopyIndex]);      
                        }                                                          
                 }     
             } 

             }// end if size()>=2





             }

             //--dictionary output function

             void printWords (ostream &out)
             {
                out<<myId<< " "<<word;     
             }



             //--equals operator for String
             bool operator == (const string &aString)const
             {
                           return word ==aString; 

             }


             //--less than operator

             bool operator <(const WordInfo &otherWordInfo) const
             { return word<otherWordInfo.word;}

             //--more than operator

             bool operator > (const WordInfo &otherWordInfo)const
             {return word>otherWordInfo.word;}

             public: 

                  vector<int> mySynonyms;
                  vector <int> myAntonyms;


                   string word;
                   int myId;


      };

      //--Definition of input operator for WordInfo
      istream & operator >>(istream &in, WordInfo &word)
      {
         word.readWords(in); 

      }



      //--Definition of output operator

      ostream & operator <<(ostream &out, WordInfo &word)
      {
            word.printWords(out);  

      }





      int main() {

          string wordFile;
          cout<<"enter name of dictionary file: "<<endl;
          getline (cin,wordFile);

          ifstream inStream (wordFile.data());

          if(!inStream.is_open())
          {
          cerr<<"cannot open "<<wordFile<<endl; 
          exit(1);                      

          }

          vector <WordInfo> wordInfoVector; 

          WordInfo aword;





          while (inStream >>aword && (!(aword=="synonyms")))
          {
              wordInfoVector.push_back(aword);


          }

          inStream.clear();









          vector <int> intVector;
          string synLine; 






          while (getline(inStream, synLine)&&(synLine!=("antonyms"))){

                aword.pushSynonyms(synLine, wordInfoVector);

                }



          int theIndex;



          string antLine;

          while (getline(inStream,antLine)){

                aword.pushAntonyms(antLine, wordInfoVector);
                }      



          cout<<endl<<"the words on the dictionary are: "<<endl;

          int h=0;          
          while (h<wordInfoVector.size()){
                cout<<wordInfoVector[h]<<endl;
                h++;
                }

          aword.dictionaryMenu(wordInfoVector);

          system("PAUSE");

          return 0;
      }

Answer 1

cin.clear() не очищает стандартный ввод. Он очищает биты ошибок, такие как eofbit, failbit и другие, и устанавливает поток в исправное состояние. Может быть, вы ожидали, что это что-то прояснит? Если пользователь набрал

yes no

Как раз перед тобой, а ты

cin >> someStringVariable;

Он будет читать до no, а поток все равно будет содержать

 no

Вызов clear затем очищает все активные биты ошибок. Тогда ваш

cin>>addAntonymAnswer1;

Будет читать no, который не был съеден предыдущим чтением, и действие немедленно возвращается, не ожидая нового ввода. Что вы должны сделать, это сделать clear с последующим игнорированием, до следующей новой строки. Вы говорите ему количество символов, которое он должен максимально игнорировать. Эта сумма должна быть максимально возможным числом:

cin.clear();
cin.ignore(std::numeric_limits<std::streamsize>::max(), '\n');

Это приведет к тому, что поток станет пустым, и последующее чтение будет ждать, пока вы что-то наберете.

---

Другая проблема возникает, если у вас есть cin >>, за которым следует getline: cin оставит все пробелы (также новые строки) после своего токена чтения, но getline прекратит чтение после того, как достигнет такой новой строки. Я вижу, вы поставили clear после почти всего. Поэтому я хочу показать вам, когда вам это нужно, а когда нет. Вам это не нужно, когда вы упорядочиваете несколько cin >>. Предположим, у вас есть в буфере: "foo \ nbar \ n". Затем вы делаете следующее чтение

cin >> a; // 1
cin >> b; // 2

После первого, ваш буфер будет содержать "\ nbar \ n". То есть символ новой строки все еще активен. Второй cin>> сначала пропустит все пробелы и символы новой строки, чтобы он мог справиться с тем, что \n находится перед bar. Теперь вы также можете упорядочить несколько вызовов getline:

getline(cin, a);
getline(cin, b);

Getline выбросит \n, которое он читает в конце строки, но не будет игнорировать символы новой строки или пробелы в начале. Итак, после первой getline буфер содержит «bar \ n». Второй getline также будет правильно читать "bar \ n". Теперь давайте рассмотрим случай, когда вам нужно очистить / игнорировать:

cin >> a;
getline(cin, b);

Первый покинет поток как "\ nbar \ n". Затем getline сразу увидит \n в начале и будет считать, что она читает пустую строку. Таким образом, он будет немедленно продолжен и ничего не будет ждать, оставляя поток как "bar \ n". Итак, если у вас есть getline после cin>>, вы должны сначала выполнить последовательность очистки / игнорирования, чтобы очистить символ новой строки. Но между getline или cin>> вы не должны этого делать.

Answer 2

В вашей программе вы спрашиваете пользователя:

Please write on a single line the ids for the synonyms of test
starting with its id, which is 8

For example, to define that the synonym of the word 'cute', which has an id 1, i
s'beautiful', which has an id 7, you should write: 1 7

In the case of test you should start with 8

Затем вы пытаетесь прочитать строку, набранную пользователем

cin>>lineOfSyns;

Однако, это только чтение до первого пробела. Таким образом, 2-й номер, введенный пользователем, все еще находится в буфере cin, когда

cin>>addAntonymAnswer1;

Строка

выполняется, поэтому данные считываются в строку addAntonymAnswer1. Пользователь никогда не получает возможность ввести «да» или «нет», и ваши тесты для этих значений не пройдены.

Вы должны рассмотреть возможность использования перегрузки string для getline() для чтения:

getline( cin, stringvar);

Это, вероятно, было бы лучше, чем использование cin.getline(), поскольку у него нет перегрузки, которая принимает string - с этой функцией-членом вам нужно читать массив символов, который гораздо менее гибок, чем чтение в string.

Answer 3

cin >> ... читает из стандартного ввода, пока не найдет символ пробела. Когда вы вводите, скажем, 8 5 для списка синонимов, 8 считывается в lineOfSyns и ничего более. Когда программа достигает cin >> addAntonymAnswer1, 5 считывается в addAntonymsAnswer1. Ваша программа ведет себя неожиданно, так как она ожидает yes или no, но получила 5.

Посмотрите на использование cin.getline() вместо >>. См., Например, разделы 18.2 и 18.3 на этой странице .

Answer 4

Он "застрял", потому что он ожидает ввода. cin прикреплен к стандартному дескриптору ввода для программы, вам нужно что-то набрать и нажать Enter.