掛け金をするゲームを作りたい

Question

c言語でじゃんけんゲームを作っています。最初に５００円所持していて、じゃんけんに勝つと３００円もらえて、負けると２００円払い、所持金が１０００円または０円になるまで続けるというルールにしたいです。以下のようなコードを書いたのですが、勝つ度、負ける度に、所持金が変化していきません。対処法を教えていただきたいです。（可能でしたら、どこが間違っているかも指摘していただきたいです。）よろしくお願いいたします。

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

int get_number(void);
int money(int a);

int get_number(void) {
	return (int)(((((double)rand()) / (double)(RAND_MAX + 1)) * 3) + 1);
}

int money(int a) {
	
	
	if (a = 1) {
		int m = 0;
			return m = m + 300;
		
	}
	else if (a = 2) {
		int m = 0;
			return m = m - 200;
		

	}
	else if (a = 0) {
		return 500;
	}
	
	
}
int main(void) {
	printf("Let's play rock,paper,sissors!!\n");
	printf("This is the rule.\n");
	printf("You have 500 yen.\n");
	printf("You have to pay 200 yen for one play.\n");
	printf("If you loose, you lost 200yen.\n");
	printf("If you win, you can get 300 yen.\n");
	printf("If you have no money or you get 1,000 yen, this game is end.\n");
	printf("Let's begin!\n");
	
	printf("You have 500 yen.\n");
	int num = 0;
	//所持金が０円または１０００円になるまで繰り返します。
	for (int i = 1; 0 < money(num) < 1000; i++) {
		printf("Please choose your hand from 1 to 3\n");
		printf("1 rock\n");
		printf("2 paper\n");
		printf("3 sissors\n");

		int h = 0;
		scanf("%d", &h);
	//もしもユーザーが１～３以外を選んでしまった時の対応。
		if (h < 1 || h>3) {
			printf("Error! Please choose from 1 to 3\n");
		}
	//正常に１～３をえらんだ時のゲームの進み。
		else {
			int com = 0;
			com = get_number();
			//あいこ。
			for (; com == h;) {
				printf("You are Even!!\n");
				printf("Please choose your hand from 1 to 3\n");
				printf("1 rock\n");
				printf("2 paper\n");
				printf("3 sissors\n");

				int h = 0;
				scanf("%d", &h);
				//勝敗がつく。
				int com = 0;
				com = get_number();
				if (com != h) {
					if (h == 1 && com == 2) {
						printf("You are loose!\n");
					}
					else if (h == 1 && com == 3) {
						printf("You are win!!\n");
					}
					else if (h == 2 && com == 1) {
						printf("You are win!!\n");
					}
					else if (h == 2 && com == 3) {
						printf("You are loose!\n");
					}
					else if (h == 3 && com == 1) {
						printf("You are loose!\n");
					}
					else if (h == 3 && com == 2) {
						printf("You are win!!\n");
					}
					break;
				}
				else {
					//NR
				}
			}

			if (h == 1 && com == 2) {
				printf("You are loose!\n");
			}
			else if (h == 1 && com == 3) {
				printf("You are win!!\n");
			}
			else if (h == 2 && com == 1) {
				printf("You are win!!\n");
			}
			else if (h == 2 && com == 3) {
				printf("You are loose!\n");
			}
			else if (h == 3 && com == 1) {
				printf("You are loose!\n");
			}
			else if (h == 3 && com == 2) {
				printf("You are win!!\n");
			}

		}
		//ゲーム終了後の所持金の計算。
		printf("If you win,please input 1.And if you loose, please input 2.\n");
		int num = 0;
		scanf("%d", &num);
		printf("As of %d stage end, you have %d yen.\n", i, money(num));
		if ( money(num) > 0 && money(num) < 1000) {
			printf("Continue your game.\n");
			
	     }
	    else {
			printf("GAME IS END! See You!\n");
			break;
	     }
	}

	return 0;
}

Answer 1

money関数

if (a = 1) {

    int m = 0;
        return m = m + 300;
}

これではa が１の場合のこの関数の戻り値は常に３００になってしまいますね

1. 所持金を表す変数をグローバル変数で定義する
2. プログラムの最初で、所持金変数に５００を入れる
3. 勝負ごとに所持金変数をどうにかする

というとこらへんでプログラムを組み直してみましょう

Comments

[h_proc]

回答ありがとうございます。
int money = 500;

int get_number(void) {
return (int)(((((double)rand()) / (double)(RAND_MAX + 1)) * 3) + 1);
}

int get_money(int a) {

if (a == 1) {
return money == money + 300;

}
else if (a == 2) {
return money == money - 200;
}
else if (a == 0) {
return 500;
}

}
のようにコードを書き直したのですが、+と-部分に左オペラントには型intが指定されています。というエラーが表示されてしまいます。グローバル関数の定義の仕方が間違っているのでしょうか。

[y_waiwai]

> return money == money + 300;

== は比較演算子です
ここは = 代入の方でしょう

Windows使ってるなら、VisualStudioを使えば、Cのコードを途中で止めて変数の内容を見たり１行づつ実行させたりとかして動作が見れます。
そゆのを入れてやってみればどうでしょう

[h_proc]

ありがとうございます。やってみたいと思います。

Answer 2

money関数のif文の条件が代入=になっていますが等値比較==ではないでしょうか

    if (a = 1) {  //a == 1では?
        int m = 0;
            return m = m + 300;

    }
    else if (a = 2) {//a == 2では?
        int m = 0;
            return m = m - 200;


    }
    else if (a = 0) { //a == 0では?
        return 500;
    }

また同じ変数の値複数回チェックする場合、switch文の方がよいような気がします。ご検討ください。