Discussion :

[titoine1978]

Bonjour,

A votre avis ce code est-t-il dangereux ?

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
void main()
{
   string  MyString;
   function(MyString);
}
 
function(string& MyString)
{
   stringstream ss;
   MyString = ss.str();
}

J'avais un bug qu'en release, une CMemoryException pas catchée. Le fait d'avoir écrit ce qui suit a résolu le problème.

Code :

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

1
2
3
4
5
6
 
function(string& MyString)
{
   stringstream ss;
   MyString = ss.str().c_str();
}

Merci.

[3DArchi]

Voici l'exemple donné dans http://www.cplusplus.com/reference/i...tream/str.html

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
 
// stringstream::str
#include <iostream>
#include <sstream>
#include <string>
using namespace std;
 
int main () {
 
  stringstream oss;
  string mystr;
 
  oss << "Sample string";
  mystr=oss.str();
 
  cout << mystr;
 
  return 0;
}

Et celui par http://www.cppreference.com/cppsstream/str.html

Code :

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

1
2
3
4
 
 ostringstream stream1;
  stream1 << "Testing!" << endl;
  cout << stream1.str();

Je ne vois pas pourquoi le passage par référence poserais problème. Normalement, l'implémentation sait gérer ce genre de chose.

[titoine1978]

Je vais essayer d'investiguer plus sur mon bug...comme toi je pense que le passage pas référence dans ce cas précis ne pose pas de problème. Je voulais en être certain.

[r0d]

Le premier code que tu as posté est ce qui peut se faire de mieux. Il est sûr et efficace (pas de copie ni de conversions inutiles). Le problème doit venir d'ailleurs.

[Nestor Fish]

D'après ce que je comprends :
Dans function, tu déclares un stringstream , et tu tentes de récupérer la string associée, mais elle n'existe pas. Tu récupères donc n'importe quoi dans MyString. Tu évite un crash en forçant un caractère de fin à MyString, ce que tu fais avec .c_str().
Tu devrais regarder du côté du constructeur. Qu'est ce que tu veux faire ?

[yan]

D'après ce que je comprends :
Dans function, tu déclares un stringstream , et tu tentes de récupérer la string associée, mais elle n'existe pas.

pourquoi?

[3DArchi]

Vraisemblablement, son problème était ailleurs. Et comme l'a dit r0d, son premier code était encore le meilleur. Qu'il déclare pour illustrer son problème une stringstream et n'y ajoute pas de données ne fera qu'avoir une chaîne de caractère vide et ne génère aucun problème.

[Nestor Fish]

Mea culpa, j'ai foncé tête baissée.

Qu'il déclare pour illustrer son problème une stringstream et n'y ajoute pas de données ne fera qu'avoir une chaîne de caractère vide et ne génère aucun problème.

Tout y est.

[Obsidian]

Bonjour,

A votre avis ce code est-t-il dangereux ?

Ça ressemble beaucoup à un objet non initialisé. Donc soit ton code est légèrement plus complexe qu'il n'y paraît (threads ?) et ton MyString disparaît dans l'intervalle où tu appelles function, soit l'implémentation de ta bibliothèque standard est buguée.

Peux-tu nous donner l'état de la pile au moment du plantage ?

Il faut bien voir que str() et c_str() sont deux choses différentes : le premier renvoie une copie de l'objet string géré par le streambuf associé à l'objet string. Le deuxième renvoie un pointeur vers une chaîne C proprement terminée par un 0.

On peut très bien imaginer, par exemple, que le fait de devoir renvoyer un pointeur oblige à initialiser la chaîne, qu'elle soit vide ou non. On peut aussi envisager le fait que le string buffer gère son contenu en stockant sa taille dans un entier non signé plutôt qu'en plaçant une sentinelle en fin de chaîne. Ainsi, si la taille est marquée comme étant nulle, il suffit de renvoyer un pointeur vers le premier '\0' en mémoire plutôt que d'initialiser un tampon. Il est donc possible que ton objet-chaîne ait été corrompu par un débordement de tampon, par exemple, et que les effets restent dormants tant que ta chaîne est vide.

Essaie de revenir à un str() normal et de mettre quelque chose dedans, ou bien poste plus de code.