Exigence et revue de code

l y a près de 10 ans, un manager m’a dit « Tu as un degré d’exigence trop élevé. » faisant référence à mes nombreux retours lors des revues de code.

A l’époque, je faisais du Cobol, mais depuis je suis passé par Java, Javascript et PHP. Et je vous rassure, mes revues de code sont toujours aussi « exigeantes ».

Un exemple

Le cas que j’aime bien présenter pour expliquer ce concept est en Java. L’architecte en charge du projet avait demandé à un petit service Web qui récupère diverses informations de la requête (j’ai oublié la raison exacte de cette demande, mais passons). Il s’agissait de lire un header, un cookie et peut-être une ou deux autres informations.

Voici (en substance) la première version du code proposé par un collègue (la technologie utilisée est du Spring MVC):