Si tratta di un bug con getline() o sto facendo qualcosa di sbagliato. Modo giusto per usare getline()?

Question

Potrebbe non essere un bug, ma non so cosa sta andando storto. La mia prima voce viene ripetuta per str1 alla seconda iterazione ed è allo stesso modo da allora. Solo la prima iterazione va bene.

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

int main() { 

cout << " \n Enter two words. \n " ; 
char c = 'y'; 
string str; 
string str1; 
while (c == 'y'){ 

    getline(cin,str); 

    getline (cin,str1); 

    cout << " \n\n str : " << str << " str1 : " << str1 ; 
    cout << " \n Continue ? \n " ; 
    cin >> c; 
} 

return 0; 
} 

L'output è:

 
Enter two words. 
hello world 
this is mr 


str : hello world str1 : this is mr 
Continue ? 
y 
hello world 


str : str1 : hello world 
Continue ? 
n 

Answer 1

Aggiungi

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

dopo vostro

cin >> c; 

Si consideri il seguente entrata:

dog 
    cat 
    y 
    owl 
    fish 
    n 

Se esaminiamo i personaggi che sono presenti nel flusso di input singolarmente, vedremo:

d o g \n c a t \n y \n o w l \n f i s h \n n \n 

La prima chiamata a getline consuma dog\n; il secondo consuma cat\n, lasciando in questo modo:

y \n o w l \n f i s h \n n \n 

La prima chiamata a cin >> c consuma solo yma non il successivo ritorno a capo, lasciando in questo modo:

\n o w l \n f i s h \n n \n 

Ora, inizia il divertimento: cosa succede durante la prossima chiamata allo getline? Perché legge fino alla prossima nuova riga, ovviamente. Quindi la prossima chiamata a getline restituisce una riga vuota e lascia owl... nel flusso di input.

La soluzione, come descritto sopra, consiste nel consumare il resto della linea di input (ora inutile).

Answer 2

Come dice rob.

Ma una correzione alternativa che sembra più bello:

// change 
char c = 'y'; 
.... 
while (c == 'y'){ 
.... 
    cin >> c; 

// Into 
std::string c = "y"; 
.... 
while (c == "y"){ 
.... 
    std::getline(cin, c); 

Quando si tratta con l'input manuale si deve fare attenzione di utilizzare l'operatore >> come questo sarà sempre lasciare il '\ n' sull'ingresso. Il che significa che puoi utilizzare un metodo che recupera il carattere '\ n' (getline()) oppure puoi rimuoverlo manualmente afterwords (ignore()).