Discussion :

[flipflip]

Bonjour à tous,

Je dev depuis maintenant deux ans une application interne pour le boulot. Au début il y avait juste quelques fonctions basique, puis les demandes d'évolutions arrivent au fur et à mesure de l'utilisation. Au début c'était un bon gros fichier php découpé en fonctions. Puis est venu une 1ere re-écriture et le passage à la POO, un peu dans la douleur pour moi car je ne suis pas dev initialement donc j'ai été obligé de pas mal me documenter et je ne suis pas convaincu d'être totalement POO ready

Je suis à la veille de pousser une grosse mise à jour technique du programme et je me pose des questions sur le futur de l'appli et aussi la possibilité qu'une autre personne intègre le dev avec moi.

Pour la partie PHP, chaque classes, fonctions sont commentés. Cela explique pourquoi, ce que qui est attendu en paramètres et le résultat renvoyé.
Pour javascript (oui il y a du jquery), j'ai essayé de reproduire une presque POO avec le même principe de commentaires.
Il y a du CSS... Mais c'est moins grave

Le décors est planté et maintenant comment encore améliorer le code ? Par exemple comment inventorier toutes les variables, contrôler que l'usage fait est bien celui attendu, qu'il n'y ai pas de doublons... Pareil pour les fonctions.

En gros c'est un audit que je souhaite faire, mais je ne sais pas trop par où commencer ni quel outils utiliser (je dev avec Atom sous Linux).

Avez-vous des pistes ou quelques lectures pour m'éclairer ?

Merci d'avance.

[grunk]

Si tu as commenter tout ton code de manière standard tu devrais pouvoir générer une doc avec PHPDoc ou Doxygen.
Ce qui aura le mérite de te faire un listing complet des classes , méthodes et variables.
Ca peut être intéressant pour un nouvel arrivant.

Ensuite coté organisation , il n'ya pas de règle , mais si on suit les PSR (PHP Standard Recommendations) on remarque assez vite que tous les projets un peu sérieux s'articulent un peu de la même facon :
- L'utilisation de composer
- des namespace
- une séparation MVC ou MVVM
- Utilisation des design pattern les plus courants

Si tu ne coche pas un de ces 4 points , il faut l'envisager.

Il faut envisager de migrer vers la version de PHP la plus récente. Un outil comme Rector permet de le faire.
Pour finir il faut passer ton code aux analyseur statiques pour améliorer la qualité intrinsèque du code et éventuellement y trouver des bugs. Un bon outil est PHPCodeSniffer

[laurentSc]

- Utilisation des design pattern les plus courants

Pourrais-tu les citer, STP ?

[grunk]

Il y'en à 22 , donc je vais pas tous les citer mais plutôt te rediriger vers un site qui en parle très bien : https://refactoring.guru/fr/design-patterns/catalog

Attention , il ne faut pas forcément toujours les utiliser , mais ce sont des solutions éprouvées à des problèmes de conception courants que tout le monde rencontre à un moment ou un autre. Il faut juste savoir qu'ils existent pour les utiliser à bon escient