Discussion :

[lpa]

Bonjour,
J'ai un soucie avec mon code: J'ai une class template ( ThirdPersonCamera ) qui est écrite dans un .h.
Cette class possède une fonction addToScene() qui n'utilise pas le template mais qui utilise un objet Scene que je ne peux pas inclure dans le ThirdPersonCamera.h car cela cause une inclusion cyclique du faite que Scene.h inclu deja ThirdPersonCamera.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
ThirdPersonCamera.h
       class Scene;
       template <class T>
	class ThirdPersonCamera : public Camera
	{
		public:
            ThirdPersonCamera(std::string name, Scene* scene, T* target)
            {
                init(name, scene, Vector3(50,10,50), target->getPosition());
                mTarget = target;
 
            }
 
            ~ThirdPersonCamera()
            {
 
            }
 
            void syncRenderablesPhysics(void)
            {
 
            }
		protected:
                  void addToScene(Scene* scene)
                 {
                      scene->addActor<ThirdPersonCamera>(this);
                  }
 
		private:
            T* mTarget;
 
	}
error: invalid use of incomplete type ‘struct Scene’|

j'ai essayé de mettre dans mon cpp:

Code :

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

1
2
3
4
5
6
7
 #include "ThirdPersonCamera.h"
#include "Scene.h"
           void ThirdPersonCamera::addToScene(Scene* scene)
            {
                scene->addActor<ThirdPersonCamera>(this);
            }
/error: ‘template<class T> class ThirdPersonCamera’ used without template parameters|

Code :

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

1
2
3
4
5
6
7
8
9
10
#include "ThirdPersonCamera.h"
#include "Scene.h"
 
    template <class T>
    void ThirdPersonCamera<T>::addToScene(Scene* scene)
    {
        scene->addActor<ThirdPersonCamera>(this);
    }
]+0x10)||undefined reference to `
ThirdPersonCamera<AnimatedMesh>::addToScene(Scene*)'|

Si quelqu'un a une idée
merci

[oodini]

Essaye la déclaration anticipée (forward declaration) de ta classe Scene.

[lpa]

Essaye la déclaration anticipée (forward declaration) de ta classe Scene.

Merci de ta réponse, j'avais oublié de le mettre c'est ce que je fait deja, mais du faite d’appeler la fonction addToScene() de cette objet j'ai l'erreur invalid use of incomplete type ‘struct Scene’|

[Emmanuel Deloget]

Merci de ta réponse, j'avais oublié de le mettre c'est ce que je fait deja, mais du faite d’appeler la fonction addToScene() de cette objet j'ai l'erreur invalid use of incomplete type ‘struct Scene’|

Laisse la déclaration de addToScene() en place, et déplace sa définition sous la définition de la classe Scene, en dehors du template (comme tu le ferais pour une classe normale).

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
 
class Scene;
 
template <class T>
class ThirdPersonCamera : public Camera
{
   ...
protected:
    void addToScene(Scene* scene);
   ...
};
 
class Scene
{
...
};
 
template <class T>
void ThirdPersonCamera<T>::addToScene(Scene *scene)
{
    scene->addActor<ThirdPersonCamera>(this);
}

L'autre solution consiste à essayer de se débarrasser de cette référence circulaire : c'est un couplage très fort, trop fort probablement. D'ailleurs, j'ai du mal à comprendre pourquoi la classe Scene doit connaître ThirdPersonCamera.

[lpa]

Laisse la déclaration de addToScene() en place, et déplace sa définition sous la définition de la classe Scene, en dehors du template (comme tu le ferais pour une classe normale).

Je n'est pas bien compris ce que tu veux me dire, je laisse la déclaration dans le fichier ThirdPersonCamera.h et je met la définition dans le fichier ThirdPersonCamera.cpp dans le quel je fait un include "Scene.h" ?
Si c'est de cela que tu parle j'ai déjà essayé cela me donne "error: ‘template<class T> class ThirdPersonCamera’ used without template parameters"

L'autre solution consiste à essayer de se débarrasser de cette référence circulaire : c'est un couplage très fort, trop fort probablement. D'ailleurs, j'ai du mal à comprendre pourquoi la classe Scene doit connaître ThirdPersonCamera.

En effet j'y est pensé mais cela me parait assez difficile a faire car Scene.h a beaucoup de dépendance qui elle même peuvent faire appelle a ThirdPersonCamera.h mais c'est surement ce que je vais faire si je ne trouve pas d'autre solution.
merci

[koala01]

Salut,

De manière générale, je constate que ThirdPersonCamera a beau etre une classe template, elle hérite, avant tout, d'une classe (non template celle-là) Camera...

Or, ta scene devrait sans doute pouvoir travailler avec n'importe quel type de caméra, et tu devrais donc pouvoir t'en sortir avec l'inclusion de... camera.h!!!

il n'y aurait donc qu'à t'assurer que camea fournit une interface suffisante pour pouvoir manoeuvrer ta ThirdPersonCamera

[lpa]

Salut,

De manière générale, je constate que ThirdPersonCamera a beau etre une classe template, elle hérite, avant tout, d'une classe (non template celle-là) Camera...

Or, ta scene devrait sans doute pouvoir travailler avec n'importe quel type de caméra, et tu devrais donc pouvoir t'en sortir avec l'inclusion de... camera.h!!!

il n'y aurait donc qu'à t'assurer que camea fournit une interface suffisante pour pouvoir manoeuvrer ta ThirdPersonCamera

Pour le moment j'ai réussi a retravailler la scene pour supprimer sa dépendance a toute camera, mais ce qui me fait un peu peur c'est que scene a pas mal de dépendance et que je ne peu plus, du coup lui ajouter de dépendance qui possèdent elle meme une dépendance envers la caméra

[koala01]

Pour le moment j'ai réussi a retravailler la scene pour supprimer sa dépendance a toute camera, mais ce qui me fait un peu peur c'est que scene a pas mal de dépendance et que je ne peu plus, du coup lui ajouter de dépendance qui possèdent elle meme une dépendance envers la caméra

De toutes manières, j'aurais vraiment tendance à penser que, si scene a vraiment autant de dépendances que ce que tu semble dire, c'est qu'il y a déjà, à la base, un sérieux problème de conception

Tu devrais, peut etre, essayer de réfléchir au fait que tu en demande beaucoup trop à ta scene, et, pourquoi pas, te rappeler quelques principes de base tels que la fameuse loi demeter ( si une classe B utilise une classe C et qu'une classe A utilise la classe B, il n'y a, a priori, aucune raison pour que la classe A aie ne serait ce qeu connaissance de l'existence de la classe C ) ou le principe de la responsabilité unique (si une classe ou une fonction a plus d'une responsabilité, c'est sans doute qu'elle en a trop ) !

Je ne sais pas quelle forme prend ton projet, mais, de manière générale, la scene devrait se "contenter" de gérer la (les) vues et de leur permettre, le cas échéant, de communiquer entre elles et ce sont les vues qui devraient se charger de la gestion des caméra.

Et, si tu rajoute du son ou de l'intelligence artificielle, cela devrait etre géré "plus haut" (au niveau de la partie ou de l'application, par exemple )

[lpa]

De toutes manières, j'aurais vraiment tendance à penser que, si scene a vraiment autant de dépendances que ce que tu semble dire, c'est qu'il y a déjà, à la base, un sérieux problème de conception

Tu devrais, peut etre, essayer de réfléchir au fait que tu en demande beaucoup trop à ta scene, et, pourquoi pas, te rappeler quelques principes de base tels que la fameuse loi demeter ( si une classe B utilise une classe C et qu'une classe A utilise la classe B, il n'y a, a priori, aucune raison pour que la classe A aie ne serait ce qeu connaissance de l'existence de la classe C ) ou le principe de la responsabilité unique (si une classe ou une fonction a plus d'une responsabilité, c'est sans doute qu'elle en a trop ) !

Je ne sais pas quelle forme prend ton projet, mais, de manière générale, la scene devrait se "contenter" de gérer la (les) vues et de leur permettre, le cas échéant, de communiquer entre elles et ce sont les vues qui devraient se charger de la gestion des caméra.

Et, si tu rajoute du son ou de l'intelligence artificielle, cela devrait etre géré "plus haut" (au niveau de la partie ou de l'application, par exemple )

Merci pour ces conseils j'ai encore du mal avec l'architecture, je vais essayer de suivre ce que tu me dit et allégé la scene au maximum.