Безконечый цикл при использовании cin и getline()

@kozo · Регистрация: 27.09.2016

Студворк — интернет-сервис помощи студентам

Здравствуйте,

Есть вот такой простенький кусок кода:

C++

                string tempid;
            string tempname;
            string tempa;
            int templ=0;
            cout<<"ADD SONG"<<endl;
            cout<<"Enter song's unique ID: ";
            cin>>tempid;
            cout<<endl;
            cout<<"Enter song's name: ";
            
            getline(cin, tempname);
            cin.ignore();
            cout<<endl;
            cout<<"Enter artist's name: ";
            
            getline(cin, tempa);
            cin.ignore();
            cout<<endl;
            cout<<"Enter song's length (in seconds): ";
            cin>>templ;
            cin.ignore();
            cout<<endl;

Он успешно считывает первые три strings, но когда доходит до cin>>templ, то начинается безконечный цикл... Что это может быть?

@DemolitionMan · 23.10.2016, 07:45

Строка 21:

Сообщение от kozo

C++

1	cin.ignore();

- Вы применяли ignore к getline в 3 случаях выше, а тут Вы применяете его к cin. Может это неправильно?

Добавлено через 12 минут
Только так:

C++

string tempid;
string tempname;
string tempa;
int templ=0;
cout<<"ADD SONG"<<endl;
cout<<"Enter song's unique ID:";
cin>>tempid;
cout<<endl;
cout<<"Enter song's name: ";
 
cin >> tempname;
cout << endl;
cout << "Enter artist's name:";
 
cin >> tempa;
cout << endl;
cout << "Enter song's length (in seconds):";
cin >> templ;
 
cout << "VVedennie znacheniya:\n";
cout << "Unique ID:" << tempid << '\n';
cout << "Song's name:" << tempname << '\n';
cout << "Artist's name:" << tempa << '\n';
cout << "Song's length:" << templ << '\n';

@kozo · 23.10.2016, 08:42 **[ТС]**

DemolitionMan, проблема в том, что некоторые имена могут иметь несколько слов,и их простым cin не прочитаешь. Что касается cin.ignore() - я пробовал его забрать после сin>>templ; всеравно безконечный цикл...
У меня этот код находится в while(char usrChar != 'q') если что, по этому и безконечный цикл. Но почему? Не могу понять

@Даценд · 23.10.2016, 09:28

kozo,
Покажите код с циклом.

@kozo · 23.10.2016, 09:30 **[ТС]**

Даценд,

C++

do {
        cout<<pName<<" PLAYLIST MENU"<<endl;
        cout<<"a - Add song\nd - Remove song\nc - Change position of song"<<endl;
        cout<<"s - Output songs by specific artist\nt - Output total time of playlist (in seconds)"<<endl;
        cout<<"o - Output full playlist\nq - Quit"<<endl;
        cout<<endl<<"Choose an option: ";
        cin>>usrChar;
        cout<<endl;
 
        
        switch(usrChar) {
            
            case 'o':
            cout<<pName<<" - OUTPUT FULL PLAYLIST"<<endl;
            if (head == 0) { 
            cout<<"Playlist is empty"<<endl<<endl;
            }
            else {
                int count =1;
                temp = head;
                 do{
                    cout<<count<<"."<<endl;
                    temp->PrintPlaylistNode();
                    temp = temp->GetNext();
                    count++;
                    cout<<endl;
                    
                }while(temp != 0);
                
            }
            
            break;
            case 'a':
            string tempid;
            string tempname;
            string tempa;
            int templ=0;
            cout<<"ADD SONG"<<endl;
            cout<<"Enter song's unique ID: ";
            cin>>tempid;
            cout<<endl;
            cout<<"Enter song's name: ";
            
            getline(cin, tempname);
            cin.ignore();
            cout<<endl;
            cout<<"Enter artist's name: ";
            
            getline(cin, tempa);
            cin.ignore();
            cout<<endl;
            cout<<"Enter song's length (in seconds): ";
            cin>>templ;
            cout<<endl;
            
 
            
            temp = new PlaylistNode(tempid, tempname, tempa, templ);
            if(head == 0) {
                head = temp;
            }
            else {
                tail->SetNext(temp);
            }
            tail = temp;
            break; 
        } 
    }while(usrChar != 'q');

