Discussion :

[LESOLEIL]

suppression message

[Satch]

bien sur que c'est correct. Quelques classes JAVA lancent des runtimes exceptions dans les constructeurs (NullPointerException le plus souvent)

D'ailleurs je me demande si une NullPointerException ne serait pas plus appropriées là.

[LESOLEIL]

euh, messaheHandler est de type MessageHandler
mais je ne sais pas si c'est un string ou pas.

[Satch]

Huu ?
Quel rapport ?

[LESOLEIL]

c'est pour le test messageHandler ..si il esgal à null, donc c'est pas un IllegalArgument?

[adiGuba]

Salut,

bien sur que c'est correct. Quelques classes JAVA lancent des runtimes exceptions dans les constructeurs (NullPointerException le plus souvent)

D'ailleurs je me demande si une NullPointerException ne serait pas plus appropriées là.

+1

Comme tu vérifies que ton objet n'est pas null cela est amplement suffisant...

Et je dirais même que si tu utilises une méthode de cet objet dans le constructeur, tu n'as même pas à traiter l'erreur toi-même :

Code :

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

1
2
3
4
5
6
7
8
9
	/**
         * ...
         * @param messageHandler ...
         * @throws NullPointerException messageHandler is null.
         */
	public OutgoingNotificationHandler(MessageHandler messageHandler){ 
	      this.messageHandler = messageHandler;
	      this.messageHandler.method();
	}

Si le paramètre messageHandler est null, alors l'appel de method() renverra un NullPointerException...
Par contre c'est toujours bien de documenter ce style d'exception dans le javadoc

a++

[LESOLEIL]

euh, je dois créer une methode qui renvoit une exception, mais je mets quoi dans cette méthode.

[Satch]

Si tu n'appelles pas de méthode sur le messageHandler dans ton constructeur, lance juste une NullPointerException s'il est null et ça suffira. Mais à condition de bien documenter comme l'a dit adiGuba.

NullPointerException est bien choisie ici, puisque tu testes si un truc es null.

Un IllegalArgumentException serait plutôt pour une méthode qui attend un entier entre 5 et 10. Si on lui donne 11, l'argument n'est pas bon.

[Actarus78]

Salut,

Pour moi ta méthode est correcte.

Tu testes un argument de ta fonction, je suppose que c est pour du debug afin d'avoir une trace lisible autre que celle retournée par la jvm. Dans ce cas, je préciserai dans mon message d'erreur le nom de mes variables qui ne doivent pas être null.

Code :

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

1
2
3
4
5
6
7
 
 
JavaDoc
 
IllegalArgumentException indicates that a method has been passed an illegal or inappropriate argument. 
 
NullPointerException thrown when an application attempts to use null in a case where an object is required

Dans ton cas tu testes bien un argument passé à ton fonction par conséquant ton IllegalArgumentException était approprié. Mais bon, ton test qu'il renvoit un NullPointerException ou un IllegalArgumentException au final il te renvera bien une exception ^^

Edit: Oui c est Ok pour tes 2 vars

[LESOLEIL]

merci pour les infos.