Discussion :

[julien25]

Bonjour!

J'ai une erreur et je ne sais pas d'où elle provient:

J'ai une classe fille qui me retoure une map<string, float> de nom Valeurdsmap.

Jusqu'ici, pas de problèmes.

Dans la classe mère (Classmere), j'ai la méthode ci dessous :
Cette methode est un get et je recopie ma map que je place en reference
Jusque la toujours pas de pb.

Code :

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

1
2
3
4
5
6
 
bool Error::get(map<string, float>*& retValeurdsmap)
{
	retValeurdsmap = this->Valeurdsmap;
	return true;
}

Dans une autre classe j'ai une méthode qui me permet de récupérer la map de ma classe mère. Visiblement tout fonctionne mais qd je veux imprimer la taille de ma map pour voir si elle est bien remplie, le programme crash.

Donc la ligne cout<<test->size(); pose problème;
Je pense que je dois dereferencer pour renvoyer ce qu'il y a derriere le pointeur mais cela ne fonctionne pas non plus.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
 
bool Local::Allocation(Classmere* myObject)
{  
	map<string, float>* test;
	myObject->get(test);// pqs de probleme, 
VS me donne  automatiquement la méthode.
	test->size();
	cout<<"test2"<<endl;
	cout<<test->size();
 
return true;
}

Merci de vos r2ponses

[3DArchi]

Salut,
Tout simplement :

Code :

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

1
2
//T*p
std::cout<<p<<std::endl;

[white_tentacle]

Attention, tu ne recopies pas la map, tu fais juste pointer test dessus (dans get).

Le code que tu montres me semble correct, donc je pense que ton pointeur est déjà corrompu dans ton objet Error.

Quand tu appelles size(), ça plante. Si ça ne plante pas au premier, c'est je pense parce que le compilo vire la ligne inutile (compilation en release).

[julien25]

Salut 3D Archi
je ne sais pas si tu as lu mon post, ce n'est pas si simple.
Je pense donc que le post était au bon endroit.
I y a un pb de référencement ou de passage de la fonction.

[julien25]

Salut white_tentacle,
oui le problème est effectivement la, je pense que je recopie mal en fait dans mon get.
Comment je peux faire? As tu une idee?
Je compile effectivement en mode release, bien vu
Je pense en fait qu'il faut que je fasse un classe abstraite.
Merci

[3DArchi]

Vu l'intitulé, je pensais que tu voulais simplement sortir l'adresse.
Vérifie que myObject est valide (!=NULL?).

[julien25]

Oui, l'intitule du titre est mal trouve. Mes excuses.
Je n'arrive tjs pas a recopier cette map même en faisant une classe abstraite. Si tu pouvais d'ailleurs remettre le post au bon endroit SVP.

[3DArchi]

Tu veux recopier la map ou récupérer le pointeur ?

[julien25]

je veux recopier ma map, c'est ce que je fais ici.
Comment as tu il acces a ma classe fille pour Valeurdsmap. Je pense qu'il faut que je lui transmette non?

Code :

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

1
2
3
4
5
bool Error::get(map<string, float>*& retValeurdsmap)
{
	retValeurdsmap = this->Valeurdsmap;
	return true;
}

[3DArchi]

Comment as tu il acces a ma classe fille pour Valeurdsmap.

je veux recopier ma map, c'est ce que je fais ici.

Si tu veux recopier la map :

Code :

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

1
2
3
4
5
6
 
bool Error::get(map<string, float>& retValeurdsmap)
{
	retValeurdsmap = this->Valeurdsmap;
	return true;
}

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
bool Local::Allocation(Classmere* myObject)
{  
	map<string, float> test;
	myObject->get(test);
	test.size();
	cout<<"test2"<<endl;
	cout<<test.size();
 
return true;
}

Pas besoin de passer par un pointeur.
Mais ça ne va que décaler ton plantage car apparemment, tu récupérais bien l'adresse avant.

[julien25]

Merci de vos réponses.
En fait l'erreur provenait d'ailleurs. Vous ne pouviez pas la voir car je ne vous avais pas donner cette partie du code.

en fait ma map était déclarer 2 fois. Une fois dans ma classe, ce c'est bien.
Mais aussi une fois a l'intérieur même de la méthode qui me cree ma map.
Ensuite, j'avais un problème de case mémoire. Je commence a toucher au point sensible de C++. En effet, j'avais mal declarer ma map. J'avais mis :
this->ValeurdsMap= ValeurdsMap

au lieu de


this->CurrentSensorMeanOfFeeder = new map<string,float>();

++

[julien25]

Pourrirez vous me donner votre point de vue entre un pointeur et une référence.
Avec un exemple parlant si possible.

Pour moi un pointeur est une adresse de case mémoire. Et une reference est un pointeur fixe mais ca ne me parle pas beaucoup.

[3DArchi]

Pourrirez vous me donner votre point de vue entre un pointeur et une référence.
Avec un exemple parlant si possible.

Pour moi un pointeur est une adresse de case mémoire. Et une reference est un pointeur fixe mais ca ne me parle pas beaucoup.

FAQ : Les références

En gros : si tu veux pouvoir changer la valeur du paramètre, utilises une référence. N'utilises un pointeur que si tu as une bonne raison de vouloir aller à une adresse... et l'usage te montrera que tu as rarement de bonne raison de le faire.

[koala01]

Salut,

Une référence est un alias de la variable d'origine.

Ce qui fait que toute modification apportée à une référence (non constante) sera répercutée sur la variable d'origine (éventuellement dans la fonction appelante).

Un pointeur est une adresse mémoire, à laquelle nous espérons le plus souvent trouver quelque chose du type indiqué.

Les pointeurs présentent de nombreuses difficultés, dont la plupart sont issues des techniques d'allocation dynamique de la mémoire.

C'est la raison pour laquelle on conseille, comme l'a si bien dit 3DArchi, de restreindre l'usage des pointeurs aux (rares) cas où l'on n'a pas d'autres solutions:

Nous pourrions presque dire "Utilisez les références quand vous pouvez, utilisez les pointeurs quand vous devez"