Discussion :

[biyoudidou]

plop,
j'essaye de faire une simple fonction pour éviter d'avoir a repasser 40 arguments quand je veux allouer un objet :

Code :

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

1
2
3
4
5
6
 
template <class U>
shared_ptr<U> CreateRes(...)
{
	return shared_ptr<U>(new U(this->GetDevice(), this->GetDeviceContext(), __VA_ARGS__));
}

mais veut pas compiler, ca me sort "error C2059: syntax error : ')'" sur le return

[Ekleog]

__VA_ARGS__ ne fonctionne que pour les macros.
Et "this->" est, en général, inutile.
Enfin, il faut utiliser make_shared autant que possible.

Donc ce code deviendrait :

Code :

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

#define CreateRes(...) std::make_shared<U>(GetDevice(), GetDeviceContext(), __VA_ARGS__)

[biyoudidou]

merci pour la réponse,
mais comme tu vois c'est une fonction membre.
La macro devrait etre déclarée dans le fichier ou elle est utilisée, et en utilisant le nom complet des arguments style game.mainwnd->GetDevice(), donc pas réutilisable et doit etre fait par l'utilisateur de la lib, c'est pas le but recherché.

[Ekleog]

merci pour la réponse,
mais comme tu vois c'est une fonction membre.
La macro devrait etre déclarée dans le fichier ou elle est utilisée, et en utilisant le nom complet des arguments style game.mainwnd->GetDevice(), donc pas réutilisable et doit etre fait par l'utilisateur de la lib, c'est pas le but recherché.

En tout cas, __VA_ARGS__ n'aura d'utilité qu'avec une macro.

Si tu n'utilises pas le c++11, tu peux te rabattre sur quelque chose comme :

Code :

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

#define CreateRes(name, ...) std::make_shared<U>((name).GetDevice(), (name).GetDeviceContext(), __VA_ARGS__)

Si tu utilises c++11 (ce que tu devrais faire dans tous les cas), voici ce que tu peux faire :

Code :

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

1
2
3
4
template <typename U, typename... Args>
shared_ptr<U> CreateRes(Args &&... args) {
   return make_shared<U>(GetDevice(), GetDeviceContext(), std::forward<Args>(args)...);
}

[biyoudidou]

okay, merci, ca répond a ma question.