Conseil pour les code reviews : Privilégiez "Je ferais" plutôt que "Il faut" En code review, évitez les formulations directives qui peuvent paraître accusatrices ou condescendantes. ❌ Formulations à éviter : • "Il faut utiliser Optional ici" • "Tu dois ajouter des tests unitaires" • "Cette méthode est trop longue" ✅ Formulations collaboratives : • "Je ferais un Optional.ofNullable() pour éviter les NPE" • "J'ajouterais quelques tests sur les cas limites" • "Je découperais cette méthode en plusieurs plus petites" Exemples concrets : ❌ "Tu devrais utiliser un Stream au lieu de cette boucle" ✅ "Je remplacerais cette boucle par un list.stream().filter().collect() pour plus de lisibilité" ❌ "Cette classe viole le principe de responsabilité unique" ✅ "J'extrairais la logique de validation dans une classe dédiée" ❌ "Il manque la gestion d'erreur" ✅ "Je proposerais d'encapsuler ça dans un try-catch ou de propager l'exception" Principe fondamental : Reviewez le code, pas le codeur. Cette approche encourage l'échange constructif et maintient un climat de confiance dans l'équipe.