Skip to content

[MAD-PT DE]Miguel Alonso Gallardo#549

Closed
malonsogallardo wants to merge 2 commits into
ironhack-labs:masterfrom
malonsogallardo:master
Closed

[MAD-PT DE]Miguel Alonso Gallardo#549
malonsogallardo wants to merge 2 commits into
ironhack-labs:masterfrom
malonsogallardo:master

Conversation

@malonsogallardo
Copy link
Copy Markdown

Buenas tardes,
Nivel de dificultad: medio.
He logrado hacer todos creo que de forma medio eficiente.
Me he quedado atascado en el bonus 2, no se como completarlo, creo que me haria falta una funcion para unir todos los caracteres y poder igualarlos, pero no he encontrado nada asi.

Un saludo y muchas gracias.

@ta-web-mad
Copy link
Copy Markdown

Incluido el .gitignore, pero quiero que te acerques y me expliques, para ver si comprendes el significado de *.vscode (soy Alex)

Trabajo con ramas?? que ha pasado? No hay más que un commit de .gitignore, y el siguiente es a master?? -> Lo pongo en pendiente de revisar

Respecto al código

Línea 14: eso no debería ser ya un else? o str1 > que str2, o str1 < str2, o todo lo demás (que solamente puede ser str1 === str2, hablando del length, obvio

Línea 21: uffff, nos metemos en terreno fangoso creando un Object.String... sabemos trabajar con clases? porque estamos creando una instancia... Creo que la solución pasa más por hacerle un split("") al string, y trabajar a partir de él. De cualquier manera, empezáis con " ", y eso no es lo que se pide en el ejercicio... hay que tener más exactitud please. Trabajemos esto con más mimo a partir de ahora en adelante

Línea 43 y 45: a ver, en primer lugar los nombres de las variables hay que tratarlos con mimo... si no el que revisa tu código por qué va a saber, 5 líneas más abajo, que z vale un lorem ipsum? Respecto a la 45, además, split(" "), no se usa esa notación de corchetes.

Una última razón para dejarlo en revisión. No hay uso de funciones de responsabilidad única! Es código espagueti que no puede reutilizarse!

@ta-web-mad ta-web-mad closed this Nov 10, 2019
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