Discussion :

[Papy214]

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 :

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
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 :

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

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 basemessage).

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

Code :

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

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 ?

Papy

[Papy214]

Je viens de contourner le problème en faisant:

Code :

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

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.

[kunfuka]

Pourquoi tu n'utilises pas les fichiers Resource ?

http://msdn.microsoft.com/fr-fr/libr...fy(VS.80).aspx

[Papy214]

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

[neptune]

Tu y étais presque:

Code :

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

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

[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

[Guulh]

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.

[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.

[neptune]

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.

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.

[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