Bonjour.

J'essaye d'appliquer les bonnes pratiques dans mes codes et j'aimerais avoir votre sur avis sur le code suivant :

https://pastebin.com/kaTRh9Ts

Respecte-t-il les règles usuelles de bonnes pratiques ?

PS : impossible de poster directement le code sinon sujet considéré comme spam.

Bonjour,

Je te recommande lire quelques articles sur la syntaxe ES6 comme :
https://putaindecode.io/fr/articles/js/es2015/arrow-functions/
https://putaindecode.io/fr/articles/js/es2015/for-of/

Il me semble que "use strict" n'est plus utile depuis ES6.

Je regarderai ton code plus en détail plus tard.

Merci pour la réponse, je lirai attentivement les liens.

Je suis intéressé d'en savoir plus sur 'use strict' et d'avoir ton avis sur le code.

Si tu veux suivre de bonnes pratiques spécifiques à la syntaxe javascript, tu peux utiliser un linter (eslint par exemple) avec un preset largement utilisé (airbnb par exemple).

Sinon, il existe de bonnes pratiques de développeur d'une manière générale, et des livres entiers traitent du sujet (le livre "Clean Code" est plutôt bon je trouve, orienté POO assez vite puisque traitant du java).

La classe Flocon dépend de variables globales, c'est mal. Sa méthode "remonter" devrait appartenir au prototype. En fait je te suggère d'utiliser le mot clé Class (introduit en ES6) pour Flocon.

Ta fonction neige (pas bon comme nom, il faut un verbe) ne devrait utiliser que des méthodes de la classe Flocon, et non pas manipuler directement ses propriétés.

Essaie d'une manière générale de respecter le principe d'une seule responsabilité par fonction, non pas quand tu es en train de coder, mais lorsque tu mets au propre ton code une fois qu'il fonctionne.