Discussion :

[coberle]

Bonjour,
J'ai un classe point

Code :

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

1
2
3
template <unsigned D> class PointT { ...
friend ostream &operator<<(ostream &oS, const PointT<D> &P);
... }

et une surcharge d'opérateur

Code :

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

1
2
3
template <unsigned D>
ostream &operator<<(ostream &oS, const PointT<D> &P)
{ ... }

Dans mon main, j'ai

Code :

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

1
2
3
 
   PointT<4> P4({1.15,3.6,1.8});
  cout << P4;

Et j'obtiens l'erreur suivant sur "cout << P4":
undefined reference to `operator<<(std::ostream&, PointT<4u> const&)'

J'avoue que je cale sur le problème ... auriez-vous une idée ?
En vous remerciant par avance.
Christian

[Neckara]

Bonjour,

Juste une petite question idiote : as-tu bien pensé à mettre le code du template dans l'header (.h) et non dans le fichier source (.cpp) ?

[coberle]

Oui - le fichier .h du template contient le header et le code (car c'est un bout de code de test que j'écrit). Le code de l'opérateur surchargé est avant mon main. En quoi cette séparation influe-t-elle ?

[Vespiras]

Bonsoir, si tu veux dire que l'implémentation de ta surcharge d'opérateur est dans ton main, tant que ton main connait le header ou est placé le prototype, ça ne change rien. Après, pour plus de clarté et une organisation "typique", il vaut mieux séparer chaque .h et .cpp en fichiers distincts par classe.

Pour ton problème, question toute bête : As-tu mis dans le header de la classe point un :

Code :

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

1
2
#include <iostream>
using namespace std;

?

[Neckara]

Bonsoir, si tu veux dire que l'implémentation de ta surcharge d'opérateur est dans ton main, tant que ton main connait le header ou est placé le prototype, ça ne change rien. Après, pour plus de clarté et une organisation "typique", il vaut mieux séparer chaque .h et .cpp en fichiers distincts par classe.

On utilise ici une fonction template.
Cf FAQ C++

Pour ton problème, question toute bête : As-tu mis dans le header de la classe point un :

Code :

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

1
2
#include <iostream>
using namespace std;

?

En effet je pense qu'il l'a fait.
Un autre lien de la FAQ s'impose.

[coberle]

Je suis finalement arrivé à la conclusion (après quelques essais de fonctions friends sur des classes non templatées) que la solution (ou le problème) se trouvait dans la déclaration de fonction friends dans les classes templatées. Après une recherche sur Google, je suis arrivé sur ce lien http://www.developpez.net/forums/d22...plates-friend/ qui m'a guidé vers la solution. En déclarant la fonction comme ceci dans la classe template, ça marche:

Code :

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

template <unsigned Q> friend ostream &operator<<(ostream &oS, const PointT<Q> &P);

(i.e. il faut rajouter

Code :

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

template <unsigned Q>

devant

Code :

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

friend

).
Encore merci à vous, Laurent et le site developpez qui est vraiment une mine !
Christian

[oodini]

Certes, mais c'est vraiment plus laxiste.
Est-ce ce que tu souhaites ?

[coberle]

En quoi est-ce plus laxiste ? Par ailleurs, je n'ai pas trouvé d'autre solution ... Bien sûr, je suis preneur d'alternatives :-)

[Neckara]

Bonjour,

En quoi est-ce plus laxiste ? Par ailleurs, je n'ai pas trouvé d'autre solution ... Bien sûr, je suis preneur d'alternatives :-)

Tu va autoriser toutes les méthodes ostream &operator<<(ostream &oS, const PointT<Q> &P); à accéder au contenu de ta classe au lieu d'autoriser uniquement l'unique méthode utilisée pour afficher ta classe.

Pourrais-tu nous montrer le fichier d'en-tête complet pour ta classe point ainsi que l'en-tête contenant ton opérateur << ?

[oodini]

Code :

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

1
2
3
template <unsigned D> class PointT { ...
friend ostream &operator<<(ostream &oS, const PointT<D> &P);
... }

Enlève ce que j'ai mis en gras.

[coberle]

En faisant comme tu le suggères, il y a 4 erreur qui sont générées, comme par exemple :
invalid use of template-name 'PointT' without an argument list;
ISO C++ forbids declaration of 'P' with no type [-fpermissive]

[coberle]

Neckara - voici le code de la classe PointT et du main. Encore merci pour ton aide (j'utilise CodeBlocks avec compil gnu/gcc)

Classe PointT (PointT.h):

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
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
 
#ifndef POINTT_H_INCLUDED
#define POINTT_H_INCLUDED
 
#include <initializer_list>
 
using namespace std;
 
template <unsigned D> class PointT
{
 public:
   PointT(initializer_list<float> coords);
   void Afficher();
 
   friend ostream &operator<<(ostream &oS, const PointT &P);
 
   static const unsigned dim;
 
 private:
    float m_coords[D]={0.0};
};
 
template <unsigned D>
const unsigned PointT<D>::dim = D;
 
template <unsigned D>
PointT<D>::PointT(initializer_list<float> coords)
{
    initializer_list<float>::iterator itr;
    int loop=min(D,coords.size());
    itr=coords.begin();
 
    for (int i=0; i<loop; i++)
        m_coords[i]=*itr++;
}
 
template <unsigned D>
void PointT<D>::Afficher()
{
   cout << "Dans afficher - " << this->m_coords[0] << endl;
}
 
#endif // POINTT_H_INCLUDED

Main (main.cpp):

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
 
#include <iostream>
#include "PointT.h"
 
using namespace std;
 
template <unsigned D>
ostream &operator<<(ostream &oS, const PointT<D> &P)
{
    oS << "Point" << D << "D - (";
    for (int i=0; i<D-1; i++)
        oS << P.m_coords[i] << ",";
 
    oS << P.m_coords[D-1] << ")" << endl;
 
    return oS;
}
 
int main()
{
    PointT<3> P({1.1,1.2,1.3});
 
    P.Afficher();
 
    cout << P; // ligne qui provoque l'erreur "undefined reference to "operator<< ..."
 
    return 0;
}

[Neckara]

Bonjour,

Pour régler ton problème de template :

Code :

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

1
2
3
4
5
6
7
8
template 
<unsigned D> class PointT;
 
template<unsigned D>
ostream &operator<<(ostream &oS, const PointT<D> &P);
 
//dans ta classe
friend ostream &operator<<<>(ostream &oS, const PointT<D> &P);

Attention, jamais de using namespace dans un header.

Je continue de regarder les autres erreurs.

EDIT : Dans ton constructeur :

Code :

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

int loop=min(D, (unsigned)coords.size());

Résultat d'exécution :

~$ ./exe
Dans afficher - 1.1
Point3D - (1.1,1.2,1.3)

Est-ce bien le résultat attendu ?

[coberle]

Merci Neckara ... ça fonctionne.
Mais je ne suis pas certain de bien comprendre le pourquoi du comment:
- pourquoi ceci avant la classe

Code :

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

1
2
3
4
5
template 
<unsigned D> class PointT;
 
template<unsigned D>
ostream &operator<<(ostream &oS, const PointT<D> &P);

- pourquoi ensuite

Code :

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

friend operator<<<> ...

PS - je crois qu'il y a une réponse ici http://stackoverflow.com/questions/4...lass-templates

[Neckara]

Merci Neckara ... ça fonctionne.
Mais je ne suis pas certain de bien comprendre le pourquoi du comment:
- pourquoi ceci avant la classe

Code :

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

1
2
3
4
5
template 
<unsigned D> class PointT;
 
template<unsigned D>
ostream &operator<<(ostream &oS, const PointT<D> &P);

C'est le principe des déclarations anticipées.

- pourquoi ensuite

Code :

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

friend operator<<<> ...

PS - je crois qu'il y a une réponse ici http://stackoverflow.com/questions/4...lass-templates

En fait tu dis que ta classe est friend avec une spécialisation de ta méthode template.
Si j'ai mis les <> c'est surtout parce que le warning me demandait de les mettre.
Mais d'après ton lien, c'est juste pour informer que la fonction est template.


Sinon ton code est correct si tu souhaites que seul main.cpp puisse utiliser la fonction template sinon, cf FAQ.

[coberle]

Merci beaucoup, c'est vraiment sympa.