ich bekomme eine List von Objekte Pointers, alle von denen von einer Basisklasse ableiten:

std::vector<Base*> obj_list;

template<unsigned int N>
class A<N> : public Base
{
public:
    int do_thing_1();
};

template<unsigned int N>
class B<N> : public Base
{
public:
    int do_thing_2();
};

Ich muss die richtige Funktion do_thing_1() oder do_thing_2() aufrufen, je nach dem welche Subklasse sie ist.

class MyClass
{
public:
void func(std::vector<Base*> &obj_list)
{
  for (std::vector<Base*>::iterator it=obj_list.begin(); it!=obj_list.end(); ++it) {
    if (dynamic_cast<A<1>*>(*it)) {
      operate(dynamic_cast<A<1>*>(*it));
    }
    else if (dynamic_cast<A<2>*>(*it)) {
      operate(dynamic_cast<A<2>*>(*it));
    }
    else if (dynamic_cast<A<3>*>(*it)) {
      operate(dynamic_cast<A<3>*>(*it));
    }
    else if (dynamic_cast<A<4>*>(*it)) {
      operate(dynamic_cast<A<4>*>(*it));
    }
    else if (dynamic_cast<A<5>*>(*it)) {
      operate(dynamic_cast<A<5>*>(*it));
    }
    else if (dynamic_cast<A<6>*>(*it)) {
      operate(dynamic_cast<A<6>*>(*it));
    }
    else if (dynamic_cast<A<7>*>(*it)) {
      operate(dynamic_cast<A<7>*>(*it));
    }
    ...
    else if (dynamic_cast<B<1>*>(*it)) {
      operate(dynamic_cast<B<1>*>(*it));
    }
    else if (dynamic_cast<B<2>*>(*it)) {
      operate(dynamic_cast<B<2>*>(*it));
    }
    else if (dynamic_cast<B<3>*>(*it)) {
      operate(dynamic_cast<B<3>*>(*it));
    }
    else if (dynamic_cast<B<4>*>(*it)) {
      operate(dynamic_cast<B<4>*>(*it));
    }
    else if (dynamic_cast<B<5>*>(*it)) {
      operate(dynamic_cast<B<5>*>(*it));
    }
    else if (dynamic_cast<B<6>*>(*it)) {
      operate(dynamic_cast<B<6>*>(*it));
    }
    else if (dynamic_cast<B<7>*>(*it)) {
      operate(dynamic_cast<B<7>*>(*it));
    }
    ...
  }
}

private:

tempate<unsigned int N>
void operate(A<N> *obj)
{
  obj->do_thing_1();
}

tempate<unsigned int N>
void operate(B<N> *obj)
{
  obj->do_thing_2();
}

}; // MyClass

Ich muss nach allen möglichen Zahlen fragen, damit die Base Pointer richtig gecastet werden kann. Das ist fast unmöglich, 2^32 Möglichkeiten aufzulisten.

Gibt es bessere Methode, diesen Code intelligenter zu schreiben?

Die Klasse A<N>, B<N>, Base sind vorgegeben. Ich darf sie nicht ändern.

Ich darf auch kein boost Library verwenden.

Vielen Dank für den Tipps vorab.

Kai

Gibt es bessere Methode, diesen Code intelligenter zu schreiben?

Nein. Der Code ist nicht zu retten.

Benutze einfach virtuelle Funktionen. Nichts anders versuchst du gerade nachzubauen...

Danke. Ich verstehe nicht ganz. Wie kann ich virtuelle Funktionen dafür verwenden, um alle möglichen dynamic_cast<...> zu sparen?

Gruß,

Kai

Kannst du wenigstens obj_list ändern?

Die Anforderung ist: meine Klasse muss die public Funktion

void func(std::vector<Base*> &obj_list);

anbieten, um andere sie aufrufen zu können.

Deshalb darf ich obj_list nicht ändern.

Gruß,

Kai

Benutze einfach virtuelle Funktionen. Nichts anders versuchst du gerade nachzubauen...

jo, die do_thing_1() und do_thing_1() virtual in die Base und alles ist gut.
geht supi schnell.

