Euh soit je comprends pas ce que me dit Sonar, soit c'est un faux positif https://mastodon.xyz/media/d40yL93NOdbM6mg91sM
@ralmn Ce n'est pas un faux positif. La façon dont tu as écrit ton code fait qu'à cet endroit, articleAdditionnelApres ne peut pas être null si articleAdditionnelAvant est null. J'utiliserai plutôt des if/elseif avec retour unique ici, c'est souvent un peu plus clair.
@Feufochmar @TheKinrar En gros la j'ai
A = null && B = null
A = null && B <> null
A <> null && B = null
et je continue si A <> null && B <> null
Je vois pas ou est le problème .... https://mastodon.xyz/media/zTU_defG_DFKV5aYKGY
@ralmn @TheKinrar Le problème, c'est que si tu quittes la fonction via un return dès que tu entres dans une des conditions, le code qui suit n'est jamais exécuté. Donc quand tu vérifies la condition (A == null && B <> null), B est déjà non-null si A est null, car sinon il aurait déjà quitté la fonction, à cause de la condition (A == null && B == null) qui précède. La deuxième vérification (B <> null) est inutile dans le deuxième cas.