char*, QString et QStringList

Version imprimable

06/07/2010, 11h54
DSGSLA

char*, QString et QStringList

Bonjour,

Je reçois un char* d'une dll externe, contenant une liste de noms séparés par des NULL et terminée par double NULL.
J'ai utilisé un code trouvé sur internet sur cette dll qui remplace les NULL par des vigules puis place les mots dans un vecteur. J'ai remplacé cette dernière partie par le remplissage d'une QStringList.
Cette QStrinList est ensuite utilisée pour paramétrer un programme lancé par un QProcess.
Ce programme renvoie une erreur mais affiche bien les bons paramètres.
Si je lance ce programme directement en ligne de commande avec les mêmes paramètres il fonctionne sans erreur.
Si dans mon code j'alimente la liste en dur (<<) avec des noms, le programme fonctionne pour ces noms mais toujours pas pour les autres.

Je pense qu'il 'a un problème dans l'alimentation de ma liste, mais comme les bons noms sont affichés dans mes sorties de debuggage il s'agit peut-être d'un problème de "code page".
Comment le résoudre ?
Puis-je créer simplement ma liste directement avec des fonctions Qt ?
06/07/2010, 11h57
Gulish

Ca doit être faisable. Peux tu nous montrer un bout de code, et un screenshot de l'erreur que ton programme renvoit ?

G.

Code:

1
2
3
4
5
6
7
8
9
10
11
(...)
            convertToCsv(s__info.jobList);
 
            char sep[] = ",";
            QStringList jobs;
            char* token = strtok(s__info.jobList,sep);
            while (token != NULL) {
                    jobs << QString(token).trimmed();
                    token = strtok(NULL,sep);
            }
(...)

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
char* convertToCsv(char* strArray) {
 
        char* ptr = strArray;
        size_t len;
 
        if (strArray != NULL) {
                //while (ptr[0] != NULL) {
                while (ptr[0] != '\0') {
                        len = strlen(ptr);
                        //if (ptr[len+1] != NULL) {
                        if (ptr[len+1] != '\0') {
                                ptr[len] = ',';
                        }
                        ptr += len+1;
                }
        }
        return strArray;
 
}

Le message d'erreur du sous-programme renvoie bien le nom "test02" mais n'a pu effectuer le traitement.

Code:

1
2
Processing "test02" - Failed
Cannot get exclusive access to executable file for job test02

Si je fais dans mon code jobs << "test02" j'ai bien en plus

Code:

1
2
3
Intializing
 
Processing "test02" - Succeeded

Et un truc du genre :

Code:

1
2
3
4
5
 
convertToCsv(s__info.jobList);
QString jobList(s__info.jobList);
QStringList jobs = jobList.split(",");
qDebug()<<jobs;

Bonjour DSGSLA

J'ai testé ton code et j'ai comparé avec un code "maison" :
Code:

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 char array[9] = { 'A', 'B', '\0', 'C', 'D', '\0', 'E', '\0', '\0' }; QString str; bool continu = true; char last = '\1'; char * ptr = array; while(continu) { if((*ptr) == '\0') { if(last == '\0') { continu = false; } else { list << str; str = QString(); } } else { str += (*ptr); } last = *ptr; ++ptr; }
Ton code fonctionne correctement et retourne le résultat attendu (testé sur la chaine de caractères indiquée dans le code)

Le problème provient peut être de ton appel à QProccess. Tu peux nous montrer ton code d'appel ?

J'ai trouvé : ce n'est pas la constitution de la liste mais le code ci-dessous (positionné après celui déjà montré) qui agit je ne sais comment sur la QStringList et qu'après ça foire.
Mis en commentaire tout passe.
La variable "force" prend la valeur 'O' ou 'N'.

Code:

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
string status;
            int retval;
            for (QStringList::iterator it=jobs.begin(); it!=jobs.end(); it++) {
 
                    hJob = __OpenJob(hProject,(char *)it->toStdString().c_str());
                    if (hJob != NULL) {
                            err = __GetInfo(hJob,__JOBSTATUS,&s__Info);
                            if (err != __NOERROR) {
                                    retval = __NOTRUNNABLE;
                            } else {
                                    retval = s__info.jobStatus;
                            }
                    } else {
                            retval = __NOTRUNNABLE;
                    }
                    if (*force == 'N') {
                        if (retval != __NOTRUNNABLE ) {
                            getJobStatus(retval,status);
                            cout << "Job " << it->toStdString() << " deja traite (" << status << ")." << endl;
                        }
                    }
                    else {
                        if (retval == __RUNNING) {
                            cout << "Job " << it->toStdString() << " en RUNNING." << endl;
                            jobs.erase(it);
                        }
                        else {
                            getJobStatus(retval,status);
                            cout << it->toStdString() << "," << retval << "," << status << endl;
                        }
                    }
            }

Peut-être que l'appel

Code:

(char *)it->toStdString().c_str()

modifie le contenu de la liste.

06/07/2010, 15h14
gbdivers
La conversion de QString en char* pose parfois quelques problèmes (en particulier à cause de l'UTF8 et l'internationnalisation). Il existe d'ailleurs plusieurs fonctions qui font des "choses" très proches mais très différentes quand même :) (toAscii(), toLatin1(), toUtf8(), toLocal8Bit()).

Pour travailler avec le C, je te conseille la macro "qPrintable" de qtglobal.h qui est faite spécialement pour ça (http://qt.developpez.com/doc/4.6/qtglobal/#qprintable)

PS : pour parcourir ta QStringList, tu peux utiliser aussi la macro "foreach" (http://qt.developpez.com/doc/4.6/qtglobal/#foreach) sans que cela soit très important pour les performances :
Code:

1 2 foreach(QString str, TaStringList) ...
06/07/2010, 15h39
DSGSLA

qPrintable renvoie un const char* et j'ai besoin d'un char*, le compilateur ne laisse pas passer ça.
06/07/2010, 15h42
Invité

Tu peux utiliser une conversion bourrin avec const_cast...

EDIT : tu peux aussi faire str.tolocal8bit().data().
06/07/2010, 15h45
gbdivers

Si tu lis la doc, il est indiqué :

Citation:

Returns str as a const char *. This is equivalent to str.toLocal8Bit().constData()

il suffit alors d'utiliser :

Code:

char* str.toLocal8Bit().data();
06/07/2010, 16h07
DSGSLA

Je vais devenir zinzin. Le simple fait de parcourir la liste après sa constitution et le sous-programme ne fonctionne plus ! Tant que j'envoie la liste sans y accéder ça marche !
06/07/2010, 16h34
gbdivers

Le seul endroit où il pourrait y avoir un problème (à mon avis) est dans la fonction __OpenJob (étrange comme nom de fonction. C'est une lib ?)

Code:

hJob = __OpenJob(hProject, (char *) it->toStdString().c_str());

1. tu transformes un const char* en char*
2. si ta fonction accède et modifie la taille de tes chaines, ça risque de planter (je ne sais pas comment QByteArray gère cela)

Si tu commentes cette ligne, as-tu encore une erreur ?
A quoi sert cette fonction ? Si c'est une fonction que tu as crées, pourquoi ne pas continuer à travailler avec QString ?
06/07/2010, 16h36
DSGSLA

J'ai trouvé, ça n'avait rien à voir avec Qt, ou le C++.
Après une des fonctions de la dll externe il faut appeler une fonction de "libération", sinon l'objet appelé par la dll reste verrouillé, ce qui plante le sous-programme.

Merci quand même pour vos réponses.