計算が正しく行われない

Question

ちょっとした数値比較ゲームを作ったのですが、ある一行を入れると正しい結果が求められません。

下にコードを載せました。
最後にコンピュータのポイントの合計と自分のポイントの合計を比較し、勝った負けたを表示しようとしたらu_pointの数字が正しく出力されません。
具体的には、合計4251873点などと出てしまいます。

check(u_point, c_point)を無くすと答えが正しく出力されます。
なぜなのでしょうか？

最後に、細かい設定などを説明するのを省略しているので質問のマナーがなっていなかったら申し訳ありません。

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

void suit(void);	/* カードの柄を決める関数 */
int card_number(void);	/* カードの数字を決める関数 */
int point(int);		/* カードのポイントを決める関数 */
void wait_enter(void);	/* エンターキーが押されるまで待つ関数 */
void check(int, int);	/* どちらのポイントの合計が多いか判定する */

int main(void){
	int c_hand1, c_hand2, c_point; 	/* コンピュータの手とポイント合計（変数） */
	int u_hand1, u_hand2, u_point;	/* 自分の手とポイント合計（変数） */
	time_t timer = time(NULL);
	srand(timer);
	c_point = 0;

	printf("コンピュータのカードは");	/* 18行目から28行目までコンピュータの動作 */
	suit();
	c_hand1 = card_number();
	printf("%d　", c_hand1);
	c_point += point(c_hand1);

	suit();
	c_hand2 = card_number();
	printf("%d　", c_hand2);
	c_point += point(c_hand2);
	printf("合計%d点\n", c_point);

	wait_enter();				/* 30行目から43行目まで自分の動作 */
	printf("1枚目");
	suit();
	u_hand1 = card_number();
	printf("%d\n\n", u_hand1);
	u_point += point(u_hand1);

	wait_enter();
	printf("2枚目");
	suit();
	u_hand2 = card_number();
	printf("%d\n\n", u_hand2);
	u_point += point(u_hand2);
	printf("合計%d点\n", u_point);

	check(u_point, c_point);		/* ここでチェック 45行が無いと、43行目では正しい結果が出力される*/

	return 0;
}

void suit(void){
	int suit;
	suit = rand() % 4;

	switch(suit){
		case 0:
			printf("スペード");
			break;
		case 1:
			printf("ハート");
			break;
		case 2:
			printf("クローバー");
			break;
		case 3:
			printf("ダイヤ");
			break;
	}
}

int card_number(void){
	return (rand() % 13) + 1;
}

int point(int x){
	int point;

	if(x == 1)
		point = 11;
	else if(x >= 10)
		point = 10;
	else
		point = x;

	return point;
}

void wait_enter(void){
	char dummy;

	printf("カードを引くならエンター");
	scanf("%c", &dummy);
}

void check(int x, int y){
	if(x > y)
		printf("あなたの勝ちです！！\n");
	else if(x < y)
		printf("あなたの負けです・・・\n");
	else
		printf("引き分け\n");
}

Answer 1

上手くいかないときは大抵自分のミス。何度も何度もこれでもかと確認する

Comments

[LouiS0616]

ある程度アタリを付けて確認できるといいですね。
変な値が出るときは初期化ミス、オーバーフロー、アドレスを表示している... など。
私も最初はセミコロンの付け忘れで数時間悩んだことがあります。

Answer 2

u_pointが初期化されていないからだと思います。
私の環境では、45行目の有無に関係なくおかしな値が出ました。

int main(void){
    int c_hand1, c_hand2, c_point=0;     /* コンピュータの手とポイント合計（変数） */
    int u_hand1, u_hand2, u_point=0;    /* 自分の手とポイント合計（変数） */
    ...

Comments

[hiky]

仰る通りです。もの凄く初歩的なミスですね、、、素早い指摘ありがとうございます。自分でも何度も上から下まで見直したつもりなのですが恥ずかしい限りです。

あと、すいません。もう一つ質問なのですが。自分の環境ではu_pointの値を初期化しなくても45行目が無い場合計算がされました。
LouiSO616さんは45行目の有無にかかわらずとのことですが、これは単に用意している環境の設定の問題ってことで理解しても良いのでしょうか？

[LouiS0616]

値を初期化していない変数を使ったときの動作は不定です。
何十回も試してそうなるのなら何らかの因果はあるのかもしれませんが、基本的には「たまたま」と説明してしまうことが多いです。

[hiky]

理解しました。本当にありがとうございました。