Skip to content

[PAR-FT-0620] - Maya Marie - #938

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

[PAR-FT-0620] - Maya Marie#938
mayadesp wants to merge 2 commits into
ironhack-labs:masterfrom
mayadesp:master

Conversation

@mayadesp

Copy link
Copy Markdown

No description provided.

@TatijanaRajcic

Copy link
Copy Markdown

Hello les filles! Merci pour le lab! Globalement tout marche bien! Quelques remarques:

  • Pour la question 3.1: vous avez un espace au début de votre nom (à enlever dans newName), ainsi qu'un à la fin qui est rajouté alors que l'on en a pas besoin puisque c'est la dernière lettre. Est-ce que vous voyez un moyen de régler ça?
  • Pour la question 3.3 : nice ! Sachez que vous pouvez aussi le faire avec une fonction qui s'appelle localeCompare! Est-ce que vous avez tenté de le faire avec une loop comme demandé? Envoyez moi un message si jamais vous le tentez!
  • Pour le bonus 1: vous auriez aussi pu utiliser split (cela aurait été un peu plus élégant haha), qui vous aurez permis de fractionner la string en éléments dans un array, et ensuite regarder la longueur de cet array (= qui serait donc le nombre de mots)
  • Pour compter le nombre de fois où "et" apparaît: si vous n'êtes pas à l'aise avec les regex, vous n'êtes pas obligées de les utiliser hehe. Vous auriez pu utiliser split puis ensuite boucler sur l'array et vérifier si l'élément sur lequel vous itérez est bien égal à "et"
    Dites moi si vous essayez tout ça!
    Bien joué 💪

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