Вот весь цикл и поблемный кусок внутри

@DemolitionMan · 23.10.2016, 10:05

Что касается ignore, я находил тему где модераторы объясняли зачем он нужен и это есть в популярных ошибках вверху. Но я забыл зачем это все нужно и где это находится. Там что-то в потоке содержится какой-то мусор(перенос строки или что-то в этом роде), я забыл короче

. Модераторы, напомните, пожалуйста, зачем нужен ignore.

Добавлено через 16 минут
Код опять какой-то бредовый получился

:

C++

string tempid;
string tempname;
string tempa;
int templ=0;
cout<<"ADD SONG"<<endl;
cout<<"Enter song's unique ID: ";
cin>>tempid;
cout<<"\nEnter song's name: ";
cin.ignore();
getline(cin, tempname);
cout<<"\nEnter artist's name: ";
getline(cin, tempa);
cout<<"\nEnter song's length (in seconds): ";
cin>>templ;
 
cout << "\nVvedennie znacheniya:\n";
cout << "Unique ID: " << tempid << '\n';
cout << "Song's name: " << tempname << '\n';
cout << "Artist's name: " << tempa << '\n';
cout << "Song's length: " << templ << '\n';

Добавлено через 2 минуты
Бред, потому что перед 10 строкой есть ignore, а перед 12 - нет.

@Даценд · 23.10.2016, 11:33

kozo,
как-то так:

C++

cout<<endl<<"Choose an option: ";
usrChar=cin.get();
cin.ignore();
cout<<endl;

и

C++

case 'a':
    string tempid;
    string tempname;
    string tempa;
    int templ=0;
    cout<<"ADD SONG"<<endl;
    cout<<"Enter song's unique ID: ";
    getline(cin, tempid);
    cout<<endl;
    cout<<"Enter song's name: ";
    getline(cin, tempname);
    cout<<endl;
    cout<<"Enter artist's name: ";
    getline(cin, tempa);
    cout<<endl;
    cout<<"Enter song's length (in seconds): ";
    cin >> templ;
    cin.ignore();
    cout<<endl;

nd2 · 23.10.2016, 16:09

Сообщение от DemolitionMan

Модераторы, напомните, пожалуйста, зачем нужен ignore.

Распространенные ошибки

@kozo · 23.10.2016, 22:22 **[ТС]**

Даценд, помогло,спасибо! Тоесть cin. ignore() нужно писать только после cin>> а не после getline()

@kozo 0 / 0 / 0 Регистрация: 27.09.2016 Сообщений: 52
	23.10.2016, 08:42 [ТС]	3
	DemolitionMan, проблема в том, что некоторые имена могут иметь несколько слов,и их простым cin не прочитаешь. Что касается cin.ignore() - я пробовал его забрать после сin>>templ; всеравно безконечный цикл... У меня этот код находится в while(char usrChar != 'q') если что, по этому и безконечный цикл. Но почему? Не могу понять 0

@Даценд 5869 / 4746 / 2940 Регистрация: 20.04.2015 Сообщений: 8,361
	23.10.2016, 09:28	4
	kozo, Покажите код с циклом. 0

nd2 3434 / 2813 / 1249 Регистрация: 29.01.2016 Сообщений: 9,426
	23.10.2016, 16:09	8
	Сообщение от DemolitionMan Модераторы, напомните, пожалуйста, зачем нужен ignore. Распространенные ошибки 0

@kozo 0 / 0 / 0 Регистрация: 27.09.2016 Сообщений: 52
	23.10.2016, 22:22 [ТС]	9
	Даценд, помогло,спасибо! Тоесть cin. ignore() нужно писать только после cin>> а не после getline() 0

Опции темы

Безконечый цикл при использовании cin и getline()

Решение