Skip to content

[MAD-PT DE] Daniel Zamarreño Avendaño#564

Closed
Z4n73 wants to merge 5 commits into
ironhack-labs:masterfrom
Z4n73:master
Closed

[MAD-PT DE] Daniel Zamarreño Avendaño#564
Z4n73 wants to merge 5 commits into
ironhack-labs:masterfrom
Z4n73:master

Conversation

@Z4n73
Copy link
Copy Markdown

@Z4n73 Z4n73 commented Nov 10, 2019

First and second iterations,

Missed to add the first commit correctly, due to the description name, sorry.

@Z4n73
Copy link
Copy Markdown
Author

Z4n73 commented Nov 10, 2019

Hi TA's!

I managed to finished all the exercises.

Overall, easy difficulty, with a single exception, i have 2 "undefined+string" concatenated when outputing the two names (driver and navigator) on console. I haven't investigated it deeply.

Farewell, Dani.

@ta-web-mad
Copy link
Copy Markdown

Buen trabajo Dani, el ejercicio está completo y bien resuelto.
Me ha gustado mucho que hayas creado funciones para resolver las diferentes iteraciones y el uso de expresiones regulares. Quizá el nombre de los parámetros de las funciones deberían ser más genéricos ya que en este ejercicio tiene sentido que se llamen driverName y navigatorName pero al fin y al cabo invertir dos strings puede ser algo que utilices en otros ejercicios y si estos parámetros se llaman name1 y name2 por ejemplo será más genérico.

Sigue trabajando así e intenta incluir el .gitignore en los proyectos que como hemos hablado en clase es una buena práctica.

#checked#closed#Diego🤙

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