Discussion :

[tROR411]

Bonjour, je voudrais savoir si la méthode que j'adopte est sûre: merci d'avance.
J'ai une class avec un constructeur qui en fonction de la présence d'un nom de fichier en paramètre va créer une poignée vers celui ci et stocker son pointeur en attribut. Je voudrais que le destructeur de cette class libère le fichier. Le soucis vient quand il s'agit de retourner ces objets, en effet le fichier va être ouvert dans la fonction et puis après le return l'objet va être copié donc l'autre aura aussi accès au même fichier mais comme l'objet temporaire du return va appeler le destructeur et fermer le fichier donc l'autre ne pourra plus y accéder.
Du coup je sais plus comment faire, faudrait il faire du c-style en allouant dynamiquement l'objet pour ne manipuler qu'un pointeur et puis delete l'objet dans un manager de ses objets ?

[JolyLoic]

C'est l'âme du C++ moderne que tu touches là !

Au retour de la fonction, l'objet n'est pas copié, mais déplacé, ce qui est une nuance importante. En effet, il n'y a pas besoin de partager le fichier entre deux objets, il y a juste besoin de le transférer de l'un à l'autre.

Si tu utilises des classes bien conçues, tu n'as rien à faire pour ça (je ne sais pas trop ce que tu entends par "poignée"):

Code :

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

1
2
3
4
class MaClasse
{
  std::ofstream monFichier;
};

Rien qu'en faisant ça, ta classe n'est pas copiable, mais elle est déplaçable, et tu peux donc écrire

Code :

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

1
2
3
4
5
6
MaClasse créeClasse(std::string const &nomFichier)
{
  MaClass résultat;
  // ...
  return résultat;
}

Si ta poignée est moins prête à l'emploi, tu peux :
- Soit créer un type autour qui te permettra de la manipuler plus sûrement (regarde des docs sur les constructeurs de déplacement et opérations d'affectation par déplacement, ainsi que sur la règle des 5)
- Soit utiliser quelquechose d'un peu moins customisé, mais déjà fait pour toi. std::unique_ptr Par exemple, pour un FILE*:

Code :

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

1
2
3
4
5
6
class FileStarDeleter 
{
  void operator[](FILE *f) {fclose(f);}
};
 
using FilePtr = std::unique_ptr<FILE, FileStarDeleter>;

Si maintenant tu te rends compte qu'en fait, tu veux vraiment partager ton objet, il existe aussi std::shared_ptr pour ça.

[tROR411]

Code :

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

1
2
3
4
5
6
MaClasse créeClasse(std::string const &nomFichier)
{
  MaClass résultat;
  // ...
  return résultat;
}

en faisant ça il n'y aura pas un appel du destructeur ? je m'en vais essayer ça.

[ternel]

Si, mais comme il y a d'abord appel au constructeur par déplacement (de la variable recevant la valeur de retour), le destructeur ne détruit qu'une coquille vide.

(si le déplacement est correctement géré dans ofstream, ce qui est bien sûr le cas)