chprome
commented
10 years ago Je te propose pour les variables JS :
- utiliser des noms qui ont du sens
- en français
- éviter les abréviations à par si elles sont évidentes
- écrire les noms en camelCase et pas en snake_case ni en spinal-case
Pour les classes / id d'éléments HTML, idem sauf la case : utiliser du spinal-case
Le 21 septembre 2013 12:05, g-prome notifications@github.com a écrit :
Il va effectivement falloir rationaliser les noms des variables (t'as commencé à le faire pour les attributs des objectifs notamment).
Je pensais m'y mettre aujourd'hui (vu que c'est mon code c'est plus simple pour moi, par exemple t'as utilisé "longueursPolygones" alors qu'en fait c'est "longueursCylindres", j'avais écrit de la merde en commentaire, faut aussi les nettoyer)
T'as des habitudes/bonnes pratiques ?
PS: c'est pas une issue.. mais j'ai pas trouvé d'autre "fil de discussion" dans GitHub
— Reply to this email directly or view it on GitHubhttps://github.com/g-prome/SimuAPN/issues/2 .
MycroftLegros
Il va effectivement falloir rationaliser les noms des variables (t'as commencé à le faire pour les attributs des objectifs notamment).
Je pensais m'y mettre aujourd'hui (vu que c'est mon code c'est plus simple pour moi, par exemple t'as utilisé "longueursPolygones" alors qu'en fait c'est "longueursCylindres", j'avais écrit de la merde en commentaire, faut aussi les nettoyer)
T'as des habitudes/bonnes pratiques ?
PS: c'est pas une issue.. mais j'ai pas trouvé d'autre "fil de discussion" dans GitHub