重複しない乱数の記述がうまく動作しません

Question

let randoms = [];
let min = 1,max = 10;

let getRandom(min,max){
   return Math.floor(Math.random()*(max - min +1)+min);
}

for (i = min; i <= max ; i++){
 while(true){
  let tmp = getRandom(min,max);
  if(!randoms.includes(tmp)){
   randoms.push(tmp);
   break;
   }
 }
}

ネットの記事や本を見て1から10までの乱数を発生させる記述を作りましたが、実行すると　Expected";"と表示されてしまいます。
もう一歩のような感じがしますが、;は不足していないように思うのですが、どこがいけないのかご教示いただけないでしょうか。よろしくお願いします。

Comments

[m.ts10806]

自身で自身のコードを解説できますか？

[miyabi_pudding]

過去質問を併せて鑑みて、
まだ基本も理解できてないとお見受けします。
基本を理解されてない質問者さんが、
while(true)なんて使っちゃいけません。（JSじゃなくても、Javaではよく使われるらしいが、それでもwhileの性質を理解しないで使うのは危険）
含め、自身のコードをしっかり理解しましょう。

[yutampp]

そのう・・・質問とは外れるんですけど。
そのロジックだと、範囲が狭いうちはいいものの、１～１０００とか大きい範囲の乱数を作ろうとすると、時間がかかりすぎると思います。
それは１０００個の穴を作って、その穴にボールが全部キレイに収まるまで、目隠しをしながらボールを投げ続けるようなものです。
その拷問じみた遊びは、終わりに近づくに連れ難易度が増すというのは分かっていただけるでしょうか。
質問から離れてしまって申し訳ないのですが、ロジックを考え直すことをおすすめします。

[miyabi_pudding]

yutamppさん>
例えがめちゃくちゃしっくりきました・・・。
下手したら無限に等しい処理数になる可能性がありますよね。

[hisayan]

皆様のご指摘の通りで、以下の記述あたりは
while(true){
let tmp = getRandom(min,max);
if(!randoms.includes(tmp)){
randoms.push(tmp);
大体こんな意味かなという感じでとらえていました。
よって今回の記述はあきらめて出直します。
ありがとうございました。

[m.ts10806]

変数と関数あたりの基本からやりなおしたほうが良いかと。やりたいことだけを追求しても書いた通りにしか動かないので基礎がごっそり抜けてる状態で動かないのは当然です(たぶんこれ初めての指摘じゃないはずなんだが)

Answer 1

const randoms = Array(10).fill(null).map((x,y)=>[Math.random(),y+1]).sort((x,y)=>x[0]-y[0]).map(x=>x[1]);
console.log(randoms);

Answer 2

let randoms = [];
let min = 1,max = 10;

function getRandom(min,max){
    return Math.floor(Math.random()*(max - min +1)+min);
}

for (i = min; i <= max ; i++){
    while(true){
        let tmp = getRandom(min,max);
        if(!randoms.includes(tmp)){
            randoms.push(tmp);
            break;
        }
    }
}
console.log(randoms);

出力結果

[ 8, 2, 7, 5, 3, 1, 6, 4, 10, 9 ]

let → function に変えただけですが一応動くっぽいですね

---

関数式を使いたいならこうでしょうか？

let randoms = [];
let min = 1,max = 10;

let getRandom = function(min,max){
    return Math.floor(Math.random()*(max - min +1)+min);
}

for (i = min; i <= max ; i++){
    while(true){
        let tmp = getRandom(min,max);
        if(!randoms.includes(tmp)){
            randoms.push(tmp);
            break;
        }
    }
}
console.log(randoms);

出力結果

[ 9, 10, 8, 2, 5, 7, 4, 1, 3, 6 ]

Comments

[m.ts10806]

インデントもこっそりキレイになってるところに全米が

[azuapricot]

（見づらかったので Shift + Tab と Tab を使って適当に直しただけ...)

[miyabi_pudding]

鑑ですね。

[hisayan]

自分にはまだ難しすぎましたが、何となく意味は分かります。ありがとうございました。

[m.ts10806]

ベストアンサーに選ばれた回答のコードは上級者向けで私もある程度調べないと理解できないコードですが、「難しすぎる」わけではないんですか？
あれが理解できるのでしたら今回の質問は出てないと思います。
「要件通り動いたからOK。なんかすごいコードもらったからOK」で済ませてません？毎回。

Answer 3

なぜ、動かないのか、それは let の使い方に誤りがあるから
リファレンス等、ちゃんと確認してください

let 文はブロックスコープの局所変数を宣言します

//誤り
let getRandom(min,max){

よく理解していないけど、最新の記述式でかっこよく書きたいというなら

let getRandom = ( min, max ) => {