Discussion :

[M.Dolly]

Bonjour à tous,

Aujourd'hui j'ai un soucis avec l'utilisation des templates, voici d'abord mon code :
Matrix.h

Code C++ :

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
#include <vector>
#include <iostream>
 
#ifndef MATRIX
#define MATRIX
 
template<typename T> 
class Matrix 
{
  private:
    std::vector<std::vector<T> > data;    
 
  public:  
    Matrix();
 
    Matrix(int swidth, int sheight);
    Matrix(int size);
 
    int getWidth() { return data.size(); }
    int getHeight() { return data.size() == 0 ? 0 : data[0].size(); }
 
    void print();    
 
    ~Matrix();
};
#endif

Matrix.cpp (simplifié)

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
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
#include "Matrix.h"
 
/*********************************
    C O N S T R U C T O R S
**********************************/
 
template<class T> 
Matrix<T>::Matrix(){
  data.empty();
  std::clog << "Construction matrice" << std::endl;
}
 
template<class T> 
Matrix<T>::Matrix(int swidth, int sheight) {
  data.resize(0);
  if(swidth > 0 && sheight > 0) {
    data.resize(swidth);
    for(unsigned int i = 0; i < data.size(); i++) {
      data[i].resize(sheight);
    }
    std::clog << "Construction matrice" << std::endl;
  }
}
 
template<class T> 
Matrix<T>::Matrix(int size) {
  if(size > 0) {
    data.resize(size);
    for(unsigned int i = 0; i < data.size(); i++) {
      data[i].resize(size);
    }
  } else {
    data.resize(0);
  }
  std::clog << "Construction matrice" << std::endl;
}
 
 
 
/*********************************
          O T H E R
**********************************/
template<class T> 
void Matrix<T>::print() {
  int height;
  height = data.size() == 0 ? 0 : data[0].size();
 
  std::cout << "Width = " << height << " height = "<< data.size() << std::endl;
 
  for(unsigned int i = 0; i < data.size(); i++) {
      for (unsigned int j = 0; j < data[i].size(); j++) {
        std::cout << data[i][j] << " ";
      }
      std::cout << std::endl;
   }
}
 
 
 
/*********************************
     D E S T R U C T O R
**********************************/
 
template<class T> 
Matrix<T>::~Matrix() {
  data.resize(0);
  std::clog << "Destruction matrice" << std::endl;
}

Et enfin le main.cpp

Code :

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

1
2
3
4
5
6
7
8
#include <iostream>
#include "Matrix.h"
 
int main() {
  Matrix<int> m1(5);
 
  return 0;
}

Voici ce me renvoie le compilateur (g++) :

Code Console :

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

1
2
3
4
5
main.o: In function `main':
/home/isen/Cours/C/perso/matrix/main.cpp:5: undefined reference to `Matrix<int>::Matrix(int)'
/home/isen/Cours/C/perso/matrix/main.cpp:5: undefined reference to `Matrix<int>::~Matrix()'
collect2: ld a retourné 1 code d'état d'exécution
make: *** [matrix] Erreur 1

Il ne semble pas reconnaître les fonctions, cela ne marche pas non plus avec m1.print();. Je dois mal utiliser les templates mais même après avoir cherché je ne trouve pas de solutions.

PS : Si je n'écrit pas template<class T> void Matrix<T>::print()(...) dans le Matrix.cpp, le compilateur me cri dessus alors que cette fonction se moque royalement du type T. Je ne vois pas pourquoi je dois écrire cela.

[backlash]

La définition des méthodes d'une classe template doivent être AUSSI dans le fichier.h où est déclarée la classe, voir faq C++.

[M.Dolly]

Merci backlash pour cette réponse rapide. Je trouve ca vraiment moche de mettre tout dans le .h il n'y a pas une autre solution ?

[koala01]

Salut,

Il est tout à fait possible de "scinder" les différentes partie du fichier en un nombre quelconque de fichiers différents, mais il faut alors prendre certaines précautions.

Tu peux, par exemple, créer un fichier "declaration" et un fichier "implémentation", mais qui sont toujours à considérer comme fichiers d'en-tête sous une forme proche de
Matrix_decl.hpp

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
#ifndef MATRIX_DECL_HPP
#define MATRIX_DECL_HPP
template<typename T> 
class Matrix 
{
  private:
    std::vector<std::vector<T> > data;    
 
  public:  
    Matrix();
 
    Matrix(int swidth, int sheight);
    Matrix(int size);
 
    int getWidth() ;
    int getHeight();
 
    void print();    
 
    ~Matrix();
};
 
#endif // MATRIX_DECL_HPP

Matrix_impl.hpp

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
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
#ifndef MATRIX_IMPL_HPP
#define  MATRIX_IMPL_HPP
/* !!!! les fonctions template DOIVENT etre déclarées inline, de manière
 * implicites ou explicites !!! 
 */
template<class T> 
inline Matrix<T>::Matrix(){
  data.empty();
  std::clog << "Construction matrice" << std::endl;
}
 
template<class T> 
inline Matrix<T>::Matrix(int swidth, int sheight) {
  data.resize(0);
  if(swidth > 0 && sheight > 0) {
    data.resize(swidth);
    for(unsigned int i = 0; i < data.size(); i++) {
      data[i].resize(sheight);
    }
    std::clog << "Construction matrice" << std::endl;
  }
}
 
template<class T> 
inline Matrix<T>::Matrix(int size) {
  if(size > 0) {
    data.resize(size);
    for(unsigned int i = 0; i < data.size(); i++) {
      data[i].resize(size);
    }
  } else {
    data.resize(0);
  }
  std::clog << "Construction matrice" << std::endl;
}
 
template<class T> 
inline void Matrix<T>::print() {
  int height;
  height = data.size() == 0 ? 0 : data[0].size();
 
  std::cout << "Width = " << height << " height = "<< data.size() << std::endl;
 
  for(unsigned int i = 0; i < data.size(); i++) {
      for (unsigned int j = 0; j < data[i].size(); j++) {
        std::cout << data[i][j] << " ";
      }
      std::cout << std::endl;
   }
}
 
 
template<class T> 
inline Matrix<T>::~Matrix() {
  data.resize(0);
  std::clog << "Destruction matrice" << std::endl;
}
 
#endif // MATRIX_IMPL_HPP

Il faudra alors s'assurer le la déclaration apparaisse systématiquement AVANT la définition des fonctions, ce qui implique qu'il faudra systématiquement inclure le fichier Matrix_decl.hpp avant le fichier Matrix_impl.hpp!

L'idéal est donc peut etre de rajouter un troisième fichier qui inclut les deux fichiers précédents dans l'ordre et qui prendrait la orme de

Code :

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

1
2
3
4
5
#ifndef MATRIX_HPP
#define MATRIX_DECL_HPP
#include <Matrix_decl.hpp>
#include <Matrix_impl.hpp>
#endif // MATRIX_HPP

Tu constatera malgré tout que cela ne revient à rien d'autre que de... déplacer dans deux fichiers ce que l'on trouve d'habitude dans un seul...

Je te laisse seul juge de l'utilité éventuelle de la chose

[oodini]

Il est courant, pour le fichier d'implémentation, d'avoir une extension *.tpp.

[koala01]

Il est courant, pour le fichier d'implémentation, d'avoir une extension *.tpp.

ou *.ipp ou... n'importe quelle extension qui ne soit pas déjà d'office réservée à un type de fichier particulier :d