Skip to content

[WEB-MAD DE] Lali Osorio #2436

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 2 commits into from

Conversation

laliiosorio
Copy link

Duda infinita del ejercicio Iteration #6: Find elements, me funcionaba perfecto cuando la palabra existia me salia true y cuando no undifined fue colocarle el else false y nunca mas volvio a ser true solo con index 0. Al final le di tantas vueltas que ni sé cual versión quedo jajajajaj Vi algo de falsy pero ni idea. también la coloque en una variable para colocar un solo return pero se quedaba el false para siempre. No sé me ocurrieron otras maneras. Anyways, ya me pegaré con los BONUS a ver qué explota!!!

PD: Mi PC peto cuando le di npm run test:watch. Me toca npm run test y ya para que no colapse.

PD2: Me vine arriba con este pull request! Jajajaja 🤯

@ta-web-mad
Copy link
Contributor

Hola Lali!
Venirse arriba con las pull request esta genial para poder ayudarte así que tú muy arriba siempre!
Lo del test:watch...no sé como ayudarte, podríamos mirarlo pero va a ser tema del ordenador, lo vemos luego si quieres.
En cuanto a tu duda con la iteración 6:
Veo que te quedaste probando pero la idea era buena:
Screenshot from 2021-06-04 12-25-19
Lo que pasa que te liaste con el bucle y con la solución que habías encontrado no lo necesitabas:
Screenshot from 2021-06-04 12-25-47
Simplemente con el includes, si le devuelves el true o false dependiendo de si el array contiene la palabra o no ya estaria! CASI!
Buen trabajo! Javascript no se te da mal como tú decias!

@stale
Copy link

stale bot commented Jul 8, 2021

This pull request has been automatically marked as stale because it didn't have any recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Jul 8, 2021
@stale
Copy link

stale bot commented Jul 10, 2021

This pull request is closed. Thank you.

@stale stale bot closed this Jul 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants