Discussion :

[roipoussiere]

Bonjour.

Je débute en c++ et malgré mes recherches je ne parviens pas à faire un trus très simple.

J'ai une classe FichierProv dont voici le .h

Code :

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

1
2
3
4
5
6
7
8
9
class FichierProv
{
private:
	ofstream fichier;
public:
	FichierProv();
	void add(string chaine);
	void close();
};

Le constructeur permet de créer un fichier

Code :

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

1
2
3
4
5
6
7
8
9
10
11
FichierProv::FichierProv()
{
	// Creation/écrasement du fichier
	fichier("test.txt", std::ios::trunc);
 
	if (!fichier)
	{
		std::cerr << "Erreur de creation du fichier" << std::endl;
		exit 1;
	}
}

La méthode add() permet d'ajouter une chaine à ce fichier

Code :

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

1
2
3
4
void FichierProv::add(string chaine)
{
	fichier << chaine << std::endl;
}

La méthode close permet de fermer ce fichier.

Code :

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

1
2
3
4
void FichierProv::close()
{
	fichier.close();
}

Je rencontre une erreur dans le constructeur :
no match for call to ‘(std::ofstream {aka std::basic_ofstream<char>}) (const char [9], const openmode&)’
Savez-vous de quoi s'agit-il ?

[PilloBuenaGente]

Bonjour, et comme ceci ?

Code :

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

1
2
3
4
5
6
7
8
9
 
FichierProv::FichierProv() : fichier("test.txt", std::ios::trunc)
{
	if (!fichier)
	{
		std::cerr << "Erreur de creation du fichier" << std::endl;
		exit 1;
	}
}

De plus pense à rajouter "std::" devant "string" car apparemment il n'y a pas de "unsing namespace std;"

[roipoussiere]

super, ça marche !
Merci !

[ternel]

Par ailleurs, je te recommande de te baser sur un principe simple nommé RAII (Ressource Acquisition Is Initialisation)

Il s'agit de structurer les classes de ressource pour avoir la correspondance valeur=ressource
L'idée, c'est de prendre/ouvrir la ressource dans le constructeur, et de la rendre dans le destructeur
c'est le cas de string, par exemple, ou de (i/o)fstream

L'effet immédiat, c'est l'absence de méthode close() / release() etc.
En effet, le code qui y serait se trouve dans le destructeur.

L'avantage, c'est que tu sais qu'un destructeur est appelé une et une seule fois: lorsque le scope de la variable se termine.

Pour plus d'information, il y a plusieurs entrées dans la faq, et quelques tutos

[Médinoc]

De telles classes exposent généralement quand même une méthode close(), au cas où l'utilisateur aurait besoin de libérer la ressource avant la fin du bloc.
Mais à part ça, le principe reste le même: Le destructeur libère automatiquement la ressource.

C'est pour ça que je préfère l'autre nom de ce principe, "RRIF": Resource Release Is Finalization. Ce trouve que ça parle plus.