De manière générale, on trouve toujours moyen d'améliorer du code... Sauf qu'en général on estime que ce n'est pas économiquement rentable. En l'occurence, c'est plutôt pour gagner du temps à l'avenir que l'on nettoie le code quand on tombe sur des choses améliorables.
De manière plus détaillée, le code est parfois issu de daCode, parfois redéveloppé depuis le passage à Templeet. Des fonctionnalités ont disparu, d'autres ont été ajoutées, puis ont disparu, etc. Donc parfois on peut supprimer des tables SQL ou des champs dans les tables, devenus inutiles. Avant les dépêches, les astuces, les forums, etc. étaient très différents, puis on a commencé à les rendre semblables niveau SQL (c'est pas un hasard si on peut traiter tous les contenus identiquement en page d'accueil), du coup du code est devenu factorisable. Des nombres magiques traînaient ça et là et on les fait disparaître au fur et à mesure. Les évolutions de Templeet ont aussi amené de nouvelles possibilités qui permettre d'écrire plus simplement le même code (ou simplement ont viré des bugs dont les contournements ne sont plus nécessaires). Enfin l'expérience aidant, on doit aussi probablement s'améliorer sur le code du site et le factoriser plus facilement. Et il reste très peu de zones du code où les développeurs actuels n'osent pas s'aventurer, ce qui est plutôt bon signe :).
Ce qui me fait penser que ça fait un moment que je n'ai pas lancé l'infame gruik_cleaner.sh sur le code du site (un script qui essaie tant bien que mal de détecter des duplications de code). Testons : si on fait une détection de blocs identiques de 200 caractères hors blancs, on trouve 5 fois le code des nouveaux tris (l'affichage sur chaque page d'index de contenus qui pourrait être inclus à partir d'un même template apparemment).
# De la clarté du code
Posté par Benoît Sibaud (site web personnel) . En réponse à la dépêche Évolutions sur LinuxFr. Évalué à 8.
De manière plus détaillée, le code est parfois issu de daCode, parfois redéveloppé depuis le passage à Templeet. Des fonctionnalités ont disparu, d'autres ont été ajoutées, puis ont disparu, etc. Donc parfois on peut supprimer des tables SQL ou des champs dans les tables, devenus inutiles. Avant les dépêches, les astuces, les forums, etc. étaient très différents, puis on a commencé à les rendre semblables niveau SQL (c'est pas un hasard si on peut traiter tous les contenus identiquement en page d'accueil), du coup du code est devenu factorisable. Des nombres magiques traînaient ça et là et on les fait disparaître au fur et à mesure. Les évolutions de Templeet ont aussi amené de nouvelles possibilités qui permettre d'écrire plus simplement le même code (ou simplement ont viré des bugs dont les contournements ne sont plus nécessaires). Enfin l'expérience aidant, on doit aussi probablement s'améliorer sur le code du site et le factoriser plus facilement. Et il reste très peu de zones du code où les développeurs actuels n'osent pas s'aventurer, ce qui est plutôt bon signe :).
Ce qui me fait penser que ça fait un moment que je n'ai pas lancé l'infame gruik_cleaner.sh sur le code du site (un script qui essaie tant bien que mal de détecter des duplications de code). Testons : si on fait une détection de blocs identiques de 200 caractères hors blancs, on trouve 5 fois le code des nouveaux tris (l'affichage sur chaque page d'index de contenus qui pourrait être inclus à partir d'un même template apparemment).