das template geht auch, aber das tut dann nicht mehr als 2^32 ifs hintereinander. nöö, das wird nicht gut. nichtmal rtti würde viel bringen. da müßte die basis auch virtuell sein, ach ist sie ja eh schon. und das muss sie eh, wenn du die objekte in einer gemeinsamen liste verwaltest.

undokumentiert, undefiniert, nur für böse buben könntest du den vptr anschauen und eine hashtable benutzen. mit dem erfolg, daß du zwar ne gescheit laufzeit hast, aber nur für den potenziellen fall eines aufrufs mal die 2^32 klassen erzeugst.

musst die basis ändern und gegebenefalls eben den chef davon überzeugen.

Virtual foo schrieb:

Benutze einfach virtuelle Funktionen. Nichts anders versuchst du gerade nachzubauen...

Die Klasse A<N>, B<N>, Base sind vorgegeben. Ich darf sie nicht ändern.

Edit: Ach, Volkard redet vom Chef-Überzeugen, hab' ich übersehen.

Die Anforderung ist: meine Klasse muss die public Funktion

Bitte die vollständige Anforderung schreiben. Immer wieder einen halben Satz zu kriegen, der alles umwirft, das ist nicht fein. Wie genau sieht die Base aus?

So halte ich die AUfgabe für Quatsch? Übungsaufgabe vom Prof? Dann fehlt mehr Kontext, um zu ahnen, auf was er hinauswill.
Aufgabe vom betrunkenen Chef? Dann wirste sie ändern können müssen.

tjroamer schrieb:

Die Anforderung ist: meine Klasse muss die public Funktion

Bitte die vollständige Anforderung schreiben. Immer wieder einen halben Satz zu kriegen, der alles umwirft, das ist nicht fein.

Wovon redest du?

tjroamer schrieb:

Die Anforderung ist: meine Klasse muss die public Funktion [...] func(std::vector<Base*> &obj_list)[...] anbieten, um andere sie aufrufen zu können.

Virtual foo schrieb:

Benutze einfach virtuelle Funktionen. Nichts anders versuchst du gerade nachzubauen...

jo, die do_thing_1() und do_thing_1() virtual in die Base und alles ist gut.
geht supi schnell.

das template geht auch, aber das tut dann nicht mehr als 2^32 ifs hintereinander. nöö, das wird nicht gut. nichtmal rtti würde viel bringen. da müßte die basis auch virtuell sein, ach ist sie ja eh schon. und das muss sie eh, wenn du die objekte in einer gemeinsamen liste verwaltest.

undokumentiert, undefiniert, nur für böse buben könntest du den vptr anschauen und eine hashtable benutzen. mit dem erfolg, daß du zwar ne gescheit laufzeit hast, aber nur für den potenziellen fall eines aufrufs mal die 2^32 klassen erzeugst.

musst die basis ändern und gegebenefalls eben den chef davon überzeugen.

die Klassen A<N>, B<N> und Base kommen von einem Library. do_thing_1() und do_thing_2 sind leider keine virtuelle Funktion in Base(). Die Base Klasse hat andere Funktionalität, die hier zu meinem Zweck nicht dargestellt sind.

Es sieht so aus, dass ich 2^32 Mal schreiben muss, um alle möglichen Fälle abzudecken. Schade!!

Trotzdem vielen Dank.

/Kai

Übungsaufgabe vom Prof? Dann fehlt mehr Kontext, um zu ahnen, auf was er hinauswill.

Wenns von einem Prof käme könnte der vielleicht einen vptr-Hack erwarten. Standardkonform geht das jedenfalls nur mit den 2^17 ifs (weil unsigned nur von 0 bis 2^16 gehen muss).

@tjroamer: Wenn du wenigstens die Signatur in std::vector<std::shared_ptr<thing_dispatcher*>> ändern könntest, hättest du noch eine kleine Chance (sofern du die Erstellung der Liste beeinflussen kannst).

Es sieht so aus, dass ich 2^32 Mal schreiben muss, um alle möglichen Fälle abzudecken.

Ach nein, da ist ein Troll am Werk. Hat sich erledigt.

