Discussion :

[CodeCRC]

Bonjour à tous,

Je dois retourner l'adresse d'un CString et le stocker dans un pointeur. Mon code est basé sur un code précédent qui à la place d'un CString utilisait un wstring. Mon code est le suivant :

Code :

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

m_pValue =  &(p_pBean->getText());

getText() me renvoyait donc à l'origine un wstring et dans mon code me renvoit un CString. Seulement voilà, je ne peux pas écrire à l'adresse stockée dans le pointeur (void *). Dès que j'écris quelque chose à cette adresse, j'ai le droit à un beau plantage en règle .
Une de mes pistes est que Wstring est une structure et donc que le & me renvoit bien l'adresse de la structure alors qu'un CString est une chaîne de caractère donc déjà un pointeur et qu'avec le code utilisé, je retourne un pointeur de pointeur...
Seulement si j'utilise ça :

Code :

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

m_pValue = p_pBean->getText();

La compilation me retourne une erreur de cast "cannot convert CString to void*".
Commençant à m'embrouiller dans tout ça (et étant débutant), je demande un peu d'aide

Merci d'avance!


Edit : Je précise (même si je ne pense pas que ça ait un interêt que lorsque je passe mon attribut text en public et que je fait un :

Code :

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

m_pValue =  &(p_pBean->text);

Ca fonctionne parfaitement.

[Matthieu Brucher]

getText() te retourne un CString. C'est une classe où les données sont stockées quelque part dans la nature. De plus, il est vraisemblable qu'il s'agisse d'un passage par valeur, donc getText retourne un objet, tu en prends l'adresse, mais l'objet est détruit à la sortie de ta fonction.
Tu devrais donc stocker un CString (comme tu aurais dû stocker un wstring aussi) et non un pointeur vers ces objets.

[CodeCRC]

Merci de ta réponse.
Cependant, je ne suis pas sûr d'avoir tout compris

GetText() me retourne bien un objet de type CString, ça je suis d'accord. J'en prend bien l'addresse puisque c'est à cette adresse que je veux ensuite écrire mais je ne comprends pas très bien quand tu dis que l'objet est détruit à la sortie de la fonction. Ce qui m'intéresse c'est juste l'adresse de l'objet...

[Matthieu Brucher]

Alors tu ne travailles pas correctement. getText() te renvoie un objet (on ne sait pas l'objet sur lequel tu appelles getText, mais vu le nom de la méthdoe, je ne me trompe pas) soit constant, soit une copie.
- si c'est un objet constant, tu ne peux pas changer le texte à l'intérieur
- si c'est uen copie, changer sa valeur ne changera pas le texte dans ton premier objet.

getText() n'est pas fait pour être une méthode qui retourne un objet modifiable dans tous les cas. Si tu veux modifier le texte, fait un getText puis un setText. Travaille proprement.

Enfin, si tu ne comprends pas pourquoi tu devrais stocker le retour de getText dans un objet plutôt que dans un pointeur, regarde les premiers cours de C++ que tu as eu sur les tableaux internes à une fonction et ce qui se passe quand on les retourne.

[3DArchi]

On fait le pari que tu as quelque chose comme ça:

Code :

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

1
2
3
4
5
 
 
//p_pBean->getText()
CString CMonBean::getText()
{...}

Donc, quant tu écris:

Code :

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

m_pValue =  &(p_pBean->getText());

tu récupères l'adresse d'une variable temporaire CString utilisée pour le retour de ta fonction. Cette variable temporaire est ensuite détruite et ton adresse est invalide!

La solution pourrait être:

Code :

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

CString* CMonBean::getText()

Mais, excuses-moi, au vue de ton niveau, je pense que cela est très dangereux. Peut-être devrais tu revoir ta conception: as-tu vraiment besoin de l'adresse de CString (est-ce pour modifier la valeur)? Comment articules-tu la durée de vie de p_pBean et celle de m_pValue?