Soucis avec getLine

Bonjour à tous. Je suis en train de me coder un simili parseur histoire de pratiquer un peu la navigation dans les fichiers.
Je bloque sur cette méthode, qui est une des premières que j'ai implantée.

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 34 35 36 37

47 bool Parseur::allerSousPartie(string nomSP, int nb) 48 { 49 //Placer le curseur au tout début du fichier 50 fichier.seekg(0, ios::beg); 51 string tmp; 52 //Parcourir les lignes jusqu'à trouver '"begin" + nomSP' nb fois 53 int cpt=0; 54 bool dansSP = false; 55 bool continuer = true; 56 cout << "test2" << endl; 57 while (continuer); 58 { 59 cout << "test2" << endl; 60 getline(fichier, tmp); 61 cout << tmp << endl; 62 if (tmp == ("begin " + nomSP) && dansSP == false) 63 { 64 cout << "trouvé!" << endl; 65 cpt++; 66 if (cpt == nb) 67 { 68 ligneDebut.push_back(fichier.tellg()); 69 dansSP = true; 70 } 71 } 72 if (tmp == ("end " + nomSP) && dansSP == true) 73 { 74 ligneFin.push_back(fichier.tellg()); 75 compteurSP++; 76 return true; 77 } 78 tmp.clear(); 79 if (fichier.eof()) 80 continuer = false; 81 } 82 return false; 83 }

---

Le soucis arrive à la ligne 60. A l'execution du programme, j'ai la chaine "test1" qui s'affiche, mais "test2" ne s'affiche pas. Je pense que ça viens d'une mauvaise utilisation de getline(), mais je n'arrive pas à voir ce qui ne va pas.
Pouvez-vous m'indiquer le soucis?

Merci

Salut,

Citation:

---

Envoyé par NLS le pingouin

Pouvez-vous m'indiquer le soucis

---

Tu as un point-virgule en trop à la fin de la ligne 57, ce qui a comme effet une boucle infinie.

MAT.

D'ailleurs pour rebondir sur la correction de Mat, pour un parseur, il serait plus judicieux de ta part de ne pas utiliser un while mais un for avec comme condition d'arrêt le retour de getline

Code:

---

1 2 3 4 5

for(string s; getline(fichier,s);) { // étudier la string en conséquence }

---

Citation:

---

Envoyé par Astraya

pour un parseur, il serait plus judicieux de ta part de ne pas utiliser un while

---

Bah, et ça ce n'est pas bon ? :?

Code:

---

1 2 3 4

std::string str; while (std::getline(fichier, s)) { // (...) }

---

Si bien sur! C'est juste une parenthèse de ma part. Dans ton cas la string est valable dans toute la fonction, moi juste dans le for, c'est juste une petite optimisation mémoire, l'instanciation tardive et la string est englobé dans un bloc( le for)

Hé hé, c'était juste histoire de chercher la petite bête… :P

Tiens, justement, si on doit en plus analyser chaque ligne :

Code:

---

1 2 3 4 5 6 7 8

// is est un std::istream quelconque   for (std::string s; std::getline(is, s); ) { std::istringstream iss(s); while (getline(iss, s, ';')) { // Un certain nombre d'opérations... } }

---

Tu le déclarerais où le std::(i)stringstream ?

Si on le déclare seulement là où on en a besoin, dans la première boucle, il est créé et détruit à chaque itération.
Par contre, si on veut éviter ça, il faut le déclarer avant, et le réinitialiser à chaque itération ; et puis sa durée de vie excède ce qui est nécessaire.

Question subsidiaire :
Pour l'exemple j'ai réutilisé la variable qui sert à lire les lignes.
Mais si on a besoin d'autres variables, avec des types différents (ou plusieurs std::string), où est-il le plus judicieux de les déclarer ?

Citation:

---

Tu le déclarerais où le std::(i)stringstream ?

---

Tu encapsules les variables nécessaires :ccool:

Code:

---

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

struct MesVariables { MesVariables() :s(),os(),i(){}   std::string s; std::ostringstream os; int i; };   for( MesVariables s; s.i<10; ++s.i) { //s.variable }

---

Ou tu utilises for_each et foncteur :yaisse2:

Hum…
On peut définir un struct dans le corps d'une fonction ?

Bien sur, elle sera uniquement valide dans la fonction.
Tu peux déclarer une classe dans une classe, dans une fonction donc dans le main compris!
D'ailleurs c'est quelque chose que je fais souvent, déclarer le foncteur + utiliser le for_each dans la même fonction.

Ahhh… je vois…
Marchi beaucoup ! :ccool:

Salut,

Il ne faut pas oublier non plus que, bien que ce ne soit pas forcément recommandé, les différentes parties de la boucle for permettent de travailler sur plusieurs choses:

Code:

---

for(int x=10,y=0; i>0,i<10;--x,++y)

---

est, par exemple, tout à fait légal ;)

Oui, ça je m'en ser(vai)s souvent.
Le seul souci, c'est que dans la première partie du for, on ne peut que déclarer que des objets du même type.

Code:

---

i>0,i<10

---

Ce ne serait pas plutot un && a la place de la virgule(en fait je ne vois pas trop son interêt) ?

Citation:

---

Envoyé par Joe Dralliam

Code:

---

i>0,i<10

---

Ce ne serait pas plutot un && a la place de la virgule(en fait je ne vois pas trop son interêt) ?

---

En fait, je voulais surout écrire i>0, j<10...

j'étais quelque peu distrait :P