VS2008 / .net / C# : throw new Exception -> provoque une Exception

salut les gars,

il m'arrive un truc bizarre et je ne sais pas comme gérer une Exception qui provoque une Exception :arf:

dans le code suivant, throw new Exception -> provoque une Exception quand le fichier ne peut pas être converti en image
Code:

1 2 3 4 5 6 7 8 9 10 try { im = Image.FromFile(fichiersRep[i]); } catch (Exception) { throw new Exception("Erreur, le fichier : (" + fichiersRep[i] + ") n'a pas pu etre converti en image"); continue; }
avec le message :

Citation:

l'exception Exception ne peut pas être géré

:roll:

pour faire un test de solidité de l'appli, j'ai renommer un fichier HTML en .gif et je pensais que le try/catch allait me mettre a l'abri de toutes les surprises ... ben non.

quelqu'un a une idée pour réellement gérer l'exception ?
ou autre soluce ?

merci

Je ne vois pas pourquoi tu utilises le continue dans cette situation. J'imagine que tu veux que l'exécution du programmes output un message d'erreur lorsqu'il y a un erreur, mais qu'il continue à rouler quand même.

Un finally pourrait régler la situation dans ton try-catch, ce qui veut dire que peut importe si un erreur a été lancé, ton finally va s'exécuter et continuer l'exécution. Bien entendu, enlève le continue.

Égallement, j'imagine que tu ne fais rien avec ton Exception. Oublie pas que si tu throw une exception, elle doit être reprise dans un autre bloc try-catch. Dans ce cas, je te conseil de toujours lancer des exceptions dans des If.

Dans ton cas, test pour voir si ta variable 'im' retourne quelque chose, sinon lance une exception.

le continue est pour zapper cette occurence de boucle puisque l'image ne peux pas etre validé !

voici le bloc complet :

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
                Image im;
                for (int i = 0 ; i < fichiersRep.Length ; i++)
                {
                    string ext = fichiersRep[i].Substring(fichiersRep[i].LastIndexOf(".") + 1).ToLower();
                    switch (ext)
                    {
                        case "bmp":
                        case "gif":
                        case "exig":
                        case "exif":
                        case "jpg":
                        case "jpeg":
                        case "png":
                        case "tif":
                        case "tiff":
                            break;
 
                        default :
                            continue;
                    }
                    try
                    {
                        im = Image.FromFile(fichiersRep[i]);
                    }
                    catch (Exception)
                    {
                        throw new Exception("Erreur, le fichier : (" + fichiersRep[i] + ") n'a pas pu etre converti en image");
                        continue;
                    }
                    imageList1.Images.Add(im);
                    listView1.Items.Add(fichiersRep[i], i);
                    im.Dispose();
                }

fichiersRep contient la liste des fichiers d'un répertoire et seul les images m'intéressent, ce code me permet d'éliminer tout ce qui n'est pas une image.

Ton continue ne sert à rien actuellement. Le fait de lancer une exception va te faire sortir de ton code.

Il faut voir le "lancer" d'exception (prochaine épreuve olympique peut être) comme une bulle de savon. Lorsque tu lances une exception celle-ci remonte jusqu'au premier bloc catch disponible. Le code après le throw ne sera donc jamais exécuté à moins d'être dans un try/catch et d'avoir un bloc finally.

Dans ton cas
Code:

1 2 3 4 5 6 7 8 9 10 try { im = Image.FromFile(fichiersRep[i]); } catch (Exception) { throw new Exception("Erreur, le fichier : (" + fichiersRep[i] + ") n'a pas pu etre converti en image"); // On ne passera jamais ici car on lance une exception avant continue; }
Plutôt que lancer une exception, tu devrais faire un log des problèmes.

ok !
j'ai toujours du mal a comprendre le fonctionnement et l'utilisation des exceptions :?
je viens de faire plusieurs tests dans un autre projet et effectivement mettre :
throw new Exception("coucou");
n'importe ou fait planter l'appli lors du debug par contre si je lance directement l'exe j'ai bien mon message d'erreur qui est affiché et qui me propose de continuer ou de quitter ....

j'ai donc fait ceci pour corriger mes erreurs :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 try { im = Image.FromFile(fichiersRep[i]); } catch (OutOfMemoryException) { MessageBox.Show("Erreur, le fichier : (" + fichiersRep[i] + ") n'a pas pu etre converti en image", "OutOfMemoryException"); continue; } catch (FileNotFoundException) { MessageBox.Show("Erreur, le fichier : (" + fichiersRep[i] + ") n'a pas pu etre trouvé", "FileNotFoundException"); continue; }
----------------
si j'ai bien compris : throw new Exception() n'est pas la pour afficher des infos a l'utilisateur mais pour envoyer des exceptions (par ex) aux utilisateurs des méthodes de nos classes qui devront les gérer dans les try/catch.

Citation:

Envoyé par Lorenzo77

si j'ai bien compris : throw new Exception() n'est pas la pour afficher des infos a l'utilisateur mais pour envoyer des exceptions (par ex) aux utilisateurs des méthodes de nos classes qui devront les gérer dans les try/catch.

C'est tout à fait ça.

Dans ton cas si tu veux faire de la gestion de log tu peux utiliser les méthodes abordé dans ce tuto