C++のifstreamの使い方で分からない所があるので、分かる方御教示ください。
下記は、C++で読み込んだファイルの中身が空だった場合を無理やり例外処理を使って書いてみたのですが、中身が存在する場合、たとえば、
line1
line2
line3
ではline1を無限に繰り返してしまいます。(1)を
ifstream instream = inStream(argv[1]);
while (instream >> input)
にするとうまくいくようなのですが、この違いがよくわかりません。
違いを教えてください。
#include <iostream>
#include <string>
#include <fstream>
using namespace std;
ifstream inStream(char *str);
int main(int argc, char *argv[]) {
if (argc < 2)
cout << "ファイル名を指定してください" << endl;
else {
try {
string input;
while (inStream(argv[1]) >> input) // (1)
cout << input << endl;
}
catch (int i) {
cerr << argv[1] << "を開けません" << endl;
}
}
return 0;
}
ifstream inStream(char *str) {
ifstream inStream(str);
if (!inStream) throw 1;
return inStream;
}
No.1ベストアンサー
- 回答日時:
ifstream inStream(str);
の行が、何をやっているか、理解してますか?
「strで指定されたファイルをストリームとしてオープンして、ファイルの読み込みポインタを先頭にする」のですよ。
そして
return inStream;
の行で、オープンしたストリームを「ifstream型の変数」として返しています。
なので
while (inStream(argv[1]) >> input)
とやると「毎回、argv[1]で指定されたファイルをオープンして、読み込み位置を1行目にする」ってのを繰り返します。
なので「何回ループしようが、1行目しか返って来ない」のです。
しかも、ストリームをクローズしないまま、何度も再オープンを繰り返しているので、何十回か繰り返した段階で、リソースを使い尽くしてオープンエラーが出る筈です。
ともかく「クローズしないでオープンを何度も繰り返す」ってのは、絶対に駄目です。
オープンしたストリームハンドラは「プログラム終了時に自動的にクローズされる」ので、明示的にクローズしない人が殆どですが、本来であれば「ストリームハンドラを変数に受け取って、最後にクローズするべき」なのです。
#include <iostream>
#include <string>
#include <fstream>
using namespace std;
ifstream inStream(char *str);
int main(int argc, char *argv[]) {
if (argc < 2)
cout << "ファイル名を指定してください" << endl;
else {
try {
string input;
//オープンしていいのは最初の1回だけ!
//ストリームハンドラは変数に受け取る
ifstream instream = inStream(argv[1]);
//受け取った変数でループを回す
while (instream >> input)
cout << input << endl;
}
//終わったらちゃんとクローズ!
instream.close();
//ここから例外処理
catch (int i) {
cerr << argv[1] << "を開けません" << endl;
}
}
return 0;
}
ifstream inStream(char *str) {
ifstream inStream(str);
if (!inStream) throw 1;
return inStream;
}
No.2
- 回答日時:
>ifstream instream = inStream(argv[1]);
>while (instream >> input)
>にするとうまくいくようなのですが、この違いがよくわかりません。
>違いを教えてください。
inStream()の中で「毎回、新規にオープンし直す」ので、結果として「毎回先頭から読み込んでいる」んです。
# 正確には…ローカル変数としてifstreamのインスタンス作成して…でしょうかね。
コードに書かれた通り、正常動作していますよ。
# ただし、オープンしたifstreamを閉じていないので、リソースリークしていると思いますが。
ってか…
>ifstream inStream(char *str) {
>ifstream inStream(str);
だと、再帰で永遠に自分呼び出ししません?(吹っ飛ぶまで)
# いや、ローカル変数名と関数名が同一だからコンパイルエラー?
お探しのQ&Aが見つからない時は、教えて!gooで質問しましょう!
似たような質問が見つかりました
- C言語・C++・C# C++プログラミングコードにポリモーフィズムを取り入れ方を教えてください。 2 2023/06/09 11:17
- C言語・C++・C# バイナリファイルをコピーするのにかかる時間を測りたいのですが実行するとFatel error:gli 2 2022/11/03 01:10
- C言語・C++・C# c言語の問題の説明、各所ごとに 5 2023/07/26 11:03
- C言語・C++・C# c言語 プログラムのエラー 1 2023/02/11 20:31
- C言語・C++・C# 宣言する関数の形が決まっている状態で、 str1とstr2の文字列をこの順に引っ付けてstrに保存し 2 2022/05/30 18:21
- C言語・C++・C# C++初心者です stirng 2 2022/09/20 20:43
- C言語・C++・C# C++のcinの動作 5 2023/02/26 00:13
- C言語・C++・C# このプログラミング誰か教えてくれませんか 1 2022/06/02 15:27
- C言語・C++・C# C言語のファイル入力が分かりません 2 2022/05/22 06:35
- C言語・C++・C# C言語 共用体について コマンドライン引数で値を2つ入力したときに、argv[2]の値をUNI u1 4 2022/04/25 20:34
関連するカテゴリからQ&Aを探す
おすすめ情報
デイリーランキングこのカテゴリの人気デイリーQ&Aランキング
-
#defineの使い方について
-
リモートデスクトップの接続元I...
-
openCVで動画が読み込めないエ...
-
iostream インクルード時に発生...
-
C++で縦の棒グラフ
-
std::map の const 修飾について
-
atlbase.h と afxwin.hの同時定義
-
wstringの内容を表示させたい
-
ヘッダーファイルがインクルー...
-
ファイル読み込みとmap処理
-
前回のクライアント側のプログ...
-
構文エラーが出ているのですが...
-
セグメンテーションフォルトが...
-
VC++ iostreamの不具合(?)
-
string型のフォーマット書式指...
-
テイラー展開とその精度について
-
コンパイルできません
-
C++ basic_ostreamの拡張
-
Cでのパケットキャプチャについて
-
mapの初期化
マンスリーランキングこのカテゴリの人気マンスリーQ&Aランキング
-
構文エラーが出ているのですが...
-
VC++で文字列から任意の文字を...
-
enumの値から定義名を文字列化...
-
switch文のエラーについて
-
空ENTERの判別
-
なぜ、C++の標準ヘッダをインク...
-
【C++】ヘッダ内でstringを格納...
-
C++での <iostream.h>と<iostre...
-
#include "fstream.h"
-
#define中の#のエスケープ
-
JPEGやPNGが読めるLoadImage関数
-
リモートデスクトップの接続元I...
-
CStringとString
-
string型のフォーマット書式指...
-
std::map の const 修飾について
-
VHDLのsignedとunsignedの違いは?
-
このプログラミング誰か教えて...
-
構造体配列のvectorへの変換と...
-
指定した文字を削除したい
-
構造体に決められた文字列を入...
おすすめ情報