Что не так с моей программой перетасовки?

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), elipses в javascript - это функция, которая появилась с последней версией EcmaScript 2015. Это «оператор распространения».

Когда вы вызываете функцию с массивом «spread», это как вызов функции с содержимым массива в виде отдельных аргументов. Например,

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

такой же, как вызов

array.push(1,2,3) 

толчок() можно добавить произвольное количество разделенных запятой аргументов в массив. Таким образом, после опустошения аргумента массива с вашими петлевыми сращиваниями вы можете нажать содержимое вновь созданного arr на пустой массив, используя оператор спреда.