Discussion :

[screetch]

bonjou, j'ai un petit problème avec mon essai de classe de formattage de chaine; j'essaye de faire la chose suivante

Code :

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

1
2
3
4
5
6
int main(void)
{
	const char *f = format<>("int value : %d\n") & 12;
	printf("%s\n", f);
	return 0;
}

le probleme c'est que format<> qui contient ici la chaîne est un temporaire qui sera détruit après l'opération, et donc f pointe potentiellement sur du caca. Je me demande si vous avez des idées brillantes pour corriger ce problème; le but serait de réussir a prolonger la vie de la chaîne pour qu'elle aie le scope de la fonction...

la classe format :

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
27
28
template< size_t SIZE = 512 >
class format
{
	typedef char ARRAY[SIZE];
public:
	ARRAY m_format;
	char *m_pattern;
public:
	void findFormat()
	{
		while(m_pattern && *m_pattern && (m_pattern[1] == '%' || m_pattern[0] != '%'))
			m_pattern++;
		if(m_pattern && !*m_pattern)
			m_pattern = 0;
	}
public:
	format(const char *f)
	{
		strncpy(m_format, f, SIZE);
		m_pattern = m_format;
		findFormat();
	}
	~format()
	{
		//*m_format = 0;
	}
	operator const char*() { return m_format; }
};

l'opérateur & a peu d'importance

des idées brillantes ?

[loufoque]

Code :

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

1
2
3
4
5
6
int main(void)
{
	format<> f("int value : %d\n") & 12;
	printf("%s\n", f.get());
	return 0;
}

[screetch]

ca ne compile pas, il faudrait faire

Code :

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

format<> f = format<>("int value : %d\n") & 12;

ce qui est un peu redondant. je cherche justement a eviter le fortmat<> en double

[loufoque]

Code :

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

1
2
3
4
5
6
int main(void)
{
	auto f = format<>("int value : %d\n") & 12;
	printf("%s\n", f.get());
	return 0;
}

?

[screetch]

Code :

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

from futur import auto;

le problème ici c'est "futur".

[loufoque]

AUTO(f, format<>("int value : %d\n") & 12) sinon si tu veux un truc qui marche sur compilateurs plus mainstreams (avec AUTO une macro)

[Luc Hermitte]

C'est en autre pour éviter ce genre de déconvenues que l'on préfère les std::string : on n'a pas à se préoccuper de la mémoire.
Accessoirement, tu devrais jeter un oeil à boost.format. En spécifiant %d, tu répètes une information que le compilateur connait déjà. Oublis le C, embrasse le C++.

[screetch]

je suis contre l'allocation dynamique pour un temporaire...

et je questionne fortement "boost::format" sur l'utilisation de l'opérateur %...

[screetch]

C'est en autre pour éviter ce genre de déconvenues que l'on préfère les std::string : on n'a pas à se préoccuper de la mémoire.

Accessoirement, je reviens la dessus pour dire que les std::string ont le même problème :

Code :

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

1
2
const char *str = std::string("blabla").c_str();
printf("%s\n", str);

ce n'est pas un problème d'allocation statique ou dynamique donc...

[white_tentacle]

Code :

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

1
2
const char *str = std::string("blabla").c_str();
printf("%s\n", str);

Pourquoi tu n'écris pas ?

Code :

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

1
2
const std::string str = std::string("blabla");
printf("%s\n", str.c_str());

Et si le but est d'éviter la copie de chaîne, trois solutions :
- utiliser une classe chaîne à comptage de référence
- passer la chaîne en paramètre in-out de ta fonction
- être un guru, écrire

Code :

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

1
2
const std::string& str = std::string("blabla");
printf("%s\n", str.c_str());

et le commenter. Cf most important const