Discussion :

[Eldergrim]

Bonjour,

J'essaie de lire le contenu d'un fichier texte dont le contenu ressemble à ceci :

Code :

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

1
2
3
12 45
5 78
2 6

Pour chaque ligne je voudrais récupérer les deux valeurs dans deux variables de type int.
J'y parviens en utilisant ce code :

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
#include <iostream>
#include <fstream>
#include <sstream>
using namespace std;
int main(int argc, char **argv) {
    string nomFichier = "test1.txt";
    ifstream fichier(nomFichier.c_str(), ios::in);
    string ligne;
    int first, second; 
 
    while(getline(fichier, ligne)) {
        cout << ligne << "\t=>\t";
        istringstream ss(ligne);
        ss >> first >> second;
        cout << first << "\t" << second << endl;
        ss.str().clear();
    }
 
    fichier.close();
    return 0;
}

Le seul "problème" est que je déclare un istringstream à chaque itération, ce qui ne me semble pas très propre ni optimisé.
Pouvez-vous m'indiquer comment modifier ce code pour ne créer un qu'un seul istringstream et le réinitialiser avec la nouvelle ligne à chaque passage dans la boucle ?

Je précise que je suis débutant en C++ et que j'ai trouvé un message, ici, qui semble répondre à ma question, mais je n'ai pas réussi à mettre en oeuvre la solution indiquée.

Merci d'avance

[jo_link_noir]

Code :

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

1
2
3
4
5
6
7
8
9
istringstream ss;
while(getline(fichier, ligne)) {
    cout << ligne << "\t=>\t";
    ss.str(ligne);
    if (!(ss >> first >> second)) {
        //erreur
    }
    cout << first << "\t" << second << endl;
}

Mais sinon, tu peux tous lire en une traite. Les flux considèrent les espaces, sauts de ligne et tabulations comme un seul caractère d'espacement.

Code :

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

1
2
3
while(fichier >> first >> second) {
    cout << first << "\t" << second << endl;
}

[Eldergrim]

En appliquant ta première proposition je ne parviens pas tout à fait à faire ce que je veux. Avec ce code ce sont toujours les données de la première ligne qui sont affichées.

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
#include <iostream>
#include <fstream>
#include <sstream>
using namespace std;
int main(int argc, char **argv) {
    string nomFichier = "test1.txt";
    ifstream fichier(nomFichier.c_str(), ios::in);
    string ligne;
    int first, second; 
 
    istringstream ss;
    while(getline(fichier, ligne)) {
        cout << ligne << "\t=>\t";
        ss.str(ligne);
        if (!(ss >> first >> second)) {/*erreur*/}
        cout << first << "\t" << second << endl;
        ss.str().clear();
    }        
    fichier.close();
    return 0;
}

La sortie donne ceci, que je mette ou pas la ligne 17 :

Code :

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

1
2
3
1 45    =>      1       45
21 2    =>      1       45
3 7     =>      1       45

En revanche la deuxième solution fonctionne très bien. Par contre je ne suis pas certain de bien comprendre pourquoi
J'ai un peu de mal à voir dans quel ordre se fait l'évaluation de la ligne while(fichier >> first >> second), pourquoi les lignes sont bien lues une à une et pas autrement, etc.
Si tu peux me donner quelques explications je suis preneur.

Merci pour tes réponses.

[jo_link_noir]

Oups, pour le premier exemple j'ai oublié de mettre ss.clear(); avant le if. Lorsque les 2 nombre sont lus, le flux passe éventuellement en oef s'il n'y a plus rien sur la ligne. Il faut donc le remettre dans un état valide (ios::clear).
Le clear() de ss.str().clear(); fait partit de std::string et ne fait strictement rien ici. str() retourne un std::string et clear() la vide...

En revanche la deuxième solution fonctionne très bien. Par contre je ne suis pas certain de bien comprendre pourquoi
J'ai un peu de mal à voir dans quel ordre se fait l'évaluation de la ligne while(fichier >> first >> second), pourquoi les lignes sont bien lues une à une et pas autrement, etc.
Si tu peux me donner quelques explications je suis preneur.

L’évaluation se fait dans le sens de la lecture first puis second puis on recommence.
Que ça lise ligne pas ligne n'est qu'une conséquence du format utilisé. Les nombres pourraient être tous sur une ligne ou un par ligne le résultat sera le même.

Quand une type scalaire (char fait exception) est lu, le flux va ignorer tous les espaces blancs. Ces caractères sont grosso modo déterminés par isspace.
Ensuite, par exemple pour un int, il va récupérer tous les caractères qui suivent et transformer ça en entier.
Si une erreur survient pendant la lecture comme un caractère invalide (comprendre différent d'un chiffre) ou la fin du fichier, alors le flux change d'état et la boucle s'arrête.
Faire if(fichier >> variable); et l'équivalent de fichier >> variable; if (fichier.good());.

[Eldergrim]

La première solution fonctionne effectivement mieux avec le ss.clear().

Pour la deuxième méthode, je vais faire quelques tests pour mieux comprendre le fonctionnement. Je n'ai pas l'habitude d'utiliser des flux, et encore moins en C++. Il faut que je pratique !

Merci pour tes explications.