Discussion :

[Idleman]

Bonjour,

Je ne suis pas très a l'aise avec les pointeurs et je tombe devant une erreur que je ne parviens pas à comprendre :

Code :

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

QStringList*' is not a class, struct, or union type

Cela se passe sur la ligne :

Code :

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

QStringList* fullColoration = Functions::readFileLine("langages/"+name+".ini");

qui fait appel a cette fonction :

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
QStringList* Functions::readFileLine(QString filePath){
    QStringList* lines = new QStringList();
    QFile file(filePath);
 
       if(file.open(QIODevice::ReadOnly | QIODevice::Text))
       {
           QTextStream flux(&file);
           while(!flux.atEnd())
           {
               lines->append(flux.readLine());
           }
           file.close();
       }
       return lines;
}

Comme tout ignorant qui se respecte, j'ai tout essayé sur la syntaxe (enlever, ajouter des * un peu partout etc) mais ce fichu machin refuse obstinément de m’obéir !

Une idée?

Amicalement,

idle

[abdou57]

Bonjour, essaie plutôt ceci:

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
QStringList* Functions::readFileLine(QString filePath){
    QStringList lines;
    QFile file(filePath);
 
       if(file.open(QIODevice::ReadOnly | QIODevice::Text))
       {
           QTextStream flux(&file);
           while(!flux.atEnd())
           {
               lines << flux.readLine();
           }
           file.close();
       }
       return &lines;
}

Bonne continuation.

[yan]

Bonjour, essaie plutôt ceci:

Code :

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

1
2
3
4
5
QStringList* Functions::readFileLine(QString filePath){
    QStringList lines;
    ...
       return &lines;
}

surtout pas
Une fois sortie de la fonction, lines est détruit => le pointeur sera invalide

@Idleman as tu essayé d'ajouter

Code :

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

#include <QStringList>

en début de ton fichier?

[abdou57]

surtout pas
Une fois sortie de la fonction, lines est détruit => le pointeur sera invalide

@abdou57 as tu essayé d'ajouter

Code :

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

#include <QStringList>

en début de ton fichier?

pour l'inclusion de QStringList la réponse est oui.

pour la destruction de lines, je n'y ai pas pensée.

merci.

[Idleman]

Bonjour,

Merci pour toutes ces réponses

Malheureusement ça n'est pas un simple oublis d'inclusion de QStringList

j'ai bien le #include <QStringList> dans mon fichier d'entêtes.

D'autres idées?

[superjaja]

Tu peux toujours essayer cela c'est plus propre je trouve.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
void Functions::readFileLine(QString filePath, QStringList& lines)
{
   QFile file(filePath);
 
   if(file.open(QIODevice::ReadOnly | QIODevice::Text))
   {
	   QTextStream flux(&file);
	   while(!flux.atEnd())
	   {
		   lines << flux.readLine();
	   }
	   file.close();
   }
}

Au moins c'est la fonction appelante qui maîtrise la durée de vie de l'objet.

PS: A valider j'ai pas testé le code mais cela devrait fonctionner...

[Idleman]

Visiblement ça fonctionne, je dis visiblement parce que le programme se lance mais n'affiche plus rien, je suppose donc qu'il est passé au problème suivant XD.

Merci beaucoup pour l'astuce, j'ignorais totalement qu'on pouvais retourner une valeur de cette façon

[Idleman]

tout compte fait, ça ne semble pas fonctionner...

J'appelle ta nouvelle fonction comme ça :

Code :

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

1
2
QStringList fullColoration;
Functions::readFileLine("langages/"+name+".ini",fullColoration);

Mais visiblement le programme n'aime pas, il me dit qu'un élément invalid a été passé a C

J'ai aussi essayé comme ça :

Code :

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

1
2
QStringList* fullColoration = new QStringList ();
Functions::readFileLine("langages/"+name+".ini",fullColoration);

mais il me dit que readFileLine n'accepte plus l'argument

bref... je suis perdu

[superjaja]

La première méthode est la bonne à condition que ta fonction readLine soit déclarée static sinon tu ne peux pas l’appeler sans instancier l'objet.

dans le .h faire:

Code :

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

static void readFileLine(QString filePath, QStringList& lines)

De meme fait comme ça:

Code :

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

1
2
3
QStringList fullColoration;
QString strPath="langages/"+name+".ini";
Functions::readFileLine(strPath,fullColoration);

[Idleman]

