Personnaliser une exception

Version imprimable

reBonjour,

Je cherche à personnaliser un message d'exception en fonction de différents différents paramètres et je me heurte à une petite (?) difficulté. Voilà mon code:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
 
    public partial class Form1 : Form
    {
 
        public Form1()
        {
            InitializeComponent();
        }
 
        private void button1_Click(object sender, EventArgs e)
        {
            try
            {
                throw new EntreeSortieException();
            }
            catch (EntreeSortieException entreeSortieException)
            {
                MessageBox.Show(entreeSortieException.Message);
            }
 
            try
            {
                throw new EntreeSortieException("Message personnalisé");
            }
            catch (EntreeSortieException entreeSortieException)
            {
                MessageBox.Show(entreeSortieException.Message);
            }
        }
    }
 
    public class EntreeSortieException : Exception
    {
        public EntreeSortieException()
            : base("Message défini dans la classe") 
        { 
        }
 
        public EntreeSortieException(string message)
            : base(message) 
        {            
        }
 
        public EntreeSortieException(string message, Exception inner) 
            : base(message, inner) 
        { 
        }
 
        protected EntreeSortieException(
          System.Runtime.Serialization.SerializationInfo info,
          System.Runtime.Serialization.StreamingContext context)
            : base(info, context) 
        { 
        }
    }

Mon application gère différents langages (anglais, français, russe, etc.)
Je dois donc ajuster chaque message d'erreur en fonction de la langue utilisée.

Je pensais donc modifier le texte du message dans

Code:

1
2
3
4
5
6
 
        public EntreeSortieException(string message)
            : base(message) 
        { 
              message = "message traduit";
        }

Mais ça ne fonctionne pas puisqu'a ce moment là, le message est déjà utilisé par base:(message).

Pour l'instant, j'ai donc inclus la fonction de traduction dans l'appel à base.
Par exemple dans une autre exception:

Code:

1
2
3
4
5
6
7
 
    public class InvalidFolderException : Exception
    {
        public InvalidFolderException(string folderName)
            : base( "{0} : " + string.Format( Locale._("invalid folder exception!") , folderName) )
        { }
    }

Le problème c'est que ça fait vite des paramètres illisibles.
Alors je me dis que j'ai peut-être loupé une étape importante dans mon apprentissage de cette chose là. Il doit bien y avoir moyen de modifier mon message à ce niveau sans le faire dans le paramètre, non ?

Votre avis sur la question ? :D

Papy

Je viens de contourner le problème en faisant:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
 
    public class InvalidFolderException : Exception
    {
        public static string TexteMessage(string foldername)
        {
            return "{0} : " + string.Format(Locale._("invalid folder exception!"), foldername);
        }
 
        public InvalidFolderException(string folderName)
            : base(TexteMessage(folderName))
        { }
    }

Mais je sais pas, je trouve pas ça très beau comme façon de coder.

16/06/2008, 12h11
kunfuka

Pourquoi tu n'utilises pas les fichiers Resource ?

http://msdn.microsoft.com/fr-fr/libr...fy(VS.80).aspx
16/06/2008, 13h39
Papy214

Parce que je ne peux pas. Cette application fait partie d'un ensemble qui utilise un système de traduction particulier.

Tu y étais presque:

Code:

1
2
3
4
public EntreeSortieException(string message)
        { 
              Message = "message traduit";
        }

16/06/2008, 14h11
Papy214

Erreur 1 La propriété ou l'indexeur 'System.Exception.Message' ne peut pas être assigné -- il est en lecture seule D:\Développements\CSharp\recup mes documents\Les exceptions\Les exceptions\Form1.cs 55 13 Les exceptions

:(

Autant pour moi... Je n'ai pas vérifié si Message était read only ou pas...
Code:

1 2 3 4 5 6 7 8 9 10 11 12 public class MyException : ApplicationException { public MyException(string message) : base(TranslateMessage(message)) { } private static string TranslateMessage(string message) { return "Message traduit"; } }
De toutes facons, je te conseillerais de faire la traduction avant de créer l'exception. Et puis, pour les exceptions personnalisées, diérive d'ApplicationException.

16/06/2008, 14h19
Guulh

Citation:

Envoyé par Papy214

Mais je sais pas, je trouve pas ça très beau comme façon de coder.

C'est comme ça que j'aurais fait aussi, sauf que j'aurai mis cette fonction auxiliaire en private voire protected.
16/06/2008, 14h27
Papy214

Neptune: Donc, ça revient à ce que j'avais finis par faire et que je trouvais pas top mais si c'est la seule solution ...

Guulh: Bon, ben je vais me contenter de ça.

Merci à tous les deux.

:D
16/06/2008, 14h39
neptune

Citation:

Envoyé par Papy214

Neptune: Donc, ça revient à ce que j'avais finis par faire et que je trouvais pas top mais si c'est la seule solution ...

Guulh: Bon, ben je vais me contenter de ça.

Merci à tous les deux.

:D

Oui, mais le plus c'est au minimum refactoriser dans une fonction. Le mieux c'est de s'occuper de la traduction avant d'envoyer le message dans l'exception.
16/06/2008, 15h29
Papy214

l'exception pourra être lancée depuis plusieurs endroits du code.
Et je veux que chaque affichage potentiel d'erreur soit parfaitement identique.
En faisant comme ça, je réduis le risque de différences entre les messages.
Sinon, j'aurais pu faire directement chaque fois que c'est nécessaire:

throw new Exception(Local._("monmessage"));

et me passer de créer une exception spécifique .

De plus, nous allons utiliser xgettext pour extraire toutes les chaines localisables de l'application. Là, une seule sera extraite. Sinon, autant de fois que d'appel à l'exception.

Maintenant, pour la traduction, j'y peux rien. C'est le client qui veut ça :cry:
16/06/2008, 15h36
neptune

Cela revient au meme, vu que tu devras écrire x fois le meme code pour la meme exception!

Et puis getttext ne donne jamais qu'un seul id pour deux chaines strictement identiques, non?