Discussion :

[CedricMocquillon]

Bonjour à tous,

Est-ce que vous pouvez me dire si le code suivant est valide par rapport à la durée de vie de la string renvoyée?

Code :

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

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
 
struct Foo
{
  Foo(char const* s) {
    long len = lstrlen(s);
    this->bstrVal	= ::SysAllocStringLen(NULL, len);
    if(len)
    {
      long rc = MultiByteToWideChar(CP_ACP, 0, s, len, bstrVal, len);
      if(rc!=-1)
        this->bstrVal[rc] = 0;
    }
  }
private:
  BSTR bstrVal;
};
 
std::string bar() {
  std::string result;
  result = "Action";
  return result;
}
 
int main() {
  Foo f(bar.c_str());
}

J'ai un doute sur le faite que la chaine renvoyée par c_str() soit encore valide lorsque je la manipule dans le constructeur de Foo.

[Bousk]

Ce code est tout à fait correct pour un compilateur normal (et pas périmé).
La string disparait à la fin de la ligne, quand la parenthèse fermante est exécutée. A l'intérieur l'objet temporaire existe.

[camboui]

Tout pareil, pas de soucis je pense, ça devrait fonctionner même avec ce bon vieux Visual Studio 6.0 qui a 20 ans d'âge

Et entre parenthèses, puisque j'en vois dans ton code, si tu dois utiliser des BSTR n'hésite pas à passer par l'encapsulation _bstr_t (#include <comdef.h>) qui va te faire tout à la fois du RAII et les conversions ANSI<->UTF16 à la volée.

[bacelar]

Manque quand même un "SysFreeString" quelque part pour éviter les fuites mémoire.
https://msdn.microsoft.com/fr-fr/library/ms891288.aspx

[CedricMocquillon]

Merci à tous pour cette confirmation!
Merci @camboui pour les _bstr_t c'est une bonne idée
@bacelar je n'avais mis qu'une sous partie la Sysfreestring est bien présent dans le code d'origine (il n'est pas de moi, on est sur la piste d'un bug...) mais je vais sans doute refactorer pour utiliser les _bstr_t proposées par camboui

[Médinoc]

Attention, dans ton code, pour bien faire la taille initiale devrait elle aussi être calculée avec MultiByteToWideChar() (en passant un buffer de sortie nul et de taille zéro) plutôt qu'avec lstrlen().
Et tu peux envisager de prendre la codepage en paramètre avec CP_ACP comme valeur par défaut.