Skip to content

Conversation

@AnaSegarra
Copy link

@AnaSegarra AnaSegarra commented Nov 6, 2019

Ejercicio terminado, incluidos los dos bonus. Me he liado un poco en el bonus de los palíndromos y no sé si habría una alternativa mejor a lo que yo he hecho. Tampoco estoy segura de si es correcto comparar strings usando operadores matemáticos para comprobar el orden lexicográfico, como he hecho en la iteración 3, o si sería mejor usar el método localeCompare().

@ta-web-mad
Copy link

Creo que el trabajo con ramas y el .gitignore contigo ya no son problemas de aquí en adelante, así que si no encuentras alguna dificultad, en principio dejaré de hacer comentarios al respecto (Alex)

Paso a revisar el código

En principio el código tiene muy buena pinta, enhorabuena!

Sólo un par de cosas:

  1. No consigo entender muy bien que intentas hacer en la línea 21
  2. Sobre tu duda, lo correcto en este caso hubiese sido splitear el nombre, referenciar al primer caracter, obtener su correspondiente código numérico ASCII y así realizar la comparación

@ta-web-mad ta-web-mad closed this Nov 10, 2019
@AnaSegarra
Copy link
Author

Gracias, Alex! Sí, en principio creo que he entendido bien lo de las ramas y el .gitignore, si me surge alguna duda ya te pregunto. Sobre la línea que dices, la verdad es que lo que hice fue un poco tontería porque primero iba a hacer let driver = hacker1.toUpperCase().split('').join(' '). Pero como era la sección de loops pues lo que hice fue recorrer el nombre original e ir añadiendo a una nueva string cada letra junto a un espacio, excepto en la última letra que no se incluye espacio (aunque no se fuera a ver, pero así no alteraba la longitud respecto al nombre original).

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