ifstream, ofstream, boucle

Bonjour à tous !
Voila, je débute en C ++, et j'ai un fichier texte se présentant sous la forme

triangle 1
valeur 1 valeur 2 valeur 3
valeur 4 valeur 5 valeur 6
triangle 2
valeur 7 valeur 8 valeur 9
valeur 10 valeur 11 valeur 12
triangle 3
valeur 13 ....etc

Bref je pense que vous avez compris, et cela fait 20 000 lignes en tout lol.

J'aimerai en fait créer un autre fichier texte reprenant exactement celui la, mais avec UN SEUL triangle c'est a dire :

triangle 1
valeur 1 valeur 2 valeur 3
valeur 4 valeur 5 valeur 6
valeur 7 valeur 8 valeur 9
valeur 10 etc...

Je pense faire un ifstream/ofstream pour scanner et écrire.

Mais ce qui m'ennui c'est qu'on ne peut pas "enlever" une ligne de texte dans une boucle ...

Qu'en pensez-vous ? Merci d'avance pour votre aide,

Cordialement,

reptilia1251

Si je comprends bien, tu veux lire ton fichier de départ, et le recopier en sautant une ligne sur trois (sauf la première), tu vas donc avoir quelque chose comme

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23

ifstream fin("tonfichier.txt"); ofstream fout("leresultat.txt"); string s; // le compteur de lignes int i=0; // lit la première ligne getline(fin,s); // sur tout le fichier while(!fin.eof()) { // i%3==0 : i est multiple de trois if(i%3!=0 || i==0) { // ecrit ta ligne fout<<s<<'\n'; } i++; // lit la ligne suivante getline(fin,s); } // ces fermetures ne sont pas nécessaires, // mais elles ne peuvent pas nuire non plus fin.close(); fout.close();

---

Francois

Merci bien fcharton !
J'ai eu qq problèmes à la compilation, mais ça marche à la fin !

Tu avais mis (!fin.eof()) mais apparemment, il y a () en trop !
Et le compilateur n'aimait pas trop fin.close(); fout.close(); Mais tu as justement précisé que ce n'était pas obligatoire !

