Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

[MAD PP] Mileydis Suárez & Víctor Bañuls #750

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from

Conversation

vicban14
Copy link

@vicban14 vicban14 commented Jan 8, 2019

Hemos llegado hasta el ejercicio finding-elements. Hemos encontrado algunas dificultades, pero hemos conseguido solventarlas. A mí (Víctor) me ha costado un poco más llegar a las conclusiones acertadas, pero al final hemos solucionado bastante.

@ta-web-mad
Copy link
Contributor

Hola chicos.

Os indico varias cosillas para que vayáis teniendo buenas prácticas:

  • Si queréis inicializar una variable string, utilizad comillas sin nada dentro. Lo que habéis hecho (" ") es un string con el carácter espacio y tiene longitud 1.
  • Cuando uséis una variable i en un for, declararla con var.
  • La función doesWordExist os está fallando porque el return que habéis escrito es lo que devuelve el forEach pero no la función. Deberíais capturar ese resultado y devolverlo fuera del forEach. Tened cuidado con esto.

Si no entendéis algo de lo que os he dicho decírnoslo a cualquier TA y os lo explicamos.

Buen trabajo, seguid así. 💪

Juan.

@ta-web-mad ta-web-mad closed this Jan 9, 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
Morty Proxy This is a proxified and sanitized view of the page, visit original site.