En tant que développeurs que nous sommes, l'état d'esprit doit rester toujours ouvert et sceptique en même temps.
Ouverte, car nous ne savons pas quand un développeur peut nous surprendre, et sceptique quant à nos propres idées car nous oublions souvent qu'en génie logiciel, il n'y a pas une seule façon correcte d'implémenter une solution. La raison d'être de nos solutions pourrait avoir du sens pour nous et n'en avoir pour les autres. Derrière une odeur de code, il pourrait y avoir une excellente idée. Peut-être que le développeur n'a pas trouvé le moyen de l'exprimer correctement.
Parce que nous (les humains) sommes terribles à communiquer, ne faites pas de fausses hypothèses, soyez prêt à demander au propriétaire du code le code que vous examinez. S'il échoue à coder l'idée selon les normes de l'entreprise, en tant que développeur principal, soyez prêt à le guider également.
Voici l'approche subjective. L'approche objective, l'OMI, est très bien expliquée dans cette question .
En plus du lien ci-dessus, l'ensemble des objectifs à atteindre (maintenabilité, lisibilité, portabilité, cohésion élevée, couplage lâche, etc.) ne sont pas nécessairement les Dix Commandements. Vous (l'équipe) devez être en mesure d'adapter ces objectifs à un point où l'équilibre entre qualité et productivité rend le travail confortable et «habitable pour les développeurs».
Je proposerais l'utilisation d'outils d'analyse de code statique pour mesurer la progression de la qualité en fonction de ces objectifs. Des outils comme SonarQube nous fournissent des portes de qualité et des profils de qualité qui peuvent être personnalisés en fonction de nos priorités. Il nous fournit également un outil de suivi des problèmes, où les développeurs peuvent être ciblés avec des problèmes liés à l'odeur de code, aux bogues, aux pratiques douteuses, etc.
Ce type d'outils peut être un bon point de départ, mais comme je l'ai dit, restez sceptique. Vous pouvez trouver certaines règles dans Sonar sans signification pour vous, alors n'hésitez pas à les ignorer ou à les supprimer de votre profil de qualité.