[BCN-PT-072022] - Daniel Moreno js-basic-algorithms W3 day 1

[danalmorse]

Cristian y Pablo, comentamos mañana una duda, he hecho merge en mi repo del branch "develop" al main y de ahi he hecho pull request al repo global.

Mi duda es que en mi repo sigue saliendo las 2 branch y no solo la main luego de haber hecho el merge pienso que se deberia haber quitado no?

comentamos mañana
saludos

[Conforcat-ta-webdev]

Hoola Daniel! 🧑🏻‍🚀 Recuerdo que ya solucionamos la duda de la rama, puedes eliminarla si lo deseas, github te da la opción de poder hacerlo. Si tienes dudas del procedimiento completo coméntanoslo y lo vemos juntos en algún descanso!

Esta genial el lab! te comento un par de cosas a que he visto que podemos mejorar, aun así esta genial!

• En las primeras iteraciones puedes usar una interpolación en vez de añadirlo mediante el "+". Se hace añadiendo los backstAsí:

const hacker1 = "Pepe"
console.log(`The driver's name is ${hacker1}`);

• En la Iteración 3 esta genial el uso de los metodos de arrays! muy pro! 👍 Pero tienes un spread operator que es un uso muy avanzado y un poco abstracto de js, esta genial buscar en internet, pero procura usar también las herramientas que hemos visto en clase para solucionar los ejercicios, ya que así practicas tus conocimientos y te ayuda a interiorizar mejor los conceptos.

• También en la iteración 3 donde hay que ordenar alfabéticamente, tienes que tener en cuenta las mayusculas, ya que pueden afectar al resultado. Con un .toLowerCase() en los nombres para partir desde los mismo valores lo solucionamos 😉

Aún así como te decía antes, esta muy bien!! sigue así! 🔥