C言語初心者です。 ジャンケンゲーム

質問

今、授業の課題でジャンケンゲームを作成していますが、なかなかできません。もし、良かったら何処が違うのか教えてください。

// main.c
#include <stdio.h>
#include <time.h>
//#include <time.h>
#include "my.h"
main(int argc, char* argv[]){
int i;
char s[7];
printf("これはジャンケンゲームです。\n");
printf("手を入力します(グー：１　チョキ：２　パー：３)\n");
scanf("%s",&s[7]);
srand((unsigned)time(NULL));
printf("%s\n",jyan(dice(3)));
if(s == jyan){
printf("あいこです。\n");
}else if(s == 1 && jyan ==2){
printf("あなたの勝ちです。\n");
}else if(s == 2 && jyan == 3){
printf("あなたの勝ちです。\n");
}else if(s == 3 && jyan ==1){
printf("あなたの勝ちです。\n");
}else{
printf("コンピュターの勝ちです。\n");
}

}

// dice.c
#include <time.h>
int dice(int n){
srand((unsigned)time(NULL));
return(rand()%n+1);

}

// jyan.c
char *jyan(int n){
static char s[][7]={"グー","チョキ","パー"};
return s[--n];

}

// my.h
#include <stdio.h>
char *jyan(int hand);
//int dice(int n);

見づらくて申し訳ありません。４つのファイルに分けて作成しています。上記のプログラムだと自分の手、コンピューターの手がランダムに出てくるのですが判定が出来ませんでした。アドバイス、よろしくお願いします。

長文になってしまい申し訳ありません。

回答No.2

BLUEPIXY · ベストアンサー

scanf("%s",&s[7]);
が、だめです。
文字を入力させるつもりなら
scanf("%s",s);
で
&s[7]だと確保した領域外になってしまいます。
＞if(s == jyan){
＞printf("あいこです。\n");
＞}else if(s == 1 && jyan ==2){
から入力とjyanの返す値は数値を想定していますが、
数値ではありません。

回答No.5

suseimei

最初の方も回答していらっしゃいますが、このレベルの課題は自分の力で乗り切るべきです。はっきり申し上げます。基本ができていません。きちんと授業に出ていましたか？

次のURLから公開されている「なぜプログラミングを学ぶのか」に目を通してください。明確な目的がないと、後が続きませんよ。

http://www.ttoyota.com/php/cppnovicecontents.php

あなたはあなたなりに努力しています。それは認めます。しかし、ソースコードを拝見する限り、毎回授業に出席していたとは思えません。基礎をしっかり学習してください。厳しいコメントですが、あなたの成長を心から期待しています。

頑張って！

回答No.4

m_mik

> if(s == jyan){
この部分の比較はいつでも成立しない状態となります。

sは入力されたデータを格納する（であろう）領域のアドレスを指しているのに対して、jyanはcharのポインタを返す関数のアドレスとなっています。
この2つのアドレスが一致することはありえません。

ここでは入力されたデータとランダムに選択されたデータとの比較を行う場所ですから、dice(3)の結果を保存しておく領域を用意して、その内容との比較を行うようにすれば、正しく処理を行えます。
ただし、dice(3)はint型であるのに対して、sはcharの配列ですので、両方の型を一致させることに注意してください。（キャストすればよいとかの話ではありません）

回答No.3

mac_res

#include <stdio.h>
#include <time.h>
#include <stdlib.h>
int
main(void)
{
char buf[BUFSIZ];
int j, you, i;
const char *jan[] = { "グー", "チョキ", "パー" };
const char *res[] = { "あなたの勝ちです。", "コンピュターの勝ちです。",
"あいこです。"
};
srand((unsigned) time(NULL));
printf("これはジャンケンゲームです。\n");
while (1) {
i = 1 + (int) (3.0 * rand() / (RAND_MAX + 1.0));
printf("\n手を入力します(グー：1　チョキ：2　パー：3) > ");
if (fgets(buf, BUFSIZ, stdin) == NULL) {
break;
}
you = atoi(buf);
if (you < 1 || you > 3) {
printf("1～3を入力してください。\n");
continue;
}
if (you == i) {
j = 2;
}
else {
switch (you) {
case 1:
if (i == 2) j = 0;
else j = 1;
break;
case 2:
if (i == 3) j = 0;
else j = 1;
break;
case 3:
if (i == 1) j = 0;
else j = 1;
break;
}
}
printf("あなたは%s。コンピュータは%s。%s\n",
jan[you - 1], jan[i - 1], res[j]);
}
return 0;
}

回答No.1

driverII

授業の課題なら、回答するのはあなたの為にならないのですが・・・

関数jyan の戻り値はchar*
比較しているのは?