Discussion :

[FstDsi]

Salut tout le monde

J'utilise des vecteurs de structure et j'ai un problème de débordement de mémoire.

Code :

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

1
2
3
4
5
6
 
struct PseudoConcept
{
 vector<int> intent;
 vector<int> extent;
};

Code :

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

1
2
3
4
5
6
7
8
 
struct InfoContext
{
PseudoConcept couple;
PseudoConcept pseudo;
vector <int> concept;
int num_concept;
};

Code :

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

1
2
 
typedef vector<InfoContext*> ContextVect;

En fait le débordement est localisé dans une boucle quand j'essaye de remplir une variable de "ContextGlobal" de type ContextVect

Le code est le suivant :

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
42
43
44
45
46
47
48
49
50
51
52
53
54
 
 
ContextVect ContextGlobal;
InfoContext ContextElement;
PseudoConcept p,c;
vector<int> cc;
 
ContextGlobal.reserve(couverture);
 
for(int n=0;n<10;n++)
{
 
 c.intent.clear();
 c.extent.clear();
 p.intent.clear();
 p.extent.clear();
 ContextElement.couple.intent.clear();
 ContextElement.couple.extent.clear();
 ContextElement.pseudo.intent.clear();
 ContextElement.pseudo.extent.clear();
 ContextElement.concept.clear();
 
 ContextGlobal[n]->couple.intent.clear();
 ContextGlobal[n]->couple.extent.clear();
 ContextGlobal[n]->pseudo.intent.clear();
 ContextGlobal[n]->pseudo.extent.clear();
 ContextGlobal[n]->concept.clear();
 
 
 c.intent.resize(1);
 c.extent.resize(1);
 c.intent.push_back(exp_courant);//exp_courant déja rempli 
 c.extent.push_back(attr_courant); //attr_courant déja rempli 
 ContextElement.couple=c;
	for (int q=0;q<20;q++)
	{
       p.intent.push_back(1);
       p.extent.push_back(0);   
	ContextElement.pseudo=p;
	cc=resultat;// resultat est un vecteur de int déjà rempli
	ContextElement.concept=cc;
	ContextElement.num_concept=1;
	ContextGlobal[n]->couple=ContextElement.couple;
	ContextGlobal[n]->pseudo=ContextElement.pseudo;
	ContextGlobal[n]->concept=ContextElement.concept;
	ContextGlobal[n]->num_concept=ContextElement.num_concept;
 
 
 
 
}
 
 
}

[Sunsawe]

Je me trompe peut être mais cette partie de code me semble étrange.

Code :

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

1
2
3
4
5
ContextGlobal[n]->couple.intent.clear();
 ContextGlobal[n]->couple.extent.clear();
 ContextGlobal[n]->pseudo.intent.clear();
 ContextGlobal[n]->pseudo.extent.clear();
 ContextGlobal[n]->concept.clear();

Tu appelles un attribut de la structure "pointée" par ContextGlobal[n], mais es tu sur d'avoir initialisé ContextGlobal[n] avant?

Le compilateur ne dirait rien puisque ton pointeur est sensé pointer un objet de type correspondant. Mais si à l'execution l'objet n'existe pas...

[FstDsi]

Je ne sais pas a quel niveau devrais-je l'initialiser.En dehors de la boucle???
En plus je dois l'initialiser à quelle valeur??Je suis débutante en C++ et toutes ces structures me bloquent.

[Sunsawe]

En fait... ce n'est pas si compliqué.
Avant de chercher où initialiser, sois sure d'avoir saisi la logique.

Avant de pouvoir accéder aux attributs d'une structure, il te faut être sur, qu'elle existe. Donc avant d'appeler:

Code :

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

ContextGlobal[n]->couple.intent.clear();

Tu dois être sure que le pointeur stocké à la position n de ton vecteur, pointe sur une structure existante.
Donc où initialiser, dépend de la logique de ton programme et d'où tu appelles la fonction qui exécute la boucle.

