Discussion :

[NLS le pingouin]

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 :

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
 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

[Mat007]

Salut,

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.

[Astraya]

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 :

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

1
2
3
4
5
 
for(string s; getline(fichier,s);)
{
     // étudier la string en conséquence
}

[Steph_ng8]

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 :

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

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

[Astraya]

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)

[Steph_ng8]

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

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

Code :

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

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 ?