Discussion :

[grabriel]

Bonjour,

On m'a demandé de faire une revue de code sur un développement en PHP J'ai regardé à droite et à gauche sur internet à la recherche d'outils de revue de code pour le php et j'en ai vu plusieurs mais pas vu de comparatif, n'ayant pas le temps de tout tester, est-ce que quelques personnes présentes sur le forum ont déjà utilisé ces outils.

Checkstyle
Php_CodeSniffer
Snoopy
trouvé là : http://www.clochix.net/post/2006/09/...mmation-en-php

ou Phpcheckstyle
http://developer.spikesource.com/wik...checkstyleNews


Je ne vois pas trop l'utilité d'outils de ce genre, pour moi un code à jour, avec des commentaires et avec lequel on peux générer une doc (Doxygen ou documentor) suffit pour être "Valide". Après qu'il respecte les conventions de Pear ou celle-ci qui ressemble au java : http://www.blueshoes.org/en/develope...ng_guidelines/
c'est certes un plus, mais c'est loin d'être le principal élément. Ce que je veux dire c'est que pour moi le plus important c'est pas le style d'indentation du code. C'est comme en html on peux pondre du code imbuvable qui passera les validations de la W3C...

Qu'en pensez vous???

Merci

[zyongh]

Je n'ai pas testé d'outils spécifique à ce sujet. J'utilise principalement la Reflexion (API OOP de PHP5). Ensuite en ce qui concerne ce que tu dis, effectivement, si le code est bien écrit et bien documenté alors je ne vois pas l'intérêt de passer du temps (c'est de l'argent) à vérifier que le code est bien indenté ou non et autres subtilités. Tant qu'on s'y retrouve quand on se replonge dedans quelques mois plus tard...

Par contre, de là à faire du code inbuvable sous prétexte qu'il est tout de même validé par W3C (ou autre), je ne suis pas d'accord. Mieux un code est écrit, plus il est facile pour celui qui repasse derrière de s'en accomoder.

Respectons les suivants...

[is_null]

C'est clairement avec un framework de test-unitaires qu'on peut efficacement tester la qualité d'un code.
Notamment depuis la version de developpement de phpunit qui permet d'analyser la couverture du code générée par xdebug.
Aussi, cette dernière version permet d'analyser les redondances de code, aidant ainsi à la re-factorisation.
Finalement, il est évident que les commentaires sont le plus important, il faut pouvoir comprendre un script ou une classe en lisant seulement les commentaire.