search

Theo-Dancoisne / pokedex

utiliser une api + vuejs
0 stars 0 forks source link

preferer l'usage des methods de tableau plutot que boucle for #2

Closed kiuKisas closed 2 years ago

kiuKisas commented 2 years ago

https://github.com/Theo-Dancoisne/pokedex/blob/80633fbe92b81712166455e5ac6871311718c92e/src/vues/top-screen.vue#L36 Préferes les methods de type map, forEach, reduce a l'utilisation d'une boucle for Le code sera bcp plus clair et ça rend implicite ce que tu veux faire https://developer.mozilla.org/fr/docs/Web/JavaScript/Reference/Global_Objects/Array

Theo-Dancoisne commented 2 years ago

✅done à noter que map et reduce ne devraient pas être utilisés car dans le contexte je ne modifie pas la variable itérée, aussi la plupart des éditeur ne détectent pas le type de variable des valeurs d'un objet (il considéra donc les valeurs d'un objet comme étant des objets), ainsi forEach n'est proposé par l'éditeur de code lors de l'écriture

kiuKisas commented 2 years ago

@Theo-Dancoisne je suis passé à coté.

Le problème de l'editeur du code c'est parce qu'il n'est pas au courant que c'est un tableau et que la fonction forEach et compagnie existe. Javascript est un language dynamique, c'est à dire que les variables ne sont pas typé, donc ton editeur ne peut pas etre au courant. C'est ce qu'ajoute partiellement typescript, et là tu aurais eu la proposition.

map et reduce ne doivent pas etre utilisé si tu modifies la variable itéré directement dans le tableau.. Et sache que je dis pas ça au pif ou pour t'embeter..

Je te laisse t'occuper de ça, je réouvre une issue: #8