[C++] problèmes avec fstream

Bonjour,
j'ai essayé pendent un moment d'utiliser Qt pour manipuler les fichiers (je m'en sert deja pour faire la GUI), mais, j'ai pas trouver de vrai bon tuto autre que la doc, et je m'y suis un peu perdu, alors j'ai decidé d'essayer fstream, enfin, ifstream.
Je risque d'avoir plein de probleme donc je prepare le coup avec ce topic :D .
Bon, pour commencer un tout bete :
Dans le .h de ma classe, je le declare :
Code:

1 2 ifstream *fichier;
Dans le .cpp, j'essaye de m'en servir :
Code:

1 2 fichier = new ifstream("poney/liste.txt", ios::in);
Mais quand je compile :

Code:

error: ISO C++ forbids declaration of 'fstream' with no type

Voila, j'ai bien penser a inclure #include <iostream> et #include <fstream>

PS : et dites, c'est quoi les balises a mettre pour la coloration syntaxique sur ce forum ?

Salut, et bienvenue sur le forum.

D'abord, il faut bien comprendre que les classes fournies par la bibliothèque standard sont toutes placées dans l'espace de nom std.

Ainsi, quand tu veux déclarer un fichier d'entrée, il faut le déclarer sous la forme de

Code:

std::ifstream

ou, éventuellement (mais en faisant attention à l'endroit où tu le fait) en plaçant la directive

Code:

using namespace std;

Il faut aussi savoir que, le plus souvent, on évitera d'avoir recours à un pointeur sur les différentes classes fournies par la bibliothèque standard: on leur préférera fortement l'utilisation des références, ne serait-ce que parce que la gestion des pointeurs et de l'allocation dynamique des éléments que cela sous-entend souvent est assez compliquée à mettre en oeuvre correctement ;)

Enfin, et pour répondre à ta dernière question, il suffit d'entourer le code les balises [ CODE ] et [ /CODE ] (en retirant les espace)

Tu peux aussi avoir recours au bouton disponible lors de l'écriture de ton message
http://demco.developpez.com/images/balisesCode.gif

std:: ?
Sinon : NE JAMAIS UTILISER DE POINTEUR DE *fstream !!
Le RAII c'est pas pour les cochons !

Erf: grilled par un message 15 fois plus long ><

Ah merci de vos reponses !
j'ai pas penser au namespace, je m'en sert pas du tout dans mon programme et c'est la premiere fois que j'essaye d'ouvrir des fichiers en C++ , c'est un peu nouveau pour moi ^^ .
J'ai un nouveau probleme !
J'ai cette erreur :
error: 'ios' has not been declared
Avec ce code :

Code:

fichier.open("poney/liste.txt", ios::in);

C'est le meme qu'au dessus, avec les meme includes ^^

Desolé, je savais pas qu'il falait pas faire de pointeur sur un fstream, pas tapé :D .
Je le referai plus, promis ! ^^

ouaip, première règle du C++: ne pas utiliser de pointeurs sauf quand tu as une très bonne raison.

Il y a une raison particulière pour laquelle tu utilises ton fstream dans le .h?

Non, j'ai lu les cours C++ sur le site et il declare tout dans le .h avant de l'utilisé dans le .cpp, donc je fais pareil, je devrai pas ?

Si tu n'as pas mis le using namespace std, utilise std::ios::in.

Sinon pour les pointeurs je ne suis pas d'accord, il y a plein de raison d'utiliser des pointeurs (compatibilité avec des libs C, argument d'une fonction C++ qui est un const char*, polymorphisme, pointeurs de fonction...).

Citation:

Sinon pour les pointeurs je ne suis pas d'accord, il y a plein de raison d'utiliser des pointeurs (compatibilité avec des libs C, argument d'une fonction C++ qui est un const char*, polymorphisme, pointeurs de fonction...).

une bonne raison quoi.

J'y avait penser mais j'ai mal vu, y avait une erreur en dessous qui y ressemblaie et j'ai cru que ça avait pas resolu le probleme, ça m'apprendra.
Bon, ça c'est bon, maintenant j'ai une question bete, c'est est-ce que je peux reutiliser fichier ?
j'ai fini avec mon liste.txt, je voudrais reutiliser fichier pour ouvrir un autre fichier, donc j'ai essayé
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 for (i = 0 ; i < m_nbcheval + 1 ; i++) { fichier.open("poney/" + lines.at(i), std::ios::in); if(fichier) { while(getline(fichier, line)) // tant que l'on peut mettre la ligne dans "contenu" { texte.fromStdString(line); contenuFichiers[i] += texte; i++; } } fichier.close(); }
Mais visiblement, reouvrir un fichier ça lui parle pas, voila l'erreure :
error: no matching function for call to 'std::basic_ifstream<char, std::char_traits<char> >::open(const QString, const std::_Ios_Openmode&)'

lines.at(i) semble renvoyer un QString. Il faut donc convertir ce QString en std::string.
Pour cela, matte la doc de Qt.

J'ai fait

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
    for (i = 0 ; i < m_nbcheval + 1 ; i++)
    {
        texte = lines.at(i);
        fichier.open("poney/" + texte.toStdString(), std::ios::in);
        if(fichier)
        {
            while(getline(fichier, line))  // tant que l'on peut mettre la ligne dans "contenu"
            {
                texte.fromStdString(line);
                contenuFichiers[i] += texte;
                i++;
            }
        }
        fichier.close();
    }

Mais j'ai toujours cette erreure ...
error: no matching function for call to 'std::basic_ifstream<char, std::char_traits<char> >::open(std::basic_string<char, std::char_traits<char>, std::allocator<char> >, const std::_Ios_Openmode&)'

Pourtant j'ai convertit, donc c'est pas ça, si ?

Salut,

je ne sais pas si il existe des moyens plus simples :s

Code:

fichier.open(("poney/" + texte.toStdString()).c_str(), std::ios::in);

Merci ça marche !
j'avais du faire une erreur ^^ .
ah c'est cool, ça m'avance bien !

Hop, j'ai rien dis, ça marche pas.
J'ai essayé de le faire en plusieurs etape pour voir ce qui ne marchait pas, mais c'est toujours le open et la meme erreure.
Je lui envoie un char* pourtant, non ?
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 for (i = 0 ; i < m_nbcheval + 1 ; i++) { texte = lines.at(i); line = "poney/" + texte.toStdString(); fichier.open(line.c_str, std::ios::in); if(fichier) { while(getline(fichier, line)) // tant que l'on peut mettre la ligne dans "contenu" { texte.fromStdString(line); contenuFichiers[i] += texte; } } fichier.close(); }
Parce que, si j'ai bien compris, open attent un char*, donc ça devrai etre bon, non ?

Decidement, la lecture de fichier, ça me reussi pas ^^ .

c_str(), pas c_str.

ah ah, ça me ferai presque rire si ça faisais pas une heure que je cherchais.
Je crois que la prochaine fois je prendrai une douche AVANT de m'y mettre le matin -_-'.
Merci !

Maintenant, dernier probleme je pense avant que tout ne marche tres bien (je suis optimiste ce matin !), quand j'essaye d'acceder a contenuFichiers[i] , j'ai une segmentation fault.
J'ai essayé d'acceder a contenuFichiers[0] pour voir, segmentation fault.
Je le declare comme ça dans le .h:

Code:

QStringList *contenuFichiers;

Et ensuite dans le .cpp :

Code:

contenuFichiers = new QStringList[i];

Sinon j'ai aussi essayé de le declaré directement dans le .cpp en faisant :

Code:

QStringList contenuFichiers[i]

mais dans les cours et tutoriels pour apprendre C++ que j'ai lu sur le site, il disait qu'il fallait a tout prix eviter de declarer un tableau avec une variable, donc j'ai fait la premiere methode, mais j'ai la meme erreur dans les deux cas.

Salut, la deuxième méthode marche bien aussi et est pratique quand on ne veut pas à avoir à gérer la destruction des tableaux.

Tu peux afficher le contenu de la variable i?

Coyotte507

i, c'est le compteur de la boucle :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 for (i = 0 ; i < m_nbcheval ; i++) { texte = lines.at(i); line = "poney/" + texte.toStdString(); fichier.open(line.c_str(), std::ios::in); if(fichier) { fichier.seekg(0, std::ios::beg); while(getline(fichier, line)) // tant que l'on peut mettre la ligne dans "contenu" { texte = QString::fromStdString(line); contenuFichiers[i] += texte; } } fichier.close(); }
Maintenant, ce code ne genere plus d'erreur, mais apres ça, quand je veux acceder a contenuFichiers, a n'importe quelle case, j'ai une segmentation fault :s .
Et j'ai verifié, c'est bien contenuFichiers qui fait la segmentation fault ^^ .
Donc si ma declaration est bonne, ça veut dire que c'est

Code:

contenuFichiers[1].at(0)

qui plante.
(j'ai mis 1 pour tester, mais n'importe quelle nombre c'est pareil).
Et j'ai mis at(0) parce que cette ligne est cencé contenir un nom, mais je suppose que la segmentation fault veut dire qu'il n'y a rien a cet endroit ?

Ah oui, j'ai oublier de preciser que contenuFichiers est une QStringList ^^

Bon, pour essayé de voir la ou ça coincait j'ai essayé d'utiliser la sortie d'erreur.
J'ai essayé d'afficher le contenu de i pour voir si ça rentrait dan la boucle while.
Eh bien non, ça n'y rentre pas, comme on pourrait s'en douter.
Donc j'ai chercher un peu, et j'ai trouver la methode clear() pour remettre les flags a leur etat d'origine, donc j'essaye ce code :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 for (i = 0 ; i < m_nbcheval ; i++) { fichier.clear(); texte = lines.at(i); line = "poney/" + texte.toStdString(); fichier.open(line.c_str(), std::ios::in); if(fichier) { while(getline(fichier, line)) // tant que l'on peut mettre la ligne dans "contenu" { texte = QString::fromStdString(line); contenuFichiers[i] += texte; fprintf(stderr, "texte : %ld\n", i); } } fichier.close(); }
Et mon but, c'est de voir s'afficher dans ma console le contenue de i a chaque passage de la boucle.
Avec ce code, je ne le voie pas.
Si je met le fprintf en dehors du while par contre, ça marche.
Donc le while retourne tout de suite 0, pourtant, la fonction clear devrait empecher ça, non ?

Salut,

L'affichage marche même avec le fprintf dans ce bloc ?
Code:

1 2 3 4 5 if (fichier) { ... }
Si c'est le cas, ça veut dire que ton fichier est vide o_O!

Oui, ça marche dans ce bloc.
Avec ton post j'ai compris ^^ .
En fait, le fichier est vide parce qu'il essaye d'ouvrir un nom de fichier qui n'existe pas.
J'ai remplacer i par line.c_str() et mis %s, et voila ce que ça m'affiche :
poney/
Donc
Code:

1 2 line = "poney/" + texte.toStdString();
deconne encore :(

J'ai fait un
Code:

1 2 fprintf(stderr, "texte : %s\n", lines.at(i).toStdString().c_str());
pour voir, et c'est vide ><.
Le probleme vient donc d'encore plus haut. Je vais trouver je vais trouver xD.

Citation:

Envoyé par Ulrar

ah ah, ça me ferai presque rire si ça faisais pas une heure que je cherchais.
Je crois que la prochaine fois je prendrai une douche AVANT de m'y mettre le matin -_-'.

Y'a pas de mal. On a souvent besoin d'un regard extérieur pour la relecture.

ça y est, j'ai trouver.
dans la partie qui me remplie lines des noms de fichier a ouvrir j'utilisais mal une fonction statique (je l'utilisais comme methode alors qu'elle retournais pas ce que je pensais).
J'ai corrigé, maintenant elle se remplie presque correctement.
Je fais
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 fichier.open("poney/liste.txt", std::ios::in); i = 0; if(fichier) { while(getline(fichier, line)) // tant que l'on peut mettre la ligne dans "contenu" { texte = QString::fromStdString(line); lines += texte; i++; fprintf(stderr, "texte : %s\n", texte.toStdString().c_str()); } }
et je vais pas recopié ce que ça m'affiche, ce serai trop long.
En resumer, tout le debut de la liste est bon, il contient bien les noms des fichiers contenu dans le fichier liste.txt
Mais apres ça, j'ai plein de fois 0 puis le premier element de la liste, ensuite plein de 1 et le deuxieme ... et ainsi de suite.
genre :
texte : element 1
texte : 0
texte : 0
...
texte : element 2
Bref. Donc j'ai fait :
Code:

1 2 fprintf(stderr, "texte : %s\n", lines.at(i).toStdString().c_str());
et la, segmentation fault =).
Pour changé

EDIT : oulala, c'est de pire en pire les erreures que je fait.
J'avais oublier d'autre printf plus bas, donc tout se mettait a la suite -_-'.
Bref, pour ce probleme c'est regler, je vais continuer ^^