Problème pour enregistrer des instances de classes dérivées dans une liste

Version imprimable

22/10/2017, 10h50
Thoma HauC

Problème pour enregistrer des instances de classes dérivées dans une liste
Bonjour,

Je reprends le C++ (ESP8266 dans l'environnement Arduino) après ne l'avoir pas pratiquer pendant une décennie et je me retrouve confronter à un problème pour lequel je ne trouve pas de solution.
Alors voilà, j'ai une classe A dont le constructeur est 'private' (pour ne pas pourvoir l'instancier) et une méthode est 'virtual' (qui sera appeler depuis une 'list' d'instance héritant de l'objet A).
Comme ceci :
Code:

1 2 3 4 5 6 7 class A { protected: A(); public: virtual String method(); };
La classe B dérive de A :
Code:

1 2 3 4 5 6 class B : public A { public: B(); String method(); };
La classe C dérive de A aussi :
Code:

1 2 3 4 5 6 class C : public A { public: C(); String method(); };
Trois instanciations des deux objets ainsi que la liste qui va stocker les instances.
Code:

1 2 3 4 B inst0(); B inst1(); C inst2(); std::list<A> liste;
ET un bout de code pour appeler 'method' pour chaque instance stockée dans la 'list'.
Code:

1 2 3 4 5 6 7 8 9 10 int main(int argc, char ** argv) { String temp; for (std::list<A>::iterator it = liste.begin(); it != liste.end(); ++it) { temp += it->method(); } }
Cela fonctionne très bien.

Maintenant, si j'ajoute dans le 'main' avant la boucle le code ci-dessous :
Code:

1 2 3 liste.push_back(inst0); liste.push_back(inst1); liste.push_back(inst2);
Le compilateur me sort une erreur "no matching function for call to 'std::list<A>::push_back(B (&)())'".
Ce qui se comprend mais comment on stocke différentes instances de classes dérivées dans une liste unique typée classe mère.

Si quelqu'un peut m'éclairer voire m'aider à trouver une solution, cela serait formidable.

D'avance merci

Thoma
22/10/2017, 16h39
oodini

Il faut un vecteur de pointeurs vers A, ou, à partir du C++11, un vecteur de std::ref<A>.

Bonjour oodini,

Effectivement, avec une liste de A * cela fonctionne. Merci.

Code:

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
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
#include <list>
 
class A
{
  protected:
    A();
  public:
    virtual String method();
};
 
class B : public A
{
  public:
    B();
    String method();
};
 
class C : public A
{
  public:
    C();
    String method();
};
 
A::A()
{
}
 
B::B() : A()
{
}
 
String B::method()
{
  return "";
}
 
C::C() : A()
{
}
 
String C::method()
{
  return "";
}
 
int main(int argc, char ** argv)
{
  String page;
  std::list<A *> liste_instances;
 
  B * inst0 = new B();
  B * inst1 = new B();
  C * inst2 = new C();
 
  liste_instances.push_back(inst0);
  liste_instances.push_back(inst1);
  liste_instances.push_back(inst2);
 
  for (std::list<A *>::iterator it = liste_instances.begin(); it != liste_instances.end(); ++it)
  {
    page += (*it)->method();
  }
 
  return 0;
}

Maintenant, où dois-je intégrer le std::ref<> de functional ?
Je supposais placer le std:ref<A>() au niveau des 'push_back' mais sans résultat, j'obtiens un "undefined reference to `vtable for A'".

Code:

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
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
#include <functional>
#include <list>
 
class A
{
  protected:
    A();
  public:
    virtual String method();
};
 
class B : public A
{
  public:
    B();
    String method();
};
 
class C : public A
{
  public:
    C();
    String method();
};
 
A::A()
{
}
 
B::B() : A()
{
}
 
String B::method()
{
  return "";
}
 
C::C() : A()
{
}
 
String C::method()
{
  return "";
}
 
int main(int argc, char ** argv)
{
  String page;
  B inst0();
  B inst1();
  C inst2();
  std::list<A> liste_instances;
 
  liste_instances.push_back(std:ref<A>(inst0));
  liste_instances.push_back(std:ref<A>(inst1));
  liste_instances.push_back(std:ref<A>(inst2));
 
  for (std::list<A>::iterator it = liste_instances.begin(); it != liste_instances.end(); ++it)
  {
    page += it->method();
  }
 
  return 0;
}

Citation:

Envoyé par Thoma HauC

Maintenant, où dois-je intégrer le std::ref<> de functional ?
Je supposais placer le std:ref<A>() au niveau des 'push_back' mais sans résultat, j'obtiens un "undefined reference to `vtable for A'".

C'est parce que ta méthode n'est pas définie pour A (si tu n'en pas besoin, déclare la en virtuelle pure).

Ci-dessous figure une code fonctionnel.
Je t'ai par ailleurs mis une autre manière d'itérer sur ton vecteur, de loin préférable à la première (dont le type de l'initialisateur devrait en outre être auto).
Code:

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 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 #include <functional> #include <iostream> #include <list> #include <string> using String = std::string; class A { protected: A() {}; public: virtual String method() const { return "A"; } }; class B : public A { public: B() {}; virtual String method() const { return "B"; } }; class C : public A { public: C() {}; virtual String method() const { return "C"; } }; int main() { String page; B inst0; B inst1; C inst2; std::list<std::reference_wrapper<A>> liste_instances; liste_instances.push_back(std::ref<A>(inst0)); liste_instances.push_back(std::ref<A>(inst1)); liste_instances.push_back(std::ref<A>(inst2)); /*----------------------------------------------------------------------------------------------------------------*/ for (std::list<std::reference_wrapper<A>>::const_iterator it = liste_instances.cbegin(); it != liste_instances.cend(); ++it) page += it->get().method(); std::cout << page << std::endl; page.clear(); /*----------------------------------------------------------------------------------------------------------------*/ for (const A& a: liste_instances) page += a.method(); std::cout << page << std::endl; return 0; } }
Le deuxième for étant préférable au premier...

24/10/2017, 16h32
Thoma HauC

Bonjour oodini,

Merci pour l'exemple, j'ai pu le transposer à mon cas.
Non sans difficulté car certains points sont encore un peu flou.

Thoma