visual studio C++ ifstream ifs() のエラー

Question

double X1,X2,X3;
char c;
ifstream ifs("test1.txt");
ifs >> X1 >> c >> X2 >> c >> X3;

double Y1,Y2,Y3;
char d;
ifstream ifs("test2.txt");
ifs >> Y1 >> d >> Y2 >> d >> Y3;

として、データファイル(複数データ）を複数回入力したいのですが、
「'ifs':再定義されています。2回以上定義されています。」
とエラーメッセージが表示されます。ifstream ifs()による複数回入力はルール違反なのでしょうか、複数回テキストファイルデータを入力するための改善方法がありましたら、お教え頂きますようお願い致します。

Answer 1

1つの変数 ifs を使いたいなら、こんな風にも書けます。

	double X1, X2, X3;
	char c;
	ifstream ifs("test1.txt");
	ifs >> X1 >> c >> X2 >> c >> X3;
	ifs.close();

	double Y1, Y2, Y3;
	char d;
	ifs.open("test2.txt");
	ifs >> Y1 >> d >> Y2 >> d >> Y3;

Answer 2

ifstream の変数名を、ifs1,ifs2とすることで解決しました。

Answer 3

改善方法

double X1,X2,X3;
char c;
{
  ifstream ifs("test1.txt");
  ifs >> X1 >> c >> X2 >> c >> X3;
}

double Y1,Y2,Y3;
char d;
{
  ifstream ifs("test2.txt");
  ifs >> Y1 >> d >> Y2 >> d >> Y3; 
}

Comments

[fana]

ほほう，無視か．

同一スコープ内で名前が衝突していること　だけでなく，
不必要に長くファイルをオープンし続けていることも改善される方法を示したつもりであったのだが．
（これなら「別の名前」を考える手間もない）

[episteme]

僕もこっち。複数のファイルが同時に開いてる必要が無いのなら。

[bldtemporal]

fana様、別の名前を付けて解決してしまいましたが、確かに名前が同じできる、この方法はスマートだと思います。今は練習のモデル計算で読み込む入力ファイル数は少ないのですが、本番では、多くなります（10個以上）ので、ご教示の方法で実施することにします。有難うございました。

Answer 4

同じ名前で再定義できない、って話なので、名前変えましょう

Comments

[bldtemporal]

y_waiwai様、
度々のご回答有難うございます。test1,test2と変えたのですが、それでは駄目でしょうか、それとも、他のところでしょうか

[y_waiwai]

ファイル名じゃなく、ifs のほうです
ifs ってのは変数名です

[y_waiwai]

doubleとcharの変数名は変えてるのに、
なんでifstreamの変数名は変えてないの？ってはなしです

[bldtemporal]

y_waiwai様、エラーメッセージは、
「'ifs'：再定義されています。2回以上初期化されています。」でした

[bldtemporal]

y_waiwai様、ほんとにプリミテブな質問で申し訳ありませんでした。ifs1,ifs2としてみたところ、実行できました。"fstream ifs"でコマンドになっていると思ってしまいました。

Answer 5

1つのブロックに同一名の変数が2つあるというだけなので、どちらかの変数名を変えましょう。

C++の入門書を読んだほうがよいと思います。

Comments

[bldtemporal]

int32_t 様、早速のご回答有難うござます。変数名を、test1,test2としたのですが、これでは変えたことにならないでしょうか、それとも他のところを変えるのでしょうか、私の酔いした入門書には、単一入力の場合は載っていますが、複数入力の方法は載っていませんでした。追って別の本を探してみます。

[int32_t]

コンパイルができて期待どおり動くのならそれで良いです。

[bldtemporal]

int32_t様、申し訳ありません。エラーメッセージは
「'ifs'：再定義されています。2回以上初期化されています。」でした。

[int32_t]

変数名 ifs を変更したのではないのですか?

ifstream ifs2("test2.txt");
ifs2 >> Y1 >> d >> Y2 >> d >> Y3;

のような感じですよ。

[bldtemporal]

int32_t様、有難うございます。y_waiwai様にお礼申し上げましたが、ifs1,ifs2とすることでコンパイルでき、実行もできました。入門書をよんでいるのですが、どうも良く分かりませんでした。