Discussion :

[mulfycrowh]

Bonjour,

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

Code :

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

1
2
3
private:
 
vector<string> Msg_Language;

J'ai un accesseur sur ce membre :

Code :

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

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 :

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

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 :

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

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 ...

[ram-0000]

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 :

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

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

[mulfycrowh]

Merci beaucoup

[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.

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 ?

[ram-0000]

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 !!

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 :

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

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);
}

[3DArchi]

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