C'était déjà le cas... , le debug fait toujours :

Code :

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

1
2
3
Démarrage de C:\Workpaces\QT\CodeZ\debug\CodeZ.exe...ASSERT failure in QList<T>::at: "index out of range", file ..\..\..\QtSDK\Desktop\Qt\4.8.0\mingw\include/QtCore/qlist.h, line 469
Invalid parameter passed to C runtime function.
Invalid parameter passed to C runtime function.

[superjaja]

Tu fais quoi après de ta QStringList ?
Tu la lit je suppose et à mon avis tu vas a un index erroné ?

Cela ne fonctionne pas comme ça ?

Code :

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

1
2
3
QStringList fullColoration;
QString strPath="langages/"+name+".ini";
Functions::readFileLine(strPath,fullColoration);

Sinon essaye au débugger c'est fort pratique...

[Idleman]

Je ne fais rien après, j'ai tous mis en commentaire le temps de résoudre le problème, effectivement l'erreur semble venir d'un index lu et non définis seulement je ne lis jamais cette liste (a moins que qt ne se soit mis a interpréter les commentaires mais j'en doute ^^)

Pour plus de clarté, voila le code intégral de la page :

language.cpp

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
#include "langage.h"
 
Langage::Langage()
{
    this->name = name;
 
    QStringList fullColoration;
    Functions::readFileLine("langages/"+name+".ini",fullColoration);
    QHash<QString, QStringList> finalTab;
    /*foreach(QString line,fullColoration){
        QStringList keyvalue =  line.split(" = ");
        QStringList values =  keyvalue.at(1).split("\",\"");
        finalTab.insert(keyvalue.at(0),values);
    }*/
 
    this->coloration = finalTab;
}
 
QHash<QString, QStringList> Langage::getColorations(){
    return this->coloration;
}
 
void Langage::setName(QString name){
    this->name = name;
}

Et la page de fonctions associées :

functions.cpp

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
38
39
40
41
#include "functions.h"
 
Functions::Functions()
{
 
 
}
 
 
void Functions::readFileLine(QString filePath, QStringList& lines)
{
 
   QFile file(filePath);
 
   if(file.open(QIODevice::ReadOnly | QIODevice::Text))
   {
       QTextStream flux(&file);
       while(!flux.atEnd())
       {
           lines << flux.readLine();
       }
       file.close();
   }
}
 
QString Functions::readFile(QString filePath){
    QFile file(filePath);
       QString texte;
       if(file.open(QIODevice::ReadOnly | QIODevice::Text))
       {
           QTextStream flux(&file);
           while(!flux.atEnd())
           {
               QString temp = flux.readLine();
               texte += temp;
 
           }
           file.close();
       }
       return texte;
}

[mitkl]

Petit aparté pour dire qu'au lieu de traiter tes fichiers INI manuellement, tu devrais plutôt utiliser QSettings qui t'évitera de recoder le système entièrement et en plus tu bénéficieras de la robustesse de la classe déjà codée.

Petit exemple ici : http://qt.developpez.com/faq/?page=m...gs-fichier-ini

[Idleman]

Merci pour l'astuce, mais les fichiers sont nommés en .ini pour des raisons historiques uniquement, ce sont en réalité des json :)

[Troudhyl]

Tu es sûr d'avoir les bases nécessaires de C++ notamment pour faire la différence entre pointeur, référence, passage par copie, etc. ? Parce que quand tu essayes des choses (passage d'un pointeur quand une référence est demandée...) on ne dirait pas. Il faudrait commencer par ça.

C'est quelle ligne de code qui provoque l'assert ?

[Idleman]

Tu es sûr d'avoir les bases nécessaires de C++ notamment pour faire la différence entre pointeur, référence, passage par copie, etc. ? Parce que quand tu essayes des choses (passage d'un pointeur quand une référence est demandée...) on ne dirait pas. Il faudrait commencer par ça.

Je ne suis pas très a l'aise avec les pointeurs

C'est quelle ligne de code qui provoque l'assert ?

Aucune ligne ne m'est retournée à part la L 469 du fichier ..\..\..\QtSDK\Desktop\Qt\4.8.0\mingw\include/QtCore/qlist.h et je ne peux pas faire de débogage point par point.

[Troudhyl]

Tu n'es pas très à l'aise, ok, deviens-le, c'est incontournable

Tu ne peux pas lancer l'appli en déboguage pour voir la pile d'appels quand l'assert sort ?