私のシャッフルプログラムに何が問題なのですか？

Question

私は以下のシャッフルプログラムを書いて"Will It Shuffle?"まで走りました。結果はコンソールで動作していることを示しています。それは配列をシャッフルしています。しかし、ウェブサイトは私にすべての赤いボックスを表示し、私のコードで何かが間違っていると思うようにしていますが、私はそれを見ません。

function shuffle (array) { 
    var arr = [], 
     length = array.length, 
     el; 

    while (length > 0) { 
     var randomEl = Math.floor(Math.random() * (length - 0) - 0); 
     if (length > 1) { 
     el = array.splice(randomEl,1); 
     } else { 
     el = array.splice(0,1); 
     } 
     arr.push(el[0]); 
     length -= 1; 
    } 

    return arr; 
} 

Answer 1

このページでは、in-placeソートが予想されるため、関数の戻り値は無視されます。あなたのコードの末尾にこれを追加した場合

は、それが期待どおりに動作：

array.push(...arr); 

ます。また、直接その場でそれを行うことができます。

function shuffle (array) { 
    var length = array.length; 
    while (length) { 
    var randomEl = Math.floor(Math.random() * length); 
    var el = array.splice(randomEl, 1); 
    array.push(el[0]); 
    --length; 
    } 
} 

Answer 2

彼らは、配列を変更し、あなた配列を変更しないでください。

元の配列を変更する必要があります。新しい配列を返さないでください。

function shuffle (array) { 
    var arr = [], 
     length = array.length, 
     el; 

    while (length > 0) { 
     var randomEl = Math.floor(Math.random() * (length - 0) - 0); 
     if (length > 1) { 
     el = array.splice(randomEl,1); 
     } else { 
     el = array.splice(0,1); 
     } 
     arr.push(el[0]); 
     length -= 1; 
    } 

    //replace array with the new items 
    //it is like using concat, but does not produce a new array, 
    //just appends it to the original which has zero items in it. 
    Array.prototype.push.apply(array, arr); 



} 

Answer 3

あなたがやっていることは、オリジナルの要素をシャッフルして新しい配列を作成することです。

しかし、戻って渡された配列を見ると、シャッフルされずに空になっていることがわかります。どうやら、これは「シャッフルされるの？あなたに求めています。

splice（）とpush（）はどちらも、それらのメソッドを呼び出す配列を常に突然変異させます。

.push（... arr）に関するご質問にお答えするために、javascriptのエリプスは、最新バージョンのEcmaScript 2015を搭載した機能です。これは「スプレッドオペレータ」です。

"spread"配列を持つ関数を呼び出すと、配列の内容を別の引数として呼び出すようなものになります。例えば、

array.push(...[1,2,3]) 

アレイにコンマで区切られた引数の任意の数を追加することができ

array.push(1,2,3) 

プッシュ（）を呼び出すと同じです。したがって、ループされたスプライスで配列の引数を空にした後、スプレッド演算子を使用して、新しく作成したarrの内容を空の配列にプッシュすることができます。