Digital Fashion - Bonnes pratiques PHP - Comments

[ping] Bonnes pratiques PHP - CactusCrew WebDev

CactusCrew WebDev — Mon, 06 Nov 2006 10:37:29 +0100

En réponse à Gameplayer, à mon travail, nous avons pris l'habitude d'indiquer le type de nos variables dans le nom de celle ci : * $S_foo = 'bar'; * $I_foo = 10; * $O_foo = new Bar(); * $A_foo = array(); * ......

Bonnes pratiques PHP - Geoffrey

Geoffrey — Mon, 30 Oct 2006 16:39:59 +0100

C'est une habitude à prendre on va dire, c'est vrai que d'un point de vue lecture, il semble plus logique de placer la variable à gauche de l'équation, et j'avoue que moi même je ne le fais pas systématiquement, mais ça peut être une source de gain de temps considérable quand on oublie malencontreusement un ou deux signes égal puisqu'assigner une variable à une constante provoque une erreur fatale :-)

Bonnes pratiques PHP - giz404

giz404 — Mon, 30 Oct 2006 15:14:35 +0100

Mettre l'opérande après la variable à tester me choque toujours. Enfin côté lisibilité c'est un peu surprenant non ?

Bonnes pratiques PHP - Geoffrey

Geoffrey — Mon, 30 Oct 2006 14:43:35 +0100

9069, c'est mon id sur http://www.parano.be/ (sur lequel j'ai publié ce petit "exercice" avant de le publier ici).

Sinon pour la notation "hongroise" (j'ai cru lire quelque part il y a quelques temps que ce n'est pas vraiment ça la notation hongroise), je ne suis pas fan non plus: je pense qu'un code bien pensé n'a pas besoin de ce genre d'artifice pour être compréhensible :-)

Bonnes pratiques PHP - Ben

Ben — Mon, 30 Oct 2006 13:00:42 +0100

Une remarque, quand je lis le code j'me pose la question, d'où vient le 9069.
C'est sûr que là c'est qu'un exemple, mais c'est une bonne pratique aussi de ne pas utiliser de valeurs obscures dans son code :)

Bonnes pratiques PHP - NiKo

NiKo — Mon, 30 Oct 2006 12:48:00 +0100

$mixedResult ? Pouah ! Au pire *pour la lisibilité* tu peux retyper la variable :

echo (string) $result;

Mais on atteint un peu les limites de PHP qui n'est pas typé fondamentalement.

Bonnes pratiques PHP - Mère Teresa

Mère Teresa — Mon, 30 Oct 2006 11:26:45 +0100

Les variables étant faiblement typées en PHP, ajouter un préfixe de type peut induire un développeur en erreur, en lui laissant croire que $booVariableName est forcément un booléen.
D'autant plus que certaines fonctions de PHP renvoient un tableau si ok et false si pas ok : comment nommer la variable ? $arrOrbooResult ?

Bonnes pratiques PHP - Gameplayer

Gameplayer — Mon, 30 Oct 2006 11:16:45 +0100

J'aime assez rajouter au niveau des noms de mes variables une information sur le type :

$strVariableName ==> chaîne
$booVariableName ==> booléen

Après ca peu parfois surcharger les noms quand on commence a utiliser des objets (préfix "obj" ? préfixe dépendant de l'objet ?)

Vous en pensez quoi ?

Bonnes pratiques PHP - Sylvio

Sylvio — Mon, 30 Oct 2006 09:49:34 +0100

Pas mal ce petit exercice.
Je vous conseille à ce propos l'excellent livre "PHP5 Best practices" (dispo en français).
Cet exercice peut-être pas mal pour un petit test lors d'un entretien d'embauche.

Bonnes pratiques PHP - Geoffrey

Geoffrey — Mon, 30 Oct 2006 08:59:45 +0100

bon ben c'est pas encore aujourd'hui qu'on me convertira aux espaces :p

Bonnes pratiques PHP - NiKo

NiKo — Mon, 30 Oct 2006 07:15:29 +0100

Pour les tabulations, c'est un peu les goûts et les couleurs, mais princiapelement parce que la taille d'une tab par défaut varie d'un soft à l'autre : j'en utilise pas mal de différents (eclipse, notepad++, vim, etc.), et pas toujours chez moi (donc je peux pas toujours toucher à la conf.)

Les espaces me garantissent un affichage conforme partout. Rien de plus énervant d'ouvrir du code qui a été indenté sous tel soft et qui rend différemment sur celui que tu utilises :|

Bonnes pratiques PHP - Geoffrey

Geoffrey — Sun, 29 Oct 2006 23:29:36 +0100

Je n'avais pas pensé au 4. de NiKo, mais effectivement, on peut considérer ça comme une bonne pratique :p

Par contre très perspicace remarque de Hades (mais mal formulée): le test de la constante 9069 en la plaçant a gauche de l'équation permet d'éviter les prises de têtes sur un truc genre:

if ($code_author = 9069) {

qui renvoient systématiquement true et qui peuvent prendre des heures à débuguer (nottament quand c'est la première fois qu'on tombe dessus).

Pour le $code_author, on assume qu'il a été déclaré plus haut :-)

Sinon, je suis curieux de savoir ce pourquoi tu n'es pas fan des tabs NiKo :o (loin de moi l'idée de lancer un troll espaces vs tabs, tout le monde sait que tabs > * hem)

Bonnes pratiques PHP - Christophe

Christophe — Sun, 29 Oct 2006 22:38:18 +0100

NiKo à tout dit.

Bonnes pratiques PHP - NiKo

NiKo — Sun, 29 Oct 2006 19:24:05 +0100

1. Déclarer ses variables (sauf que là $code_author n'a pas été déclaré ^^)
2. Tester le typage
3. Utiliser des noms de variables parlants
4. Utiliser des noms de variables anglophones
5. Pas de shorts tags
6. Indenter son code (perso je suis pas fan des tabulations)
7. Respecter un standard de codage (genre Pear)

Bonnes pratiques PHP - Hades_L

Hades_L — Sun, 29 Oct 2006 19:20:24 +0100

utilisation du tag <?php au lieu du short_tag
Déclaration/initialisation de variable avant de l'utiliser ($code_is_ugly)
test de l'opérande avant la variable (9069 avant $code_author)
vérification des types (===)

J'en vois pas plus :/ (à supposer que j'ai relevé de bonne choses déjà)