Libération de la mémoire

Version imprimable

30/09/2010, 15h46
tutudanslarue

Libération de la mémoire

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!
30/09/2010, 16h02
johnlamericain

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

Citation:

Envoyé par johnlamericain

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:

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:

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:

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

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

Code:

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?

30/09/2010, 17h32
johnlamericain
Tout simplement tes pointeurs.
Code:

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).
30/09/2010, 17h57
tutudanslarue
Citation:
Envoyé par johnlamericain

Tout simplement tes pointeurs.

Code:

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:

l_cbxCurrent=NULL;

suivi d'un

Code:

delete

et la même chose pour

Code:

l_lstValues

cela ne change absolument rien. La ressource utilisé n'a pas changé :aie:
30/09/2010, 18h03
johnlamericain

Puis-je voir ton code de cette partie ?

Citation:

Envoyé par johnlamericain

Puis-je voir ton code de cette partie ?

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
	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));
	}
	l_cbxCurrent=NULL;
	delete l_cbxCurrent;
 
	return l_lstValues;
		l_lstValues=NULL;
	delete l_lstValues;

Je me dit que ça vient peut être aussi des

Code:

new QComboBox());

Mais je sais pas comment je pourrais les supprimer...

Citation:
Envoyé par tutudanslarue
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 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)); } l_cbxCurrent=NULL; delete l_cbxCurrent; return l_lstValues; l_lstValues=NULL; delete l_lstValues;
Le problème de ton code, c'est qu'à chaque fois que ta fonction arrive à une ligne return elle retourne et n'exécute pas ce qu'il y a après.

A chaque fois que la partie default de ton switch est executé, tu perds de la mémoire....

De plus la suppression de l_lstValues doit être fait par le pointeur qui récupére la sortie de ta fonction. Une référence serait surement mieux !

Citation:

Envoyé par tutudanslarue

Je me dit que ça vient peut être aussi des

Code:

new QComboBox());

Mais je sais pas comment je pourrais les supprimer...

Oui également, ça dépends de ce que tu en fais dans ta fonction. Utilise un passage par référence (pas de pointeur), c'est plus simple et tu n'as pas de problème de mémoire.

01/10/2010, 10h04
yan
Code:

QStringList* l_lstValues = new QStringList();

pourquoi un pointeur :cry::cry::cry:
Code:

1 2 3 4 return l_lstValues; l_lstValues=NULL; delete l_lstValues;
A ton avis, que fait le return :P
01/10/2010, 10h42
tutudanslarue
Citation:

Envoyé par johnlamericain

Le problème de ton code, c'est qu'à chaque fois que ta fonction arrive à une ligne return elle retourne et n'exécute pas ce qu'il y a après.

A chaque fois que la partie default de ton switch est executé, tu perds de la mémoire....

De plus la suppression de l_lstValues doit être fait par le pointeur qui récupére la sortie de ta fonction. Une référence serait surement mieux !

Oui également, ça dépends de ce que tu en fais dans ta fonction. Utilise un passage par référence (pas de pointeur), c'est plus simple et tu n'as pas de problème de mémoire.

Je ne comprends pas ce que tu veux quand tu dit que je dois passer en référence, tu parles de new QComboBox()?
Citation:
Envoyé par yan

Code:

QStringList* l_lstValues = new QStringList();

pourquoi un pointeur :cry::cry::cry:

Code:

1 2 3 4 return l_lstValues; l_lstValues=NULL; delete l_lstValues;

A ton avis, que fait le return :P
J'ai modifié enlevé le pointeur , la fonction renvoie un Qstringlist, ça ne change toujours rien! exactement la même mémoire utilisé
01/10/2010, 11h52
tutudanslarue

Bon...j'ai changé un pointeur en valeur normale, et puis j'ai supprimé les

Code:

new QComboBox()

...ça ne me plaisait pas , puisque

Code:

->value(

est censé être une lecture de valeur et non une création...ben ça à l'air de fonctionner :mouarf: . Faut que je regarde si ça fait pas planter le programme ailleurs mais l'origine du problème est bien là.
01/10/2010, 17h33
tutudanslarue

J'ai quand même l'impression que QT gère très mal les instances, je ne comprends pas pourquoi les objets ne sont pas supprimés...
Existe t'il des méthodes pour supprimer manuellement des objets que ne sont pas des pointeurs? Merci!
01/10/2010, 18h15
johnlamericain

Citation:

Envoyé par tutudanslarue

J'ai quand même l'impression que QT gère très mal les instances, je ne comprends pas pourquoi les objets ne sont pas supprimés...
Existe t'il des méthodes pour supprimer manuellement des objets que ne sont pas des pointeurs? Merci!

je pense que tu viens probablement d'un autre language tel que Java ou C# qui gère la mémoire différement du C++, ai-je raison ?

Il s'agit de la gestion de la mémoire d'un point de vue C++, Qt n'a rien a voir la dedans !
04/10/2010, 09h19
tutudanslarue

Citation:

Envoyé par johnlamericain

je pense que tu viens probablement d'un autre language tel que Java ou C# qui gère la mémoire différement du C++, ai-je raison ?

Il s'agit de la gestion de la mémoire d'un point de vue C++, Qt n'a rien a voir la dedans !

JAVA ;)
04/10/2010, 10h14
johnlamericain

Citation:

Envoyé par tutudanslarue

JAVA ;)

Dans ce cas tu devrais te tourner vers des cours C++ sur les pointeurs.

Quelques liens

Gérer ses ressources de manière robuste en C++
Les pointeurs du C et du C++
Penser en C++