このプログラムを関数で小分けしたいんですが・・・

質問

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

int main(void)

{
int n;
int player; // プレイヤーの手
int enemy; // 相手の手

srand(0); // 乱数列を初期化

do
{
puts("じゃん　けん (グー:1 チョキ:2 パー:3) ");
scanf("%d", &player);

//相手の手を計算
enemy = rand() % 3 + 1;

// 勝ち負けを判定
if(enemy == player)
{
printf("あいこ～\n");
}
else if(player == 1 && enemy == 2)
{
printf("Win!\n");
}
else if(player == 2 && enemy == 3)
{
printf("Win!\n");
}
else if(player == 3 && enemy == 1)
{
printf("Win!\n");
}
else
{
printf("Lose・・・\n");
}

printf("0を押したら終了。続けるなら適当な数字");
scanf("%d", &n);
// n が 0 以外だったら続ける
}while(n != 0);

return 0;
}

このように、じゃんけんをするプログラムを作ったのですが
メインだけでは出来たのですが、相手の手の判定や勝ち負けの判定などを
上手く関数でコンパクトにまとめたいんですが、
関数を殆ど使った事が無いので良く分かりません。
このプログラムを上手くコンパクトに関数に分けてくれませんか？

回答No.2

ssk38 · ベストアンサー

こんなもんでしょうか。
勝ち負け判定（の表示以外）と相手の手決定を関数化しました。
そうするとsrandが浮いてしまったので、相手の手のところに一緒にして、
はじめだけ呼ばれるようにしました。

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

int aitenote() {
static int initialized = 0;
if (initialized == 0) {
srand(0); // 乱数列を初期化
initialized = 1;
}
return rand() % 3 + 1;
}

int katimake(int player, int enemy) {

if(enemy == player)
{
return 0;
}
else if((player == 1 && enemy == 2)
|| (player == 2 && enemy == 3)
|| (player == 3 && enemy == 1))
{
return 1;
}
else
{
return -1;
}

}

int main(void)

{
int n;
int player; // プレイヤーの手
int enemy; // 相手の手

do
{
puts("じゃん　けん (グー:1 チョキ:2 パー:3) ");
scanf("%d", &player);

//相手の手を計算
enemy = aitenote();

// 勝ち負けを判定
switch (katimake(player, enemy)) {
case 0:printf("あいこ～\n");break;
case 1:printf("Win!\n");break;
case -1:printf("Lose・・・\n");break;
}

printf("0を押したら終了。続けるなら適当な数字");
scanf("%d", &n);

// n が 0 以外だったら続ける
}while(n != 0);

return 0;
}

お礼

ありがとうございます。
分かりやすくまとめて頂き、助かりました。

回答No.1

Tacosan

このプログラムだけでも
srand, puts, scanf, rand, printf
と 5つも関数を使っているのに「関数を殆ど使ったことがない」とはこれいかに.

「関数を殆ど作ったことがない」と読み替えたとしても, そうだとしたらそもそも C でプログラムが書けない.

「よくわかりません」というのも何がどう「よくわからない」のかわからないんだけどね. 「よくわからない」だから, だいたいは理解してるんでしょ? だとしたら, 理解できているところと理解できていないところをちゃんと書いた方が答えやすいよ.