Pointeur non modifié à l'interieur d'une fonction ? o_O

Bonjour,
Je me resous à poster parce que mon code ne se comporte vraiment pas comment je pense qu'il devrait se comporter. J'ai du louper quelque chose d'énorme mais je ne vois pas...

Voila une fonction :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14

bool isActionKnown(const char* actionName, Action *a) { map<string,Action*>::iterator result; result = knownActions.find( actionName ); if (result == knownActions.end() ) return false; else { a = knownActions[actionName]; cout<<"Value inside the function : "<<a->name<<endl; return true; } }

---

Et voilà un bout de code qui l'appelle :

Code:

---

1 2 3 4 5

Action* A = new Action("test"); cout<<"Value before the function : "<<A->name<<endl; k.isActionKnown("empty",A); cout<<"Value after the function : "<<A->name<<endl;

---

Et la trace du programme est :

Code:

---

1 2 3 4

Value before the function : test Value inside the function : empty Value after the function : test

---

Alors ma question est : pourquoi je n'ai pas "value after the function : empty"...
Parce que l'objet qui est passé est une copie (donc une copie du pointeur ?)
Le truc c'est qu'on ne peut pas passer un pointeur par reference...
:help:

Oui, c'est ce que j'ai essayé de faire, mais le compilo me jette :

"pointeur non conforme vers la reference"

(enfin, j'ai résolu mon problème autrement, mais ça m'intrigue quand même... je vois pas comment faire a part en utilisant un pointeur vers ce pointeur...)

Dans ta fonction, tu obtiens un nouveau pointeur sur un Action :

Code:

---

a = knownActions[actionName];

---

Or, l'adresse mémoire de a a changé et non pas seulement son contenu.

Soit tu passes le pointeur par référence par qu'il mette à jour l'adresse du pointeur :

Code:

---

bool isActionKnown(const char* actionName, Action &*a)

---

Soit tu modifies vers quoi pointe a et ne touche pas à son adresse mémoire.

Code:

---

*a = ce_qu_il_faut

---

Au passage, utilise std::string en paramètre au lieu de const char*

A moins que Action soit polymorphique, je te conseillerait de stocker par valeur et non pas par pointeur. Ca fera un code plus propre et pas besoin d'allocation dynamique (du moins sur la partie que tu montres)

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

bool isActionKnown(string actionName, Action &a) { map<string,Action>::iterator result; result = knownActions.find( actionName ); if (result == knownActions.end() ) return false; else { a = knownActions[actionName]; cout<<"Value inside the function : "<<a.name<<endl; return true; } }

---

Code:

---

1 2 3 4

Action A("test"); cout<<"Value before the function : "<<A.name<<endl; k.isActionKnown("empty",A); cout<<"Value after the function : "<<A.name<<endl;

---

En fait le code que je montre est juste un bout réarrangé. Mes objets Action sont uniques, mais référencés dans plusieurs autres structures (y compris dans Action), donc je n'ai pas trop le choix...

Par contre :

Code:

---

bool isActionKnown(const char* actionName, Action &*a)

---

j'avais essayé, mais le compilateur me jette. Il n'aime pas la référence sur un pointeur...


EDIT :
Autant pour moi, j'ai inversé la position des deux opérateurs. Ca ca marche :

Code:

---

bool isActionKnown(const char* actionName, Action *&a)

---

Merci !

Citation:

---

Envoyé par stephane.lallee

Le truc c'est qu'on ne peut pas passer un pointeur par reference...

---

ah ? depuis quand ?

Code:

---

1 2 3 4 5 6 7 8 9 10 11

void myFunc(Action*& ptr) { ptr = new Action; }   int main(int, char**) { Action* ptr = NULL; myFunc(ptr); // ptr != NULL }

---

en fait ton problème est là :

Citation:

---

Envoyé par stephane.lallee

Code:

---

bool isActionKnown(const char* actionName, Action &*a)

---

j'avais essayé, mais le compilateur me jette. Il n'aime pas la référence sur un pointeur...

---

ce qu'il n'aime pas c'est ton pointeur sur une référence :aie:

Modifie le contenu du pointeur et non pas le pointeur.

Voici un exemple simple illustrant ton problème:

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 25 26

#include <iostream>   int gl_val = 2;   void f(int * val) { val = &gl_val; }   void g(int * val) { *val = gl_val; }   int main(int argc, char *argv[]) { int val = 1;   f(&val); std::cout << "f: " << val << std::endl;   g(&val); std::cout << "g: " << val << std::endl;   return 0; }

---

Citation:

---

Envoyé par Trunks

Modifie le contenu du pointeur et non pas le pointeur.

---

Oui oui, j'ai bien compris ce que tu essaie de me dire, mais vraiment moi ce que je veux c'est modifier le pointeur, afin d'avoir une seule instance de mon objet et de récupérer son adresse. Je ne veux pas modifier cet objet :)

Mais sinon ca marche bien avec le passage par référence du pointeur, et non le pointeur sur une référence... Merci à vous deux :king:

*n'avait jamais pensé que les pointeurs avaient eux aussi des adresses :roll:*

Citation:

---

Envoyé par shenron666

ce qu'il n'aime pas c'est ton pointeur sur une référence :aie:

---

J'avais inversé les 2 symboles ^^ (j'avais fait ça de tête sans tester :roll:)


stephane.lallee, tu as le bouton Résolu ...

Citation:

---

Envoyé par stephane.lallee

*n'avait jamais pensé que les pointeurs avaient eux aussi des adresses :roll:*

---

en toute logique, le pointeur est bien stocké quelque part ;)