じゃんけんゲームのif文を簡潔にしたい

Question

前提・実現したいこと

C言語でコンピューターとプレーヤーでじゃんけんをするプログラムを作りました。
０がグーで、１がチョキ、２がパーです。
配列は使わないで、コンピュータの手は乱数で決めて、プレイヤーはキーボードから手を入力します。

発生している問題・エラーメッセージ

きちんと動くプログラムはできたのですが、if文が冗長であるような気がしています。
もう少し簡潔に書ければそうしたいのですが、あまりいい案が思いつきません。
何かもっと簡潔に書くことができるアドバイスなどあればいただきたいです。

該当のソースコード

printf("プレイヤーは");

if(player==0){
 printf("グー");
}
else if(player==1){
 printf("チョキ");
}
else{
 printf("パー");
}

printf("で、コンピューターは");

if(comp==0){
 printf("グー");
}
else if(comp==1){
 printf("チョキ");
}
else{
 printf("パー");
}

judge=(player-comp+3)%3;

if(judge==0){
 printf("なので、引き分け\n");
}
else if(judge==1){
 printf("なので、コンピューターの勝ち\n");
}
else{
printf("なので、プレイヤーの勝ち\n");
}

Comments

[Zuishin]

配列を使えばいいのでは？
printf("%s\n", hand[player]);

[fana]

> 配列は使わないで
のような「縛りプレイの範囲で」簡潔にしたい，という話？

[mn6262]

はい！そうです。

[HogeAnimalLover]

縛りプレイするならば別に今のままで良いと思うけど。

[Zuishin]

また嫌がらせのクイズか。

[Zuishin]

> C言語でコンピューターとプレーヤーでじゃんけんをするプログラムを作りました。

作ってないでしょ。宿題でしょ。自分でやれ。

[mn6262]

不快に思われたのでしたら、申し訳ありませんでした。
ですが、自分でプログラムを作ったのは本当です。
作ったうえで、もっといいものにすることができるのならしたいと思って質問をさせていただきました。

[Zuishin]

自分で作るのにそんな変な縛りを入れる必要は無いし、

> 回答ありがとうございます。
はい…。本当に面倒なのですが、配列禁止とのことでして（泣）

他人から禁止されるいわれもない。

[fana]

頭空っぽにして，9パターン分岐のswitchを書くというのがある意味「簡潔」かもしれない．

Answer 1

えーと、if～else if～else～ を何度も書きたくないってことかな?

#include <stdio.h>

const char* hand(int type) {
  if ( type == 0) {
    return "グー";
  } else if ( type == 1) {
    return "チョキ";
  } else {
    return "パー";
  }
}

const char* judge(int player, int comp) {
  int judge = (player - comp + 3) % 3;
  if (judge == 0) {
    return "引き分け";
  } else if (judge == 1) {
    return "コンピューターの勝ち";
  } else {
    return "プレイヤーの勝ち";
  }
}

/*
   おためし
 */
int main() {
  int player;
  int comp;
  for ( player = 0; player < 3; ++player ) {
    for ( comp = 0; comp < 3; ++comp ) {
      printf("プレイヤーは%sで、コンピューターは%sなので、%s\n",
             hand(player), hand(comp), judge(player,comp));
    }
  }
  return 0;
}

Comments

[mn6262]

回答ありがとうございました！
わざわざコードまで書いていただきとても参考になりました。
今回はポインタ等の使用も制限されているので、使うことが出来ませんが、もう少し勉強して使えるようになった時の参考にさせていただこうと思います。

Answer 2

配列使わない縛りらしいので……

• ｛グー，チョキ，パー｝の表示は２回やってる→関数にでもしたらどうか
• ifは三項演算子にでもしたら見た目には簡潔と言えるかも？（判断は人に依るか）

くらいかな．

Comments

[mn6262]

回答ありがとうございます。
いただいたアドバイスを参考にして改善してみようと思います。

[fana]

「配列使わない縛り」に対抗するために，
メンバのアライメントが確定する形で書いた構造体を用いてアドレスオフセットでどうの！
とかも考えられるが，それ「簡潔」か？　と言われると違う気がする．

[mn6262]

まだプログラミングを勉強し始めたばかりで初歩的なことしか出来ませんが、いつかそういった対抗策が思いつくように努力していこうと思います！
ありがとうございました！

Answer 3

どこの if 文が冗長と考えているのか分かりませんが、

  char* hands[] = {
    "グー",
    "チョキ",
    "パー"
  };

printf("%s", hands[player]);

とか？

って、配列使わないのか……

ならば無理矢理だが（ｗ

#include <stdio.h>
#cinlude <string.h>

char* hands = "グー　チョキパー　";  // グーとチョキの間、パーの後ろには全角空白
char buffer[10];

memset(buffer, '\0', 10);
strncpy(buffer, hands+(player * strlen("チョキ")), strlen("チョキ"));
printf("%s", buffer);

いわゆる文字列を使えない、ならこれでもアウトですが一応（ｗ

Comments

[mn6262]

回答ありがとうございます。
はい…。本当に面倒なのですが、配列禁止とのことでして（泣）

[mn6262]

何度もありがとうございました！
とても参考になりました！