Taille template de chaîne de caractères

Version imprimable

Hello,
J'ai un problème "assez" simple que je n'arrive pas à résoudre. J'essaie de sérialiser en XML (avec TinyXML) des chaînes de caractères dont la taille est arbitraire. J'ai donc le code suivant :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 template<unsigned long Size> void get_value(const TiXmlElement* node, char text[Size]) { std::string temp = node->GetText(); strncpy(temp.c_str(), text, Size); } template<unsigned long Size> void set_value(TiXmlElement* node, const char text[Size]) { TiXmlText* text_node = new TiXmlText(text); text_node->SetCDATA(true); node->LinkEndChild(text_node); }
La deuxième fonction passe, mais pas la première. En fait, le compilateur (icc 10.1) n'arrive pas à trouver get_value, par exemple avec le code suivant (issu de mes tests unitaires) :
Code:

1 2 char new_text[16]; SOAPRPC::get_value(pNode, new_text);
vous avez une idée de mon erreur toute bête ?

03/06/2009, 18h27
koala01
Salut,

As tu, tout bêtement, essayé en précisant la taille :question:

Quelque chose comme
Code:

1 2 char new_text[16]; SOAPRPC::get_value<16>(pNode, new_text);
(ou un typedef associé, why not:question:)
03/06/2009, 18h28
Nogane
Bonsoir,
Ça ne marche pas chez moi non plus. (Les deux functions en fait). Avec minGW 4.3

La seul alternative que j'aurais serais quelque chose comme ca:
Code:

1 2 3 4 5 6 template<class T> void get_value(const TiXmlElement* node, T text) { std::string temp = node->GetText(); std::strncpy(&temp[0],text, sizeof(text)); }
Ça marche mais je comprend bien que ce n'est pas top.

Au fait, "strncpy(temp.c_str(), text, Size);" ne compile pas avec minGW car c_str() renvoie un "const char*" alors que strncpy attend un char*.

Bizarre qu'il ne déduise pas.
Et avec un code de ce genre :
Code:

1 2 void get_value(TiXmlElement* node, const char (&array)[N]) { /* ... */ }

03/06/2009, 18h50
Matthieu Brucher
Citation:
Envoyé par koala01

Salut,

As tu, tout bêtement, essayé en précisant la taille :question:

Quelque chose comme

Code:

1 2 char new_text[16]; SOAPRPC::get_value<16>(pNode, new_text);

(ou un typedef associé, why not:question:)
Le problème, c'est que c'est appelé par un code qui est généré automatiquement avec du gccxml. Il y a plusieurs types possibles, et je me base sur le compilateur pour faire la résolution des différentes surcharges et spécialisations.
Citation:
Envoyé par Nogane

Bonsoir,
Ça ne marche pas chez moi non plus. (Les deux functions en fait). Avec minGW 4.3

La seul alternative que j'aurais serais quelque chose comme ca:

Code:

1 2 3 4 5 6 template<class T> void get_value(const TiXmlElement* node, T text) { std::string temp = node->GetText(); std::strncpy(&temp[0],text, sizeof(text)); }

Ça marche mais je comprend bien que ce n'est pas top.

Au fait, "strncpy(temp.c_str(), text, Size);" ne compile pas avec minGW car c_str() renvoie un "const char*" alors que strncpy attend un char*.
Tiens, bizarre que ça ne passe pas avec Mingw. Et bizarre aussi pour strncpy, puisque j'utilise les en-têtes gcc, et je n'ai pas eu de problème. Peut-être plus un bug chez eux ?
Citation:
Envoyé par Alp

Bizarre qu'il ne déduise pas.
Et avec un code de ce genre :

Code:

1 2 void get_value(TiXmlElement* node, const char (&array)[N]) { /* ... */ }
Euh... Faudra que j'essaie. Mais bon, ça ne me plaît pas trop comme écriture :|
03/06/2009, 20h04
Alp

J'ai oublié template <int N> biensûr dans mon code :?
03/06/2009, 20h06
Matthieu Brucher

