commentaires et correction

[asecher]

No description provided.

[vitreene]

Bonjour, merci de vos commentaires à propos de mon rendu. Je suis surpris par votre emploi des fonctions fléchées pour éviter la lourdeur du binding : c’est très judicieux ! Je ne me rappelle pas avoir vu d’exemples employant des fonctions fléchées dans ce contexte. En tout cas c’est noté. (je lis qu’il y a quelques précautions <https://medium.com/@machnicki/handle-events-in-react-with-arrow-functions-ede88184bbb> à prendre) …et mon beautyfier manque de pertinence ;-)

…

Le 22 nov. 2017 à 11:33, Alexis Sécher ***@***.***> a écrit : You can view, comment on, or merge this pull request online at: #1 Commit Summary • commentaires et correction File Changes • M README.md (7) • A quiz-react.zip (0) • M src/App.js (93) • M src/components/Layout.js (28) • M src/components/Question/choix-check.js (123) • M src/components/Question/choix-radio.js (131) • M src/components/Question/index.js (51) • M src/components/Valider.js (38) • M src/components/navigation.js (140) • M src/components/titre.js (11) • M src/index.js (15) Patch Links: • https://github.com/vitreene/quizz/pull/1.patch • https://github.com/vitreene/quizz/pull/1.diff — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread.

[asecher]

Par rapport aux fonctions fléchées, la principale précaution à prendre c'est qu'il ne faut pas l'utiliser en anonyme directement dans les props. Comme suit:

<MyComponent  onClick={e => console.log(e.t)} />

(Le props est différent pour React à chaque render, pas top sur les performances)

Pour le beautyfier Prettier est vraiment pas mal.

[vitreene]

'il ne faut pas l'utiliser en anonyme directement dans les props.

C’est tout à fait ça. Je vais d’ici la fin de semaine intégrer la correction sur le state du composant navigation. Je suppose que d’ici là vous n’avez plus besoin de l’exercice en tout cas ?

…

Le 22 nov. 2017 à 16:40, Alexis Sécher ***@***.***> a écrit : Par rapport aux fonctions fléchées, la principale précaution à prendre c'est qu'il ne faut pas l'utiliser en anonyme directement dans les props. Comme suit: <MyComponent onClick={e => console.log(e.t)} /> (Le props est différent pour React à chaque render, pas top sur les performances) Pour le beautyfier Prettier est vraiment pas mal. — You are receiving this because you commented. Reply to this email directly, view it on GitHub, or mute the thread.