javascriptでランダムな数値を出力する方法

ボタンを押すと13～18のランダムな数字を出力したい場合、どのようにやればいいのかを考えております。
自分なりに考えてはみましたが、どのやり方が自然、不自然、ありえないのか？
皆様のご意見をいただけたら幸いです。
ボタン2やボタン3は、13～18という範囲が今後変更になる際にやりやすい方法として考えてみました。特にボタン3は編集はしやすいのですが、どうにも無駄な動きが多い気がしまして、これらコードを見て、納得なのか、ありえないな！変！と思うのか、他にどんなやり方があるのか？ご意見をいただけたら幸いです。

html
1<!DOCTYPE>
2<html lang="ja">
3  <head>
4    <meta charset="utf-8">
5    <link rel="stylesheet" href="styles.css">
6  </head>
7  <body>
8    <div id="target">?</div>
9    <button id="btn1">ボタン1</button>
10    <button id="btn2">ボタン2</button>
11    <button id="btn3">ボタン3</button>
12    <script>
13      {
14        'use strict'
15        const target = document.getElementById("target");
16        const btn1 = document.getElementById("btn1");
17        const btn2 = document.getElementById("btn2");
18        const btn3 = document.getElementById("btn3");
19
20        btn1.addEventListener('click', ()=>{
21          const result = [13,14,15,16,17,18];
22          const num = Math.floor(Math.random() * result.length);
23          target.textContent = result[num];
24        });
25
26        btn2.addEventListener('click', ()=>{
27          const num_length = Math.floor(Math.random() * 6);//0-5
28          const answer2 = num_length + 13//13-18
29          target.textContent = answer2;
30        });
31
32        btn3.addEventListener('click', ()=>{	
33          let answer3;
34          do {
35            answer3 = Math.floor(Math.random() * 18 + 1 );//1-18
36          }while(answer3 < 13)
37            target.textContent = answer3;
38        });
39
40
41      }
42    </script>
43  </body>
44</html>

行動規範の内容に同意します

回答3件

ベストアンサー

「２者間のランダムな整数を得る」のでしたら既にリファレンスにもサンプルコードが載っていて、
ロジックがほぼ確立しています。

[Math.random()

](https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/Global_Objects/Math/random)の[例](https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/Global_Objects/Math/random#Examples)

自身で考えるのももちろん大事なことですが、実際は「ランダムな数値を出して終わり」ではないと思うので、もし自身で考えられたロジックが問題なく動くものなのでしたら「どのようにリクエストがくるのか」そして「その結果をどう使っていくか」という観点から考えられてはいかがでしょうか。

投稿2019/04/23 05:46

編集2019/04/23 05:47

m.ts10806

総合スコア80850

rickey

2019/04/23 05:54

普段調べて、コピペばかりだったので、自分でやってみました。その結果をどう使っていくのかってとても大切ですね。ありがとうございました。

m.ts10806

2019/04/23 06:12

良い心がけと思います。効率非効率は置いといて想定の結果がでる(要件を満たす)のが第一なので、ちゃんと結果がでているのなら「どれもありえる」というのが実際かと思います。なにかを作り上げるのに方法はひとつではありません

行動規範の内容に同意します

ボタン1でよいと思います

参考までに、多少効率は悪いけどユニークにデータを拾う

javascript
1const min=13,max=18,inc=1;
2const result = Array((max-min+1)/inc).fill(null).map((x,y)=>y*inc+min).map(x=>[x,Math.random()]).sort((x,y)=>x[1]-y[1]).map(x=>x[0]);
3console.log(result.shift());