T'as essayé ?
Ce qui me chiffonne, c'est que le compilateur comprend bien que les deux aruments sont un const TiXmlElement* et un char[16] (par exemple), mais il n'arrive pas à instancier le code. Tu as essayé si ça marche avec ta solution ?

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
#include <iostream>
 
using namespace std;
 
template <int N>
void get_value(const char (&array)[N])
{ 
  cout << "get_value<" << N << "> called" << endl;
}
 
int main()
{
  char test[10];
  get_value(test);
  return 0;
}

renvoie

Code:

1
2
$ ./test
get_value<10> called

03/06/2009, 20h32
Matthieu Brucher

OK, je vais tester ça demain :D

Pourquoi il y a le (&array) ? (l'array doit être modifié, donc j'enlèverai le const ;))
03/06/2009, 20h57
Alp

Citation:

Envoyé par Matthieu Brucher

OK, je vais tester ça demain :D

Pourquoi il y a le (&array) ? (l'array doit être modifié, donc j'enlèverai le const ;))

J'ai tiré l'idée d'ici : http://hermitte.free.fr/Info/C++/lib..._help.hpp.html
Il faudra demander à Luc :mrgreen:
03/06/2009, 21h04
Matthieu Brucher

Ah !! Trop facile comme ça :D

Parce qu'en argument de fonction par valeur, un tableau de taille définie ou non, ça devient en interne un pointeur.

Code mettant ça en évidence :
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 void f(int i[2]) { } /*void f(int i[3]) { } void f(int i[]) { } void f(int *i) { }*/ void g(int (&i)[2]) { } void g(int (&i)[3]) { } /*void g(int (&i)[]) { }*/
Décommente une seule des fonctions f, et tu auras une erreur de fonction définie 2 fois...

03/06/2009, 22h02
Alp

Citation:

Envoyé par JolyLoic

Décommente une seule des fonctions f, et tu auras une erreur de fonction définie 2 fois...

Alors que ça ne sera pas le cas pour les fonctions g, c'est ça ?
03/06/2009, 22h07
Matthieu Brucher

Citation:

Envoyé par JolyLoic

Parce qu'en argument de fonction par valeur, un tableau de taille définie ou non, ça devient en interne un pointeur.

OK pour ça, il n'y a pas de problème, c'est plus la syntaxe qui m'étonnait.
03/06/2009, 22h18
Goten

Par contre : (&array)[] ça compile pas si?

Mais c'est vrai que cette syntaxe (je l'utilise des fois pour remplir les vector puisque ça permet de récupérer des pointeurs sur le début et la fin) m'a toujours un peu étonné...
03/06/2009, 22h26
JolyLoic

Citation:

Envoyé par Alp

Alors que ça ne sera pas le cas pour les fonctions g, c'est ça ?

Elles sont décommentées dans l'exemple, et le code compile...

Citation:

Envoyé par Goten

Par contre : (&array)[] ça compile pas si?

Non, en effet (d'où le fait qu'elle soit commentée chez moi). Un tel tableau est sensé avoir une taille.
04/06/2009, 09h15
Matthieu Brucher

Effectivement, ça fonctionne très bien. Et pour la deuxième fonction, j'imagine que c'est ma surcharge avec un std::string qui était utilisée, et non le const char[]...

Merci pour la solution ;)
08/06/2009, 11h58
Médinoc

Attention aussi avec l'emploi de strncpy() : Souviens-toi que s'il y a troncature, le buffer n'est pas terminé par un caractère nul...
08/06/2009, 13h17
Matthieu Brucher

Tout à fait ;)
C'est pour ça que j'ai fini par passer un std::string, mais comme c'était un refactoring progressif, je ne voulais pas tout détruire d'un coup !
08/06/2009, 14h42
loufoque

Dans la deuxième fonction, Size ne sert à rien...

Et c'est bien entendu parce qu'il manquait le & que la première ne marchait pas.

Pour le set_value tu pourrais utiliser une range, ça coûte rien.
Pour le get_value par contre, soit tu prends un output iterator soit tu retournes une std::string.