Discussion :

[menakikou]

Bonjour,

je suis sur la qualimétrie, j'ai remarqué le fait d'utiliser cette exeception "NullPointerException "pouvait générer des problèmes.

Je ne comprend pas pourquoi ?

Par exemple dans cette ligne de code :

Code :

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

if (value == null) throw new NullPointerException("La valeur à ajouter est nul.");

Que peut il se passe ? La machine virtuelle peux planter ?

Si je remplace NullPointerException par IllegalArgumentExeption , est ce que ce serai mieux ? On m'a dit de remplacer par IllegalArgumentExeption quelle différence vas t il se passer ?

Merci d'avance , je sais je pose beaucoup trop de question

[Pill_S]

Hello,

non, il n'y a pratiquement aucune différence entre ces deux exceptions. NPE est, traditionnellement, levée lorsque l'on tente d'utiliser un champ ou méthode d'une référence null. J'aurais donc tendance à conseiller d'utiliser IAE, mais bon, il y a très peu de différence...

[menakikou]

J'utilise en fait un plugins sous eclipse "PMD" qui corrige mon code, et quand je lève l'exéception NullPointerException . il me mets


Avoid throwing a NullPointerException - it's confusing because most people will assume that the
virtual machine threw it. Consider using an IllegalArgumentException instead; this will be
clearly seen as a programmer-initiated exception.

Je vois toujours pas pourquoi il est préferable de lever IllegalArgumentException comme exeption au lieu de NullPointerException ?

Merci

[benjamin_musique]

Salut,

la réponse me semble limpide pour ma part :

- une NPE est une erreur technique, qui est habituellement remontée lorsque le cas d'utilisation (ici: ajouter une valeur nulle) n'est pas géré. C'est une erreur purement technique

- toi, tu veux gérer ce cas, c'est pourquoi on te conseille d'utiliser une IllegalArgumentException pour ton message d'erreur qui est déjà plus "fonctionnelle" (par opposition à technique)

Concrètement, on est d'accord que ça revient au même, c'est juste une "best practice", tu en fais ce que tu veux.

[henolivier]

prenons en exemple ton élément value, si tu fait ça alors que value est null :

Code :

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

if ( value.equals("") ) return;

ton programme a l'exécution va lancer automatiquement une NPE pour avertir que tu ne peux pas appeler la méthode equals sur ton élément value.

Le fait de mettre

Code :

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

if (value == null) throw new NullPointerException("La valeur à ajouter est nul.");

Revient a la même exception qu'au dessus mais est gérée par toi mémé,
le conseil de PMD ici est de différencier une exception que tu lances toi directement car l'argument est incorrect d'une exception lancable par la JVM directement,
la finalité est la même (enfin, en fonction des catch de ton application) mais permettra en fait a quelqu'un voyant les logs de comprendre plus facilement le type de problème et surtout de comprendre que tu gères l'exception et que ce n'est pas la JVM qui a lance celle-ci

En écrivant ça :

Code :

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

if (value == null) throw new IllegalArgumentException("La valeur à ajouter est nul.");

Si l'exception est lancée, le logs sera lisible comme étant un problème sur un argument de méthode (et non pas une exception non gérée par ton application)

[menakikou]

Merci j'ai compris