C言語 ループがうまくいかない。

Question

前提・実現したいこと

C言語で数当てゲームを作成しています。
その際に
続けますか？（１：続ける　０：終了）：
の画面まで表示されるのですが１や０を入力してもプログラムが止まってしまいます。

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

３桁の数を当ててください！
1桁目の予想を０～９の数字で入力してください：3
2桁目の予想を０～９の数字で入力してください：4
3桁目の予想を０～９の数字で入力してください：5
0ヒット！　1ブロー！
続けますか？（１：続ける　０：終了）：1

該当のソースコード

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

typedef char String[1024];

int main(void)
{
   srand((unsigned)time(NULL));
   printf("***数当てゲーム（レベル２）***\n");
   printf("３桁の数を当ててください！\n");

   const int TIMES = 3;
   int answer [TIMES];
   int input [TIMES];

   for (int i = 0; i < TIMES; i++)
   {
      answer[i] = rand () % 10;
   }
   
   do {
      int hit = 0;
      int blow = 0;

   for (int j = 0; j < TIMES; j++)
   {
      printf("%d桁目の予想を０～９の数字で入力してください：", j + 1);
      String inputStr;
      scanf("%s", inputStr);
      input[j] = atoi(inputStr);
   }

   for (int i = 0; i < TIMES; i++)
   {
      if (answer[i] == input[i])
      {
         hit++;
      }
   for (int j = 0; j < TIMES; j++)
   {
      if (input[i] == answer[j] && i != j)
      {
         blow++;
      }
      
   }
   }
   
   
   printf("%dヒット！　%dブロー！\n", hit, blow);
   if (hit == 3)
   {
      printf("正解です\n");
      break;
   } else {
      printf("続けますか？（１：続ける　０：終了）：");
      String retryStr;
      scanf("%s, retryStr");
      if (atoi(retryStr) == 0)
      {
         printf("正解は%d%d%dでした！\n", answer[0], answer[1],answer[2]);
         break;
      }
      
   }
   }while (true);
   
   return 0;
}

試したこと

何度も見直したが、問題点がわかりませんでした。

補足情報（FW/ツールのバージョンなど）

VScodeで作成しています。
コンパイルはgccを使っています。

Comments

[K_3578]

CなんだったらC#タグは不要です。外しましょう。

[ryouk]

ありがとうございます。

[K_3578]

>ありがとうございます。
うん、お礼なんか良いのでタグ検索の邪魔になるから今からでも編集で外してください。

Answer 1

scanf("%s, retryStr");

scanf("%s", retryStr); じゃなくて?

Comments

[ryouk]

ありがとうございます！
無事解決できました。

Answer 2

元のソースコードをコンパイルすると何らかの警告が出ると思います。
手元のコンパイラ(gcc)では次のような警告が出ました。

hoge.c: In function ‘main’:
hoge.c:60:15: warning: format ‘%s’ expects a matching ‘char *’ argument [-Wformat=]
       scanf("%s, retryStr");

scanfの行で文字列を格納する変数を指定するのを忘れているのが原因だと思います。

--- hogeorg.c   2021-05-06 14:18:10.007693111 +0900
+++ hoge.c      2021-05-06 14:17:28.699615707 +0900
@@ -57,7 +57,7 @@
    } else {
       printf("続けますか？（１：続ける　０：終了）：");
       String retryStr;
-      scanf("%s, retryStr");
+      scanf("%s, retryStr", retryStr);
       if (atoi(retryStr) == 0)
       {
          printf("正解は%d%d%dでした！\n", answer[0], answer[1],answer[2]);

Answer 3

C言語のコードを書くなら、デバッグできる環境を整えましょう。
Eclipseや、WindowsならVisualStudioなど。
コードの任意の場所で実行を止め、変数のナカミを見ることができます。そこから１行づつ実行して、コードの流れを見れるようになります
そうすれば、アテズッポでコードを書かなくて済むようになります。

Comments

[ryouk]

ありがとうございます。
VScodeとはまた別にVisualStudioをダウンロードするということでしょうか？

[y_waiwai]

VScodeとGCCの組み合わせでもデバッグ環境は整えられます
また、フリーソフトで教育用のデバッグ環境をインストールできるやつもあったりします
その手の記事がたくさんあるので、試してみてはどうでしょう

[ryouk]

わかりました！
ありがとうございます。