ループが失敗している間に、intと文字列C++を読み込むとき

Question

私はユーザから複数のStringsとintsを読み込もうとしています。私は、名前と年齢のペアを取って、ユーザーが「done」を入力するまでそれを続けます。しかし、私のdo-whileが早くクラッシュして、なぜ私は理解できませんか？

int number; 
string name; 
do 
{ 
cout << "Your name: " ; 
getline(cin, name);  
cout <<name<< " age: " ; 
cin >> number; 
} 
while (name!="done"); 

編集：「完了」と入力した後も、年齢にも「完了」と入力する必要があります。なぜですか？

Answer 1

VS 2010でプログラムを実行しようとしましたが、有効な番号を入力しても、プログラムは次の名前を読み飛ばしてスキップします。

cin >> number私は自然に数字の後に入力した '\ n'を飲み込まないようです。

cin >> numberの後にgetchar（）の呼び出しを追加しようとしましたが、プログラムが驚いたことに期待通りに動作し始めました。

cin >> numberの後にclean（）/ ignore（）を入力しても問題ないとか、getline（）を使って（そして解析して）数値を読み取っても結構です。

「完了」の年齢を入力しない場合は、入力した直後にループから抜け出す必要があります。私の最終的なコード：

#include <iostream> 
#include <string> 

using namespace std; 

int main() 
{ 
    int number; 
    string name; 
    while(true) 
    { 
     cout << "Your name: " ; 
     getline(cin, name); 
     if(name == "done") 
      break; 
     cout <<name<< " age: " ; 
     cin >> number; 
     cin.clear(); 
     cin.ignore(std::numeric_limits<streamsize>::max(),'\n'); 
    } 
} 

Answer 2

誰かが無効な年齢、つまり数字ではないものを入力した場合は、あなたのcinのフラグを消去し、残りの文字も消去する必要があります。

if(!(cin >> number) 
{ 
    cin.clear(); 
} 
cin.ignore(std::numeric_limits<streamsize>::max()); 

数値を読み取った場合は、改行を無視する必要があります。

Answer 3

それはageとしてint型を期待しながら、あなたはそれを文字列を与えるので。最初の反復の終わりに

、あなたは 入力ageへキーを入力し、余分な改行 が cinに入れた入力されたとして。したがって、2回目の反復では、 改行が空の行として表示され、 nameが空の文字列として設定されました。その後、プログラムは を ageとしました。十分な注意を払わないと、文字列を入力してプログラムがクラッシュします。

1つの提案：交換するには ：

getline(cin, name); 

と：

do { 
    getline(cin, name);  
} while (name.empty()); 

Answer 4

#include <iostream> 
#include <string> 

int main() 
{ 
    std::string name; 
    int age; 
    while (true) 
    { 
     std::cout << "Please enter your name: "; 
     std::cin >> name; 
     if ("done" == name) 
      break; 
     std::cout << name << ", please enter your age: "; 
     std::cin >> age; 
     std::cout << name << ", you are " << age << " years old\n"; 
    } 
    std::cout << "Bye.\n"; 
    return 0; 
} 

getline()と>>の使用をミキシング問題となる可能性があります。可能ならばそれを避けるのがベストです。両方ともgetline()を使用し、stringstream、おそらくはatoi()を使用してintに変換することもできますが、あまり好きではありません。

Answer 5

は、std名前空間を使用し

この方法

int number; 
string name; 
do 
{ 
std::cout<<"Your name: " ; 
std::cin>>name;  
std::cout<<name<<" age: " ; 
std::cin>>number; 
} 
while (name!="done"); 

またはあなたの入力は、それがクラッシュいただきました。この

using namspace std; 
int number; 
    string name; 
    do 
    { 
    std::cout<<"Your name: " ; 
    std::cin>>name;  
    std::cout<<name<<" age: " ; 
    std::cin>>number; 
    } 
    while (name!="done");