tester l'existence d'un fichier et plus

Marsh Posté le 28-05-2010 à 10:20:12

Bonjour,
je suis en train d'apprendre à utiliser les strings et les flux en c++ et tout ne fonctionne pas comme je voudrais. Voici mon code

Code :

#include <iostream>
#include <fstream>
#include <string>
using namespace std;
char text[100];
bool is_readable(const string &file) //indique si un fichier est lisible (et donc si il existe)
{
ifstream fichier(file.c_str());
return !fichier.fail();
}
void main()
{
string answer;
string output_file("fichier.dat" );
ofstream output(output_file.c_str());
if(is_readable(output_file.c_str()))
{
cout << output_file << " exists. Do you want to erase this file (type \"erase\" ) or to create a new one (type \"create\" )?" << endl;
cin >> answer;
while (strcmp(answer.c_str(),"erase" )==1 || strcmp(answer.c_str(),"create" )==1) //strcmp renvoie 1 quand les chaines sont différentes
{
cout << "Choice is incorrect, type \"erase\" or \"create\"." << endl;
cout << answer << endl;
cin >> answer;
}
if(strcmp(answer.c_str(),"erase" )==0)
{
ofstream output(output_file.c_str());
}
if(strcmp(answer.c_str(),"create" )==0)
{
cout << "Type the new file name" << endl;
cin >> output_file;
output.close();
ofstream output(output_file.c_str());
}
}
}//fin de main

En gros, je voudrais tester si le fichier "fichier.dat" existe déjà et si c'est le cas permettre à l'utilisateur de l'écraser ou de proposer un nouveau nom. Qu'est ce que je fais mal ?

Message édité par pamputt le 28-05-2010 à 15:02:52

Reply

Marsh Posté le 28-05-2010 à 10:20:12

Reply

Marsh Posté le 28-05-2010 à 15:12:46

Tu fais un horrible mélangee de C / CPP.
Et ton proto de main est faux.
Et tu devrais lire le man de strcmp.

Reply

Marsh Posté le 28-05-2010 à 15:18:26

Pour l'horrible mélange de C/C++, c'est bien possible mais comme je l'ai dit au début je suis en train d'apprendre donc tous les conseils sont les bienvenus pour que j'améliore cela.
Le proto de main est peut-être faux, en effet car j'ai créé un truc vite fait. Tu peux me dire ce qui ne va pas.
Enfin, je viens de lire le man de strcmp, il dit

Citation :

SYNOPSIS
#include <string.h>

int strcmp(const char *s1, const char *s2);

int strncmp(const char *s1, const char *s2, size_t n);

DESCRIPTION
La fonction strcmp() compare les deux chaînes s1 et s2. Elle renvoie un
entier négatif, nul, ou positif, si s1 est respectivement inférieure, égale
ou supérieure à s2.

La fonction strncmp() est identique sauf qu'elle ne compare que les n (au
plus) premiers caractères de s1 et s2.

VALEUR RENVOYÉE
Les fonctions strcmp() et strncmp() renvoient un entier inférieur, égal ou
supérieur à zéro si s1 (ou ses n premiers octets) est respectivement
inférieure, égale ou supérieure à s2.

Je ne vois pas ce qui cloche.

Reply

Marsh Posté le 28-05-2010 à 15:47:17

Code :

while (strcmp(answer.c_str(),"erase" )==1 || strcmp(answer.c_str(),"create" )==1) //strcmp renvoie 1 quand les chaines sont différentes

Mais en CPP tu peux faire beaucoup plus simplement un test

Code :

while (answer != "erase" || answer != "create" )

Reply

Marsh Posté le 28-05-2010 à 17:48:57

et reagrde du coté de boost::filesystem pour ce genre de truc

Reply

Marsh Posté le 01-06-2010 à 10:40:30

Bon, j'ai modifié mon code pour corriger quelques problèmes. Maintenant j'ai ceci

Code :

#include <iostream>
#include <fstream>
#include <string>
using namespace std;
bool is_readable(const string &file) //indique si un fichier est lisible (et donc si il existe)
{
ifstream fichier(file.c_str());
return !fichier.fail();
}
void main()
{
string answer;
string output_file("fichier.dat" );
ofstream output;
if(is_readable(output_file.c_str()))
{
cout << output_file << " exists. Do you want to erase this file (type \"erase\" ) or to create a new one (type \"create\" )?" << endl;
cin >> answer;
while (answer!="erase" && answer!="create" )
{
cout << "Choice is incorrect, type \"erase\" or \"create\"." << endl;
cout << answer << endl;
cin >> answer;
}
if(answer=="erase" )
{
ofstream output(output_file.c_str());
}
if(answer=="create" )
{
cout << "Type the new file name" << endl;
cin >> output_file;
ofstream output(output_file.c_str());
}
}
output << "Hello" << endl;
}//fin de main

Mon problème c'est que le flux de sortie output ne fonctionne pas. Je m'explique. Si je lui dit d'écraser le fichier fichier.dat alors il va bien l'écraser mais il ne va pas écrire Hello dans ce fichier. fichier.dat sera vide. Idem si je dis de créer un nouveau fichier, alors le fichier sera bien créé mais rien ne sera écrit dedans. Est ce que vous avez une idée sur la manière de corriger cela ?

Message édité par pamputt le 01-06-2010 à 10:43:20

Reply

Marsh Posté le 01-06-2010 à 11:10:32

Bon en fait c'est bon, j'ai réussi à faire ce que je voulais. Tout fonctionne bien maintenant. Voici mon code final si ça en intéresse d'autre

Code :

#include <iostream>
#include <fstream>
#include <string>
using namespace std;
bool is_readable(const string &file) //indique si un fichier est lisible (et donc si il existe)
{
ifstream fichier(file.c_str());
return !fichier.fail();
}
void main()
{
string answer;
string output_file("fichier.dat" );
ofstream output;
if(is_readable(output_file.c_str()))
{
cout << output_file << " exists. Do you want to erase this file (type \"erase\" ) or to create a new one (type \"create\" )?" << endl;
cin >> answer;
while (answer!="erase" && answer!="create" )
{
cout << "Choice is incorrect, type \"erase\" or \"create\"." << endl;
cout << answer << endl;
cin >> answer;
}
if(answer=="erase" )
{
output.open(output_file.c_str());
}
if(answer=="create" )
{
cout << "Type the new file name" << endl;
cin >> output_file;
output.open(output_file.c_str());
}
}
output << "Hello" << endl;
}//fin de main

Reply

tester l'existence d'un fichier et plus

Sujets relatifs:

Leave a Replay