Corrección Bug Jeduan

[landinezrichard]

Buen día.

En el vídeo: Funciones orientadas a eventos, Jeduan Tiene un Bug, el cual no corrige, el bug se corrige en el archivo "/lib/avengers.js" en la función "shuffle":

Es cuestión de matemáticas y de usar bien los signos de agrupación, como paréntesis y llaves. El error era de Jeduan, en la función shuffle uso:

let index = Math.floor(Math.random() * arr.length - 1);

Lo cual puede generar un numero igual al .length del array, y en esa posición no hay ningún elemento, pero está función igual hace el cambio de posición de los elementos, y despues, cuando esta buscando los thumbnails, se encuentra con un elemento undefined en la posición x del array y manda el error al catch. Para solucionar esto, simplemente agrupamos:

let index = Math.floor(Math.random() * (arr.length - 1));

Por eso hay que usar los signos de agrupación, para generar lo que queremos, ya que el resultado puede cambiar, con otro ejemplo, seria:

5_9+7 = 52
(5_9)+7 = 52
5*(9+7) = 80

Se obtienen dos valores diferentes, de acuerdo a como los agrupemos.

[jeduan]

Gracias por tus comentarios y tu buen ojo. Te invito a mandar Pull Requests para que futuros usuarios del repo tengan esta información.