C++ Einfügen einer Klasse in einem map-container

Ich habe eine map in C++ und ich möchte Eingangs meiner Klasse als Wert und einen string als Schlüssel.
Wenn ich versuche, bekomme ich eine Fehlermeldung 'Scene_Branding' : illegal use of this type as an expression
Ich bekomme eine ungültige Verwendung dieses Typs als Ausdruck, und ich kann nicht scheinen, um herauszufinden, warum. Hier ist etwas code.

 string CurrentScene = "Scene_Branding";
 map<string, Scene> Scenes;
 Scenes.insert(std::make_pair("Scene_Branding", Scene_Branding));  //<-- Illegal Error parameter 2

und hier ist die Szene Branding header..

#ifndef Scene_Branding_H
#define Scene_Branding_H

#include "Scene.h"
#include <iostream>
#include <string>


class Scene_Branding : Scene
{
public:
 Scene_Branding();
 ~Scene_Branding();
 void Draw();
};

#endif

und hier ist die Szene header.. 

#ifndef Scene_H
#define Scene_H

#include <iostream>
#include <string>

class Scene
{
public:
 Scene();
 ~Scene();
 virtual void Draw();

};

#endif

und hier ist es-cpp-Dateien.

Szene cpp.

#include "Scene.h"

Scene::Scene()
{

}
Scene::~Scene()
{

}
void Scene::Draw(){
 std::cout << "Hey";
}

Scene_Branding cpp

#include "Scene_Branding.h"

Scene_Branding::Scene_Branding()
{

}

Scene_Branding::~Scene_Branding()
{

}

void Scene_Branding::Draw()
{
 std::cout << "Drawing from Scene_branding";
}
  • beide Antworten sind richtig, darauf hinzuweisen, dass, müssen Sie eine Instanz der Scene_Branding nicht nur der Typ. ein Hinweis darauf, machen Sie Ihre Basisklasse destructor virtual
  • Meinst du das wirklich private Vererbung?
InformationsquelleAutor Fouf | 2010-02-17
  1. 22

    Ersten, nicht speichern Sie die Objekte selbst in der Karte, speichern Zeiger auf deine Objekte.

    Zweitens, müssen Sie eine Instanz von Scene_Branding std::make_pair, nicht die Klasse selbst.

    BEARBEITEN:
    Hier ist, wie Sie gehen über die Speicherung von Zeigern:

     string CurrentScene = "Scene_Branding";
 map<string, Scene*> Scenes;
 Scenes.insert(std::make_pair("Scene_Branding", new Scene_Branding()));

    Aber, da Sie aufgefordert, diese Art von Frage, ich empfehle Ihnen, Lesen Sie ein gutes c++ Buch für weitere greifen Konzepte wie Zeiger.

    • Das speichern von Objekten ist in Ordnung, solange das Objekt der Klasse final ist.
    • Aha, ich habe versucht, aber ich war irrtümlich der Platzierung Scene_Branding .. trotzdem, danke.
    • könnten Sie ein wenig zu klären. Was bedeutet es, "solange die Objekt-Klasse ist endgültig? Du meinst Schlüsselwort? Und warum diese Letzte tun werden?
    • bedeutet, es gibt keine Unterklassen, um die Scene Klasse. Wenn es Unterklassen dann mit map<string, Scene> verursachen object-slicing. Eine Finale Klasse verhindert diese situation.
    InformationsquelleAutor erelender
  2. 7

    Versuchen:

    Scenes.insert(std::make_pair("Scene_Branding", Scene_Branding()));
    • Mich zu schlagen!!
    • Dies funktioniert, aber der code ist so nicht gedacht und cout ist, ausgeben Hey, und nicht die Zeichnung aus Scene_Branding.. hmm aber danke für die Korrektur.
    • Dies ist, weil eine Instanz der Szene in der Karte gespeichert werden, nicht eine Instanz der Scene_Branding. Dies wird als Objekt schneiden, kurz, es bewirkt, dass Sie verlieren Informationen, die erklärt, in Scene_Branding. Speichern von Zeigern wird nicht dieses Problem verursachen.
    • Wie würde ich mich über die Rückgabe ein pointer? Ich glaube, ich bekomme nur Fehler wie error C2664: '- Szene: Szene(const Szene &)' : cannot convert parameter 1 from 'Scene_Branding (__cdecl *const )(void)' in 'const Szene &'
    • -1 weil der slicing-Fehler. Ich denke, dass Sie nicht sehen, dass der OP will einen container von polymorphen Objekten. 🙂
    InformationsquelleAutor Alexander Gessler
  3. 3

    Denke ich, dass Sie nicht wollen, zu tun.

    1. es gibt keine runtime-type-mapping in C++, Sie Objekte speichern, nicht Arten.
    2. nicht speichern polymorphe Typen in STL-Containern, verwenden Sie boost::ptr_map statt, wenn es euer Wunsch ist

    So, der "neue" code:

    class Scene
{
public:
  virtual ~Scene();                  //Virtual Destructor, it's a base class
  virtual Scene* clone() const = 0;  //Polymorphic construction
private:
  //whatever you wish
};

class Scene_Branding: public Scene
{
public:
  virtual Scene_Branding* clone() const { return new Scene_Branding(); }
};

    Und die neue Art zu speichern diese:

    const std::string SceneBrandingKey = "Scene_Branding";

typedef boost::ptr_map<std::string, Scene> scenes_type;

scenes_type Scenes;
Scenes.insert(SceneBrandingKey, new Scene_Branding());

    Und Sie können es verwenden, wie folgt:

    Scenes["Scene_Branding"].process(); //Note: use '.' not '->'

    Die nette Sache über Boost-Zeiger-Container ist, dass es wurde gedacht, für polymorphe Typen, mit Ausnahme von Sicherheit und allem, und doch imitiert das Verhalten bzw. die Schnittstelle der STL, so dass Sie nicht verloren gehen 🙂

    • Ich denke, dass OP ein wenig newbie zu sein, Tauchen Sie ein in steigern noch.
    • Ich bin mir nicht sicher, aber IMHO nicht um virtuelle, sondern Scene_Branding* clone() const sollte es virtuellen Szene* clone() const 😉
    • Es ist lustig, wie jeder denkt, dass C++ - Neulinge lernen müssen, Dinge zu tun, die die fehleranfällige manuelle Weg, bevor die Verwendung von Bibliotheken, die das Leben leichter machen.
    • +1 Nach der OP lernt über die Tücken der manuellen Speicherverwaltung, er/Sie wird Sie zu schätzen wissen shared_ptr und ptr_container.
    • Natürlich, die Dinge, die das Leben leichter machen, sind aber besser, um high-level " - Sachen effizient, verstehen muss man das low-level Zeug, denkst du nicht? Nicht einen lamborghini für einen neuen Treiber.
    • Nein, es ist richtig, weil ein Scene_Branding* ist umwandelbar zu einem Scene* (es heißt kovariante Rückgabe). Es bedeutet, dass durch die Anrufung des virtual - Methode erhalten Sie eine Scene* aber wenn du explizit die Scene_Branding::clone dann erhalten Sie eine Scene_Branding* > wenn Sie mehr Informationen eingeben, erhalten Sie eine weitere genaue Art zurück 🙂
    • Ich nur teilweise Zustimmen. Ich werde nicht Tauchen Sie ein in template metaprogramming oder Präprozessor-Programmierung doch, es ist zu früh für die OP. Auf der anderen Seite, warum sollte ich nicht zeigen, einfach-zu-benutzen Sie Bibliotheken ? Ich verstehe nicht, warum die Leute immer auf die Lehre über char* Manipulationen in C++, weil std::string sind aufwendiger, wenn Sie Zeichenfolgen in fast jeder anderen Sprache...
    • Ich nur teilweise Zustimmen, auch :). Wenn Sie jemand lehren, wie die Verwendung von std::string, er lernt nur, wie die Verwendung von std::string. Auf der anderen Seite die Lehre char* manupilations es ihm erlaubt, das Konzept der Zeiger manupilation auch für andere Typen. Ich denke, eine Sprache zu lernen, sollte über how stuff works unter der Haube, nicht wie man alles verwenden. Wenn man lernt, wie Sachen arbeiten, andere werden Folgen.

    InformationsquelleAutor Matthieu M.
  4. 0

    Müssen Sie die Zeiger in die Klasse, wenn generische map-Container anderes Ergebnis möglich ist, wird ein neues Objekt leer der Klasse... mit Zeigern arbeiten!

    std::map<std::string, Type*> map_;

    Einfügen

    map_["key"] = *Type;

    HINWEIS: wenn-fpermisive compiler-flag nicht gesetzt ist, müssen Sie pass *Geben Sie nicht const beim einfügen in die map_ container.

    Beste Grüße,

    Javier,

    InformationsquelleAutor Javier Gutiérrez-Maturana Sánc
Schreibe einen Kommentar