Discussion :

[valefor]

Bonjour.

Je dois coder un arbre avec possibilité de l'afficher.

Pour cela j'ai une classe de base A et une classe dérivée B, par exemple :

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
 
class A
{
protected:
  int m_val;
  A * m_ptr;
};
 
class B : public A
{
public:
  void draw(void)
  {
    A * ptr = m_ptr;
    ptr->m_val = 0;
  }
};

Le problème quand je compile est le suivant :

Code :

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

1
2
3
4
 
foo.cpp: In member function ‘void B::draw()’:
foo.cpp:6: error: ‘int A::m_val’ is protected
foo.cpp:16: error: within this context

C'est peut-être un problème de conception, mais j'aurai aimé garder quelque chose de simple et esthétique. Les solutions que j'envisage sont :

(1) déclarer une fonction draw friend dans A (pas estéthique à mon goùt).

(2) Caster m_ptr en B* au lieu de A* dans la fonction doSmth (c'est pas sécurisé du tout).

(3) rendre les membres de A public et faire un heritage privé (problème suivant le nombre de niveau d'héritage et je perds les fonctions publiques de A).

(4) Faire des accesseurs aux membres protected de A (beaucoup de travail mais certaienemt le plus élégant)

Je vous consulte pour savoir si vous voyez une meilleure conception ou une meilleure solution.

Merci d'avance.

[Aleph69]

Bonjour,

je partage ton avis : la solution la plus "propre" me semble être

(4) Faire des accesseurs aux membres protected de A (beaucoup de travail mais certaienemt le plus élégant)

Eventuellement, tu peux déclarer ces accesseurs comme des fonctions membres protégées.

Si tu veux gagner du temps, puisque la classe B hérite publiquement de A, peut-être peux-tu aussi déclarer la classe B comme amie de la classe A?

Code :

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

1
2
3
4
5
6
7
8
 
class A
{
protected:
  int m_val;
  A * m_ptr;
  friend class B;
};

L'un des problèmes de cette solution est que tu devras potentiellement le faire pour toutes les classes héritant de A ou B et qui veulent explicitement accéder à des données membres d'objets de type A.

[Médinoc]

Personnellement, pour accéder à une variable membre protected d'une instance autre que 'this' de la classe mère, j'utilise un accesseur statique dans celle-ci:
http://yaronet.com/posts.php?sl=&s=109391&p=1&h=11#11

[Aleph69]

Re,

@Médinoc, pour ma culture personnelle : pourquoi faire un accesseur statique? C'est plus rapide?

[Médinoc]

Il est statique, parce qu'on veut qu'il soit protected, qu'il a besoin d'accéder à autre chose que this, et qu'il n'a pas besoin d'accéder à this.

[Aleph69]

Re,

Il est statique, parce qu'on veut qu'il soit protected, qu'il a besoin d'accéder à autre chose que this, et qu'il n'a pas besoin d'accéder à this.

Ah oui d'accord!
Et c'est mieux que de faire un mutateur protégé?

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
 
class A
{
protected:
  int m_val;
  A * m_ptr;
  void setVal(int const& val)
  {
      m_val=val;
  }
};
 
class B : public A
{
public:
  void draw(void)
  {
    A * ptr = m_ptr;
    ptr->setVal(0);
  }
};

[Médinoc]

^Ça compile, ça?

Edit: Ben non.

Code :

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

error C2248: 'A::setVal' : cannot access protected member declared in class 'A'

[Aleph69]

Re,

ah bah non, je me suis retrouvé dans la même situation que valefor...

[valefor]

Merci pour vos commentaires ! Je vais devoir me fendre d'accesseurs, c'est po juste

[Aleph69]

Bonjour,

je reviens sur cette technique :

Personnellement, pour accéder à une variable membre protected d'une instance autre que 'this' de la classe mère, j'utilise un accesseur statique dans celle-ci:
http://yaronet.com/posts.php?sl=&s=109391&p=1&h=11#11

Je me demande si cela ne serait pas une bonne chose qu'elle apparaisse dans la faq?

[Médinoc]

Hmm, possible.