C++でforループがうまくいきません

Question

###前提・実現したいこと
C++で簡単なタイピングゲームを作っています。
ファイルからランダムな行を読み込んで、","で分割して分割した2番目の単語を入力させるものです。
###発生している問題・エラーメッセージ
ループ中、時々p_word_afterの指す値がNULLになります。この時、p_word_beforeの指す文字列は一つ前のループのときと同じになっています。
ちなみに、rand()%55の値が同じである場合はp_word_afterに値が入っているので、これは原因ではないと思います。これはどういった現象なんでしょうか。
###該当のソースコード

int key=0,count,score=0,mistake=0,ran=0,line,add,word_num;
int level=0;
char *advise;
unsigned i;
char line_text[256];
char word_before[50][128];
char word_after[50][128];
char *p_word_before;
char *p_word_after;
FILE *fp;
fp=fopen(<<filename>>,"r");
for(count=50;count>0;count--){
	rewind(fp);
	p_word_before=word_before[count];
	p_word_after=word_after[count];
	int len;
	srand((unsigned)time(NULL));
	line=0;
	for(line=rand()%55;line>0;line--){
		fgets(line_text,256,fp);
	}
	p_word_before=strtok(line_text,",");
	p_word_after=strtok(NULL,",");
	system("cls");
	add=0;
	while(*(p_word_after+add)!='\n'){
		add++;
	}
	*(p_word_after+add)='\0';
	printf("%s (%s) 残り%d単語\n",p_word_before,p_word_after,count);
	i=0;
	len=strlen(p_word_after);
	while(i<len){
		if(_kbhit()){
			key=_getch();
			if(key==0||key==224)key=_getch();
			if(key==p_word_after[i]){
				printf("checked %c\n",key);
				i++;
				score++;
			}else{
				mistake++;
			}
			fflush(stdin);
		}
	}
}

###補足情報(言語/FW/ツール等のバージョンなど)
IDE : VisualStudio2012 for Desktop

Comments

[HogeAnimalLover]

それはどこの行です発生していますか？strtokで発生しているならばこの仕様を調べれば良いと思います。（検索対象が見つからない場合にNULLが返る）

Answer 1

こんにちは。

ざっと見る限り、yasunori_bamさんが記載されている現象が起こるソースのようには見えません。

rand()%55が0の時はfgets(line_text,256,fp);が実行されませんので、p_word_beforeに「前回」のfor(count)ループ時の値が入ることはありえますが、その時、p_word_afterにも前回と同じ値が入る筈なのでNULLにはならない筈です。

また、rand()%55が例えば5の時、p_word_afterに値が入ったり、NULLが入ったりするのですね？
それもちょっと考えにくいです。・・・①

ただ、もし、サブ・スレッドを使っていてサブ・スレッドでもstrtok()を使っていたら、何が起こるか分かりませんので、ご説明されたような事象が起きる可能性はあります。strtok()はスレッド安全ではありませんので。
しかし、ご提示された内容からサブ・スレッドを使っているとは思えません。

試しに、下記の直後で、line_text、p_word_before、p_word_afterを表示してみては如何でしょうか？
何か掴めそうな気がします。

p_word_before=strtok(line_text,",");

p_word_after=strtok(NULL,",");

ところで、①の現象はどのようにして確認されましたか？ どうもその確認にミスがありそうな予感がします。
また、ファイルは55行以上存在し、その全ての行にはp_word_afterがNULLになることはない文字列が入っていることは確認できていますか？ 例えば、p_word_afterがNULLにならない行を55行コピペしておけば簡単に条件を作れると思います。

---

因みに下記はミスですね？（特に影響はないと思いますが。）

p_word_before=word_before[count];

p_word_after=word_after[count];

countは50から始まりますが、 word_before[50][128], word_after[50][128]ですので、word_before[49][], word_after[49][]までしかありません。

Comments

[catsforepaw]

> その時、p_word_afterにも前回と同じ値が入る筈なのでNULLにはならない筈です。

strtok関数は渡された文字列バッファに書き込みを行います（区切り文字のところにnull終端を書き込む）。ですので、ファイルから読まなかった場合、バッファの内容は前回とは違うものになります。

[Chironian]

あああ、そういえばそうでした。フォローありがとうございます。
ということは、rand()%55が0の時に限って発生しそうですね。

しかし、この質問を開いた時はまだcatsforepawさんの書き込みなかったのに．．．
書き込む前にチェックするようにしているのですが、時々忘れちゃいます。orz

[yasunori_bam]

回答有り難うございます。①の現象はVisualStudioのデバッグモードで変数の値を確認しながら実行して結果を得ました。
ご提案いただいた通り、p_word_before,p_word_after,line_textを表示してみたところ、rand()%55のときline_textに前のループ(for(count+1))のときの分割後のline_textが入っていました。ほんとにありがとうごうざいました！

[catsforepaw]

Chironian さん

> 書き込む前にチェックするようにしているのですが、時々忘れちゃいます。

私もです……。

Answer 2

ぱっと見て気になったことが一点。

for(count=50;count>0;count--){

この記述だと、ループの中ではcountが50から始まって最後に1になります。

p_word_before=word_before[count];
p_word_after=word_after[count];

それぞれの配列は[50]で確保されているため、添え字に指定可能な範囲は0～49です。ループ初回でcountに50が入っているので、配列の範囲を超えています。

---

しっかり見て気になった点を。

for(line=rand()%55;line>0;line--){
fgets(line_text,256,fp);
}

rand()%55は0になる可能性があります。その場合、条件判定でいきなりループ終了になるため、ループ内のfgetsが実行されません。
そうなると、line_textに正しい文字列が入らず、strtok関数が正しい結果を返さないのだと思います。

line=rand()%55 + 1とすれば良いと思います。

---

補足
strtok関数は、返すべき文字列がないとNULLを返します。前述のループでfgetsを実行しないまま抜けると、前回のバッファの内容が残っているわけですが、区切り文字が消されてしまっているので最初のstrtokで文字列全体を返してしまい、次のstrtokでは返すべき文字列がないという状況になったのだと思われます。