for文が重くて実行されない

Question

前提・実現したいこと

確率を計算するためにfor文を使ったコードを書いたのですが、重くて実行されないのでどこを改善すればいいのかを教えていただきたいです。

10回で試してみたら動いたので、エラーはないかと思うのですがコード自体におかしな点があったらすみません。

前提
コンピュータがランダムに生成した数字を当てるプログラムです。
user_array にはユーザーが選んだ09の数字が9個、配列で格納されています。
コンピュータは重複ありで09の数字を生成。
試行回数1000回を想定。

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

1000回の繰り返し処理が実行されない。

該当のソースコード

var traials_num = 1000;

  var win_count = {straight: 0, box: 0, lose: 0};
  var com_array = [];
  var num = 10;
  var length = 9;

  for(var i = 0; i < traials_num; i++){

    for (var i = 0; i <= length; i++){
      com_array.push(Math.floor(Math.random() * num));
    }

    if(user_array.every((u,i) => u == com_array[i])){
      win_count.straight++;
    }else if(user_array.sort().join() == com_array.sort().join()){
      win_count.box++;
    }else{
      win_count.lose++;
    }
  }
  straight.innerHTML = '大当たり:' + (win_count.straight / traials_num * 100).toFixed(2) + '%';
  box.innerHTML = '小当たり:' + (win_count.box / traials_num * 100).toFixed(2) + '%';
  lose.innerHTML = '残念:' + (win_count.lose / traials_num * 100).toFixed(2)  +'%';
  

Comments

[maisumakun]

user_arrayの定義はどこでしょうか？

[退会済みユーザー]

そちらはこの部分より上で定義しており、console.logで正常に表示されていたので省略してしまいました。

他の方の回答で解決しました。

質問を見てくださり、ありがとうございました。

Answer 1

forの入れ子において、同じ変数名iを、varで定義しているからです。
つまり、親のforのiを、子のforで、また0に初期化してしまっているため、
単純に、1000のlength（今回の場合は9）乗回計算を行っている可能性があります。
（10のlength（今回の場合は9）乗は1,000,000,000回なので、まだ計算できるレベル）

親と子で、変数名を変えるか、
letで初期化するようにしましょう。
（letならば、for文に関しては、同名であっても、スコープは被らないため、違う変数として初期化される）

Comments

[退会済みユーザー]

ご丁寧にありがとうございます。

[xail2222]

細かいところすみません。ちょっと気になったのだけど
10*10で100だけど
10の10乗は10000000000では？

[miyabi_pudding]

気づいて修正しました　汗
xail2222さん、ご指摘ありがとうございます。

[xail2222]

んー。length+1回をtraials_num回回すのだから、traials_num*10で10000回でいいんでないですか？
なぜ乗？

あー。間違った初めの処理だとその回数計算しているってことか、わかりました。すみません

[xail2222]

何か気になってたので、確かめてみたのですが

for (var i = 0; i <= length; i++){
com_array.push(Math.floor(Math.random() * num));
}

の処理を実行した時点でi=length+1=10になるので
traials_num=10の場合、i < traials_numを一回で満たさなくなり10回のループとなり
traials_numが12以上の場合、なんどやってもi=10となって無限ループになるのではないでしょうか。

解決している問題なので、どうでもいいことかもしれないですが。。。

[miyabi_pudding]

無限ループっぽいですね。
質問者さん曰く、10の設定だと、処理が完了できる、という話だったので、
無限ループではないと踏んで回答いたしました。
とかく、質問の構文では、処理回数が極大になるので、
変数名や初期化の宣言子を変えねばならないという、根本問題は変わりません。

[xail2222]

はい。問題の本質は変わらないですね。気になってしまったのでコメントしてしまいました。すみません。
返信ありがとうございました。

Answer 2

for(var i = 0; i < traials_num; i++){

    for (var i = 0; i <= length; i++){

二重ループで同じ変数を使っています。これではダメです。

Comments

[退会済みユーザー]

ありがとうございます。

入れ子を初めて使ったのでしっかり勉強し直します。