Dans les faits, oui, l'initialisation doit se faire avant l'accès à la structure.

Mais par sécurité, tu pourrais ajouter ceci:

Code :

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

1
2
3
4
5
6
7
8
if(ContextGlobal[n] != NULL)
{
ContextGlobal[n]->couple.intent.clear();
 ContextGlobal[n]->couple.extent.clear();
 ContextGlobal[n]->pseudo.intent.clear();
 ContextGlobal[n]->pseudo.extent.clear();
 ContextGlobal[n]->concept.clear();
}

Ce test va simplement s'assurer que le pointeur n'est pas null avant que le programme n'accède à la structure.


Autre chose, que cherches tu à faire avec ce code:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
for (int q=0;q<20;q++)
	{
       p.intent.push_back(1);
       p.extent.push_back(0);   
	ContextElement.pseudo=p;
	cc=resultat;// resultat est un vecteur de int déjà rempli
	ContextElement.concept=cc;
	ContextElement.num_concept=1;
	ContextGlobal[n]->couple=ContextElement.couple;
	ContextGlobal[n]->pseudo=ContextElement.pseudo;
	ContextGlobal[n]->concept=ContextElement.concept;
	ContextGlobal[n]->num_concept=ContextElement.num_concept;
}

Parce que là, tu remplaces successivement les attributs de ContextGlobal[n] par ceux du dernier ContextElement. Tu pourrais tout aussi bien sortir les 4 dernières lignes de la boucle et faire le remplacement une seule fois avec le dernier ContextElement, le résultat serait le même (et tu économiserais des opérations inutiles).

[FstDsi]

J'ai essayé la solution que tu m'as proposé mais ça se plante tjs.

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
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
 
ContextVect ContextGlobal;
InfoContext ContextElement;
PseudoConcept p,c;
vector<int> cc;
 
ContextGlobal.reserve(couverture);
 
for(int n=0;n<10;n++)
{
 
 c.intent.clear();
 c.extent.clear();
 p.intent.clear();
 p.extent.clear();
 ContextElement.couple.intent.clear();
 ContextElement.couple.extent.clear();
 ContextElement.pseudo.intent.clear();
 ContextElement.pseudo.extent.clear();
 ContextElement.concept.clear();
 
  if(ContextGlobal[n] != NULL)
{
 ContextGlobal[n]->couple.intent.clear();
 ContextGlobal[n]->couple.extent.clear();
 ContextGlobal[n]->pseudo.intent.clear();
 ContextGlobal[n]->pseudo.extent.clear();
 ContextGlobal[n]->concept.clear();
}
 
 
 c.intent.resize(1);
 c.extent.resize(1);
 c.intent.push_back(exp_courant);//exp_courant déja rempli 
 c.extent.push_back(attr_courant); //attr_courant déja rempli 
 ContextElement.couple=c;
	for (int q=0;q<20;q++)
	{
       p.intent.push_back(1);
       p.extent.push_back(0);   
	ContextElement.pseudo=p;
	cc=resultat;// resultat est un vecteur de int déjà rempli
	ContextElement.concept=cc;
	ContextElement.num_concept=1;
 
}
 
//Cette partie est en dehors de la boucle je me suis trompée tt a leur
 
// Le problème vient exactement de ces lignes.Quand je les met en commentaire le programme ne se plante plus.
 
        ContextGlobal[n]->couple=ContextElement.couple;
	ContextGlobal[n]->pseudo=ContextElement.pseudo;
	ContextGlobal[n]->concept=ContextElement.concept;
	ContextGlobal[n]->num_concept=ContextElement.num_concept;
 
}

[Sunsawe]

Ma première explication serait exactement la même que la dernière avec exactement la même solution.

Il se peut que ce soit là, que tu cherches à accéder à un objet non existant. Essai de mettre le même test pour ces lignes.

Est ce que tu en comprends l'utilité d'ailleurs? (il ne faut pas que ça te paraisse magic, mais plutot logique)