Skip to content

[MAD-PT DE] Ana Segarra - #545

Closed
AnaSegarra wants to merge 15 commits into
ironhack-labs:masterfrom
AnaSegarra:master
Closed

[MAD-PT DE] Ana Segarra#545
AnaSegarra wants to merge 15 commits into
ironhack-labs:masterfrom
AnaSegarra:master

Conversation

@AnaSegarra

@AnaSegarra AnaSegarra commented Nov 6, 2019

Copy link
Copy Markdown

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
Copy Markdown

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
Copy Markdown
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