Discussion :

[jeedev]

Bonjours à tous,

j'ai du mal à avoir une bonne stratégie pour gérer les exception dans ma classe alors j'explique.

J'ai une interface INatureServiceMetier

Code :

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

1
2
3
public interface INatureServiceMetier extends INatureService {
public Nature creerNature(Utilisateur utilisateur, Nature nature) throws ?????;
}

La méthode creerNature peut levé les exceptions suivantes :
1- Si l'utilisateur n'a pas les privilèges de créer une nature
2- Si la nature existe déja dans la table nature
3- Si utilisateur est null
4- Si nature est null
5- S'il y a une erreur technique tel que le serveur de base de données est arrêté



Question :
est ce que je doit créer une exception pour chaque cas de 1 à 5 que j'ai numéroté ou il y a une autre façon de faire les choses.

merci pour votre aide

[adiGuba]

Salut,


Dans un premier temps il te faudra définir pour chaque type d'erreur que tu rencontres l'importance de l'exception, afin de forcer ou non la gestion de ces exceptions.

Tu as en effet deux possibilitées :

• Utiliser des checked-exceptions (qui hérite de Exception), qui forceront l'utilisateur de la méthode à traiter l'exception (soit via un try/catch, soit par un nouveau throws).
  Tu utiliseras ceci pour les cas d'erreur "probable" qui devront être traité.
• Utiliser des unchecked-exceptions (qui hérite de RuntimeException), qui pourront être ignoré sans problème par le développeurs (dans ce cas elles font planter le thread courant en remontant).
  Tu peux utiliser ceci pour les erreurs moins "fréquente". Par exemple pour tout ce qui correspond à des erreurs de développement...

est ce que je doit créer une exception pour chaque cas de 1 à 5 que j'ai numéroté ou il y a une autre façon de faire les choses.

Déjà il faudrait si possible que tu réutilises les exceptions existantes.

Par exemple les cas 3 & 4 vont remonter une NullPointerException, ce qui est le plus approprié : l'exception est bien connu des développeurs (ils corrigeront l'erreur plus vite) et c'est une unchecked-exception (pas besoin de try/catch).



Pour les cas 1, 2 & 5, perso je remonterait directement l'exception que tu reçoit dans ta méthode (je suppose que tu dois en recevoir une). En effet pourquoi créer une nouvelle exception ???



Sinon, si vraiment tu veux créer tes propres exceptions, tu peux utiliser une arborescence avec une classe commune. Cela permet par la suite des possibilité de traitement groupé (via la classe commune) ou non (via les sous-classes).

a++

[jeedev]

merci pour les explications,

Voici ce que j'ai fait

1- J'ai créer une classe SrcException (Classe mère pour mes exception)

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
public class SrcException extends Exception {
 
    public SrcException() {
        super();
    }
 
    public SrcException(String message, Throwable cause) {
        super(message, cause);
    }
 
    public SrcException(String message) {
        super(message);
    }
 
    public SrcException(Throwable cause) {
        super(cause);
    }
 
}

2- j'ai créé les autres exceptions

Code :

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

1
2
3
 
public class JuridictionInsuffisanteException extends SrcException {}
public class NatureExisteDejaException extends SrcException {}

3- j'ai créé mon service

Code :

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

1
2
3
4
 
public interface INatureService {
public Journee creerNature(Nature nature, Utilisateur utilisateur) throws JuridictionInsuffisanteException, NatureDejaExistException, SrcException; 
}

4- dans l'implémentation

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 
public interface NatureService implements INatureService {
 
public Journee creerNature(Nature nature, Utilisateur utilisateur) throws JuridictionInsuffisanteException, NatureDejaExistException, SrcException{
   try{
 
       }
   catch(JuridictionInsuffisanteException e){
   }
   catch(NatureDejaExistException e){
   }
   catch(SrcException e){
    }
}
}

est ce que je suis dans le bon sens ?