Discussion :

[BugFactory]

Bonjour,

Je débute avec Sonar Qube et je me demande comment régler proprement ce faux positif.

J'ai un code similaire à celui-ci :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
for(Element element : getList()) {
    if(element.isActive()) {
        // etc
    }
}

Sonar Qube indique un code smell dans if(element.isActive()) : element pourrait être null, ce qui déclencherait un NullPointerException.

Mais element ne peut pas être null, la liste retournée par getList ne contient jamais d'élément null. Et si c'était le cas, c'est un bogue de getList et dans ce cas NullPointerException est le comportement désiré.

Je ne veux pas ajouter un test rien que pour faire plaisir à Sonar. Un test inutile diminue la lisibilité du code, ce qui est exactement le contraire de ce pourquoi on utilise Sonar.

Comment indiquer à Sonar qu'element n'est jamais null?

[Marco46]

La question serait plutôt comment indiquer à ton compilateur que ta variable ne peut jamais être null et est-ce que ta version de Sonar supporte cette fonctionnalité.

J'ai pas fait de Java depuis quelques années mais j'avais à dispo des annotations @NotNull et @Nullable pour gérer ça. Je ne sais pas quel est l'état de l'art en 2019.

[BugFactory]

Merci pour la réponse, mais @Nonnull ne peut s'appliquer qu'à la liste elle-même et pas ses éléments.