Discussion :

[tomtom7]

Bonjour,
J'ai 3 fois la même fonction avec des prototypes différents :

HRESULT AddAttributeString(int value, bool right = true, const std::string &sep = "");
HRESULT AddAttributeString(const char *value, bool right = true, const std::string &sep = "");
HRESULT AddAttributeString(double value, bool right = true, bool highPrecision = false, const std::string &sep = "");

J'utilise ces fonctions à plein d'endroits différents et ça marche bien partout sauf lors d'un appel :

Code :

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

1
2
3
 
int i =2;
AddAttributeString(n,false,",");

Lors de la compilation j'ai :
error C2666: 'AddAttributeString' : 2 overloads have similar conversions

J'ai essayé de caster n en int, en long...
Si je modifie son type en unsigned int ça compile mais la fonction appelée pendant l'exécution est celle qui prend un double (alors que le reste des paramètres ne matche pas du tout...)

J'ai temporairement résolu le problème en réécrivant une fonction qui s'appelle différemment mais je ne comprends toujours pas pourquoi l'échec de la compilation.

Si quelqu'un a une idée...

[Matthieu Brucher]

C'est ta surcharge qui ne marche pas, pas le polymorphisme

Bon, n, c'est quoi comme type ?
La première version prend dont un int, un bool et un string.
La deuxième un const char* - pourquoi pas un string ?? -, un bool et un string
La troisième prend un double, 2 bool et un string.

La troisième ne peut donc pas être appelée, sauf s'il transforme un bool en string... Gros inconvénient des paramè-tres par défaut

[tomtom7]

n est un int.
Je suis complétement d'accord que les paramètres ne correspondent pas.
Ce qui est bizarre c'est que le 1er paramètre matche bien, enfin en théorie...

J'ai mis les 3 protos mais celui qui prend un char* n'est à priori pas en cause. Le compilo dit bien qu'il ne sait pas quoi choisir entre la méthode qui prend un double et celle qui prend un int...

[harsh]

Est ce la seule erreur de compil (peut etre que ton proto ne correspond pas a la declaration, parfois c'est une erreur con, dans quel cas le compilo te l'indique peut etre plus loin).

Sinon envoie le bout de code complet exact qui plante a la compile, idem pour tous tes proto et tous le barratin que te sors le compilo

[tomtom7]

Merci harsh.

J'étais persuadé que c'était le 1er paramètre qui posait problème mais le compilo me donnait ça :

xml_stuff.cpp(94) : error C2666: 'xml::CXmlWriter::AddAttributeString' : 2 overloads have similar conversions
xml_writer.h(50): could be 'HRESULT xml::CXmlWriter::AddAttributeString(double,bool,bool,const std::string &)'
xml_writer.h(48): or 'HRESULT xml::CXmlWriter::AddAttributeString(int,bool,const std::string &)'
while trying to match the argument list '(int, bool, const char [2])'

J'ai remplacé

Code :

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

1
2
 
AddAttributeString(n,false,",");

par

Code :

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

1
2
 
AddAttributeString(n,false,boost::lexical_cast<std::string>(","));

Et ça marche.

Merkiii

[Matthieu Brucher]

Tu peux te passer du lexical cast en mettant directement std::string(",") normalement , ça passe.

[tomtom7]

Ouais. J'ai mis ça pour vérifier que ça passait bien c'est tout.
J'ai changé les protos pour du char* à la place du std::string&.

En tout cas merci.

[loufoque]

T'utilises MSVC6 ou quoi ?

[Luc Hermitte]

J'avais croisé un truc surprenant dans le passé. Un const char * préfère être converti en bool qu'en std::string. De là à ce que cela explique des choses ...

Sinon, la surcharge correspond à une des deux formes du polymorphisme ad'hoc. L'autre, ce sont les convertions implicites. Dans l'absolu, il avait bien un problème de polymorphisme.

[tomtom7]

Dans l'absolu, il avait bien un problème de polymorphisme.

C'est ce que je pense...