3. Frage zu Objekten, Referenzen und Vererbung

Frage zu Objekten, Referenzen und Vererbung

  • R

    Ja, da ich das Label aber mit:

    MenuC file("File", TRUE);

    im Hauptprogramm anlege, sollte es doch so lange existieren, bis file out of scope geht, oder nicht?

    Evtl. wäre es hier noch besser, wenn ich den Labelstring per Referenz übergebe, statt per Copy.

    Ciao

    0
  • L

    Der Text existiert solange bis das temporäre Objekt string( "File" ) out of scope geht. Das ist aber nach dem Semikolon in dieser Zeile. Ab diesem Zeitpunkt ist in einer Struktur tief in der API ein ungültiger Zeiger abgelegt.

    0
  • R

    Hm, das kapier ich jetzt nicht ganz, da das Objekt, welches mit:

    MenuC file("File", TRUE);

    angelegt wird doch so lange existiert, bis der umgebende Scope (Methode etc.) verlassen wird? Dachte ich zumindest? Oder nicht?

    0
  • L

    Das Objekt file ja, der String, der aus dem Literal "File" erzeugt wird, nicht.

    Was erwartet der File-Konstruktor? Ein std::string.
    Was gibst Du ihm? Ein const char*.

    Was passiert? Der Compiler entdeckt eine implizite Umwandlung von const char* nach string, bei der der Text kopiert wird. Der Konstruktor von MenuEntryC bekommt einen konstanten std::string, aus dem Du c_str() benutzt. Danach ist der Konstruktor beendet, der konstante std::string, der gebraucht wurde um const char* an MenuEntryC zu übergeben wird gelöscht, und Du stehst da mit einem Pointer, der aus einem c_str() eines jetzt nicht mehr gültigen Objektes stammt.

    0
  • R

    Wie kann ich dann in diesem Fall den String richtig initialisieren, damit er dauerhaft erhalten bleibt, bis file out of scope geht?

    0
  • T

    Übergib doch einfach direkt ein "const char*":

    MenuEntryC::MenuEntryC(const int type, const char* pLabel) 
{ 
    cout << "Create MenuEntry: " << this << " Label: " << pLabel << endl; 

    newMenu.nm_Label = pLabel;
    newMenu.nm_Type = type; 
    newMenu.nm_CommKey = 0; 
    newMenu.nm_Flags = 0; 
    newMenu.nm_MutualExclude = 0; 
    newMenu.nm_UserData = NULL; 
}

    Edit: bei den abgeleiteten Klassen (z.B. MenuC) dann natürlich auch...

    0
  • R

    Das geht natürlich auch, wollte aber möglichst mit C++/STL-Konstrukten arbeiten,
    wobei der const char* hier bestimmt performanter ist, mich aber immer an C erinnert.

    Aber ich muss mich wohl damit abfinden, dass die Gemeinsamkeiten zw. C und C++ wohl größer sind als mir lieb wäre. Ich wäre in C++ auch gern ohne Zeiger ausgekommen, aber dann mutieren die STL-Container ja zu Speichermonstern, da immer kopiert wird.

    Ciao

    0
  • R

    Nur der Vollständigkeit halber:

    const char * hat nicht funktioniert, da STRPTR ein char * ist.
    Nachdem ich die Parameter in char * geändert habe, hats bestens funktioniert.

    0
  • T

    Trotzdem würde ich die Parameter als "const char *" übergeben und nur an der entsprechenden Zuweisung einen cast vornehmen:

    newMenu.nm_Label = const_cast<char *>(pLabel); // bzw <STRPTR>
    0
  • R

    Danke, werde ich probieren!

    Da fehlt mir halt die Erfahrung mit C++. Dachte der const_cast geht nur in die andere Richtung (also wenn ich was konstant machen will, was bisher noch nicht als konstant deklariert war).

    0
Anmelden zum Antworten