• [^] # Re: return early pattern

    Posté par . En réponse au journal La plus belle ligne de code. Évalué à 3.

    tout simplement parce que le diff la merge request ne passait pas verticalement avec toute cette indentation.

    Autant les pyramids of doom sont un VRAI probleme, et je matraque constamment a mes équipes que guard else c'est pas fait pour le chien (oui, on fait du swift, et guard a en cadeau bonux une garantie du compilateur que le block return ou fatalError()), que la lisibilité et réduire le contexte a garder en tete en relecture est plus important qu'une règle arbitraire "un seul return par fonction", autant l'excuse du "le diff est pas lisible" est assez bidon.

    Utilise un outil different si ton outil est pas capable de faire des trucs de base. J'ai eu le meme débat (assez houleux) a propos de nos règles PMD qui interdit les "space terminated strings", l'excuse donnée étant que le mec qui a mit au point les règles et qui faisait pas mal de review utilisait un outil des bois qui était pas foutu de filtrer les whitespace des diffs.

    Bon, apres, si tu fais surtout du python, effectivement, c'est une autre histoire.