Discussion :

[v4np13]

Bonjour,

dans le code d'une application que l'on a récupéré en maintenance, on a du code comme ceci :

Code java :

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

if ((!text.equals("") && (text !=null)){...}

Ce code n'est pas détecté comme une source possible de NullPointerException dans Sonar.

Sauriez-vous comment rajouter cette règle simplement ?

D'avance merci pour votre réponse,

[Influenza]

Bonjour,

J'aurais parié que la règle "Broken Null Check" serait levée ?
Est-ce que tu l'as activée ?

Mathieu

[v4np13]

Bonjour,

cette règle est bien activée et en critical, mais elle ne couvre pas ce cas :

The null check is broken since it will throw a Nullpointer itself. The reason is that a method is called on the object when it is null. It is likely that you used || instead of && or vice versa.

Elle couvrirait plutôt ces cas :

Code :

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

1
2
if ((!text.equals("") || (text !=null)){...}
if ((text !=null) || (!text.equals("")){...}

[catmary]

Bonjour,

As tu eu une réponse à ta question stp ?
Car cela m'intéresse aussi !

Bonjour,

dans le code d'une application que l'on a récupéré en maintenance, on a du code comme ceci :

Code java :

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

if ((!text.equals("") && (text !=null)){...}

Ce code n'est pas détecté comme une source possible de NullPointerException dans Sonar.

Sauriez-vous comment rajouter cette règle simplement ?

D'avance merci pour votre réponse,

[v4np13]

Salut,

non, hélas. J'aurai d'autres règles à ajouter depuis mais pas trouvé. Peut-être est-ce couvert par une version plus récente ? J'utilise toujours la même version pour éviter de casser certains builds. J'avais upgradé une fois avant mon topic et ça s'était assez mal passé sur un ou deux projets. C'est assez sensible à faire.

[50Nio]

Des règles qui correspondent à ton cas sont :
"Misplaced Null Check"
"Correcteness - NullCheck of value previously dereferenced"

De plus, on peut écrire ton code plus simplement :

Code :

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

if ((!"".equals(text)){...}

Ca permet d'éviter le check sur null.

Ca peut se vérifier avec la règle "String literals should be placed on the left side when checking for equality"