C言語で無限ループをbreakすることができない

Question

C言語を勉強中に分からないことがあったので，質問させていただきます．

まず，以下のような実装を求められました．

「whileの無限ループを作り、その中で、１から１０までの乱数を発生させて、表示しなさい。その際、値が１０になったら、ループを抜け、”終了します”と表示してプログラムを終えるようにしなさい。」

これに対して私は次のようなコードを書きました．

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

int main(void){

int n;

while(1){
  srand((unsigned)time(NULL));
  n = rand()%10+1;
  printf("%d\n", n);
    if(n==10){
      printf("終了します\n");
      break;
      }
  }
}

しかし，このコードの実行結果は1から10のどれか１つがずっと表示され続ける無限ループになってしまいます．私の書いたコードのどこが間違っているのでしょうか？

Answer 1

srand関数をwhile文に入る前に書いたらどうでしょうか？

Answer 2

srand関数というのは、rand関数の初期化用の関数です
ですんでsrandを実行したあとのrandの返り値は常に一定となってしまいます
ということで、srandは、最初に一回だけ実行し、その後のループではrandだけを実行し続ければ、乱数（っぽい）数字列が生成されます

Answer 3

rand()は、バラバラな数字の羅列の先頭から順番に一つ値を取り出す関数です。
ビンゴゲームのカードの左上から順に数字を読み上げるようなイメージですね。

一方、srand()はビンゴゲームの例えで言うとカードを作る(カードに数字を並べる)ような役割です。
で、そのカードがどのような数字の並びになるかは引数に渡した値で決まります。
同じ値を渡すと同じ並びのカードができてしまいます。
普通はプログラム起動時にそのときの時刻(毎回違うはずですよね)を渡してプログラムの中で一度だけ呼び出せば良いです。
あとはrand()で先頭から順番に値を取り出して使えば良いですね。

ところが質問者さんのコードだと無限ループの中で何度もsrand()を呼び出すことになってしまい、且つそのループが超高速で回るため
srand()に同じ時刻を何度も渡すことになり、同じ数字の並びのビンゴカードを何度も作り直すようなことになっています。
rand()で同じビンゴカードの左上から値を取得すれば、そりゃ毎回同じ値になってしまいますよね。

---

細かいことを言うと上記のままというわけでは無いのですが、初学者にとってイメージしやすいかと思い私はビンゴカードの例えをよく用いています。

Answer 4

参考情報

• rand関数でランダムな値を生成する

http://c.zombie-hunting-club.com/entry/2017/11/04/134724
time, srand rand の使い方が説明されています。