Conversation
| // ITERATION 2 | ||
|
|
||
| function selectRandom() {} | ||
| function selectRandom(room, weapon, suspect) { |
There was a problem hiding this comment.
Es importante hacerlo en una sola función, ya que de esta manera estas repitiendo código. Fijate que haces un Math.floor y la siguiente lineas accedes al array y lo pusheas.
Sería mucho más fácil si pasandole un array la función te devuelve un elemento.
|
|
||
|
|
||
|
|
||
| function pickMystery() {} |
There was a problem hiding this comment.
Esta función debería llamar a selectRandom asignar la respuestas a diferentes constantes. Por ejemplo; suspect, room, weapon y después de eso pasarle un objeto ya compuesto a revealMystery. De esta manera revealMystery no necesita saber en que posición del array viene cada cosa sino acceder directamente al objeto.
Imagina que llamas al parámetro mystery.
Podrias acceder de la siguiente manera: mystery.suspect.firstName, mystery.suspect.lastName, mystery.weapon.name y mystery.room.name
j0nl1
left a comment
j0nl1
left a comment
There was a problem hiding this comment.
Te he dejado unos comentarios con una sugerencia de cambios para que pruebes esta implementación y veas la diferencia.
|
This pull request has been automatically marked as stale because it didn't have any recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
2 participants
en vez de hacer 2 funciones una para seleccionar y otra para crear el array lo hice todo en una.