Discussion :

[kain_tn]

Bonjour,


J'ai une classe A qui possède deux constructeurs:

Code :

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

1
2
 
A(const int error, const std::string& message = "", const char* filename = __FILE__, const int line = __LINE__);

et

Code :

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

1
2
 
A(const std::string& message, const char* filename = __FILE__, const int line = __LINE__);

Pour me simplifier l'écriture, j'aurais aimé écrire un truc du genre:

Code :

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

1
2
 
#define A(a) A(a, __FILE__, __LINE__)

et

Code :

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

1
2
 
#define A(a,b) A(a,b,__FILE__,__LINE__)

Mais ce n'est pas possible car c'est une redéfinition de ma macro A...

J'ai donc tenté une écriture à base de templates:

Code :

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

1
2
3
4
5
6
7
8
9
10
 
 template <typename T, typename A, typename B>
 T ctor(A a, B b){
	return T(a, b, __FILE__, __LINE__);
 }
 
 template <typename T, typename A>
 T ctor(A a){
	return T(a, __FILE__, __LINE__);
 }

Mais l'ennui c'est que les __FILE__ et __LINE__ correspondent aux lignes de déclaration de mes templates et non aux lignes d'appel à ctor<A>(0, "toto") ou ctor<A>("titi") dans le code...

J'avoue que je sèche un petit peu...

[koala01]

Salut,

Tu as déjà un problème bien plus embêtant d'ambiguïté de tes constructeur, parce que tu définis pour les deux une valeur par défaut pour chacun des paramètres requis...

En effet, le premier constructeur fait que les signatures admises, du fait des valeurs par défaut, sont:

• A()
• A(int const)
• A(int const, std::string const&)
• A(int const, std::string const &, char * const)
• A(int const, std::string const &, char * const, int const)

Et le second constructeur fait que les signatures admises sont:

• A()
• A(std::string const&)
• A(std::string const &, char * const)
• A(std::string const &, char * const, int const)

Tu remarquera que les deux constructeurs ont pour effet de permettre une signature ne prenant aucun argument

De ce fait, lorsque tu décide d'invoquer

Code :

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

A mona;

le compilateur est incapable de déterminer s'il doit invoquer le premier ou le second constructeur...

[kain_tn]

Salut,

Tu as déjà un problème bien plus embêtant d'ambiguïté de tes constructeur, parce que tu définis pour les deux une valeur par défaut pour chacun des paramètres requis...

Ah oui en effet. C'est corrigé
En revanche, ça ne change rien à mon problème de #define, qui vient du fait qu'il considère #define A(a,b) comme une redéfinition de #define A(a)...

[aoyou]

En revanche, ça ne change rien à mon problème de #define, qui vient du fait qu'il considère #define A(a,b) comme une redéfinition de #define A(a)...

Malheureusement, il n'y a pas de miracle. On ne peut pas surcharger une macro. Le seul moyen pour toi de t'en sortir serait d'avoir des constructeurs avec le même nombre d'arguments mais dont les types différeraient.

[white_tentacle]

Je ne comprends pas pourquoi tu as besoin de macros. Les paramètres par défaut ne te suffisent pas ? J'ai loupé un truc ?

[gb_68]

Bonjour,

Je ne comprends pas pourquoi tu as besoin de macros. Les paramètres par défaut ne te suffisent pas ? J'ai loupé un truc ?

line contiendra la ligne où est déclarée le constructeur hors je pense que c'est la ligne d'invocation du constructeur qui est souhaitée ici.
Pour les macros tu peux tenter quelque chose avec les arguments variables de macros :

Code :

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

#define A( ... ) (__VA_ARGS__,__FILE__,__LINE__)

mais ce genre de macros ne résoudra pas tous les problèmes :

Code :

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

1
2
A * pA = new A(0,""); // Ok
A a(0,""); // pas Ok

Sinon, tu peux te servir d'une macro pour réduire __FILE__,__LINE__ :

Code :

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

1
2
3
4
5
#define FANDL __FILE__,__LINE__
// ...
    A a(1,"1",FANDL);
    A b("2",FANDL);
    A * pA = new A(3,"3",FANDL);

[kain_tn]

Je ne comprends pas pourquoi tu as besoin de macros. Les paramètres par défaut ne te suffisent pas ? J'ai loupé un truc ?

C'est pour ne pas avoir à passer les "__FILE__, __LINE__" à chaque appel en fait.

[kain_tn]

Sinon, tu peux te servir d'une macro pour réduire __FILE__,__LINE__ :

Code :

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

1
2
3
4
5
#define FANDL __FILE__,__LINE__
// ...
    A a(1,"1",FANDL);
    A b("2",FANDL);
    A * pA = new A(3,"3",FANDL);

Oui, c'est ce que j'ai fini par faire.
Merci à tous