Discussion :

[tutudanslarue]

Bonjour à tous! J'ai un souci au niveau de l'utilisation de la mémoire, si quelqu'un à une idée de la cause du problème ce serait super!

En gros je fais appel à des boucles for imbriqués, et dans ces boucles for j'instancie des objets à chaque fois. ces boucles sont dans une méthode, seulement le souci c'est qu'à la sortie de ces méthodes l'espace utilisé est toujours présent. Du coup un nouvel appel à la méthode crée un nouvel espace de mémoire utilisé!

Les variables locales sont elle réellement détruites à la sortie des méthodes?
Comment pourrais je supprimmer manuellement ces variables? Merci!

[johnlamericain]

Il serait plus facile de te répondre avec le code de ces boucles. Peux-tu nous le fournir ?

[tutudanslarue]

Il serait plus facile de te répondre avec le code de ces boucles. Peux-tu nous le fournir ?

oui biensur

alors on a une première méthode dans laquel il y a un for:

Code :

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

1
2
3
4
5
6
7
			for (int index = 0; index < l_xConf.size(); index++)
			{
 
				ChangerAttribut(l_xConf.at(index));
 
				pxDlgAttente->stepItProgressBar();
			}

dans ChangerAttribut:

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
	for (int indexLigne = 0; indexLigne < m_lstAttributsLcdv->rowCount(); indexLigne++) 
	{
		l_saCodesLcdv = GetComboBoxValues(ECmbxCodes, indexLigne);
		l_saValeursLcdv = GetComboBoxValues(ECmbxValeurs,indexLigne);
 
		for (int index = 0; index < l_saCodesLcdv->size(); index++) 
		{
			if (p_sClasseValeurLCDV == l_saCodesLcdv->at(index))
			{
				[...]
 
				l_iIndiceLigne = indexLigne;
			}
			if (l_iIndiceLigne != -1) break;
		}
		if (l_iIndiceLigne != -1) break;
	}

En commentant décommentant, je m'apperçois que la mémoire utilisé augmente à ce niveau là:

Code :

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

1
2
		l_saCodesLcdv = GetComboBoxValues(ECmbxCodes, indexLigne);
		l_saValeursLcdv = GetComboBoxValues(ECmbxValeurs,indexLigne);

du coup voilà la méthode GetComboBoxValues():

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
	QComboBox* l_cbxCurrent;
	switch(p_eType)
	{
		case ECmbxCodes:
			l_cbxCurrent = m_mapComboBoxCodes->value(GetNomAttributAvecLigne(p_iLigne),new QComboBox());
			break;
		case ECmbxValeurs:
			l_cbxCurrent = m_mapComboBoxValeurs->value(GetNomAttributAvecLigne(p_iLigne),new QComboBox());
			break;
		default:
			return NULL;
	}
 
	QStringList* l_lstValues = new QStringList();
	for(int i = 0 ; i < l_cbxCurrent->count() ; i++)
	{
		l_lstValues->append(l_cbxCurrent->itemText(i));
	}
	return l_lstValues;

Je me dit que ça vient du grand nombre de variable qui est utilisé, mais normalement ceux ci devraient être detruit une fois la méthode terminé non?

[johnlamericain]

Tout simplement tes pointeurs.

Code :

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

1
2
QComboBox* l_cbxCurrent;
QStringList* l_lstValues = new QStringList();

sont probablement jamais détruits (ou au moins celui qui n'est pas retourné par la fonction).

[tutudanslarue]

Tout simplement tes pointeurs.

Code :

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

1
2
QComboBox* l_cbxCurrent;
QStringList* l_lstValues = new QStringList();

sont probablement jamais détruits (ou au moins celui qui n'est pas retourné par la fonction).

Ben j'ai rajouté un

Code :

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

l_cbxCurrent=NULL;

suivi d'un

Code :

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

delete

et la même chose pour

Code :

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

l_lstValues

cela ne change absolument rien. La ressource utilisé n'a pas changé

[johnlamericain]

Puis-je voir ton code de cette partie ?