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 @Feufochmar @TheKinrar Le "problème" est, comme souligné, que B ne peut que être non nul à la seconde partie du test de la seconde ligne (sinon tu n'y arriverai pas, du fait de l'évaluation booléenne en court-circuit). D'où le verdict "test toujours vrai, inutile".
Si tu arrives à la première condition de la troisième ligne, c'est que A n'est pas nul (sinon, l'un des deux précédents tests aurait retourné vrai). D'où le même verdict, le bidule est inutile.
@NuitsDeChine @TheKinrar @Feufochmar Merci a vous trois :)
j'ai enfin compris :)
@TheKinrar @Feufochmar @ralmn Je précise (500 caractères, finalement, c'est pas toujours assez) "test inutile car toujours vrai et inclus dans un ET"