Map von Pointern auf Membervariablen

[Nexus]

Student83 schrieb:

Macht der Code nicht genau das?

Sagen wir ja: Nein.

Du dereferenzierst den Zeiger und weist ihm das Objekt zu, anstatt den Zeiger selbst neu zu setzen. Schau dir vielleicht nochmals die Semantik der Dereferenzierung genau an.

Student83 schrieb:

Wären evtl. virtuelle Funktionen hier brauchbar?

Kommt drauf an, was du willst. Aber virtuelle Funktionen müssen die gleiche Signatur haben (kovariante Parameter sind nicht möglich). In meinen Augen wäre eine virtuelle Funktion ohne Parameter sinnvoller (wenn überhaupt). Ausserdem versuchst du, RValues an Referenzen auf nicht-konstante Objekte zu binden, was verboten ist. Und nicht zuletzt heisst es virtual ...

[Student83]

Evtl. verstehen wir uns falsch. Genau das will ich doch.

Ein Pointer referenziert ein Objekt. Wenn du vom Pointer wieder zum Objekt willst, musst du den Pointer "dereferenzieren".

Die Pointer werden beim Applikationsstart auf die Membervariablen referenziert, zeigen also auf sie. Und das soll sich die ganze Lebensdauer des Programms über nicht mehr ändern. Ich möchte die Werte der Member auf die gezeigt wird verändern, muss also den Zeiger dafür dereferenzieren.
Nur das klappt nicht da wie ihr schon gesagt habt hier Slicing passiert.
Ich habe also meine Klasse

class Control
{
private:
    Double width;
    Double height;
};

Control::Control()
{
    Object* zeigerAufWidth = &width;
    Object* zeigerAufHeight = &height;
    //Irgendwie dem Member width einen Wert zuweisen (über den Pointer)
    //Irgendwie dem Member height einen Wert zuweisen (über den Pointer)
}

Was mir fehlt ist die Möglichkeit, wie ich über die Zeiger den Membern width und height einen anderen Wert übergebe.

[Nexus]

Das sind absolute Grundlagen. Lies dir diesen Artikel zu Zeigern durch.

[Student83]

Ok, bringt mich aber nicht weiter. Du siehst das Problem nicht. Das Problem ist das Slicing.
Dies hier geht:

Double d = 320;
Double *e = &d;
std::wcerr << L"d before: '" << d << L"'" << std::endl;
*e = Double(640);
std::wcerr << L"d after: '" << d << L"'" << std::endl;

Aber nur wenn man vorher weis dass man Double hat. Ich erstelle die Objecte zur Laufzeit und weis nur dass sie die gemeinsame Basisklasse Object haben.
Darum sieht mein Code so aus:

Double d = 320;
Object *e = &d; // Achtung Hier Object* nicht Double*
std::wcerr << L"d before: '" << d << L"'" << std::endl;
*e = Double(640);
std::wcerr << L"d after: '" << d << L"'" << std::endl;

Mit der Folge dass Slicing auftritt. Wie kann ich dieses Slicing verhindern/umgehen? Das ist meine Frage.

[Student83]

Oh man ich glaubs nicht. Ich musste *e nur in Klammern setzen.

(*e) = Double(640);

Jetzt geht es. Hat das irgend einen Grund?

[314159265358979]

Das ist definitiv nicht der Fehler, da das in diesem Fall egal ist.

[Student83]

Habs gerade gemerkt. War Schwachsinn meinerseits.

[Nexus]

Du musst unterscheiden zwischen

*ptr = obj;

und

ptr = &obj;

Sobald du diesen Unterschied siehst, sollte sich der Rest ergeben. Slicing entsteht nur bei Kopien oder Zuweisungen, nicht bei Zeigern und Referenzen.

[Student83]

Naja, im ersten Fall weist man dem Object auf das ptr zeigt den Wert von obj zu und im zweiten lässt man den ptr auf obj zeigen.
Leider wurde ich mit Tomaten vor den Augen geboren, denn ich sehe trotzdem keine Lösung für mein Problem ;).
Denn ich möchte ja einen Wert zuweisen. Also auf gut Deutsch. Was ich vorhabe geht nicht?
Könnte ich das Slicing evtl. mit einem dynamic_cast verhindern? Die Sache mit der virtual-Funktion funktioniert nämlich leider wie du schon gesagt hast auch nicht.

[Nexus]

Ich dachte, das hätte sich geklärt. Zeiger auf Membervariablen kann ja nicht das Problem sein, oder? Damit kein Slicing vorkommt, musst du eben Verweise und nicht Kopien benutzen.