Merci encore (je teste ça demain sur le grand fichier, j'espère qu'il n'y aura pas de problème),

A +

Citation:

---

Envoyé par reptilia1251

Tu avais mis (!fin.eof()) mais apparemment, il y a () en trop !

---

C'est étrange, eof() est une fonction standard de basic_ios, je crois...

Citation:

---

Envoyé par reptilia1251

Et le compilateur n'aimait pas trop fin.close(); fout.close(); Mais tu as justement précisé que ce n'était pas obligatoire !

---

Ca aussi... il doit te manquer quelque chose quelque part...

Il te disait quoi, ton compilateur?

Francois

Soit dit en passant on utilise pas eof() pour tester la fin d'un fichier dans une condition de sortie de boucle.
On préférera utiliser getline() pour plus de détail sur le pourquoi de la chose voir la faq.

Mais ça n'empêche que t'as un problème c'est bien eof() et non eof.

Pour les appelles à close() oublie les ils sont dans ce cas inutile. (d'ailleurs fcharton la fait remarqué... x) )

Edit : Goten m'a doublé. :aie:
On peut simplifier un peu la logique de lecture du fichier, en utilisant le code de la faq :

Citation:

---

Envoyé par FAQ

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22

#include <string> #include <fstream> #include <iostream>   int main() { // le constructeur de ifstream permet d'ouvrir un fichier en lecture std::ifstream fichier( "fichier.txt" );   if ( fichier ) // ce test échoue si le fichier n'est pas ouvert { std::string ligne; // variable contenant chaque ligne lue   // cette boucle s'arrête dès qu'une erreur de lecture survient while ( std::getline( fichier, ligne ) ) { // afficher la ligne à l'écran std::cout << ligne << std::endl; } } }

---

---

Pour les erreurs de compilations, peut-être manque-il des includes ? :koi:
Ici, il faut <iostream> <fstream> et <string>.

Citation:

---

Envoyé par Goten

Pour les appelles à close() oublie les ils sont dans ce cas inutile. (d'ailleurs fcharton la fait remarqué... x) )

---

Perso, j'ai tendance à toujours écrire mes close(), et souvent même à leur ajouter des clear()... Cela garantit

1- que le fichier est fermé dès qu'on n'en a plus besoin (généralement ce n'est pas grave, mais ca peut avoir son importance)
2- que l'on peut éventuellement réutiliser le même ifstream sur un autre fichier (le genre de chose qu'on fait dans l'urgence, quand on découvre qu'il nous manque une donnée dans un second fichier, et qui amène bien des déconvenues si les bits d'erreur du fichier précédent sont restés en place)

Cela indique également à un relecteur quand on lit un fichier, et quand on en a fini... Dans des processus de traitement un peu complexes, c'est généralement très utile (dans ce domaine, je conseillerais également l'écriture explicite des open(), plutôt que via l'instanciation).


Pour eof(), je suis bien sur d'accord, mais il me semble qu'il faudrait alors ajouter (à la sortie de la boucle) un test minimal de sortie, pour savoir si la boucle a fini sur une erreur ou sur une fin de fichier. Ce n'est pas tout à fait pareil.

Personnellement, j'aurais plutôt tendance à mettre les contrôles de lecture dans la boucle, et à garder eof() en test de sortie (quitte à avoir un break, un throw ou un goto pour la sortie sur erreur de lecture). Ca me parait plus propre à la relecture (on dit alors "lire le fichier jusqu'à la fin"). Mais je conçois que c'est avant tout une question de goût.

Francois

Bonjour,
je rencontre effectivement des problèmes.
Je met ce qui suis dans le fichier triangle.txt :
triangle 1
1 2 3
4 5 6
triangle 2
7 8 9
10 11 12
triangle 3
13 14 15
16 17 18

et j'écris le programme :

Code:

---

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 <stdio.h> #include <iostream> #include <fstream> #include <string>   int main ()   {   ifstream fin("triangle.txt"); ofstream fout("resultri.txt"); string s;// le compteur de lignes int i=0;// lit la première ligne getline(fin,s);   while(!fin.eof())   {   if(i%3!=0 || i==0) {   fout<<s<<"\n"; } i++;   getline(fin,s); }   fin.close(); fout.close();     }

---

je compile en tapant gcc -Wall triFree.C -o resultatri

et j'obtien plein d'erreurs de type :

Code:

---

1 2 3 4 5

In function `int main()': triFree.C:10: error: `ifstream' was not declared in this scope triFree.C:10: error: expected `;' before "fin" triFree.C:11: error: `ofstream' was not declared in this scope triFree.C:11: error: expected `;' before "fout"

---

je ne comprend pas ! J'ai mis les bons include pourtant...

Merci pour votre aide !

Bonjour,
La bibliothèque standard du C++ est contenue dans le namespace "std".

Code:

---

1 2 3 4 5

std::ifstream fin("triangle.txt"); std::ofstream fout("resultri.txt"); std::string s;// le compteur de lignes // etc

---

Bonjour,
En effet j'avais oublié les std,
j'ai donc rajouté ces derniers comme suit :

Code:

---

1 2 3 4 5

std :: ifstream fin("triangle.txt"); std :: ofstream fout("resultri.txt"); std :: string s;// le compteur de lignes   std :: int i=0;// lit la première ligne

---

les messages d'erreurs sont (j'ai appelé mon fichier triFree.C)

Code:

---

1 2 3 4 5 6 7

triFree.C: In function `int main()': triFree.C:15: error: expected unqualified-id before "int" triFree.C:15: error: expected `;' before "int" triFree.C:23: error: `i' was not declared in this scope triFree.C:25: error: `fout' is not a member of `std' triFree.C:27: error: `i' was not declared in this scope triFree.C:27: warning: unused variable 'i'

---

On dirait vraiment qu'il voit rien du tout...

Citation:

---

On dirait vraiment qu'il voit rien du tout...

---

Non, non, le compilateur voit très bien...

Code:

---

std::int i = 0; // gné ?

---

ps: utilise les balises [code*][/code*] (sans l'étoile) stp, c'est plus simple à lire.

Hmm, dans ce cas , j'ai des messages de ce type, ce que j'avais deja en mettant un
using namespace std (et sans rajouté les std après bien sur)

Code:

---

1 2 3 4 5 6

/tmp/ccLp3Ig4.o(.text+0x52): In function `std::__verify_grouping(char const*, unsigned long, std::basic_string<char, std::char_traits<char>, std::allocator<char> > const&)': : undefined reference to `std::basic_string<char, std::char_traits<char>, std::allocator<char> >::size() const' /tmp/ccLp3Ig4.o(.text+0x182): In function `std::__verify_grouping(char const*, unsigned long, std::basic_string<char, std::char_traits<char>, std::allocator<char> > const&)': : undefined reference to `std::basic_string<char, std::char_traits<char>, std::allocator<char> >::operator[](unsigned long) const' /tmp/ccLp3Ig4.o(.text+0x2a2): In function `std::__verify_grouping(char const*, unsigned long, std::basic_string<char, std::char_traits<char>, std::allocator<char> > const&)':

---

"int" n'est pas un type de std. C'est un type de base.
Bref, tu ne peux pas mettre std:: devant un "int" (ni même un double, long, char, ...).

Oui oui, j'ai bien compris ! mais même en enlevant le std, ça ne fonctionne pas :cry:

Tu peux reposter le code complet que tu essayes de compiler en ce moment ?
Il doit y avoir une erreur de syntaxe quelque part pour que le compilo s'emmêle à ce point...

En attendant, pour clarifier l'utilisation des std:: et autre using namespace, il y a deux solutions :
Soit :

Code:

---

1 2 3 4 5 6 7 8 9 10 11

#include <string> #include <iostream>   using namespace std;   int main() { string s = "hello wordl"; cout << s << endl; return 0; }

---

soit :

Code:

---

1 2 3 4 5 6 7 8 9

#include <string> #include <iostream>   int main() { std::string s = "hello wordl"; std::cout << s << std::endl; return 0; }

---

Olala, même le hello world ne marche pas... en tapant ça :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

#include <string> #include <iostream>   using namespace std;   int main() { string s = "hello wordl"; cout << s << endl; return 0; }

---

je tape : gcc -wall helloworld.C -o hello

il me renvoi pleins d'erreurs de type

Code:

---

1 2 3 4 5 6 7 8

undefined reference to `std::basic_string<char, std::char_traits<char>, std::allocator<char> >::operator[](unsigned long) const' /tmp/ccB9b13I.o(.text+0x4a2): In function `main': : undefined reference to `std::allocator<char>::allocator()' /tmp/ccB9b13I.o(.text+0x4d2): In function `main': : undefined reference to `std::basic_string<char, std::char_traits<char>, std::allocator<char> >::basic_string(char const*, std::allocator<char> const&)' /tmp/ccB9b13I.o(.text+0x512): In function `main': : undefined reference to `std::allocator<char>::~allocator()'

---

Okkkkkkkkkkkkkkay, je pense avoir compris. :ccool:

Citation:

---

gcc -wall helloworld.C -o hello

---

gcc c'est pour le C ! Pour le C++ il me semble qu'il faut taper :

Citation:

---

g++ -wall helloworld.cpp -o hello

---

(A confirmer, je maitrise pas très bien GCC).

Wouhou, merci ! C'est bien ça, le hello world marche bien.
C'est vraiment mes premiers pas :-S

J'essaye maintenant pour les triangles !

Wouhou, ça marche aussi pour les triangles !

Merci encore !

"gcc" c'est la suite de compilation qui peut compiler tous les langages supportés.

Par défaut pour déterminer le langage à compiler "gcc" utilise l'extension des fichiers.

Si l'extension est non standard il est possible d'indiquer explicitement le langage via l'option "-x" :

Code:

---

gcc -x c++ -wall helloworld.C -o hello

---

Citation:

---

Envoyé par reptilia1251

Bonjour,
En effet j'avais oublié les std,
j'ai donc rajouté ces derniers comme suit :

Code:

---

1 2 3 4 5

std :: ifstream fin("triangle.txt"); std :: ofstream fout("resultri.txt"); std :: string s;// le compteur de lignes std :: int i=0;// lit la première ligne

---

---

Euh, je n'en suit pas sûr, mais il ne faut pas mettre d'espace entre les ::