JavaScriptにて、配列の中の値を抽出する数を複数パターン設定したい

Question

JavaScriptで、配列の中からランダムに値を抽出し、かつ、抽出する数も3つか4つに設定したいです。

(問題のある記述は、下記の一番下のコードだと考えています。)

下記のようにコードを書いたのですが、「||」は、左側がtrueであれば左側が選択されるとのことで、出力される数が全て3つになってしまいます。

３つか4つのように設定したいのですが、どのように記述すればよいでしょうか。
よろしくお願いいたします。

//配列からランダムで選択する関数
function random(array, num) {
  var a = array;
  var t = [];
  var r = [];
  var l = a.length;
  var n = num < l ? num : l;
  while (n-- > 0) {
    var i = Math.random() * l | 0;
    r[n] = t[i] || a[i];
    --l;
    t[i] = t[l] || a[l];
  }
  return r;
}

var user_array = ['1', '2', '3', '4', '5', '6'];

random(user_array, 3 || 4);

Answer 1

こんにちは

以下、ランダム抽出のコードを自作することからは離れる回答になります。配列に対する操作で便利なライブラリ Lodash を利用すると、 目的とする

ランダムに値を抽出し、かつ、抽出する数も3つか4つ

の配列は、 _.sample と _.sampleSize を使って以下で得られます。

_.sampleSize(user_array, _.sample([3, 4]))

以下は、上記を10回試行してみるサンプルです。

• 動作確認用のサンプル: https://jsfiddle.net/jun68ykt/vuz47e8d/1/

ご自身でランダム抽出のプログラムを書くことは大変よいと思います。そして、ご自身でも書けた上でならば、次は、このような便利なライブラリを使うことを検討してみてもよいかもしれません。

以上参考になれば幸いです。

追記

ご質問に挙げられている function random(array, num) を使えば、

random(user_array, 3 || 4);

と書くことで期待していた動作を、以下によって実現できます。

random(user_array, random([3,4], 1));

以下は、上記を 10回試行するサンプルです。

• 動作確認用のサンプル: https://jsfiddle.net/jun68ykt/t0149jrw/1/

上記のコードを実行するときに、まず random([3,4], 1) が、 [3] または [4] という、要素が1個の配列を返します。仮に [3] が返ってきたすると、次に

random(user_array, [3]);

を実行しようとします。関数randomの第2引数numは、配列ではなく数値を期待しているので、エラーになりそうですが、期待どおりに動きます。これは以下で確認できるような型変換が暗黙に行われるからです。

$ node
> [4] == 4
true
> [4] === 4
false
> Number([4])
4
> let x = [4];
undefined
> x -- > 0
true
> x
3
> x -- > 0
true
> x
2
> const ary = [0, 10, 20, 30, 40, 50, 60];
undefined
> ary
[ 0, 10, 20, 30, 40, 50, 60 ]
> ary[[4]]
40
> ary[[4]] = 99
99
> ary
[ 0, 10, 20, 30, 99, 50, 60 ]
>

Comments

[YutaEmoto]

ありがとうございます！！ライブラリの紹介に加え、とても詳細に記載していただき、ベストアンサーとさせていただきました！
numに配列を渡すのは盲点でした。。本当にありがとうございます！！

[jun68ykt]

どういたしまして。解決されたようで、よかったです ????

Answer 2

配列をシャッフルしたあと引数のnum1かnum2の個数分を
配列からsliceで抽出します。
3 || 4 は必ず 3 になるので使えないです。

// 配列からランダムで選択する関数
function random(array, num1, num2) {
  for (var i = array.length - 1; i > 0; i--){
    var r = Math.floor(Math.random() * (i + 1));
    var tmp = array[i];
    array[i] = array[r];
    array[r] = tmp;
  }
  var num = Math.random() < 0.5 ? num1 : num2;
  return array.slice(-num);
}
var user_array = ['1', '2', '3', '4', '5', '6'];
var result = random(user_array, 3, 4);
console.log(result);

Answer 3

自分だったらこんな感じに

function random(array, num) {
  //略
}

function get_choice_num() {
  //3か4を返す
}

var user_array = ['1', '2', '3', '4', '5', '6'];

random(user_array, get_choice_num());

Comments

[YutaEmoto]

ありがとうございます！
複数出力する関数を作るというのはとてもシンプルで良いですね！
最近始めたばかりで一つの関数の中で全て完結させようとしてしまっていたので、とても参考になります。

Answer 4

例えばこんな。これだけだと重複したものも出るけど。

let nums = [1,2,5,8,9,123,23,15,6];
let loop=3
if(Math.random() >= 0.5){
    loop+=1;
}
for(loopの回数loop){
    console.log(nums[Math.floor( Math.random() *  nums.length)]);
}