Vaduz blog

NO GAME NO LIFE

JavaScript で配列をシャッフルする

なんか prototype に shuffle 生やしたら空でも length が 1 になって困ったので普通に関数にした

function shuffle(arr) {
    var i = arr.length;
    while (i--) {
        var j = Math.floor( Math.random() * (i + 1) );
        var t = arr[i]
        arr[i] = arr[j];
        arr[j] = t;
    }
    return arr;
};

function shuffle(arr) {

var i = arr.length;

while (i--) {

var j = Math.floor( Math.random() * (i + 1) );

var t = arr[i]

arr[i] = arr[j];

arr[j] = t;

}

return arr;

};

# 追記: コメントでの指摘をうけて修正しました。

By Vaduz | 2014/07/23 | Programming | 3 Comments |

3 thoughts on “JavaScript で配列をシャッフルする”

shmorimo 2014/07/27 at 08:07

これだと偏りませんか？
https://gist.github.com/hanabokuro/450513ac2d6ed3e98c62

Reply ↓
1. shmorimo 2014/07/27 at 08:10
  
  Math.floor( Math.random() * (i + 1) );
  な気がします
  
  Reply ↓
  1. Vaduz Post author2014/07/29 at 19:27
    
    おお、ほんとですね。
    配列の最初の方に後ろの方が若干偏ってしまいますね。
    
    Reply ↓

3 thoughts on “JavaScript で配列をシャッフルする”

shmorimo へ返信する コメントをキャンセル

shmorimo へ返信するコメントをキャンセル