Discussion :

[Seb33300]

Salut � tous

je ne suis pas sur d'avoir bien compris le fonctionnemet des try catch

Par exemple je veux charger une image dans un picturebox.
Je souhaiterais que si celle si est trouv�e, elle s'affiche, sinon que rien ne se passe

j'ai donc fait ce code :

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
try{
	pictureBox1->Image = System::Drawing::Image::FromFile(repPic);
 }catch(exception e){}

Mais si je charge un chemin d'image erron� (donc image non existante), le programme s'arrete et me renvoi une exception :/

quelqu'un peut m'aider ?

Merci

[nico-pyright(c)]

il ne faut pas utiliser de try catch, mais plutot tester l'existence du fichier
voir dans la faq C++/CLI http://dotnet.developpez.com/faq/cpp...file#existfile

nb : cette syntaxe est incorrecte et provoquera une erreur de compilation

[M�dinoc]

C'est marrant, la doc de File::Exists() dit explicitement que �a ne suffit pas: Il faut toujours un traitement d'exception derri�re.

Seb33300: C'est ta syntaxe de traitement d'exception qui est incorrecte: Une exception C++ (std::exception) n'est pas la m�me chose qu'une exception .Net (System::Exception)
Essaie ainsi:

Code C++/CLI non-test� :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
6
7
8
 
try {
	pictureBox1->Image = System::Drawing::Image::FromFile(repPic);
} catch(OutOfMemoryException ^e) {
	//Traiter l'erreur ?
} catch(FileNotFoundException ^e) {
	//Traiter l'erreur ?
}

[nico-pyright(c)]

c'est vrai que c'est marrant, qu'est-ce qu'on s'�clate.

D�j�, relis la question :

Je souhaiterais que si celle si est trouv�e, elle s'affiche, sinon que rien ne se passe

sa solution est d'utiliser un try catch pour essayer de l'afficher et si ca l�ve une exception, ca ne fait rien (meme si la syntaxe est incorrecte).

La bonne solution est de tester l'existence du fichier avant d'essayer de le charger.

Donc de dire qu'il faut encapsuler File::Exists dans un try catch n'a rien � voir avec la quesion


Ensuite, je ne vois aucun endroit de la doc qui dit qu'il faut encapsuler File::Exists dans un try catch, le seul moment o� cela parle d'exception, c'est en disant qu'il ne va pas en lever dans le cas d'un probl�me de permission :

the caller does not have sufficient permissions to read the specified file, no exception is thrown and the method returns false regardless of the existence of path.

Ensuite, si je pousse le vice � me dire que la doc est incompl�te, je prends mon reflector pr�f�r� et je regarde l'impl�mentation de file::exists.
Et que vois-je ?

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
 
public static bool Exists(string path)
{
    try
    {
        if (path == null)
        {
            return false;
        }
        if (path.Length == 0)
        {
            return false;
        }
        path = Path.GetFullPathInternal (path);
        new FileIOPermission(FileIOPermissionAccess.Read, new string[] { path }, false, false).Demand();
        return InternalExists(path);
    }
    catch (ArgumentException)
    {
    }
    catch (NotSupportedException)
    {
    }
    catch (SecurityException)
    {
    }
    catch (IOException)
    {
    }
    catch (UnauthorizedAccessException)
    {
    }
    return false;
}

un try catch qui �num�re les exceptions, et notamment, celle qui nous interesse le plus, IOException. Tout est catch� pour retourner un false en cons�quence.

V�rifie tes sources et arrete de chercher la ptite bete tout le temps

[M�dinoc]

Be aware that another process can potentially do something with the file in between the time you call the Exists method and perform another operation on the file, such as Delete. A recommended programming practice is to wrap the Exists method, and the operations you take on the file, in a try...catch block as shown in the example. This helps to narrow the scope for potential conflicts. The Exists method can only help to ensure that the file will be available, it cannot guarantee it.

[nico-pyright(c)]

ce qui veut bien dire que ce n'est pas le File::Exists qu'il faut entourer de try catch.
Ce qui est certain, c'est que les acc�s aux fichiers doivent se faire dans un try catch.
Enfin bref.