Skip to content

[PAR - WDFT] Remy/Jonathan - #664

Closed
RemyPham wants to merge 1 commit into
ironhack-labs:masterfrom
RemyPham:master
Closed

[PAR - WDFT] Remy/Jonathan#664
RemyPham wants to merge 1 commit into
ironhack-labs:masterfrom
RemyPham:master

Conversation

@RemyPham

Copy link
Copy Markdown

Sending you this for now. Still working on the bonus 2 this weekend.

@TatijanaRajcic

TatijanaRajcic commented Jan 13, 2020

Copy link
Copy Markdown

Bon travail à tous les deux :)

Votre code est bon, je propose simplement des alternatives:

  • Pour spliter un mot comme J O H N vous auriez pu utiliser les fonctions split et join ensemble:
function splitWord (word) {
  return word.toUpperCase().split("").join(" ")
}
splitWord("John")
  • Pour mettre un mot à l'envers vous auriez pu utiliser la fonction reverse;
function reverseWord(word) {
  return word.split("").reverse().join("");
}
  • Pour votre lorem ipsum: attention, votre code ne marche pas pour les "et," et "et." . Il y a plusieurs façons de procéder: vous pourriez "cleaner" la phrase d'origine en utilisant un split combiné à un regex puis utiliser filter pour retourner un array qui ne contient que des "et" . Le count serait alors le length de cet array. C'est une façon de faire mais il y en a d'autres :) En tous cas attention àces cas particuliers!
  • N'hésitez pas à venir me voir si vous voulez tenter le bonus 2 :)))
  • N'hésitez pas à mettre votre code dans des fonctions comme vous l'avez fait pour le lorem ipsum, c'est beaucoup plus clair :)
    Bon travail! :)

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