Discussion :

[shenron666]

Bonjour, je me permet de demander de l'aide pour un problème que je rencontre dans l'un de mes programmes

j'ai créé une classe de base, un objet général :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
class CObject
{
public:
      CUIObject(void):m_pParent(NULL) {}
      virtual ~CUIObject(void) {}
 
   protected:
      void SetParent(CUIObject* pParent) { m_pParent = pParent; }
 
   protected:
      CObject*   m_pParent;
};

et ensuite je déclare une classe dérivée :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
class CObjetComplexe : public CObject
{
   public:
      CObjetComplexe(void) {}
      virtual ~CObjetComplexe(void) {}
 
      void AddObjet(CObject* pObjet);
 
   protected:
      std::vector<CObject*>   m_arObjects;
};

dans CObjetComplexe je voudrais pouvoir indiquer à l'objet ajouté qui est son parent :

Code :

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

1
2
3
4
5
void CObjetComplexe::AddObjet(CObject* pObjet)
{
   m_arObjects.Add(pObjet);
   pObjet->SetParent(this);
}

mon problème est que je voudrais encapsuler la fonction SetParent dans CObject pour qu'elle ne soit pas accessible de l'extérieur mais dans AddObjet de CObjetComplexe j'ai une erreur en appelant SetParent si elle n'est pas publique

auriez-vous une solution pour que SetParent soit une méthode accessible par CObject et ses héritiers mais pas par les autres ?

[BertrandA]

Le design me paraît maladroit : CObjetComplexe est composé d'une collection d'objets CObject. L'affirmation "CObjetComplexe EST UN CObject" est fausse.
Conclusion : l'héritage n'est pas approprié ici.

mon problème est que je voudrais encapsuler la fonction SetParent dans CObject

C'est là qu'elle devrait être effectivement, comme membre public.

[shenron666]

Le design me paraît maladroit : CObjetComplexe est composé d'une collection d'objets CObject. L'affirmation "CObjetComplexe EST UN CObject" est fausse.
Conclusion : l'héritage n'est pas approprié ici.

Sachant que CObject est une base et que je vais créer des héritiers qui vont eux même contenir d'autres objets qui ne seront pas les mêmes je ne comprend pas comment je pourrai faire sans utiliser l'héritage

j'ai CWnd qui est un CObject dérivé auquel j'ai ajouté la possibilité d'ajouter d'autes objets
et j'ai CButton qui est un autre CObject dérivé qui n'aura pas la possibilité de contenir de CObjet

je pourrai ajouter des CWnd et des CButton à un objet CWnd mais pas à CButton
CObject a une méthode virtuelle pure appelée "Draw" qui est redéclarée par chaque dérivé et qui "indique" la façon dont le CObject va être dessiné

C'est là qu'elle devrait être effectivement, comme membre public.

c'est justement ce que je ne veut pas, je cherche à faire que SetParent ne soit utilisable que par CObject et ses héritiers

si je laisse SetParent en public dans CObject je n'ai pas de problème mais je voudrais justement que SetParent ne soit pas public tout en donnant la possibilité de pouvoir indiquer à un CObject (qui peut-etre un CWnd ou un CButton) qui est son parent

j'ai un peu de mal à m'expliquer, veuillez m'en excuser

[Laurent Gomila]

Et en mettant SetParent en protected ?

[shenron666]

Et en mettant SetParent en protected ?

j'obtient l'erreur suivante :
C2248: 'CObject::SetParent'*: impossible d'accéder à protected membre déclaré(e) dans la classe 'CObject'

et c'est à cause de l'appel : pObjet->SetParent(this) par un héritier

la seule solution que j'ai trouvé jusqu'à présent mais qui ne me convient pas (je sais je suis exigeant) c'est de mettre dans la section protected (SetParent de CObject est en protected aussi)
friend class CObject;
dans chaque héritier et de remplacer
pObjet->SetParent(this);
par
m_pParent = this;

si je ne trouve pas d'autre alternative je finirai par m'y plier

[Jan Rendek]

Ce que tu cherches à faire n'est pas prévu dans le language C++. La restriction d'accès au méthodes d'une classe ne se décline qu'en 3 niveaux : public, protected et private, avec des exceptions possibles (classes ou fonctions amies).
D'autres langages implantes ce que tu décris (Eiffel par exemple).
Dans ce cas présent, je laisserais la méthode publique. Un objet extérieur doit pouvoir fixer le père d'un CObject. Ce type de construction est utilisé dans la bibliotèque Qt (structure arborescente entre tous les composants graphiques) et les méthodes de rattachement sont publique.

[shenron666]

Merci à tous pour vos réponses, finalement je vais suivre mon idée première et donc le conseil de VoidSeer et laisser la méthode en public

je pense que c'est la meilleure solution surtout si après je veux pouvoir modifier le parent d'un objet sans trop de contraintes

[Rely]

la seule solution que j'ai trouvé jusqu'à présent mais qui ne me convient pas (je sais je suis exigeant) c'est de mettre dans la section protected (SetParent de CObject est en protected aussi)
friend class CObject;
dans chaque héritier et de remplacer
pObjet->SetParent(this);
par
m_pParent = this;

si je ne trouve pas d'autre alternative je finirai par m'y plier

tu peux aussi déclarer SetParent en static avec un paramètre supplémentaire (si tu trpouvess cela plus élégant)

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
 
class CUIObject 
{ 
public: 
      CUIObject(void):m_pParent(NULL) {} 
      virtual ~CUIObject(void) {} 
 
   protected: 
      static void SetParent(CUIObject* pObject, CUIObject* pParent) {
          pObject->m_pParent = pParent; 
      } 
 
   protected: 
      CObject*   m_pParent; 
};

En tout cas je trouve plus élégant!

[Rely]

Pour compléter mon dernier post j'ajouterai

Code :

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

1
2
3
4
5
6
 
void CObjetComplexe::AddObjet(CUIObject* pObjet) 
{ 
   m_arObjects.Add(pObjet); 
   SetParent(pObject, this); 
}

[shenron666]

Merci pour cette proposition intéressante
je me sens bête sur ce coup là de ne pas y avoir pensé