数字当てゲームを作りたい

Question

数字当てゲームを作っています。コンピューターに０～９のランダムな数を一つ表示させ、利用者には数を予測してもらい、キーボードから入力してもらいます。キーボードからの入力値とコンピューターから表示される数字の差により、利用者にポイントを与えたり、引いたりして、持ち点が1000点または0点になったら終了させます。このようなプログラムを書きました。しかし、いつまでたっても持ち点が５０点のままです。一体どこが間違っているのでしょうか。教えていただけると嬉しいです。よろしくお願いいたします。

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

int rand_n(void);//これは乱数を表示する関数

int rand_n(void) {

	int m = 0;
	m = 0 + (int)(rand()*(9 - 0 + 1.0) / (1.0 + RAND_MAX));
	
	return m;

}
int guess(int n , int m);//入力された数字とコンピュータが出した数字の差を計算してくれる関数

int guess(int n, int m) {
	int s = 0;
	
	if (n > m) {
		s = n - m;
		
	}
	else {
		s = m - n;
		
	}
	return s;
}
int hold(int s, int f);

int hold(int s,int f) {//持ち点を計算してくれる関数
	    

		//for (int i = 0; f < 1000; i++) {
			
		if (s == 0) {
			f = f + 400;
		}
		else if (s == 1) {
			f = f + 300;
		}
		else if (s == 2) {
			f = f + 250;
		}
		else if (s == 3) {
			f = f + 150;
		}
		else if (s == 4) {
			f = f + 100;
		}
		else if (s == 5) {
			f = f + 50;
		}
		else if (s == 6) {
			f = f - 10;
		}
		else if (s == 7) {
			f = f - 20;
		}
		else if (s == 8) {
			f = f - 30;
		}
		else if (s == 9) {
			f = f - 50;
		}

		//printf("%d回目終了！あなたの持ち点は%d点です。\n", i+1, f);
	//}
	return f;
}


int main(void) {
	printf("GESS NUMBER\n");

	printf("You have 50 points\n");

	//キーボードからの入力を受け付ける
	int f = 50;
	for (int i = 0; f < 1000; i++) {
		int n = 0;
		printf("予測した数字（０～９）を入力してください。\n");

		scanf("%d", &n);

		rand_n();//コンピューターがえらんだ数字を表示する

		printf("コンピューターが選んだ数字は%d\n", rand_n());

		int t = 0;

		t = guess(n, rand_n());//入力値との差を計算する

		hold(t,f);//持ち点の計算をする。

		printf("%d回目終了！あなたの持ち点は%d点です。\n", i + 1, f);
	}
	printf("Yor point is 1000!\n");

	printf("Quit this game\n");

	return 0;
}

Comments

[cateye]

