C言語で21ゲームを作りました。アドバイスください。

21ゲームを作りました。
無駄に処理しているところやバグをこれ以上自分ではわかりません。
ご教授ください。

C
1#include <stdio.h>
2#include <stdlib.h>
3#include <time.h>
4#include <stdbool.h>
5int main(void) {
6    int input = 0, cpu = 0, answer = 0;
7    bool p_lose = false, c_lose = false;
8    
9    printf("--------21ゲーム--------\n");
10    while (answer < 21) {
11        srand((unsigned)time(NULL));
12        printf("1から3までの数字を入力してください。\n");
13        printf("あなた　: ");
14        scanf("%d", &input);
15        if(input > 3)
16            continue;
17        else
18            answer += input;
19        
20        if (answer == 20) c_lose = true;
21        
22        cpu = rand() % 3 + 1;
23        printf("あいて　: %d\n", cpu);
24        answer += cpu;
25        printf("現在の数字%d\n\n",answer);
26        if (answer == 20) p_lose = true;
27        
28        if (p_lose == true || c_lose == true)
29            break;
30
31    }
32    if (p_lose == true){
33        printf("あいての勝ち\n");
34    } else if (c_lose == true){
35        printf("あなたの勝ち\n");
36    }
37    return 0;
38}
39

maisumakun

2018/03/13 23:41

もう少し具体的に、「このような場合に動作がおかしい」など、質問する点を明確にしてもらえればと思います。

ttyp03

2018/03/13 23:41

バグの詳細を記入してください。

Rock_s6nmnl

2018/03/13 23:49

エラーは出ていません。無駄に処理しているところや、バグがもしあったら教えていただきたいという質問です。分かりづらくてすみません。

ttyp03

2018/03/13 23:52

丸投げの質問にあたると思うので推奨されません。もう少し「ここの処理はどうにかならないか」とかならギリギリOKかと思いますが。

ttyp03

2018/03/13 23:53

ちなみにエラーとバグは意味合いが違うので気をつけましょう。

行動規範の内容に同意します

回答7件

srandは全体で1回だけでいいです。乱数シードが毎回設定され直すというのは、処理的にも無駄ですが乱数の偏りが発生しやすくなるため推奨されません。
bool値をifで使うなら「bool値 == true」と「bool値」と書くのは意味が一緒です。普通trueと比較せずにそのままbool値を結果として使います。
whileをbreakするifが末尾にあるなら、whileの条件に組み込むほうがスマートです。なぜならwhileが終わる条件が1箇所見るだけではっきりするようになるからです。
無駄とは違いますがCPUの最後の一手すらランダムなのはCPUの出来が悪すぎかと思います。20までの距離が3以下である場合、必ず20を選択するようにしないと敵AIとして不出来すぎます。

投稿2018/03/14 00:42

masaya_ohashi

総合スコア9206

入力が**「1から3まで」**の要件を満たしていません。現状だと0や負数も入力できてしまい、ゲームとして成り立ちません。

投稿2018/03/14 01:53

catsforepaw

総合スコア5938

他の方からある通り丸投げ感のある質問ではありますが、こういうの嫌いではないので回答します。

とりあえずソースコードを見た感じで、以下は修正した方がいいです。

20とか3とかは定数にする。
c_lose、p_loseの二種類の変数がありますが、コンピュータが負けるかプレイヤーが負けるかは同時に起こることはないでしょうから、この手のフラグは一つにする。

ひとまずここまで。他にもあれば追記します。

投稿2018/03/14 00:36

bassbone

総合スコア767

ゲームのルールと流れに関してバグといえそうな問題３点、
他の方が指摘していないコード上の細かいことを２点、指摘します。

(a) 勝ち負けが決まる条件をif (answer == 20)とするのは正しいと言えない。 if (answer == 21)とすべきではないでしょうか。

こちらが 20 にすれば相手は 1 を出すしかないから勝ち…ですが、勝敗がきまるのは、それだけでしょうか。例えば 19 の時点で 2 を選んだら負けです。この場合 answer は 20 になりませんから、勝敗を正しく判定できません。
要は、21までプレイさせること。

じゃ、そもそも 2 を選んで負けるような、マヌケなプレーヤーはいるのか？いるでしょう。理解していない人、うっかり 2 を出してしまうそそっかしい人、そしてなにより乱数で数を選ぶコンピュータ。

