for文とif文の使い方

Question

じゃんけんゲームのプログラムをfor文用いて作るという課題が出されました。分からないながら、このようにソースコードをかいてみたのですが、案の定作動しませんでした。
コンピューターの手は乱数により、ランダムに提示され、人間の手はキーボード入力するようになっていて、勝敗がつくまで、じゃんけんが続くようにしたいです。
キーボードから２が入力された場合のソースコードを添付させていただきました。ご指導よろしくお願いいたします。

#include <stdio.h>

int get_number(void)
{
	int jam = 0;
	jam = ((((int)rand() / (int)(RAND_MAX + 1))) * 3 )+1;
	return jam;
}
int main (void){
printf("Let's play rock,paper,scissors!\n");
		printf("Choose your hand from 1 to 3.\n");
		printf("1 rock\n");
		printf("2 paper\n");
		printf("3 scissors\n");
		int h = 0;
		scanf("%d\n", &h);
if (h = 2) {
			int res = 0;
			res = get_number();
			for (; res == 2;) {
				printf("You are even.\n");
				printf("Choose your hand from 1 to 3.\n");
				printf("1 rock\n");
				printf("2 paper\n");
				printf("3 scissors\n");
				int h = 0;
				scanf("%d", &h);
				break;
			}
			for (; res == 1;) {
				printf("You are win!!\n");
				break;
			}
			for (; res == 3;) {
				printf("You are loose!!\n");
				break;
			}
		}
return 0;
}

Answer 1

参考として、モックの1例を記載します。

整数以外の入力が無い前提であれば、大体、以下のようになるかと思います。

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

int get_number(void)
{
	int jam = rand()%3+1;
	return jam;
}

void echo(void) {
	printf("Choose your hand from 1 to 3.\n");
	printf("1 rock\n");
	printf("2 paper\n");
	printf("3 scissors\n");
}

int main (void) {
	int h, res;
	
	for(;;) {
		echo();
		scanf("%d", &h);
		
		if (!(h >= 1 && h <= 3)) {
			printf("1～3を入力してください。\n");
			continue;
		}
		
		// コンピュータの手をランダムに取得
		res = get_number();
		
		if (res == h) {
			// あいこの場合
			printf("You are even.\n");
		} else if (勝ちの条件(複数ある場合は、||で複数記載)) {
			// ユーザの勝ちの場合
			printf("You are win!!\n");
			break;
		}  else {
			// ユーザの負けの場合
			printf("You are loose!!\n");
			break;
		}
	}
	return 0;
}

1点、課題内容によりますが、ユーザビリティ的にコンピュータの手は出力しなくて良いのかな？と思いました。

Comments

[h_proc]

回答ありがとうございます。参考にして、もう一度書き直してみます。

[h_proc]

場合分けをfor 文を使って行うことはできないでしょうか。

[aikon_marimo]

課題の内容的にif文を使ってはいけない、ということでしょうか？
for文の条件式に書けば出来なくはないですが、わざわざfor文でやる意味はないです。

[h_proc]

ありがとうございます。

[Zuishin]

確かに for は使ってますけどこれでは点数が低いと思います。

正しくないと言っているわけではありません。正しい処理をしているかどうかまでは読んでいません。

九九の宿題を足し算で答えるようなもので、答えは合っているが課題の意図とはずれているということです。

for を使えという課題は、初期化・継続条件・継続処理の三つが正しく理解できているかどうかを見るためのものです。この回答ではそれを見ることができません。

[Zuishin]

また細かいことですが、無限ループを抜ける手段が無いので、return 0 に到達できません。
到達できない部分のあるコードは保守性を損ないます。コンパイラによっては警告が出ます。
無限ループを抜ける手段を用意するか、main の戻り値を void とするのが良いと思います。

上記は正しくありませんでした。ループを抜ける手段は用意されているので訂正します。

[Zuishin]

あと課題の目的は学習です。人にやらせて丸写ししたのでは無意味です。
宿題代行サイトへの規制が検討されている昨今、このような回答はあまり感心しません。

Answer 2

if (h = 2) {
は、hが定義されていないので、実行エラーではなく、コンパイルエラーになります。
定義されたとしたら
if (h = 2) {
ではなく
if (h == 2) {
が正しいはずです。
まず、ソースを提示する場合は、コンパイルが正常に完了したものを提示するようにしましょう。
(#include 文も含めて全て提示してください。回答者は提示されたソースを回答者の環境でそのままコピペして実行し、確認します。その為、そのままコピペして、コンパイルエラーにならないものを提示してください)
コンパイルエラーが取れない場合は、「コンパイルエラーが取れない」という質問にしてください。

Comments

[h_proc]

回答ありがとうございます。以後気を付けます。

[h_proc]

大変申し訳ありません。冒頭でｈを定義しても、作動しませんでした。（質問内容に冒頭のコードを追加しました。）もし可能でしたら、もう一度教えていただきたいと思います。よろしくお願い致します。

Answer 3

変数宣言とかしてないのでビルドできない気がしますがそれは一旦おいておいて

forを用いた課題ということでifは使ってはいけないことはないはずなのでforではなくifで条件判定しましょう。

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

この関数で相手の手を返してる(多分13を返す)と思いますが、rand()は0RAND_MAXまでの整数を返すはずなので、この計算式ではおかしいはずです。余りを利用すればできるはずです。

あいこ判定の間ループ処理をしてあいこでなければループを抜け勝ちかまけ判定を入れればよいかと思います。

Comments

[h_proc]

ありがとうございます。