Discussion :

[Alp]

Bonjour à tous,
Ben voilà j'essayais d'implémenter un petit foncteur de multiplication, en version template.
Ca donne ça :

Code :

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

1
2
3
4
5
6
7
8
9
 
class mult 
{
      public:
      template<typename T> T& operator() (T& t1,T& t2)
      {
                        return t1*t2;
      }
};

ensutie dans main :

Code :

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

1
2
3
4
5
6
7
int main()
{
    mult m;
    cout << m(5,9) << endl;
    cout << m(1.50,40.00) << endl;
    cin.get();
}

Et j'obtiens ces erreurs :

In function `int main()':
no match for call to `(mult) (int, int)'
no match for call to `(mult) (double, int)'

Quelques explications sur ces erreurs ne seraient pas de trop
Merci de votre attention et de votre temps.

[JolyLoic]

Tu passes tes arguments sous forme de T&, et non T const &, donc à l'appel de la fonction, tu ne peux pas lui passer une valeur, juste une variable modifiable. C'est pour le cas int, int

Pour le second cas, si tu veux en plus gérer deux types dfférents, fait deux paramètres template : template<class T1, class T2> ?? operator() (T& t1,T& t2)

Quel est le type de la valeur de retour dans ce cas ? Soit tu forces l'utilisateur à le spécifier, soit tu utilises une classe de traits.

[Alp]

Merci pour ces explications.
Comment faire une class de traits? Quel rôle jouerait-elle? Elle détecterait le type à retourner ?

Edit :

Code :

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

1
2
3
4
5
6
7
8
9
 
class mult 
{
      public:
      template<typename T> T& operator() (T const& t1,T const& t2)
      {
                        return t1*t2;
      }
};

me retourne les memes erreurs avec les mêmes appels
Quand à la multiplication de double et int, c'était une erreur de ma part, j'ai corrigé en mettant 40.0.
Je verrai plus tard pour des multiplications entre différents types.

[amaury pouly]

je ne vois pas en quoi mette const T& résoidrait le problème:
const T&
veut dire: valeur passée par référence non modifiable
or on ne passe pas la référence d'un nombre
il faut mettre:
T
et pas
const T&

[Alp]

En effet :

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
 
class mult 
{
      public:
      template<typename T> T operator() (T t1,T t2)
      {
                        return t1*t2;
      }
};
 
int main()
{
    mult m;
    cout << m(5,9) << endl;
    cout << m(1.50,40.00) << endl;
    cin.get();
}

Cela marche.
Mais faire une copie de chaque objet ... pour des int ca va, mais si je définis mes propres trucs ... Il y aurait pas une astuce que j'aurais manqué? :o

[Laurent Gomila]

je ne vois pas en quoi mette const T& résoidrait le problème:
const T&
veut dire: valeur passée par référence non modifiable
or on ne passe pas la référence d'un nombre
il faut mettre:
T
et pas
const T&

Ca dépend ce que l'on compte passer comme paramètres. S'il ne s'agit que de types primitifs ou très légers on peut en effet se contenter d'un passage par valeur, sinon il faut prévoir le passage par référence constante.
Pour ça aussi les traits peuvent bien aider (voir boost::type_traits si je me souviens bien).

http://c.developpez.com/faq/cpp/?pag...sage_parametre

[Luc Hermitte]

Tu retournais un temporaire via référence...

[Alp]

Ah oui en effet :/ j'risquais pas d'aller bien loin.
Merci en tout cas.
Bonne progr'