volkard schrieb:

tjroamer schrieb:

Die Anforderung ist: meine Klasse muss die public Funktion

Bitte die vollständige Anforderung schreiben. Immer wieder einen halben Satz zu kriegen, der alles umwirft, das ist nicht fein.

Wovon redest du?

tjroamer schrieb:

Die Anforderung ist: meine Klasse muss die public Funktion [...] func(std::vector<Base*> &obj_list)[...] anbieten, um andere sie aufrufen zu können.

Sorry, vielleicht habe ich nicht klar geschrieben. Jetzt ist die konkrete Anforderung:

**Meine Klasse muss den Usern die Funktion

void func(std::vector<Base*> *obj_list);

zur Verfügung stellen. Die Funktion muss je nach der Subklasse do_thing_1() oder do_thing_2() richtig aufrufen.
**

Gruß,

Kai

Sone schrieb:

volkard schrieb:

tjroamer schrieb:

Die Anforderung ist: meine Klasse muss die public Funktion

Bitte die vollständige Anforderung schreiben. Immer wieder einen halben Satz zu kriegen, der alles umwirft, das ist nicht fein.

Wovon redest du?

tjroamer schrieb:

Die Anforderung ist: meine Klasse muss die public Funktion [...] func(std::vector<Base*> &obj_list)[...] anbieten, um andere sie aufrufen zu können.

Sorry, vielleicht habe ich nicht klar geschrieben. Jetzt ist die konkrete Anforderung:

**Meine Klasse muss den Usern die Funktion

void func(std::vector<Base*> *obj_list);

zur Verfügung stellen. Die Funktion muss je nach der Subklasse do_thing_1() oder do_thing_2() richtig aufrufen.
**

Gruß,

Kai

Süßer, wir haben den Code von Base immer noch nicht. Sind wenigstens do_thing_1 und do_thing_2 dort virtuell? Dann würde doch RTTI reichen.

volkard schrieb:

Übungsaufgabe vom Prof? Dann fehlt mehr Kontext, um zu ahnen, auf was er hinauswill.

Wenns von einem Prof käme könnte der vielleicht einen vptr-Hack erwarten. Standardkonform geht das jedenfalls nur mit den 2^17 ifs (weil unsigned nur von 0 bis 2^16 gehen muss).

@tjroamer: Wenn du wenigstens die Signatur in std::vector<std::shared_ptr<thing_dispatcher*>> ändern könntest, hättest du noch eine kleine Chance (sofern du die Erstellung der Liste beeinflussen kannst).

Es sieht so aus, dass ich 2^32 Mal schreiben muss, um alle möglichen Fälle abzudecken.

Ach nein, da ist ein Troll am Werk. Hat sich erledigt.

ich denke, ich muss aufgeben, der Parameter std::vector<Base*> ist leider nicht änderbar. Vielen Dank an euch allen für eure Zeit und gute Nacht!

Gruß,

Kai

tjroamer schrieb:

Sone schrieb:

volkard schrieb:

tjroamer schrieb:

Die Anforderung ist: meine Klasse muss die public Funktion

Bitte die vollständige Anforderung schreiben. Immer wieder einen halben Satz zu kriegen, der alles umwirft, das ist nicht fein.

Wovon redest du?

tjroamer schrieb:

Die Anforderung ist: meine Klasse muss die public Funktion [...] func(std::vector<Base*> &obj_list)[...] anbieten, um andere sie aufrufen zu können.

Sorry, vielleicht habe ich nicht klar geschrieben. Jetzt ist die konkrete Anforderung:

**Meine Klasse muss den Usern die Funktion

void func(std::vector<Base*> *obj_list);

zur Verfügung stellen. Die Funktion muss je nach der Subklasse do_thing_1() oder do_thing_2() richtig aufrufen.
**

Gruß,

Kai

Süßer, wir haben den Code von Base immer noch nicht. Sind wenigstens do_thing_1 und do_thing_2 dort virtuell? Dann würde doch RTTI reichen.

die Klasse Base und A<N>, B<N> sehen so aus:

class Base
{
public:
  Base() {}
  virtual const char* kind();
  virtual const char* name();
  virtual ~Base();
};