Kannst du nochmals dein momentanes Problem darstellen, am besten mit möglichst wenig drum herum, sodass man auf Anhieb verstehst, was du tun willst?

[Student83]

Double d = 320;
Object *e = &d;
std::wcerr << L"d before: '" << d << L"'" << std::endl;
*e = Double(640);
std::wcerr << L"d after: '" << d << L"'" << std::endl;

Oder etwas komplizierter mit map:

Double d = 320; //Diese Variable soll später zur Laufzeit verändert werden
std::map<std::wstring, Object*> map; //dazu wird ein Zeiger auf sie mit einem Keywort vom Typ String
map.insert(std::pair<std::wstring, Object*>(L"d", &d)); //in der Map eingefügt
std::wcerr << L"d before: '" << d << L"'" << std::endl; // Ausgabe Wert d vorher
std::map<std::wstring, Object*>::iterator it = map.find(L"d"); //Zeiger auf d in Liste ausfindig machen
if (it != map.end()) // und wenn gefunden
{
	*(*it).second = Double(640); //hier irgendwie den Wert Double(640) der Variablen d zuweisen
}
std::wcerr << L"d after: '" << d << L"'" << std::endl; //Ausgabe geändertes d
//Leider funktioniert das so nicht und d ist immer 320!

[Nexus]

Wie schwierig kann eine vernünftige Problembeschreibung eigentlich sein? Ich habe die Frage nicht gestellt, nur damit du den gleichen Code nochmals ohne Erklärung postest. Abgesehen davon gibt es Dinge, die ich schon im ersten Post auf der ersten Seite erwähnt habe und die du immer noch ignorierst.

Sorry, aber irgendwann wird es mir zu blöd, immer nachzufragen, weil du dein Problem nicht richtig beschreiben kannst. Wenn es wirklich nur um das Slicing geht, wurde dir meiner Meinung nach die Antwort schon längst gegeben.

[Student83]

Ganz ruhig, der Code lässt sich halt nicht weiter vereinfachen. Und das Problem habe ich jetzt auch schon mehrfach benannt. Habe jetzt nochmal Kommentare hinzugefügt. ich hoffe es wird jetzt klar was gewollt ist.
Was wäre denn die Antwort auf das Slicing Problem?

[Gast]

So, was soll denn passieren, wenn der Object-Zeiger nicht auf die Object-Basis eines Doubles zeigt?

[Student83]

Darüber mache ich mir gedanken wenn es soweit ist. Man könnte zum Beispiel ganz einfach erstmal schauen ob es denn auch ein Double ist:

std::wstring type = ((*it).second)->GetType().Name();
std::wcerr << type << std::endl;

Das gibt sogar Double aus ;). Jetzt aber bitte nicht vom Thema abschweifen sonst blickt hier im Thread echt niemand mehr durch.

[Gast]

Aber du hast schon verstanden, warum dein Code nicht so funktioniert, wie du es gerne hättest, ja? Was ist eigentlich die aktuelle Frage?

[Student83]

Wie kann ich einer Membervariablen einer Klasse dynamisch zur Laufzeit einen Wert zuweisen wenn ich nur den Namen bzw. Wert als String aus einer Datei ausgelesen bekomme. Das ist meine Frage.
Mein Lösungsansatz war nun dieser, dass ich Referenzen bzw. da man diese nicht in einer Map speichern kann, Pointer auf diese zeigen lasse um dann über den Zeiger den Wert dynamisch zuzuweisen. Da wusste ich allerdings noch nicht dass das wegen Slicing nicht geht.
Darum wollte ich jetzt wissen ob es dafür eine Lösung gibt.

[Gast]

class Object {
public:
    virtual void Assign( const Object& other ); // oder meinetwegen operator=
};

class Double : public Object {
public:
    void Assign( const Object& other ) {
         assert( other.GetType().Name() == "Double" ); // Oder was du zu gedenken tust
         *this = static_cast< const Double& >( other ); // oder dynamic_cast, aber dann bräuchtest du die Typnamen auch gar nicht mitschleifen
    }
};

[Student83]

Es funktioniert. Danke dir für die Lösung. War auch ne schwierige Geburt ;). Ist zwar nicht 100% optimal weil jetzt jede Klasse diese Assign-Funktion mitbekommt und ich diese für jede Klasse individuell anpassen muss. Aber anders wird es wohl nicht gehen. Der Tipp mit dem überschreiben des =-Operators ist auch gut.

[Gast]

template< typename T >
void Assign( Object& lhs, const T& rhs )
{
	assert( lhs.GetType() == rhs.GetType() );
	static_cast<T&>(lhs) = rhs;
}

Ginge auch Wir, und du selbst, wissen ja noch nicht, was du am Ende nun damit machen willst