Comprehénsion de this
Résolu
Bonjour,
Je voulais savoir si mon code était juste?
Mon but, lancer une Exception quand un étudiant veux changer de série alors qu'il y est déjà ou qu'il est délégué de classe.
Je voulais savoir si mon code était juste?
Mon but, lancer une Exception quand un étudiant veux changer de série alors qu'il y est déjà ou qu'il est délégué de classe.
public void changerSerie(Serie serie){
if (this.serie == serie) throw new IllegalArgumentException("L'étudiant est déjà dans cette série");
if (this == this.serie.getDelegue()){
throw new IllegalArgumentException("Un délégué ne peut pas changer de série");
} else
this.serie = serie;
}
1 réponse
-
ModérateurBonjour,
A priori c'est bon.
Remarques :
Si tu as un attribut qui s'appelleserie
, évites d'appeler également ton paramètre de méthodeserie
, ça te complique la vie pour rien.
Il n'est pas vraiment nécessaire de lever une exception si l'étudiant est déjà dans la série, car au final ça ferait juste rien, mais en sortie de méthode tu auras bien la série voulue de configurée.
En général dans ces cas là on renvoie un booléen, qui vaut true si l'objet a changé.
NB. Ne pas oublier le cas où on passerait null en valeur.
Soit c'est autorisé et il faut vérifier queserie != null
avant de faireserie.getDelegue()
.
Soit c'est interdit et il faut lever une exception pour ce cas.public void changerSerie(Serie nouvelleSerie) { if (nouvelleSerie == null) throw new IllegalArgumentException("null n'est pas une série valide"); if (serie == nouvelleSerie) return false; if (serie != null && serie.getDelegue() == this) throw new IllegalStateException("Un délégué ne peut pas changer de série"); serie = nouvelleSerie; return true; }
