Skip to content

tarea semana 3 dia 1 Gabriela Mora - #27

Open
gwmg13 wants to merge 1 commit into
Conforcat-Frontend-PT-Jul:masterfrom
gwmg13:master
Open

tarea semana 3 dia 1 Gabriela Mora#27
gwmg13 wants to merge 1 commit into
Conforcat-Frontend-PT-Jul:masterfrom
gwmg13:master

Conversation

@gwmg13

@gwmg13 gwmg13 commented Aug 13, 2022

Copy link
Copy Markdown

No description provided.

@Conforcat-ta-webdev

Copy link
Copy Markdown

Hooola Gabriela! 👋

El lab esta muy bien! te comento un par de cosas que he visto:

  • El uso de var actualmente esta desaconsejado ya que "contamina" el codigo con diferentes scopes que se nos pueden torcer a lo largo del desarrollo de una aplicación. Como vimos en clase, lo ideal es usar siempre const y let.
  • En la Iteración 3 donde ordenamos alfabéticamente recuerda que las mayusculas tambien tienen peso en este filtro, ya que .localeCompare() nos compara con la tabla ASCII y las mayusculas van detras de todas las minusculas, por lo que "zapato" iría antes que "Alfredo". Lo podemos arreglar aplicando a nuestros hacker1 y hacker2 un .toLowerCase() así parten de las mismas condiciones.
  • Tienes algunas aplicaciones de código avanzado en el Lab, esta muy bien que busquéis en internet, pero intentad siempre aplicar las herramientas que hemos visto en clase para así ayudaros a asimilar los conceptos que hemos visto. 😊

Aún así esta genial! sigue así 🔥

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants