Discussion :

[Aleph69]

Bonjour,

j'essaie de lancer et capturer la bonne exception en enveloppant un code C qui appelle fopen.

Concrètement, j'ai une classe pour gérer un type de fichier, disons File (admettant pour donnée membre un pointeur sur FILE nommé content_) dont le constructeur s'écrit

Code :

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

1
2
3
4
5
6
  File::File(std::string const & filename)
  : content_(fopen(filename.c_str(), "r"))
  {
    if(!content_)
      throw new std::ios_base::failure("Unable to open the specified file");
  }

Je fais appel à ce constructeur pour un fichier n'existant pas, en tentant de capturer l'exception que je lance dans le constructeur :

Code :

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

1
2
3
4
5
6
7
8
  try
  {
    File * does_not_exist = new File("Does not exist");
  }
  catch(std::ios_base::failue const & e)
  {
    std::cout << e.what() << std::endl;
  }

Malheureusement, je n'attrape pas l'exception avec cette ligne de code.
En revanche, si je remplace mon appel à catch par catch(...), une exception est bien attrapée.

Est-ce que vous sauriez pourquoi?

EDIT : cela ne fonctionne pas non plus si j'essaie d'attraper une instance de std::exception.

Merci!

[Aleph69]

Comme prévu, c'était une erreur de ma part.
Le problème venait du lancer d'exception, qu'il fallait faire par valeur.

[JolyLoic]

Toujours lancer une exception par valeur, et l'attraper par référence.

Sinon, tu es conscient que dans ton code, en cas d'exception, tu as non seulement une fuite mémoire, mais une fuite de handle de fichier (pour peu qu'une exception soit lancée après une ouverture ayant réussi) ?

[Aleph69]

Salut Loïc,

merci pour ta réponse.
Je veux bien que tu détailles un peu ta réponse.

Mon raisonnement naïf est le suivant :
- fopen ne lève pas d'exception;
- je lève une exception si et seulement si content_ vaut NULL, c'est-à-dire si et seulement aucun fichier n'a été ouvert : donc aucun fichier n'a à être fermé;
- si je ne lève pas d'exception, content_ ne vaut pas NULL et le fichier sera fermé dans le destructeur de FIle (RAII).

Du coup, je ne vois ni la fuite mémoire, ni la fuite de handle de fichier.
Un peu d'aide est donc la bienvenue!

Et encore merci!

[prgasp77]

EDIT : Attention, j'ai écris ce message alors que j'avais 39°C de fièvre. Comme relevé ci-après, certains passages sont faux.

Bonjour,

la ligne File * does_not_exist = new File("Does not exist"); est très riche, et sous son apparence simple procède à plusieurs opérations :

1. new réserve sizeof(File) octets ;
2. un objet File est construit à cet emplacement mémoire (un std::string est construit puis détruit dans le processus) ;
3. l'adresse de la plage réservée est affectée à does_not_exist.

(N.B. : 2 et 3 peuvent être inversés / parallélisés).

Ainsi, si 2 lève un exception, la plage mémoire réservée en 1 n'est jamais libérée.

[Aleph69]

Salut!

Merci pour ta réponse.
Ce bout de code est juste un exemple jouet!
Si c'est lui qui pose problème, alors pas d'inquiétude!