Version imprimable
Bonjour,
dans le code d'une application que l'on a récupéré en maintenance, on a du code comme ceci :
Ce code n'est pas détecté comme une source possible de NullPointerException dans Sonar.Code:if ((!text.equals("") && (text !=null)){...}
Sauriez-vous comment rajouter cette règle simplement ?
D'avance merci pour votre réponse,
Bonjour,
J'aurais parié que la règle "Broken Null Check" serait levée ?
Est-ce que tu l'as activée ?
Mathieu
Bonjour,
cette règle est bien activée et en critical, mais elle ne couvre pas ce cas :
Elle couvrirait plutôt ces cas :Citation:
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.
Code:
2
Bonjour,
As tu eu une réponse à ta question stp ?
Car cela m'intéresse aussi !
Citation:
Envoyé par 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. :?
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 :
Ca permet d'éviter le check sur null.Code:if ((!"".equals(text)){...}
Ca peut se vérifier avec la règle "String literals should be placed on the left side when checking for equality"