Design et gestion d'erreur

Bonjour,
Je me pose une question de conception.

J'ai une application avec une interface graphique et un modèle derrière.

Pour faire simple disons que je gère une série d’événement (qui ont une date et un type).
Mon interface graphique permet à l'utilisateur d'ajouter des événements.

Un évènement peut ne pas être ajouter au modèle pour différentes raisons().

Je cherche donc un mécanisme intelligent pour gérer cela

Voila la conclusion a la quelle je suis arrivé

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

//faire un énumération pour les type d'erreur   public enum AddEvenementErreur { BEFORE_BEGINNING, AFTER_END, TOO_MANY }   faire un exception qui a comme attibut le type d'erreur public class ModeleException extends Exception{ private AddEvenementErreur type; ... ... ... }

---

et dans mon modèle d'avoir une méthode

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

public void addEvenement(Evenement E ) throw ModeleException { //faire des test pour savoir si on peut ajouter l'événement if (test1) { throw new ModeleException(BEFORE_BEGINNING) if (test2) { throw new ModeleException (AFTER_END) if (test3 ){ throw new ModeleException (TOO_MANY)   //ajouter l événement .... .... .... ...   }

---

et dans mon UI d'avoir le suivant

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

try{ addEvenement(e);   } catch (ModeleException e ) {   String error_to_dysplay ; if e.type = BEFORE_BEGINNING {error_to_dysplay = "blabla"} if e.type = AFTER_END {error_to_dysplay = "testets"} if e.type = TOO_MANY {error_to_dysplay = "testets"}   messageDialog.openError(shell,"ERREUR",error_to_dysplay}   }

---

Pensez vous que c'est une bonne manière de réglez le problème ?
Car sur internet on peux trouver des personnes qui disent que les Exception ne sont pas faite pour gérer les entrées des utilisateurs(alors que c'est ce que je fais) mais je ne vois pas de système moins compliqué.

En vous remerciant.

Cordialement

Bonjour,

Je trouve l'utilisation des exceptions tout à fait appropriée, mais j'utiliserais une approche un tout petit peu différente.

Au lieu d'utiliser un enum comme "flag" pour différencier les erreurs j'utiliserai différentes classes (les phrases sont à adapter) :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18

public class ModeleException extends Exception{ } public class BeforeBeginingException extends ModeleException{ public BeforeBeginingException() { super("Impossible d'ajouter un événement avant le début"); } } public class AfterEndException extends ModeleException{ public AfterEndException() { super("Impossible d'ajouter un événement après la fin"); } } public class TooManyException extends ModeleException{ public TooManyException() { super("Impossible d'ajouter un événement car il y en a trop"); } }

---

Dans le modèle :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14

public void addEvenement(Evenement E ) throw ModeleException { //faire des test pour savoir si on peut ajouter l'événement if (test1) { throw new BeforeBeginingException();} if (test2) { throw new AfterEndException ();} if (test3 ){ throw new TooManyException ();}   //ajouter l événement .... .... .... ...   }

---

Et dans l'ihm :

Code:

---

1 2 3 4 5

try{ addEvenement(e); } catch (ModeleException e ) { messageDialog.openError(shell,"ERREUR",e.getMessage()); }

---

Ce que je préfère avec cette méthode est que si tu ajoutes une nouvelle exception dans ton modèle tu n'a pas besoin d'aller modifier la vue. Avec ta méthode si tu rajoutes un nouveau cas à ton enum, tu dois aller dans la vue pour rajouter un if...

Romain.