C++で予期していない動きをしているのですがなぜですか？

Question

自分がしたいことが数字を入力したら正負を判断してyを入力したらもう一度その処理を繰り返すプログラムを書きたいのですが

実行してみると文字を入力したら1回目は正負を判断してくれてyを入力したら0が永遠にコンソールされてしまいます。
コードが間違っているのですか？
参考書通りにコードを書いたつもりなんですが

#include <bits/stdc++.h>
using namespace std;


int main (){
  string str;
  do{
    int n;
    cin >> n;
    if(n < 0){
      cout << "minus";
    }
    else if(n > 0){
      cout << "plus";
    }
    else{
      cout << 0;
    }
    cin >> str;
  }while(|str == "y");
}

Comments

[tatamyiwathy]

}while(|str == "y");
☝strの前の | は必要ですか？

[gakihiro]

要らないですね

Answer 1

再度、確認してみました。

質問主さんのコードは、動きとしては、
数字→y→数字→y→…→数字→y以外→終了　（想定した動き）
数字以外→0を吐いて終了　（想定しない動き）
数字→y→数字以外→0を無限ループ　（想定しない動き）
となりました。
ということで、上記を念頭に置いて、別途試したところ、
int の変数に対して、文字や文字列を与えるとおかしくなるようです。
（それ以降、cinが入力待ちで止まらなくなり、変数には強制的に0が入る。）
（また、適切に改行されていないため、Enter押しても改行しない事があり勘違いの元なのかもしれません。）

Answer 2

延々と0が表示されるのは、何か変な入力をしてしまってcinがfail状態になってるだけだと思います。
下記のコードで数字以外が入力された場合にFailが出力され続けることが確認できるはずです。

#include <iostream>
using namespace std;
int main(void){
    int n;
    while (true) {
        cin >> n;
        if (cin.fail()) {
            cout << "Fail\n";
        }
        cout << n << '\n';
    }
}

公開するようなプログラムであれば不正な入力された場合を想定してコードを描く必要もあるかもしれませんが、C++の勉強してるだけであればコードはそのままで不正な入力しないように気を付けるだけでいいんじゃないでしょうか。

Answer 3

#include <iostream>
#include <string>

int main (){
  using namespace std;
  string str;
  while ( true ) {
    cin >> str;
    if ( str == "y" ) break; // 'y' ならおしまい
    try {
      int n = stoi(str); // 文字列をintに変換
      if(n < 0){
        cout << "minus\n";
      }
      else if(n > 0){
        cout << "plus\n";
      } else {
        cout << 0 << endl;
      }
    } catch (...) { // intに変換できんかったら...
      cerr << "unknown input: " << str << endl;
    }
  }
}