Discussion :

[marcous]

Bonsoir
j'ai crée ce bout de code pour apprendre la gestion de fichiers
j'aimerai vos conseils pour amélioré

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
 
#include <iostream>
#include <fstream>
#include <string>
using namespace std;
 
int main()
{
    cout<<"Entrez le nom du fichier ( ne oubliez de mettre .txt) : ";
    string f_nom;
    cin>>f_nom;
    ofstream ajouteProduits(f_nom,ios_base::app);
    char c ='n';
    string nomProduit;
    int quantiteProduit;
    do
    {
 
     cout<<"Veuillez-saisir un nouveau produit : ";
     cin>>nomProduit;
     cout<<"veuillez saisir le nombre de produits  :";
     cin>>quantiteProduit;
     cout<<"Voulez -vous rajouter un autre produits(o ou n) ?" ;
     cin>>c;
    ajouteProduits<<nomProduit<<" "<<quantiteProduit<<endl;
    }
    while(c=='o');
 
    ajouteProduits.close();
    cout<<"le fichier "<<f_nom<<" est sauvegarder"<<endl;
 
    return 0;
}

[Médinoc]

Tu devrais envisager d'utiliser std::getline() pour saisir les chaînes (un nom de fichier, ou un nom de produit, peuvent contenir des espaces).

Aussi, tu peux te passer du close() avec un bloc:

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
 
#include <iostream>
#include <fstream>
#include <string>
using namespace std;
 
int main()
{
    cout<<"Entrez le nom du fichier ( ne oubliez de mettre .txt) : ";
    string f_nom;
    cin>>f_nom;
    {
        ofstream ajouteProduits(f_nom,ios_base::app);
        char c ='n';
        string nomProduit;
        int quantiteProduit;
        do
        {
            cout<<"Veuillez-saisir un nouveau produit : ";
            cin>>nomProduit;
            cout<<"veuillez saisir le nombre de produits  :";
            cin>>quantiteProduit;
            cout<<"Voulez -vous rajouter un autre produits(o ou n) ?" ;
            cin>>c;
            ajouteProduits<<nomProduit<<" "<<quantiteProduit<<endl;
        }
        while(c=='o');
    }
    cout << "le fichier " << f_nom << " est sauvegarde" << endl;
 
    return 0;
}

Autre chose, tu devrais tester que l'ouverture du fichier a réussi (un if(ajouteProduits) sert à ça).

[marcous]

bonjour
et merci de tes conseils

si je mets un

Code :

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

if(ajouteProduits)

je dois le placer où? peux t'on mettre faire

Code :

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

IF {Do {}while }

???


si je mets un getline()
je peux pas rentrer un nom car j'ai le

Code :

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

cout<<"Voulez -vous rajouter un autre produits(o ou n) ?" ;

qui arrive direct derriere et donc impossible de rentrer quelque chose .

[Médinoc]

Normalement, un truc comme ceci marche:

Code C++ :

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
35
36
37
38
39
40
41
42
43
44
45
// http://www.developpez.net/forums/d1470759/c-cpp/cpp/debuter/j-ai-cree-bout-code-apprendre-gestion-fichiers/#post7971835
 
#include <iostream>
#include <fstream>
#include <string>
using namespace std;
 
int main()
{
	cout<<"Entrez le nom du fichier (avec l'extension .txt) : ";
	string f_nom;
	getline(cin, f_nom);
	bool succes = false;
	{
		ofstream ajouteProduits(f_nom.c_str(), ios_base::app);
		if(ajouteProduits)
		{
			char c ='n';
			string nomProduit;
			int quantiteProduit;
			do
			{
				cout<<"Veuillez-saisir un nouveau produit : ";
				getline(cin, nomProduit);
				cout<<"veuillez saisir le nombre de produits  :";
				cin>>quantiteProduit;
				cout<<"Voulez -vous rajouter un autre produit (o ou n) ?" ;
				cin>>c;
				string dummy; //Pour s'occuper du \n laissé derrière
				getline(cin, dummy);
				ajouteProduits<<nomProduit<<" "<<quantiteProduit<<endl;
			}
			while(c=='o');
			succes = true;
		}
		else
			cout << "Echec de l'ouverture du fichier." << endl;
	}
	if(succes)
		cout << "le fichier " << f_nom << " est sauvegarde" << endl;
	else
		cout << "Echec du programme." << endl;
 
	return 0;
}

[marcous]

merci

que veux dire dummy?

merci encore de ton aide

[Médinoc]

"dummy" est un mot anglais assez difficile à traduire.

Mais en gros, dans le cas présent, c'est une string pour rien. Ce n'est pas son contenu qui nous intéresse, c'est juste le fait de la lire sur l'entrée standard.