(b) ゲームの終盤になって、例えば 19 の時に 3 を選べるのはおかしいが、このプログラムだと、コンピュータも人も、3 を選べてしまう。
終盤、 21 に近づいたら選べる数を制限したら良いと思います。即ち

18 までは 1 〜 3 を選べる
19 の時点で 1, 2 を選べる
20 の時点で選べるのは 1 だけ

人が 20 にした時点で人の勝ちが決まるが、コンピュータはもう一手選んでからゲームが終了する

のに対し、

コンピュータが 20 にした時点でコンピュータの勝ちが決まり、直ちにゲームが終了する

ゲームの流れが非対称で、いただけません。これもバグ。

(d)

C
1        if(input > 3)
2            continue;
3        else
4            answer += input;

ここの else は不要です。これで十分だし、わかりやすいはず。

C
1        if(input > 3)
2            continue;
3
4        answer += input;

(e) 変数の全てに初期値が必要でしょうか？

C
1    int input = 0, cpu = 0, answer = 0;
2    bool p_lose = false, c_lose = false;

この中で、input と cpu に初期値を与える必要はありません。
初期値を書いてもバグではありませんし、積極的に初期値を書く人もいるかもしれませんが、「とにかく初期化しちゃえば良いんでしょ？」みたいな、安易な感じが私は好きではありません。０でない初期値だってあるのだから、０にしておけば安心、とはならない。

投稿2018/03/14 15:50

rubato6809

総合スコア1380

自分が考えたアルゴリズムを説明します。
1.まず、while文の条件文のところですが、とりあえずに「1」（無限ループ）にしておき、while文の中で
コールした数字が21に達したときbreakでループを止めます。
2.入力の時 1から3以外の整数の場合、再入力のためにdo～while文を使います。
3.コールした数字を表示するときfor文で表示します。(ここはwhile文でもOK!)
4.自分とcpuのどちらが21をコールするか判定する変数(下のソースではwin_flagとしています。)を
用意し、最終結果を表示します。

C言語
1
2#include <stdio.h>
3#include <stdlib.h>
4#include <time.h>
5
6#define BUFSIZE 256
7#define MAX_NUMBER 21
8#define MAX_ANSWER 3
9#define MIN_ANSWER 1
10#define MY_WIN 1
11#define MY_LOSE 0
12
13int main(void)
14{
15	char buf[BUFSIZE];
16	int number = 0;
17	int my_answer;
18	int cpu;
19	int i;
20	int win_flag; //1ならば自分の勝ち,0ならばcpuの勝ち,
21	srand((unsigned)time(NULL));
22	printf("--------------21ゲーム---------------\n");
23	
24	while(1){
25		do{
26			printf("%d～%dまでの自然数を入力\n",MIN_ANSWER,MAX_ANSWER);
27			printf("-->");
28			fgets(buf,sizeof(buf),stdin);
29			my_answer = atoi(buf);
30			if(my_answer < 1 || my_answer > MAX_ANSWER){
31				printf("%d～%dまでの自然数ではありません.\n",MIN_ANSWER,MAX_ANSWER);
32			}
33		}while(my_answer < MIN_ANSWER || my_answer > MAX_ANSWER);
34		printf("あなたが言った数字:");
35		for(i = number + 1; i < number + 1 + my_answer; i++){
36			if(i <= MAX_NUMBER){
37				printf("%d ",i);
38			}
39		}
40		printf("\n");
41		number += my_answer;
42		if(number >= MAX_NUMBER){
43			win_flag = MY_LOSE;
44			break;
45		}
46		if(number == 17){ /*この部分はcpuの思考能力を上げるためのコード*/
47			cpu = MAX_ANSWER;
48		}
49		else if(number == 18){ 
50			cpu = 2;
51		}
52		else if(number == 19){
53			cpu = MIN_ANSWER;
54		}
55		else{
56			cpu = rand() % MAX_ANSWER + 1;
57		}
58		printf("cpuが言った数字:");
59		for(i = number + 1; i < number + 1 + cpu; i++){
60			if(i <= MAX_NUMBER){
61				printf("%d ",i);
62			}
63		}
64		printf("\n");
65		number += cpu;
66		if(number >= MAX_NUMBER){
67			win_flag = MY_WIN;
68			break;
69		}
70	}
71	if(win_flag == MY_WIN){
72		printf("おめでとう!!! あなたの勝ちです。yeah!!!\n");
73	}
74	else{
75		printf("cpuの勝ち 残念...\n");
76	}
77	
78	return 0;
79}