[Coding style] Vos solutions pour un code lisible ?

[NicoMSEvent]

Rien de ce qui suit n'est à considérer comme une attaque personnelle. Je vais juste expliquer rapidement pourquoi je trouve le code illisible (sans faire de ronds de jambe).

la critique est constructive, je l'accepte (voir ma signature :p)
Tout d'abord j'ai pondu ce code en 4eme vitesse, je ne l'ai meme pas testé. (oups, je pense avoir mélangé syntaxe C++ et php :p)

[*]La langue : mélanger français et anglais dans le code, c'est curieux (cf. isVide()) ;

un bon point pour toi ^^
je repère plus facilement mes ajouts de code comme ça (par rapport aux objets standards en anglais, même si c'est une mauvaise excuse :p)

[*]La notation Hongroise est très verbeuse pour ne rien apporter : le nommage des variables suffit. Un prix, on sait tous ce que c'est. De plus, je ne comprends pas trop comment l (long int) et f (float) peuvent cohabiter ;

$p_ pour les paramètres, $l_ pour les variables locales, $g_ pour les variables globales...

[*]Le nommage n'est pas cohérent : on a une variable $p_f_monnaiearendre, où sont passés les underscore ?

quand j'écris vite, j'ai tendance a oublier mes normes

De la même manière, pourquoi l'attribut ReservePieces s'appelle-t-il ainsi alors que tes autres variables sont de la forme reserve_pieces ? Et pourquoi n'a-t-il pas d'accesseur ?

pour moi, le plus intéressant du code était dans la ligne empiler - dépiler (pas pris la peine d'avoir un accesseur pour la variable dans l'objet... je sais, c'est pas bien ^^)

[*]C'est quoi Distributeur ? C'est ta classe ? Si oui, pourquoi appeler ses méthodes avec -> plutôt que :: ? A moins que je n'ai déjà oublié la syntaxe de PHP… :heuuu:

Distributeur est un objet instancié (et puis je pense m'être emmêlé les pinceaux entre la syntaxe C++ et PHP)

[*]Le code n'est pas assez aéré, il manque des espaces entre les opérateurs et les opérandes ;

tu as raison, c'est plus clair comme ça. j'ai des habitudes difficiles a changer

[*]Le commentaire n'est pas une phrase correcte ;

les informations principales sont contenues dans la ligne, je n'ai pas cherché a faire une phrase :$

Une petite traduction, une uniformisation des noms et paf, ça devient un bon code (de mon point de vue, je le rappelle).

en effet, c'est plus lisible. (même si je trouve que la notation hongroise permet d'avoir une vue plus claire de ce qui se trouve dans telle ou telle variable, et d'ou elle vient... si c'est un parametre ou pas par exemple)

Je soulève juste que le fait de passer un objet qui est retourné par une fonction, directement en paramètre dans une autre fonction, n'est pas pour moi une habitude qui rend le code moins lisible (que du contraire)