Skip to content

[BCN-PT-082022] Leandro Balaguer - #12

Open
leandrobalaguer wants to merge 7 commits into
Conforcat-Frontend-PT-Jul:masterfrom
leandrobalaguer:master
Open

[BCN-PT-082022] Leandro Balaguer#12
leandrobalaguer wants to merge 7 commits into
Conforcat-Frontend-PT-Jul:masterfrom
leandrobalaguer:master

Conversation

@leandrobalaguer

Copy link
Copy Markdown

No description provided.

@leandrobalaguer leandrobalaguer changed the title Leandro_Balaguer_WEEK_3_Lab_First_3_iterations [BCN-PT-082022] Leandro Balaguer Aug 9, 2022
@Conforcat-ta-webdev

Copy link
Copy Markdown

Hola Leandro 😸
Algunos comentarios de tu código 🦜 :

  • La primera iteración está muy bien 🥇. Sin embargo, la segunda iteración está incompleta. Para esta iteración se espera el uso de condicionales dependiendo del largor de los nombres. Se espera algo similar a esto:
if (condition1) {
    console.log(
      "The driver has the longest name, it has " +
        `number` +
        " characters."
    );
  } else if (condition2) {
    console.log(
      "The navigator has the longest name, it has " +
        `number`+
        " characters."
    );
... Y tendrías que completarla.

Esta es una de las muchas formas en las que podrías intentarlo. Anímate a probarlo 👯

  • La iteración 3.3 tiene algunas cosas que podríamos comentar. Para seguir el principio de Don't Repeat Yourself, no es necesario declarar una variable con los nombres ya que los tenemos desde la iteración 1. Basta con llamarlos.
  • Con los operadores < y > puedes realizar la comparación ya que se rigen con el estándar ASCII, esa parte está muy bien. Lo único, frente a los condicionales, luego de un if, dependiendo de los casos, se recomienda usar else if para los demás y else si es la condición por default; esto viene mejor en lugar de agrupar solo if.
  • Como estaríamos reutilizando los nombres, ya no tendrías que agruparlos en un array y por lo tanto el método sort no sería necesario. Esto simplifica el código. Las comparaciones las realizarías directamente con los las variables que has creado para los nombres.

Si tienes alguna inquietud o duda frente a lo que te estamos comentando, lo podemos ver 👀

Esperamos que te animes a realizar los bonus para seguir practicando.

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