Problème pour convertir un entier en String

Version imprimable

15/04/2010, 15h36
K-you

Problème pour convertir un entier en String
Bonjour,

J'utilise std::ostringstream de la bibliothèque sstream pour faire passer mes entiers sous la forme de chaîne de caractères.

Dans un programme tout simple, pas de souci, l'exemple suivant fonctionne :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 #include <iostream> #include <sstream> int main() { std::ostringstream out; int solution = 8; out << solution; std::cout << out.str() << std::endl; return 0; }
Mais quand j'intègre ça dans mon application multi-threadée, ça ne marche plus. J'utilise les Thread boost, j'appelle donc une méthode static de cette façon :

Déclaration dans MaClasse.h
Code:

1 2 static void thread_MonThread(MaClasse * MaClasse);
Appel dans MaClasse.c
Code:

1 2 boost::thread thread_MonThread(boost::bind(&MaClasse::thread_MonThread,this));
dans ma MaClasse.
Voici l'allure de mon Thread (pour l'instant très simple) :
Code:

1 2 3 4 5 6 7 8 void MaClasse::thread_MonThread(MaClasse * pMaClasse) { std::ostringstream out; int val = 8; out << val; std::string str = out.str(); }
et à la ligne

Code:

out << val;

j'ai un seg fault. Je ne comprend pas pourquoi, je dois mal gérer quelque chose. J'ai essayé de faire passer ma variable "out" en attribut de MaClasse et d'appeler une fonction privée qui retourne la String résultat. Mais même résultat : seg fault. Si qqun sait d'ou cette erreur peut venir, je suis preneur...
15/04/2010, 16h07
camboui

Il n'y a pas de '\0' à la fin des stringstream. Tu dois l'ajouter avant l'appel de la méthode str().
15/04/2010, 16h13
K-you

Donc en gros la ligne bug parce qu'il essaie d'affecter à mon String une chaîne qui se termine que très longtemps après ce qu'elle devrait (quand par chance il croise le caractère \0 dans la mémoire) ?

Alors pourquoi dans l'exemple simple il n'y a pas de problème ?
15/04/2010, 16h22
K-you

J'ai testé et c'est vrai que ça fonctionne en ajoutant le caractère, mais si tu peux m'expliquer, ça serait parfait.

En tout cas merci à toi.
15/04/2010, 16h29
nikko34

euh pour moi c'est sensé marcher sans "\0" à la fin, vu que ça renvoie une copie d'un std::string, .str().

Non? ou je me trompe?
15/04/2010, 16h41
K-you

Oui c'est vrai que ça marche sans aussi. Le bug venait d'ailleur (une libération de mémoire appelée d'ailleur, et qui prenait effet pendant le traitement du thread de ma classe, forcément...)

Faut pas confondre les (i/o)stringstream avec les std::string.

Les premiers sont de la merdouillabaise, sans vouloir relancer la polémique sur les stream (que je n'utilise plus que pour faire du logging).

Les seconds sont bien plus sympas.
Et j'argumente avec ce petit code:

Code:

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
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
template<typename _I>
struct make_unsigned
{
};
 
 
template<>
struct make_unsigned<int>
{
	typedef unsigned int type;
};
 
 
template<>
struct make_unsigned<long>
{
	typedef unsigned long type;
};
 
 
template<>
struct make_unsigned<long long>
{
	typedef unsigned long long type;
};
 
 
template<class _C, class _I>
inline std::basic_string<_C, std::char_traits<_C>, std::allocator<_C> >
	integral_to_basic_string(_I value)
{
	int const _MAX_DIGITS=( sizeof(_I)==4 ? 12 : (sizeof(_I)==8 ? 24 : -1) );
	_C buf[_MAX_DIGITS];
	_C *bufp=buf;
	int neg=(value<0?1:0);
	if (neg)
		value=-value;
	do
	{
		//do we really need to cast to unsigned ?
		*bufp++=(_C)((make_unsigned<_I>::type)value % 10);
		(*(make_unsigned<_I>::type *)&value)/=10;
	}
	while (value>0);
	std::basic_string<_C, std::char_traits<_C>, std::allocator<_C> > s;
	s.resize(bufp-buf+neg);
	std::basic_string<_C, std::char_traits<_C>, std::allocator<_C> >::iterator
		it_s=s.begin();
	if (neg)
		*it_s++='-';
	while (bufp!=buf)
	{
		_C c=*--bufp;
		*it_s++=(_C)(c+'0');
	}
	return s;
}

Exemples d'utilisation:

Code:

1
2
3
4
inline std::string i2stringa(int value)
	{ return integral_to_basic_string<char,int>(value); }
inline std::wstring i2stringw(long long value)
	{ return integral_to_basic_string<wchar_t,long long>(value); }

:mouarf:

15/04/2010, 16h50
camboui

Vous me faites douter maintenant, je ne sais plus s'il y a un '\0' ou pas à la fin des stringstream lors de l'appel à .str().
Où sont les experts des streams ?
15/04/2010, 17h01
nikko34

j'ai jamais eu à rajouter quoique ce soit pour que ça marche... (et c'est utile)

http://cpp.developpez.com/faq/cpp/?p...RINGS_numtostr
15/04/2010, 17h03
Sylvain Togni

Citation:

Envoyé par camboui

Vous me faites douter maintenant, je ne sais plus s'il y a un '\0' ou pas à la fin des stringstream lors de l'appel à .str().
Où sont les experts des streams ?

Non il n'y a pas de '\0' a la fin d'un stringstream, pas plus qu'à la fin d'une std::string.

Mais en C++ il n'y a pas besoin de '\0' à la fin des chaînes, c'est en C qu'il y en a besoin.
15/04/2010, 17h05
camboui

Oubliez tout ce que j'ai dit, ce sont des anneries (moi et ma connaissance des streams...)

Mais je ferais ceci

Code:

std::cout << out.str().c_str() << std::endl;

au lieu de ceci

Code:

std::cout << out.str() << std::endl;
15/04/2010, 17h07
camboui

Citation:

Envoyé par Sylvain Togni

Non il n'y a pas de '\0' a la fin d'un stringstream, pas plus qu'à la fin d'une std::string.

Mais en C++ il n'y a pas besoin de '\0' à la fin des chaînes, c'est en C qu'il y en a besoin.

Quand on appelle la méthode c_str() de string le '\0' est garanti à la fin.
15/04/2010, 17h08
nikko34

ouais mais t'en a pas besoin pour std::cout non plus, il prend très bien les std::string
15/04/2010, 17h19
camboui

J'ai toujours un pied dans MSVC6 qui ne les prend pas bien... :roll: