Discussion :

[SAKDOSS]

Bonjour tout le monde,

J'ai un petit soucis avec les template.

Je cherche à créer un vecteur contenant des elements d'une classe Test qui est une classe template.

L'idée serait de pouvoir mettre dans se vecteur des instance de Test qui puissent être de type différent (ex : Test<int> ; Test<string> ; ...).

Après avoir un peu cherché j'ai vu qu'il était conseillé de passer par une classe de base (appelons la TestDeBase) et de créer un vecteur de TestDeBase*.

J'ai donc créé les fichiers suivants :

TestDeBase.cpp :

Code :

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

1
2
3
4
class TestDeBase{
	public :
		TestDeBase(){}
};

Test.cpp :

Code :

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

1
2
3
4
5
6
7
#include"TestDeBase.cpp"
 
template<typename T>
class Test : TestDeBase{
	public :
		Test() : TestDeBase(){}
};

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
#include<stdlib.h>
#include<stdio.h>
#include<vector>
using namespace std;
#include"Test.cpp"
 
int main(int argc, char *argv[]){
	vector<TestDeBase *> vect();
	Test<int> test();	
	vect.push_back(&test);
 
	return EXIT_SUCCESS;
}

Hélas le compilateur ne semble pas apprécier :

Code :

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

1
2
main.cpp: In function 'int main(int, char**)':
main.cpp:11: error: request for member 'push_back' in 'vect', which is of non-class type 'std::vector<TestDeBase*, std::allocator<TestDeBase*> > ()()'

Je m'y suis mal pris ?
Ce n'est pas la bonne méthode ?
Vous avez plus simple ?

Si vous avez la réponse à une de ces questions merci d'avance !

[Mat007]

Salut,

Retire les () à droite des déclarations de vect et test.

MAT.

[hiura]

Car tu déclares des fonctions et pas des variables avec cette syntaxe.

[Alp]

Ce que tu cherches à faire s'appelle "Type Erasure".
Le meilleur article que j'ai lu à ce sujet, de loin : http://www.artima.com/cppsource/type_erasure.html

[SAKDOSS]

Effectivement pour les ()

@Alp : Je n'ai pas lu en détail encore, mais la solution proposée par le lien, semble utiliser la librairie boost. Il n'y aurait pas une solution utilisant les librairies standard ?
Je vais continuer de le lire quand j'aurais plus de temps.
EDIT : ah il y a peut être une solution sans le boost mais je n'ai vraiment plus de temps là :p

En attendant je continue sur mon programme. Après suppression des parenthèses j'obtient le main.cpp suivant :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
#include<stdlib.h>
#include<stdio.h>
#include<vector>
using namespace std;
#include"Test.cpp"
 
int main(int argc, char *argv[]){
 
	vector<TestDeBase *> vect;
	Test<int> test;	
	vect.push_back(&test);
 
	return EXIT_SUCCESS;
}

Et l'erreur de compilation suivante :

Code :

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

1
2
3
4
gcc main.cpp -o pl
main.cpp: In function ‘int main(int, char**)’:
main.cpp:11: erreur: ‘TestDeBase’ is an inaccessible base of ‘Test<int>’
[zales@gmlin34 TesttemplateC]$

[white_tentacle]

Pour que ton héritage soit polymorphique, il doit être public (sinon, violation manifeste du LSP)

Donc :

Code :

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

1
2
3
4
5
template<typename T>
class Test : public TestDeBase{
	public :
		Test() : TestDeBase(){}
};

Tu noteras le simple rajout du mot "public" lors de la déclaration d'héritage, car l'héritage par défaut est privé en C++.

[koala01]

Salut,

Au passage, on n'inclus jamais un fichier d'implémentation ( *.cpp)

Je présumes que ton fichier Test.cpp contient en réalité les définitions des fonctions membres de ta classe test, qui est template, et que tu sais qu'il faut disposer de cette implémentation, mais, pour éviter toute confusion:

Soit on définit les fonctions au sein du fichier d'en-tête (*.hpp), soit on les définit au sein d'un fichier séparé, dont l'extension peut être tout ce que tu veux, mais pas *.cpp (tpp ou impl me semblent sympa )

Ce sera alors le fichier sépéré (tpp ou impl) qui sera inclus dans le fichier d'en-tête, et le fichier d'en-tête qui sera inclus lorsque tu as besoin de ta classe template:
fichier "test.hpp"

Code :

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

1
2
3
4
5
6
template <class T>
class Test
{
    /*...*/ 
};
#inlcude "impl/test.impl"

fichier "test.impl"

Code :

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

/* l'ensemble des définitions de fonctions de Test */

fichier "main.cpp"

Code :

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

1
2
3
4
5
6
#include "test.hpp"
/* autres includes éventuels */
int main()
{
    /*...*/
}

[Alp]

@Alp : Je n'ai pas lu en détail encore, mais la solution proposée par le lien, semble utiliser la librairie boost. Il n'y aurait pas une solution utilisant les librairies standard ?
Je vais continuer de le lire quand j'aurais plus de temps.
EDIT : ah il y a peut être une solution sans le boost mais je n'ai vraiment plus de temps là :p

Non, cela ne nécessite absolument pas Boost. C'est juste qu'un des modules de Boost illustre parfaitement la technique de Type Erasure, et qui revient exactemet à ce que tu essayes de faire. C'est donc une illustration, n'ais pas peur dès qu'il y a des "boost::"