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.