Skip to content

Instantly share code, notes, and snippets.

@eperedo
Last active June 15, 2017 22:07
Show Gist options
  • Star 0 You must be signed in to star a gist
  • Fork 0 You must be signed in to fork a gist
  • Save eperedo/efde7cb5f554fedd2aead57622ab3ef1 to your computer and use it in GitHub Desktop.
Save eperedo/efde7cb5f554fedd2aead57622ab3ef1 to your computer and use it in GitHub Desktop.
Chut quiz javascriptero
@JoseJuan81
Copy link

JoseJuan81 commented Jun 10, 2017

https://jsbin.com/vewaputexi/edit?html,js,console

EDUARDO: Faltó el punto 1 del ejercicio:

  1. Analizar la función y ESCRIBIR todas las posibles maneras de mejorarla.

Al ejecutar el código

console.log(filterByState(2));

no devuelve el resultado esperado

@JoseJuan81
Copy link

JoseJuan81 commented Jun 13, 2017

https://jsbin.com/xerecikazu/edit?html,js,console

Realicé el punto 1 del ejercicio y corregí el resultado para el valor 2. adicionalmente minifiqué el código.

EDUARDO:

  1. Minificar el código es una buena práctica, pero es un proceso que se realiza exclusivamente cuando se hace deploy a producción.
  2. La función tal como la planteas cumple el objetivo de ser más clara, pero hay un detalle, al extraer las variables FUERA de la función pierdes la opción de reutilizar esa función en cualquier otro lado de tu aplicación. Tendrías que declararlas en cualquier otro archivo que quieras reutilizar esa función.

@eperedo
Copy link
Author

eperedo commented Jun 15, 2017

SOLUCION: Dudas, sugerencias, insultos o cualquier cosa me avisan.
http://jsbin.com/nugugiyupu/1/edit?js,console

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment