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 ?

[Astraya]

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

Tu encapsules les variables nécessaires

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

[Steph_ng8]

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

[Astraya]

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.

[Steph_ng8]

Ahhh… je vois…
Marchi beaucoup !

[koala01]

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 :

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

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

est, par exemple, tout à fait légal

[Steph_ng8]

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.

[Invité]

Code :

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

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

[koala01]

Code :

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

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