Discussion :

[alpha_one_x86]

Bonjour, j'ai un probléme sur ce code:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
QStringList env=theApplication->environment();
	//remove the value and add to path
	for (int i = 0; i < env.size(); ++i) {
		if(env.at(i).startsWith("QMAKESPEC="))
			env.removeAt(i);
		if(env.at(i).startsWith("QTDIR="))
			env.removeAt(i);
		if(env.at(i).startsWith("PATH="))
		//QString(';')+QtAbsolutePath+QDir::separator()+"bin;"+MingwAbsolutePath+QDir::separator()+"bin"
			env.at(i)=QString("test");
	}

Il me fait:

passing 'const QString' as 'this' argument of 'QString& QString::operator=(const QString&)' discards qualifiers

Alors que je n'est aucun const, ou est mon erreur et d'ou proviens elle?
Merci d'avance.

[yan]

Alors que je n'est aucun const, ou est mon erreur et d'ou proviens elle?

comme le dit la doc at() retourne une refrence const
http://qt.developpez.com/doc/4.4/qlist/#at

et tes test/traitement sur la QStringList est très très très dangereuse ...

Code :

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

1
2
3
4
5
6
7
8
9
 
    if(env.at(i).startsWith("QMAKESPEC="))
        env.removeAt(i);
 
    if(env.at(i).startsWith("QTDIR="))
        env.removeAt(i);
 
    if(env.at(i).startsWith("PATH="))
	env.at(i)=QString("test");

exemple 1:

• i = env.size() -1.
• env.at(i).startsWith("QMAKESPEC=") == true => env.removeAt(i);
• i est devenue un indice invalide => env.at(i).startsWith("QTDIR=") => exception ou crash

exemple 2 :

• env.at(i).startsWith("QMAKESPEC=") == false
• env.at(i).startsWith("QTDIR=")) == true => env.removeAt(i); seulement le nouvelle élément en i commence par QMAKESPEC
• i++ => tu na pas viré l'argument commençant par QMAKESPEC

[alpha_one_x86]

Alors comment quand même faire cette modification?
J'ai utilisé des [] ça à l'aire de marcher.
Voila le nouveau code que j'ai fait:

Code :

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

1
2
3
4
5
6
7
8
9
	for (int i = 0; i < env.size(); ++i) {
		if(env.at(i).startsWith("QMAKESPEC=") || env.at(i).startsWith("QTDIR="))
		{
			i--;
			env.removeAt(i);
		}
		if(env.at(i).startsWith("PATH="))
			env[i]=QString(';')+QtAbsolutePath+QDir::separator()+"bin;"+MingwAbsolutePath+QDir::separator()+"bin";
	}

ça m'as l'aire d'étre bon, tu en pense quoi?

[yan]

Code :

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

1
2
3
 
i--;
env.removeAt(i);

et tu pense virer quoi !!!!

en utilisant les iterator et avec ce que j'ai compris, je ferais un truc comme :

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
QStringList env=theApplication->environment();
QStringList ::iterator it = env.begin();
 
while (it != env.end());
{
   if( it->startsWith("QMAKESPEC=") ||  it->startsWith("QTDIR="))
   {
       it = env.erase(it);
   }
   else if ( it->startsWith("PATH=") 
   {
      *it += QString(';') 
                + QtAbsolutePath + QDir::separator() + "bin;"
                + MingwAbsolutePath + QDir::separator() + "bin";
 
       ++it;
   }
   else
   {
       ++it;
   }
}