タイピングゲームの作成、乱数について

Question

###前提・実現したいこと
Cでタイピングゲームのような物を作っているのですが、今のコードのままだとランダムに表示させた問題を正解したときに再度同じ問題が出てしまいます
既に使用した配列の中身を排除していく方法がわからず苦戦しています
どうすればいいでしょうか？

###発生している問題・エラーメッセージ
同じ問題が出てしまう

###該当のソースコード

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <time.h>

int main()
{
	//単語設定
	char *word[] = {"banana","rinngo",......};//例です
	char typing[256];
	
	int count;	//ループ用変数
	clock_t start,end;
	
	srand((unsigned)time(NULL));	//乱数発生
	
	for(count = 0; count < 10; count++){	//ループ処理
	start = clock();	//時間計測スタート
		
		int wordchange = rand () % 20;
		
		while(*word != typing){	//比較処理
			puts(*(word + wordchange));
			gets(typing);

			if(strcmp(*(word + wordchange),typing) == 0){
				puts("\n\"right!\"\n");	//正解
				break;
			}
			else{
				puts("\n\"Nooooo!!!!\"\n");	//不正解
			}
		}
	}
	end = clock();	//時間計測終了
	printf("あなたは %d秒 でした。\n",end/CLOCKS_PER_SEC);	//結果表示、CLOCKS_PER_SECで変換
	


	return 0;
}

Answer 1

調べればやり方はいくつか出てくるのでしょうけど、考え方のひとつを紹介します。
まずword配列参照用の配列（idxとしておきます）を用意します。
word配列の要素数が20個なら同じく20個です。
初期値は、0～19です（idx[0]=0,idx[1]=1,...）
乱数の範囲は0～19で、最大値を1問ごとに減らしていきます。
word配列の参照は、idx配列を介して行います。
int wordchange = rand () % randmax;
word[idx[wordchange]]
1問終わるごとに、idx配列の対象要素を削除し後ろを詰めます。
例えば、乱数の結果、2だったとすると、idx配列を次のように組み直します。
idx[0]=0
idx[1]=1
idx[2]=3
idx[3]=4
...

それからstart = clock()はfor文の外に出さないと1問ごとの時間になってしまいますよ。

修正版コード

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <time.h>

int main()
{
	//単語設定
	char *word[] = {"1","2","3","4","5","6","7","8","9","10","11","12","13","14","15","16","17","18","19","20"};
	char typing[256];
	clock_t start,end;
	char idx[20];
	int i;
	int randmax = 20;
	int wordchange;

	// idx配列の初期化
	for(i = 0; i < 20; i++) idx[i] = i;

	srand((unsigned)time(NULL));	//乱数発生

	int count;	//ループ用変数

	//時間計測スタート
	start = clock();
	for(count = 0; count < 10; count++){	//ループ処理
		wordchange = rand () % randmax;

		while(*word != typing){	//比較処理
			puts(word[idx[wordchange]]);
			gets(typing);

			if(strcmp(word[idx[wordchange]],typing) == 0){
				puts("\n\"right!\"\n");	//正解
				break;
			}
			else{
				puts("\n\"Nooooo!!!!\"\n");	//不正解
			}
		}
		// idx配列を詰める
		for(i = wordchange; i < 20; i++) idx[i] = idx[i + 1];
		randmax--;
	}
	end = clock();	//時間計測終了
	printf("あなたは %d秒 でした。\n",end/CLOCKS_PER_SEC);	//結果表示、CLOCKS_PER_SECで秒に変換

	return 0;
}

Comments

[yowappu]

素早い回答、ありがとうございます。
word配列参照用の配列を用意する、の部分なのですが
｝がついていませんとエラーメッセージが出てうまくいきません

start = clock()の部分、ありがとうございます、さっそく直しました

[ttyp03]

修正したコードを貼ってもらってもよいですか。

[yowappu]

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <time.h>

int main()
{
//単語設定
char *word[] = {"1","2"......};
char typing[256];
clock_t start,end;
char idx[20];
idx[0]=0;
idx[1]=1;
idx[2]=2;
idx[3]=3;
idx[4]=4;
idx[5]=5;
idx[6]=6;
idx[7]=7;
idx[8]=8;
idx[9]=9;
idx[10]=10;
idx[11]=11;
idx[12]=12;
idx[13]=13;
idx[14]=14;
idx[15]=15;
idx[16]=16;
idx[17]=17;
idx[18]=18;
idx[19]=19;

srand((unsigned)time(NULL)); //乱数発生

int count; //ループ用変数

//時間計測スタート
for(count = 0; count < 10; count++){ //ループ処理
start = clock();
int wordchange = rand () % RAND_MAX;

while(*word != typing){ //比較処理
puts(word[idx[wordchange]]);
gets(typing);

if(strcmp(word[idx[wordchange]],typing) == 0){
puts("\n\"right!\"\n"); //正解
break;
}
else{
puts("\n\"Nooooo!!!!\"\n"); //不正解
}
}
}
end = clock(); //時間計測終了
printf("あなたは %d秒 でした。\n",end/CLOCKS_PER_SEC); //結果表示、CLOCKS_PER_SECで秒に変換

return 0;
}

修正をし、プログラム自体は動くようになったのですが、1問目を解いた後に強制終了するようになってしまいました。

[ttyp03]

コードありがとうございます。
問題点を修正したコードを本文の方に貼っておきました。
参考にしてください。
直した箇所は以下です。
・idx配列の初期化をループ化
・RAND_MAXを変数化
・変数wordchangeを関数先頭で宣言（ループの外で使うため）
・idx配列を詰める処理を追加
・randmaxをデクリメントする処理を追加
・計測スタートをループの外に移動

[ttyp03]

それから先ほどのコードはとりあえず動く状態にしたので、本来なら20などハードコーディングせずに、マクロで定義したり、配列の要素数から求めるべきです。
今のコードの理解ができたら、そういったところも見直してみてください。

[yowappu]

丁寧な回答たいへんありがとうございます

教えていただいたのにも関わらず配列を詰める処理などを考えていませんでした・・
今回教えていただいたことを参考にしもっと勉強していきたいと思います。
ありがとうございました