Discussion :

[Matthieu Brucher]

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 :

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

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 :

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

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

vous avez une idée de mon erreur toute bête ?

[koala01]

Salut,

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

Quelque chose comme

Code :

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

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

(ou un typedef associé, why not)

[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 :

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

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*.

[Alp]

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

Code :

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

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

[Matthieu Brucher]

Salut,

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

Quelque chose comme

Code :

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

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

(ou un typedef associé, why not)

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.

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 :

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

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 ?

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

Code :

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

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

[Alp]

J'ai oublié template <int N> biensûr dans mon code

[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 ?

[Alp]

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
#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 :

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

1
2
$ ./test
get_value<10> called

[Matthieu Brucher]

OK, je vais tester ça demain

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

[Alp]

OK, je vais tester ça demain

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

[Matthieu Brucher]

Ah !! Trop facile comme ça

[JolyLoic]

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 :

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
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...

[Alp]

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 ?

[Matthieu Brucher]

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.

[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é...

[JolyLoic]

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...

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.

[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

[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...

[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 !

[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.