Discussion :

[touftouf57]

Bonjour,
Je souhaiterais mettre dans un winform, ou plutot dans le controleur, un méthode privé qui s'occupera de traiter les exceptions.
A l'intérieur je voudrais faire un switch sur le type de l'exception (xyzException, abcException....). Mais je ne trouve pas la syntaxe concernant le switch.
voila ce que j'ai essayé:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
private void GestionException(Exception e) {
            switch (e.GetType()) {
                case typeof(StoreProcException):
                    if (MessageBox.Show(e.Message + "\nVoulez-vous la recréer?", e.GetType().ToString(), MessageBoxButtons.YesNo) == DialogResult.Yes)
                        CreerTable.creerStoreProcManquante(((StoreProcException)e).NomStoreProcManquante);
                    break;
.......
                default:
                    break;
 
            }
        }

Avec ce code, cela ne veut pas compiler, déjà au niveau du switch (la valeur d'un type intégral est attendue). Et au niveau du case (une valeur de constante est attendue).

J'ai essayé les Type.getType(e),typeof(e), Type.GetTypeCode(StoreProcException)..., sans succès.
J'ai chercher sur la toile, et en appliquant les codes proposés cela ne fonctionne pas.

Quelqu'un aurait-il un idée?
Merci

[StormimOn]

Le mot clé switch ne fonctionne qu'avec des valeurs intégrales (chaînes de caractères, entiers, énumérations).

Pour faire ce que tu veux, utilise le mot clé is.

[touftouf57]

OOk,

donc je suis condamné à faire des if en cascade, et n'utiliser le if que pour des enum, entier, string.

Merci

[tomlev]

Le mot clé switch ne fonctionne qu'avec des valeurs intégrales (chaînes de caractères, entiers, énumérations).

Techniquement, string n'est pas un type intégral... mais il est aussi supporté par switch

donc je suis condamné à faire des if en cascade

Ben si tu fais ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
if (e is StoreProcException)
{
    ...
}
else if (e is AutreException)
{
    ...
}
else
{
    ...
}

C'est pas tellement plus lourd à écrire qu'un switch...

[touftouf57]

Tu as tout as fait raison. Et c'est vrai que niveau "écriture" ça se tiens.

Mais au départ je pensais qu'on pouvait switcher sur le type. Je suis passé par les if et c'est ok.

Merci.

[Mehdi_ab]

Bonjour,


y a une solution de contournement pour pouvoir utiliser l swich, c'est le faite d'appeler le tostring() du Type et l comparer avec l tostring du typeof

Code :

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

1
2
3
4
5
6
7
8
swich (e.GetType().ToString()){
 
case typeof(montype).ToString():
 
break;
 
 
}

[_PascalC_]

typeof(montype).ToString() n'étant pas de nature constante ça me suprendrait très fortement que ça compile ton exemple...

[tomlev]

typeof(montype).ToString() n'étant pas de nature constante ça me suprendrait très fortement que ça compile ton exemple...

je confirme, ça ne compile pas... de toutes façons c'est un peu crade comme méthode

[NazOok]

Bonjour, ce sujet date un peu mais je rencontre actuellement ce problème !

Après avoir recherché sur le net, je suis tombé sur pas mal de réponse ou la solution proposé est bien la cascade de "if else if".. Mais je suis aussi tombé quelques fois sur cette réponse ou il s'agit d'écrire deux petites classes :
http://codingly.com/2009/10/19/switch-sur-un-type-en-c/

Je voulais donc votre avis sur cette méthode ? La lisibilité me parait un peu lourde mais au niveau performance, est-il mieux d'utiliser la cascade de "if else" ou cette méthode ?

Merci d'avance :-)

[touftouf57]

Si les types correspondent à des classes sur lesquelles tu as la main, j'utiliserais plutot le DP Visitor.

De cette façon, il n'y a plus de "If". C'est le compilateur qui t'oriente vers la bonne signature de méthode.
Visitor

[NazOok]

D'accord, je viens de me documenter et je vois un peu la chose !
Mais dans mon cas perso, le traitement est vraiment court et redondant, ma méthode avec if else étant déjà codée je vais la laisser comme ça..

Mais je garde ce pattern sous le coude, et j'essaierais de la mettre en place plus tard, il pourra m'être utile !
Merci beaucoup.

[pitahaya]

D'accord, je viens de me documenter et je vois un peu la chose !
Mais dans mon cas perso, le traitement est vraiment court et redondant, ma méthode avec if else étant déjà codée je vais la laisser comme ça..

Mais je garde ce pattern sous le coude, et j'essaierais de la mettre en place plus tard, il pourra m'être utile !
Merci beaucoup.

C'est une mauvaise pratique de tester les types. Utilises le polymorphisme. Ou si tu as des besoins spécifiques utilises, par exemple, le pattern "Etat" (cf Design Patterns).

J'ajouterai faire des if (truc is typemachin) est extrêmement lent !!