ガチャのシミュレーターを作りたいが乱数に偏りがある

Question

###前提・実現したいこと
ソシャゲのガチャのシミュレーターを作りたいが乱数の生成がうまくいかない。
うまく乱数を生成するためにはどうしたらいいのでしょうか。

###発生している問題・エラーメッセージ
確率1.5%で当たる抽選を

rand()%200+1;

で乱数を生成させ、乱数が198~200の範囲ならばあたるようにすることで作ったが、1000万回ほど回してもあたりを引かない。

###該当のソースコード

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

#define PN 10000000

int c[PN+1],i;

void main(){

srand(time(NULL));

for(i=1;i<=PN;i++){
c[i]=rand()%200+1;
}

for(i=1;i<=PN;i++){
if(197<c[i]<=200){

printf("atari \n");

}
}

}

Comments

[ozwk]

「rand()%200+1;で乱数を生成させ」...記載のコードは0が2つほど多くなってますが?

[ttyp03]

c[CN+1] … CNなんて定義はありませんが？

[ozwk]

あと色々試行錯誤したのはわかりますが、「1000万回ほど回してもあたりを引かない。 」コード、つまり動くコードを貼ってもらえますか?質問の内容とコードをあわせてください。

Answer 1

こんにちは。

書いている間に解決しちゃいましたが、ツッコミどころ満載ですね。
ちょっと厳しいと思いますが、突っ込んでみました。ごめんなさい。

1. インデントはきちんと付けましょう。
2. int c[CN+1];のCNは未定義です。PNの間違いでしょう。
3. main()はint型を返却する必要があります。
4. iが未定義です。
5. 設計はrand()%200+1なのにコードはrand()%20000+1となってます。
6. if(197<c[i]<=200)は常に成立してしまいます。「あたりを引かない」と言う報告と矛盾してます。
7. if(197<c[i]<=200)はif((197<c[i]) && (c[i]<=200))と書くのが正解と思います。
8. 0スタートに慣れましょう。本質的でない+1を減らせます。

修正してみました。

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

#define PN 10000000

int c[PN];

int main()
{
    srand(time(NULL));
    for(int i=0;i<PN;i++)
    {
        c[i]=rand()%200;
    }

    for(int i=0;i<PN;i++)
    {
        if((197<=c[i]) && (c[i]<200))
        {
            printf("atari\n");
        }
    }

    return 0;
}

でも、'''で囲む初心者さんが少ない中、きちんと'''で囲んでくれているのはたいへんありがたいです。
頑張って下さい。

Answer 2

実際に動かしてませんが、たぶんif文の書き方じゃないでしょうか。

if(197<c[i]<=200){
↓
if(197 < c[i] && c[i] <= 200){