getlineの後に、fille>>countryをすると無限ループになる(C++)

Question

自分の住んでいる地域について情報を保存したファイルから、データを読み込んで表示したいです。

C++で、getlineの後に、file>>を使用して、while loopの２周目に入ると、
無限ループになってしまいます。

cin.ignore()や、cin.clear()をつかってリピートを防ごうとしたのですが、
改善できませんでした。

アドバイスをいただけないでしょうか？

###sample.txtの中身は以下です

Tokyo 1丁目 111-222
1

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

    int main(){

        string address;
        bool japan;
        ifstream file;
	    file.open("sample.txt");

	    while(!file.eof()){
		    getline(file, address);
		    file >> japan;
        }
   }

Comments

[dodox86]

"C#"は完全に別のプログラミング言語なので、タグは外してください。また、恐らく"C"のタグも必要ありません。

[cateye]

コンパイルできる、ソースを上げて下さい。
また、環境（OS,開発環境等）も記述して下さい。
・・・addressやcountryの型が分かりません・・・

Answer 1

アドバイスをいただけないでしょうか？

ifstream の file を使っているのに、cin の ignore や clear は無意味です。

file.eof() が false だからといって、getline や file >> が成功するとは限りません。
次のように getline や file >> の返した値をチェックしないといけないでしょう。

	while (getline(file, address) && (file >> country)) {
		...
	}

あるいは、

	while (true) {
		getline(file, address);
		if (file.eof()) break;
		file >> country;
		if (file.eof()) break;
		...
	}

getline を実行すると、'\n' まで読み込んで '\n' を捨てますが、
file >> country; を実行した後は、'\n' は入力に残ったままなので、
次の getline で、address は "" になるでしょう。

sample.txt の内容も質問に追加してください。

追記
修正された質問のコードに、
cout << "address [" << address << "], japan: " << japan << endl;
を追加して実行すると

address [Tokyo 1丁目 111-222], japan: 1
address [], japan: 1

無限ループにはなっていません。

コードを次のようにすると、

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

int main()
{
	string address;
	bool japan;
	ifstream file;
	file.open("sample.txt");

	while (getline(file, address) && (file >> japan)) {
		cout << "address [" << address << "], japan: " << japan << endl;
	}
}

address [Tokyo 1丁目 111-222], japan: 1 だけが読めました。

追記2

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

int main()
{
	string address, tmp;
	bool japan;
	ifstream file;
	file.open("sample.txt");

	while (getline(file, address) && getline(file, tmp)) {
		japan = tmp == "1";
		cout << "address [" << address << "], japan: " << japan << endl;
	}
}

Comments

[alizona]

'\n'に対応するために、getline(file, tmp);を　file>>country; の下に記載したのですが、
他に何かスマートな方法はありますでしょうか？

[alizona]

申し訳ありませんでした。txtファイルの中身に、もう一セットの情報をいれると無限ルーぷになりました。

[kazuma-s]

なぜ、現象の再現する sample.txt を提示しなかったのですか？
sample.txt を
------------------------------
Tokyo 1丁目 111-222
1
Osaka 2丁目 123-456
0
---------------------
として、私の示したコードを実行すると、2つめは読み込めませんでしたが、
無限ループにはならずに終了しました。
2つ以上のデータを読めるコードを追加します。

[alizona]

申し訳ありません。

非常に助かりました。
ありがとうございました。