template<unsigned int N>
class A: public Base
{
public:
  A() {}
  void do_thing_1()
  {
    // do thing_1 depending on the N
  }

  virtual const char* kind() { return "A"; }
  virtual const char* name() { return "A"; }
};

template<unsigned int N>
class B: public Base
{
public:
  B() {}
  void do_thing_2()
  {
    // do thing_2 depending on the N
  }

  virtual const char* kind() { return "B"; }
  virtual const char* name() { return "B"; }
};

do_thing_1() und do_thing_2() sind keine virtual

Übrigens, die sind so klein? In dem Fall sei doch ein ganz Böser Bezlebub' und schreib die zwei Zeilen da rein - ich nehme die Verantwortung auf mich:

virtual void do_thing_1() = 0;
virtual void do_thing_2() = 0;

Aber Hase! Da sind ja weder do_thing_1 oder ~_2 in der Basisklasse deklariert, geschweige denn virtuell!

Übrigens, die sind so klein? In dem Fall sei doch ein ganz Böser Bezlebub' und schreib die zwei Zeilen da rein - ich nehme die Verantwortung auf mich:

virtual void do_thing_1() = 0;
virtual void do_thing_2() = 0;

anscheinend gibt es keinen anderen Weg. OK, ich versuche auf diesen Weg zu gehen. Vielen Dank.

Gruß,

Kai

Das ist fast unmöglich, 2^32 Möglichkeiten aufzulisten.

Das ist nicht dein Problem. Dein Problem ist, dass du 2^32+ verschiedene Funktionen hast.

tjroamer schrieb:

Das ist fast unmöglich, 2^32 Möglichkeiten aufzulisten.

Das ist nicht dein Problem. Dein Problem ist, dass du 2^32+ verschiedene Funktionen hast.

Fast ein wenig ironisch - Stroustrup schreibt noch, Templates sind gut, um den Maschinencode zu verkürzen
Edit: Oder so ähnlich.

tjroamer schrieb:

Das ist fast unmöglich, 2^32 Möglichkeiten aufzulisten.

Das ist nicht dein Problem. Dein Problem ist, dass du 2^32+ verschiedene Funktionen hast.

die Funktion operate() ist als template definiert. Hier ist kein Problem. Der User kann folgenden Code schreiben:

A<13> a_13();
B<1000> b_1000();

std::vector<Base*> obj_list;
obj_list.push_back(&a_13);
obj_list.push_back(&b_1000);

MyClass myclass();

myclass.func(obj_list);

MyClass muss solchen Code beherrschen. Das ist mein Problem.

Gruß,

/Kai

A<13> a_13();//er wird
B<1000> b_1000();//sich wundern

std::vector<Base*> obj_list;
obj_list.push_back(&a_13);//hier
obj_list.push_back(&b_1000);

MyClass myclass();

myclass.func(obj_list);

gibt es hier einen Tippfehler oder wie?

Gruß,

Kai

Edit:

tjroamer schrieb:

volkard schrieb:

A<13> a_13();//er wird
B<1000> b_1000();//sich wundern

std::vector<Base*> obj_list;
obj_list.push_back(&a_13);//hier
obj_list.push_back(&b_1000);

MyClass myclass();

myclass.func(obj_list);

gibt es hier einen Tippfehler oder wie?

Das sind Funktionsdeklarationen.
Guckstdu.

Ich sehe es sofort. :stolz:

Edit:

tjroamer schrieb:

volkard schrieb:

A<13> a_13();//er wird
B<1000> b_1000();//sich wundern

std::vector<Base*> obj_list;
obj_list.push_back(&a_13);//hier
obj_list.push_back(&b_1000);

MyClass myclass();

myclass.func(obj_list);

gibt es hier einen Tippfehler oder wie?

Das sind Funktionsdeklarationen.
Guckstdu.

OK. Wahrscheinlich habe ich zu einfach geschrieben. Ich wollte schreiben, a_13 und b_1000 sind zwei Klassenobjekten.

sollte sein:

A<13> a_13;
B<1000> b_1000;

Gruß,

/Kai