Discussion :

[Arkaan]

Bonjour,

Je viens de débuter avec Qt et j'aimerais lire un fichier texte ligne par ligne.
Jusque là rien de bien sorcier je fais tout comme décrit dans la documentation
et ça compile.
Sauf que le programme ne lit que la première ligne du programme et pas la suite.
Du coup je me demande ce qu'il manque pas une incrémentation mais je n'en ai
pas vu dans les exemples. Pourriez m'aider à trouver où mon code cloche.

Merci

Le bout de code où se trouve l'ouverture et l'utilisation du fichier:

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
QFile file("data.txt");
 
if (file.open(QIODevice::ReadOnly | QIODevice::Text)) {
    QTextStream flux(&file);
    while(! file.atEnd()) {
        QString temp = flux.readLine();
 
        QStringList liste = temp.split(" : ");
        QString nomMachine = liste.at(0);
        QString adresse = liste.at(1);
 
        if(nomMachine == notreCode) {
            pageWeb =new Fenetre2(adresse,this);
            pageWeb->exec();
            control = true;
            break;
        }
    }
}

[Invité]

Bonjour,

je ne peux pas te répondre......mais comme tu es débutant je peux te donnez quelque liens qui sont sur le site actuel :

http://www.developpez.net/forums/d98...ces-debutants/

http://www.developpez.net/forums/d11...aux-exercices/

Le premier lien te conduira vers des exercices de programmation, le deuxiémes te conduira à leur réponses...

[gbdivers]

Bonjour Arkaan et bienvenue sur le forum

Pas de problème particulier avec ton code, c'est un code classique pour lire un fichier texte.
Quelques remarques :
- si ta première ligne contient le bon nom de machine, le break va arrêter ta boucle. Peut être que cela peut expliquer pourquoi tu lis qu'une ligne. Ajoute des qWarning() pour suivre le code
- perso, j'aime pas trop le break, ça diminue la lecture de ton code : quand on lit la ligne while, on se dit "la boucle parcours la totalité du fichier", on ne sait pas à ce moment là que la boucle peut s'arrêter avant. Ton code fait peut de lignes donc on voit rapidement qu'il y a un break mais sur code plus complexe, on le verrait pas forcement. J'aurai écris :

Code :

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

1
2
3
4
5
6
7
while(!file.atEnd() & !control) {
    ...
    if(nomMachine == notreCode) {
        ...
        control = true;
    }
}

- tu écris at(0) et at(1) sans tester avant la taille de la QStringList, pas bonne idée (même si tu sais que ton fichier contient toujours au moins 2 éléments par ligne) :
QStringList liste = temp.split(" : ");

Code :

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

1
2
3
if (liste.size() >= 2) {
    QString nomMachine = liste.at(0);
    QString adresse = liste.at(1);

Bonne continuation

@brother2007
Je comprends pas ton message. Tu es un peu hors sujet, non ?

[Arkaan]

Merci pour vos réponses!

J'ai fait ce que tu suggérais gbdivers à savoir que j'ai retiré le break et et que j'ai mis une
vérification sur la taille de la QStringList cependant le problème demeure.

Avec les qWarning il apparait que le programme ne rentre qu'une fois dans la boucle.
C'est pourquoi il ne vérifie que la première ligne. Du coup je regarde du côté des la
condition d'arrêt de la boucle puisque ça semble venir de là.

Merci encore

[jbb2811]

le

Code :

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

1
2
 
pageWeb->exec();

est bloquant donc si la 1ere ligne match, tu ne passe aux suivantes qu'une fois la fenêtre fermée.

[Arkaan]

ok merci pour l'information!

En fait j'ai réussi en utilisant du C++ "classique".
Pour ceux qui aurait eu le même problème voilà ma solution elle vaut ce qu'elle vaut
mais ça marche

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
 
ifstream fichier("data.txt");
 
if(fichier) {
    string ligne;
 
     while(getline(fichier, ligne)){
            QString lignebis = ligne.c_str();
            QStringList liste = lignebis.split(" : ");
            if (liste.size() >= 2) {
                QString nomMachine = liste.at(0);
                QString adresse = liste.at(1);
                if(nomMachine == notreCode) {
                    pageWeb =new Fenetre2(adresse,this);
                    pageWeb->exec();
                    control = true;
                }
            }
       }

pour le principe je lit le fichier ligne par ligne en extrayant les lignes sous forme de string
je les converti en QString ensuite pour les exploiter.

[gbdivers]

Euh, c'est le même code avec ifstream, sauf que tu as viré ton break. Si tu voulais que ta boucle continue, tu pouvais virer ton break aussi dans le premier code