Skip to content

[BCN-PT-082022] Francisco Pilosio - #11

Open
FranciscoPilosio wants to merge 4 commits into
Conforcat-Frontend-PT-Jul:masterfrom
FranciscoPilosio:master
Open

[BCN-PT-082022] Francisco Pilosio#11
FranciscoPilosio wants to merge 4 commits into
Conforcat-Frontend-PT-Jul:masterfrom
FranciscoPilosio:master

Conversation

@FranciscoPilosio

Copy link
Copy Markdown

Iteration 1 done

@FranciscoPilosio FranciscoPilosio changed the title [BCN-PT] Francisco Pilosio [BCN-PT-082022] Francisco Pilosio Aug 9, 2022
@Conforcat-ta-webdev

Copy link
Copy Markdown

Hoola Francisco! 👋

Recuerda que el link que debes compartir en el campus es el de esta pull, porfa! 😇

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

  • En las primeras iteraciones en vez de usar el "+" podemos usar una interpolación. Como lo has hecho tu también esta bien, pero para que veas que hay otras maneras (esta se usa mucho mas que los +):
const hacker1 = "Name"
console.log(`The driver's name is ${hacker1}`);
  • Esta muy bien el uso de los métodos! ✅
  • En la iteración 3 donde nos piden que ordenemos alfabéticamente tenemos un pequeño error. Los comparadóres < y > Nos ayuda a comparar dos strings alfabéticamente dependiendo de la posición que ocupen en la tabla ASCII. Nos devuelve true o false dependiendo de si el primero va antes o después, es decir, si a < b === true entonces a irá alfabéticamente antes que b. En tu ejercicio lo tienes al revés, cambiando los símbolos podemos solucionarlo. te dejo aquí un ejemplo de como sería:
const driver = "alpha"
const navigator = "beta"


if (driver.toLowerCase() < navigator.toLowerCase()) {
  console.log("The driver's name goes first.")
}
else if (driver.toLowerCase() > navigator.toLowerCase()) {
  console.log("Yo, the navigator goes first definitely.")
}
else {
  console.log("What?! You both have the same name?")
}

**Además recuerda que las mayusculas en ese orden alfabético en la tabla de ASCII están después de todas las minúsculas, por lo que "Alfredo" iría detrás de "zapato", por eso controlamos esto mediante un .toLowerCase(), así todo esta en las mismas condiciones.

Pese a esto, esta genial! Sigue así! 🔥

@FranciscoPilosio

Copy link
Copy Markdown
Author

Buenas,

Sorry ... 99% seguro que para los otros labs mande el link de la pull, haha.

Genial, muchas gracias por el feedback. Ya arregle la iteration 3 para que quede bien.

Nos vemos el martes!

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