int f = 50;の直後にif (f == 0) {が有るのはなぜでしょう？・・・この判定は必ず偽になりますよね？
で、return 0 + (int)(rand()*(9 - 0 + 1.0) / (1.0 + RAND_MAX));
単純に０〜９を発生させるだけならrand()%10ではだめですか？
また、rand_n()を毎回呼んでると乱数の値が変わりますが、意図したことでしょうか？　rand_n() - gが負数になったらどうなりますか？

[jimbe]

> int rand_n(void);//これは乱数を表示する関数
> int rand_n(void) { （略)
関数を先に宣言致しますのは, コード上, 実際の定義の前にその関数を使用する別の関数が書かれている場合と記憶しています.
このように続けられるのは不要で, 宣言は削除されても問題無いかと思います.
また, rand_n 関数は '表示' (printf)を行っておりません.
コメントが実際と異なると, それが正しいかどうかを判断するため, 開発に向いていた思考が止まり易くなってしまいます.
間違ったコメントは削除し, コードがコメントとして機能するように, 変数名やメソッド名を決定頂くほうが宜しいかと思います.
例: s → score, guess() → difference()

> int guess(int n , int m);//入力された数字とコンピュータが出した数字の差を計算してくれる関数
n と m の差でしたら, Math.abs() メソッドが御座います.
int guess(int n, int m) { return Math.abs(n, m); } //入力された数字とコンピュータが出した数字の差を計算してくれる関数
で済み, コードの見通しが多少良くなるかと思います.

> int hold(int s,int f) {//持ち点を計算してくれる関数
s が 0～9 の範囲を取り, f を加減算されるのでしたら, 配列のご使用がシンプルかと思います.
int[] values = { 400,300,250,150,100,50,-10,-20,-30,-50 }
int hold(int s,int f) { return f + values[s]; } //持ち点を計算してくれる関数

なお, hold 関数は計算結果を **戻り値** として返しております. パラメータとして渡している f は, 呼び出し側では変化致しません. (これは f を '値渡し' しているためです. )
計算結果を f に入れ直されるのであれば,
f = hold(t,f);
とする必要がございます.

[cateye]

あらW・・・やっぱそうだよね(ﾟдﾟ)(｡_｡)(ﾟдﾟ)(｡_｡) ｳﾝｳﾝ
ちなみに、うちのコンパイラ(clang version 8.0.0)は、-Weverything有効にしてるんで・・・
t176063.c:12:5: warning: no previous prototype for function 'rand_n' [-Wmissing-prototypes]
int rand_n(void) { return rand() % 10; }
^
t176063.c:14:5: warning: no previous prototype for function 'guess' [-Wmissing-prototypes]
int guess(int f)
^
2 warnings generated.
「プロトタイプ何とかしろ」みたいな文句言うんで・・・
Math.abs() ？・・javaなので、私はint p = abs(rand_n() - g);にしています。

[h_proc]

ありがとうございます。コメントの付け方など、至らない点が多くあることが分かりました。皆さんのコメントを参考に修正したいと思います。

[jimbe]

ああ, 申し訳ありません. 言語を複数同時に使ってしまいまして, 混乱しておりました.

Answer 1

• testとdebugのやり方を習得しよう。

じゃないとトラブルのたんびに誰かにお伺いをたてねばならん。

• やりたいことを箇条書きあるいは疑似コードで表現せよ

  得点 = 50
  回数 = 0
　while ( 得点 > 0 && 得点 < 1000 ) {
    回数++
    問題 = 0～9 の乱数
    回答 = 入力値
    得点 += 問題 と 回答 の 差に基づく評価値
    回数 と 得点 を表示する
  }

んでもってコレ↑をCで表現せよ。

Comments

[cateye]

乱数の発生をループの外側でやるか内側でやるかよくわからない・・・
（一回ごとの点数が有るから内側説が有望かも）
あと、while ( 得点 > 0 || 得点 < 1000 ) {　→ while ( 得点 > 0 && 得点 < 1000 ) {

[episteme]

外でやるんなら「何回の入力でHitするか」が得点になると思うんよね。
...指摘ありがとです。条件のトコ、修正しときました。

Answer 2

BAが出たようなので・・・

#include <stdio.h>
#include <stdlib.h>
//
//0〜9の乱数を発生させる。
int rand_n(void);
//入力された数字とコンピュータが出した数字を比較して点数を返す。
int guess(int);
//
/*************************************
*************************************/
//
int rand_n(void) { return rand() % 10; }
//
int guess(int f)
{
    static int       cnt     = 1;
    static const int point[] = {
        // 10回ぐらいでプレイヤーが勝つ
//        400, 300, 250, 150, 100, 50, -10, -20, -30, -50
        // 30回ぐらいでプレイヤーが勝つ
        200, 100, 50, 25, 10, 0, -25, -50, -100, -200

    };
    int  g;
    char buf[32];

    printf("Stage%d: Yor point is %d\n", cnt, f);
    puts("数字を予測してください。");
    //
    fgets(buf, sizeof buf, stdin);
    sscanf(buf, "%d", &g);
    // 緊急脱出用＾＾；
    if (0 > g || g > 9) {
        puts("quit!");
        exit(1);
    }
    //
    printf("Selected number is %d\n\n", g);
    //
    int p = abs(rand_n() - g);  // abs()は結果が負になった時の対策
    // for DEBUG
    if (p == 0) {
        puts("equal");
    }
//    printf("%d\n", p);
    //
    f += point[p];
    //
    cnt++;
    //
    return f;
}

int main(void)
{
    puts("guess NUMBER");

    puts("０～９の数字がランダムに選択されるので、\n"
         "選択された数字を当ててください。\n");

    puts("You have 50 points\n");

    int f = 50;
    do{
        f = guess(f);
    }while (f > 0 && f < 1000);

    //
    if (f <= 0) {
        puts("your loss ;-p");
    } else {
        puts("Your wins v^^");
    }
    //
    puts("Quit this game!");

    return 0;
}

　一応動作確認はしていますが、分かるでしょうか？
何かいい変数名がないかちょっと考えたのですが、このままとします。

Comments

[cateye]

ちなみに得点についてはjimbeさんの言うようにテーブルにしています。“連続した数値でデータを引用をする”という時はテーブルに出来ないか？とか考えましょうｄ＾＾　（まぁ、今回は０〜９と範囲が狭いのでswitchもありかｗ）　また、私がコメントに書いた負の数になった時どうするか？は解決しましたか？・・・ヒントはjimbeさんが書いてくれています。みなさんの書いてくれたことを精査して必要な情報を引き出しましょう　ｄ＾＾

[cateye]

長くなるので最後だけ・・・
Stage6: Yor point is 135
数字を予測してください。
1
Selected number is 1

Stage7: Yor point is 145
数字を予測してください。
11
quit!
「緊急脱出」は、ゲームが終わらないかやめたい時用

Answer 3

まず乱数とは何か知るべきです。

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


int rand_n(void) {
    return 0 + (int)(rand()*(9 - 0 + 1.0) / (1.0 + RAND_MAX));
}

int main(void) {
    printf("%d\n", rand_n());
    printf("%d\n", rand_n());
    printf("%d\n", rand_n());
    
    return 0;
}

実行結果 Wandbox

8
3
7

rand_nを呼び出す度、毎回違う値が返ってきます。
ご提示のプログラムについても同じで、途中で何回も答えが変わっています。

ゲームとして成立させるなら、
一回だけrand_nを呼び出し、その返り値を答えとして変数に格納しておくべきです。

しかし、キーボードからの入力値とコンピューターから表示される数字の差が正しく計算されておらず、試しに表示させた（31行目）ところ、-18456と表示されてしまいました。

printf("%d\n", g - rand());

前述のとおり乱数の値は毎回異なりますし、
randはrand_nよりもっと広い範囲の整数を返しますから、何ら不思議では無いです。

その他、怪しい箇所

上から順に列挙します。
0. グローバル変数のfとローカル変数のfが共存している。
0. 不要なプロトタイプ宣言が並んでいる。
0. rand_nの実装について、なぜ0を加算しているのか。
0. gess ⇒ guess
0. fを初期化した直後に値を判定しても意味が無い。
0. for (int i = 0; f == 100; i++) 継続条件が仕様に反する。

ゲームを作るのは確かに楽しいですが、もうちょっと簡単なものから始めてはいかがですか。
特にループの挙動について入念な復習が必要です。

ゲーム中、値の入力を一回しか受け付けていないことに気付いているでしょうか。

質問編集を受けて

rand_nを呼び出す度に答えが変わっています。
一回だけrand_nを呼び出し、その返り値を答えとして変数に格納しておくべきです。

Comments

[h_proc]

ありがとうございます。もう一度考え直してみます。