Discussion :

[bigquick]

Salut,

Voilà, je travaille sous Visual C++ 6 (qui n'est pas très fort en template ), et du coup je voudrais savoir si c'est mon code qui n'est pas bon, ou lui

J'ai une fonction pour instancier un dérivé de Composant, qui compile très bien:

Code :

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

1
2
3
4
5
6
7
8
 template <typename T>
Composant* ManagerComposant::instancier(std::string _strFichier)
{
	// static_cast le template en composant : erreur de compilation si le cast n'est pas possible
	Composant* pComposant = static_cast<Composant*>(new T);
 
	// .... beaucoup d'autre code ....
}

mais à l'utilisation ça se passe moins bien :

Code :

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

return instancier<Vecteur>(_strFichier);

J'obtiens l'erreur : error C2275: 'Vecteur' : illegal use of this type as an expression

Est-ce que ça vous à l'air correct ?
merci!

(ps: j'utilise une fonction parcequ'elle instancie le composant et fait plein de traitements dessus, qui sont indépendants du type réél du composant)

[Laurent Gomila]

Ton code est correct, c'est bien VC6 qui a du mal avec les fonctions membres templates. J'avais eu exactement le même problème.

[bigquick]

Ah ok, merci
Du coup je vais devoir trouver une autre solution alors si j'ai bien compris...

A moins que aies trouvé un contournement sympa quand tu avais le même problème ?

[Luc Hermitte]

Pour un truc pareil, il n'y a guère de risque à définir les créateurs comme des fonctions libres. Et ensuite passer le composant fraichement créé à la fonction membre de ton gestionnaire.

[xterminhate]

N'importe quel compilateur gratuit fait mieux que VC6 aujourd'hui. Peut être qu'il faut penser à changer de compilo plutot que de manière de coder...

[xterminhate]

Code :

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

1
2
3
4
5
6
7
8
 template <typename T>
Composant* ManagerComposant::instancier(std::string _strFichier)
{
	// static_cast le template en composant : erreur de compilation si le cast n'est pas possible
	Composant* pComposant = static_cast<Composant*>(new T);
 
	// .... beaucoup d'autre code ....
}

Et bien, j'ai du mal à voir ce à quoi le static cast sert. Soit le paramètre modèle à pour base Composant et ca compile sans cast. Soit ce n'est pas le cas, et ca ne compilera jamais. Le static cast n'aide en rien.

Pour adapter le traitement dans ManagerComposant, tu peux utiliser BOOST::type_traits -> ::boost::is_base_and_derived<T,U>::value

[bigquick]

Tu sais, en entreprise chacun ne compile pas sur la plateforme / compilateur de son choix... sinon ca deviendrait vite la foire à la saucisse
Je voulais pas VC6, on m'a forcé

Pour ce qui est du static_cast, en fait je n'ai jamais vraiment compris à quoi il sert... je le mets histoire d'explicier le cast, mais quand est-ce que c'est vraiment utile ?

Et merci Luc, c'est ce que j'ai fait du coup, ça fonctionne beaucoup mieux maintenant

[Herode]

Pour ce qui est du static_cast, en fait je n'ai jamais vraiment compris à quoi il sert... je le mets histoire d'explicier le cast, mais quand est-ce que c'est vraiment utile ?

Ca sert surtout à expliciter
A mon avis, il est surtout utile dans la mesure où il facilite la maintenance, ce qui est une raison largement suffisante. En outre, le recours systématique aux opérateurs de transtypage permet de lutter contre un code du type :

Code :

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

1
2
3
class A{}; 
class B{};
A* pa = (A*) new B;

qui, lui, est susceptible de compiler...

[bigquick]

OK c'est noté
Je vais continuer à l'utiliser alors ....

Merci à tous