C++相談室 part48

丸コピでスマソ。
fi.open(fileChar, ios::in | ios::binary);
if (fi){
while(charaNum < 100){
charaNum++;
fi.seekg(1, ios::cur);
}
fi.seekg(0, ios::beg);

while(!fi.eof()) //ファイルの最後にきたら終了
{
sprintf(cha2, "%s", cha1);
fi.read(cha1, 1);
sprintf(chan, "%s%s", cha2, cha1);
if (strcmp(chan, "le") == 0){//改行
strs.erase(strs.end() - 1);
strvec.push_back(new string(strs, 0, strs.size()));
strs = "";
fi.seekg(2, ios::cur);
}else{
strs.append(cha1);//文字を保存
}
}
fi.getline(chars, 255);
strvec.push_back(new string(chars));
}
fi.close();

980 ：デフォルトの名無しさん：2006/04/27(木) 15:23:39

つバッファオーバーフロー

981 ：デフォルトの名無しさん：2006/04/27(木) 15:31:04

初期化してないcha1の使いまわしでcha2が溢れてるだけじゃねーの？

982 ：970：2006/04/27(木) 15:32:19

再度読み込む際は、これのあるオブジェクトごとdeleteしてるので、
バッファオーバーフローは無いと思うんですが。
一応格納する文字列のサイズは読み込む際は毎回０になってます。

983 ：970：2006/04/27(木) 15:33:19

sprintf(cha1, "");
sprintf(cha2, "");
で一応初期化はしてます。

984 ：デフォルトの名無しさん：2006/04/27(木) 15:33:33

chaではじまる変数の宣言と読み込もうとしたファイルのサイズも晒せ

985 ：970：2006/04/27(木) 15:37:15

vector<string*> strvec;//ファイル内容行ごとに格納
string strs;//一行読み込み
int charaNum;//文字数
char cha;
char cha1[2];
char cha2[2];
char chan[4];
char fileChar[100];
ifstream fi;//ファイル入力
ファイルサイズは34.6kb～65.9kbです。

986 ：デフォルトの名無しさん：2006/04/27(木) 15:48:50

sprintf(cha1, "");がまずい

最初の一バイト目だけ'\0'で初期化されるけど、ニバイト目にはゴミが入ってる可能性がある。
fi.read(cha1, 1); で一バイト目に文字を入れたのはいいけど、ニバイト目に運良く'\0'が入っていれば次の
sprintf(chan, "%s%s", cha2, cha1); で期待通りの動作になる。

でも、ニバイト目に'\0'じゃないなにか ('A'とか)がはいってると
sprintf(chan, "%s%s", cha2, cha1); で readで読んだ一文字目 + 'A' + '\0'が現れるまで延々と続く
ゴミ文字列がchanにコピーされる

chanのサイズが4バイト分しかないんで、3文字+'\0'を越える長さの文字列を入れると悲しいことが起こる。

readした直後に cha1[1] = '\0';とかやっとけば問題ない

二文字目にたまたま何が入ってるかは、おまいのプログラムのつくりによるんだが
一回目は'\0'が入ってて、二回目以降は違うものが入ってるんだと思う

987 ：970：2006/04/27(木) 16:09:04

ありがとうございました。
おかげ様で上手く行きそうです。
助かりました。
こんな落とし穴があったとは。

988 ：デフォルトの名無しさん：2006/04/27(木) 17:57:09

std::ifstreamやofstreamなどのファイルストリームが
デストラクタで自動的にcloseされるようになっていない
のには何か理由があるのですか？

989 ：デフォルトの名無しさん：2006/04/27(木) 20:24:00

>>988
デストラクタでcloseされるよ。

990 ：デフォルトの名無しさん：2006/04/28(金) 01:13:15

>>970
最後まで読んでstd::ios::eofbitとかstd::ios::failbitが立ってしまってんだろ。

C++のファイルストリームは、Cとは違い、一度これらのフラグが立ったら、
自動ではクリアされない。そして、クリアしないと入出力が停止される。

clear()汁。

991 ：デフォルトの名無しさん：2006/04/28(金) 07:47:59

俺の股間もよく failbit がたってしまいます。
clear() しても今度はすぐにストリームが空になって
eofbit たってしまいます。

992 ：デフォルトの名無しさん：2006/04/28(金) 08:29:50

これだからスパゲティちんこは･･･