人狼作成:乱数で同じ数が出ないようにする

Question

前提・実現したいこと

人狼のシステムをC言語(C++などではなくただのC)で作りたい。

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

役職決めでランダムな1～4の値をrandを使って各一回ずつ出したいのですが、srandを使うと同じ数が出てしまい、srandなしでは毎回同じ値になってしまいます。

該当のソースコード

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

void Jinrou4();
void shuffle();

int main()
{
    int n;
    int role;
    puts("プレイする人数を入力してください。(4~8)");　　　　　　　/*ここは無視してください*/
    scanf("%d",&n);
    if(n==4)
    {
        Jinrou4();
    }else
    {
        puts("入力された値が正しくありません");
        
    }
}

void Jinrou4(int argc, char **argv)
{
    int player;
    puts("役職を決めます。");
    
    for ( int player = 0; player <= 3; player++)
    {
        
        printf("あなたはplayer%dです。\n",player+1);
        puts("Enterを押してください。");
        int values[] = { 1, 2, 3, 4 };
        int size = sizeof(values) / sizeof(int);
        shuffle(values, size);
        if (values[player]==1)
        {
            puts("あなたは人狼です\n");
            
        }else if(values[player]==2||3)
        {
            puts("あなたは市民です\n");
           
        }else if(values[player]==4)
        {
            puts("あなたは占い師です\n");
            
        }
        printf("確認したらENTERを押してください");
        if(getchar()=='\n'){
            printf("\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n");
        }
        
    }
}

void shuffle(int array[], int size)
{
    srand((unsigned) time(NULL));
    int i = size;
    while (i > 1) {
        int j = rand() % i;
        i--;
        int t = array[i];
        array[i] = array[j];
        array[j] = t;
    }
}

試したこと

ネットで調べ、Jinrou4の関数にsrandを入れるなどしてみましたが上手くいきませんでした。

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

vscodeとコマンドプロンプトを使っています。
初めての質問なので至らない点はご容赦ください。

Answer 1

     for ( int player = 0; player <= 3; player++)
      // ...
        int values[] = { 1, 2, 3, 4 };
        int size = sizeof(values) / sizeof(int);
        shuffle(values, size);

これでは、player毎に、valueの値が書き換えられるので、重複する可能性が大きいです。

        int values[] = { 1, 2, 3, 4 };
        int size = sizeof(values) / sizeof(int);
        shuffle(values, size);

を for() の前に出しましょう。

Comments

[shimi43]

回答ありがとうございます！
修正させていただきました。

Answer 2

srandは一旦置いておいて、次の部分が明らかにおかしいので修正してみて下さい。
ともすればこちらが真の問題かもしれません。

else if(values[player]==2||3)

↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓

else if(values[player]==2 || values[player]==3)

また、srandはmain関数の先頭の方で一度だけ呼び出すのが無難です。

Comments

[shimi43]

回答ありがとうございます
確かに明らかにおかしいですね…
ありがとうございます！

[shimi43]

ご指摘いただいた点を直しましたがやはり上手くいきませんでした。
srandを使ってしまうとやはり一度使った値を使わないことは難しいのでしょうか？

[LouiS0616]

valuesの生成及びシャッフルを、ループに突入する前に済ませてしまって下さい。

[shimi43]

返信ありがとうございます。
そのようにしたところうまく作動しました。
ありがとうございますm(_　_)ｍ

[LouiS0616]

解決したようで何よりです。
ところで、なぜシャッフルをループ前にしなければならなかったのか、理由はお分かりでしょうか。

Answer 3

srandを複数回呼び出さないでください。

Comments

[shimi43]

main関数の最初で呼び出すことにしてみます
回答ありがとうございます