Retour d'un LPCSTR

Bonjour,

J'ai créé une classe Language qui contient un membre :

Code:

---

1 2 3

private:   vector<string> Msg_Language;

---

J'ai un accesseur sur ce membre :

Code:

---

1 2 3 4

string CLanguage::Get_MsgLanguage(int idx) const { return Msg_Language[idx]; }

---

D'autre part, j'ai créé une fonction de traduction du message :

Code:

---

1 2 3 4 5 6 7

template<typename T> LPCSTR get_translation(int ID_Msg,T tParameter) { static CLanguage& My_Language = CLanguage::getInstance(""); // Getting the single language instance string s = My_Language.Get_MsgLanguage(ID_Msg); return s.c_str(); }

---

Et j'utilise cette fonction dans :

Code:

---

SetDlgItemText(hwndDlg,IDC_ENTER_NEW_RESOLUTION,get_translation(ID_MSG_ENTER_NEW_RESOLUTION,""));

---

Le problème est le suivant :

La variable s est tout à fait cohérente au debugging.
Par contre, la valeur retournée par get_translation elle ne l'est pas. Ma boîte de dialogue affiche du charabia ...
Même chose sur une variable intermédiaire de test : LPCSTR ss = get_translation

Si vous avez des idées ...

Merci à tous ...

Je pense que tu as un problème de mariage UNICODE/non unicode.

ton vector est un vector de string (donc non unicode).

Je parie que ton projet est quant à lui compilé en unicode.

Essaye d'utiliser un vector de wstring pour que tout le monde ait la même taille de caractères.

Autre chose, ton accesseur, il retourne une string, il pourrait retourner une référence de string (car la string est déjà allouée par ton vector)

Code:

---

1 2 3 4

const wstring & CLanguage::Get_MsgLanguage(int idx) const { return Msg_Language[idx]; }

---

Merci beaucoup

Dans ta fonction de traduction, s est une variable temporaire. Tu retourne un pointeur interne à cette variable (le c_str()) puis cette variable temporaire est détruite : Ton pointeur devient alors invalide.

En général, on utilise c_str à la dernière seconde, au moment où l'on a besoin de la chaîne. Pourquoi ne pas faire retourner à get_translation une string ?

Citation:

---

Envoyé par JolyLoic

Dans ta fonction de traduction, s est une variable temporaire. Tu retourne un pointeur interne à cette variable (le c_str()) puis cette variable temporaire est détruite : Ton pointeur devient alors invalide.

---

Ha tien, pas vu !!

Citation:

---

Envoyé par JolyLoic

Pourquoi ne pas faire retourner à get_translation une string ?

---

Ou même une référence constante de string puisque la string est déjà allouée dans le vecteur

Code:

---

1 2 3 4 5 6

template<typename T> const wstring & get_translation(int ID_Msg,T tParameter) { static CLanguage& My_Language = CLanguage::getInstance(""); // Getting the single language instance return My_Language.Get_MsgLanguage(ID_Msg); }

---

Salut,
C'est parce que tu as simplifié ton exemple sinon quel est l'intérêt du paramètre générique T ?

Bonjour et merci pour toutes ces réponses,

get_translation appartient à Toolbox.h qui est comme son nom l'indique une boîte à outils et je ne sais pas encore créer une bibliothèque de fonctions qui n'inclut pas de template. C'est bête ...:D
Car si je mets cete fonction dans un .cpp je ne peux pas faire #include ...cpp