class ZentrierteSchrift
{
    private:
        char *Text; /* Zeiger auf den Text */
        GLfloat x; /* Hälfte der Breite des Textes (negativ) */
        GLfloat y; /* Viertel der "Hoehe des Textes" */
    public:
        ZentrierteSchrift() : Text(0) {} /* Zeiger mit 0 initialisieren */
        ZentrierteSchrift(const char *Text) : Text(0)
        {
            Aktualisiere(Text);
        }
        ~ZentrierteSchrift() { delete[] Text; } /* Speicher freigeben */
        void Aktualisiere(const char *was) /* Welcher Text soll eigentlich gezeichnet werden? */
        {
            delete[] Text; /* Da Text am Anfang 0 ist, geschieht dann noch nichts. */
            Text = new char[strlen(was) + 1]; /* Reserviere 1 Zeichen mehr für 0-Terminierung */
            strcpy(Text,was); /* Kopiert Text inklusive 0-Terminierung */
            SIZE Groesse; /* Lege SIZE Struktur für WinAPI Funktion an */
            GetTextExtentPoint32(hDC,Text,strlen(Text),&Groesse); /* Groesse ermitteln */
            x = static_cast<GLfloat>(Groesse.cx) / -2; /* Hälfte und negativ */
            y = static_cast<GLfloat>(Groesse.cy) / 4; /* Viertel, da Windows auch noch Zeilenabstand mit einberechnet */
        }
        void Ausgabe()
        {
// nicht so wichtig
        }
};

mfg.

mach mal so:

class ZentrierteSchrift {
         /*...*/   
        ZentrierteSchrift(const char *s) {
            Aktualisiere(s);
        }
        void Aktualisiere(const char *was) 
        {
            delete[] Text;  
            Text = new char[strlen(was) + 1]; 
            strncpy(Text, was, strlen(was));
            /*...*/
        }
};

Problem war Null-Pointer, den du im Element-Initialiser angegeben hast.

Hallo,

mach mal so:

class ZentrierteSchrift {
         /*...*/   
        ZentrierteSchrift(const char *s) {
            Aktualisiere(s);
        }
        void Aktualisiere(const char *was) 
        {
            delete[] Text;  
            Text = new char[strlen(was) + 1]; 
            strncpy(Text, was, strlen(was));
            /*...*/
        }
};

Problem war Null-Pointer, den du im Element-Initialiser angegeben hast.

Warum ist der Null-Pointer ein Problem??

Und wenn ich das ganze durch strncpy ersetze, kopiere ich ja das 0-Terminierungszeichen nicht mit, oder?

mfg.

Warum ist der Null-Pointer ein Problem??

Verfolge doch mal genau, was dein Konstruktor tut. Du übergibst nen Null-Pointer an Aktualisiere (und rufst dann strlen und strcpy darauf auf), das macht (für mich) alles wenig Sinn.
Btw. welches Verhalten zeigt sich denn genau? Dann kann ich vllt. genaueres sagen.

Und wenn ich das ganze durch strncpy ersetze, kopiere ich ja das 0-Terminierungszeichen nicht mit, oder?

So wie ich's geschrieben hab nicht. Aber das könnte man leicht ändern, indem man einfach Text[n] = '\0' schreibt oder ein Element mehr kopiert.
Wäre nicht schlecht, wenn du noch ne size_t Variable für die Größe hättest, dann müsstest du nicht immer mit strlen rumeiern.

joomoo schrieb:

Warum ist der Null-Pointer ein Problem??

Verfolge doch mal genau, was dein Konstruktor tut. Du übergibst nen Null-Pointer an Aktualisiere (und rufst dann strlen und strcpy darauf auf), das macht (für mich) alles wenig Sinn.
Btw. welches Verhalten zeigt sich denn genau? Dann kann ich vllt. genaueres sagen.

Stimmt! Ist mir gar nicht aufgefallen. Ich hab ausversehen den Parameter und den privaten Zeiger gleich benannt, beide Text. Ich werde das mal ändern und gucken ob es was hilft.

Aber das strcpy kann ich auch so lassen oder?

mfg.

Stimmt! Ist mir gar nicht aufgefallen. Ich hab ausversehen den Parameter und den privaten Zeiger gleich benannt, beide Text.

Bingo.

joomoo schrieb:

Aber das strcpy kann ich auch so lassen oder?

Lieber nicht, das ist unsicher und schreibt, wenn's blöd läuft bzw. der Zielstring zu kurz ist, wild im Speicher rum und das führt zu undefiniertem Verhalten. Freunde dich mit strncpy an (und mit strncat).

Die Klasse Menu hat mehrere vectoren, einige enthalten GLfloat Objekte und einer ZentrierteSchrift Objekte. Beim hinzufügen neuer Elemente zum Vector mit den ZentrierteSchrift Objekten ändern sich plötzlich die im vector<GLfloat> Werte zu ziemlich komischen Zahlen wie: 67200099052103200000.000000 oder so ähnlich. Auf jeden Fall sehr groß. Manchmal passiert es auch dass sich das Programm mit einem Laufzeitfehler verabschiedet. Für mich spricht das alles nach einem undefinierten Verhalten weil ich irgentwo einen Fehler gemacht habe. Und ich glaube dass er in dieser Klasse liegt. Es ist zum verzweifeln.

mfg.

Text[ strncpy(Text, was, strlen(was)) ] = '\0';

Machs doch einfach so:

Text[ strncpy(Text, was, strlen(was)) ] = '\0';

Der Rückgabe wert von strncpy ist glaub ich ein Zeiger auf den String und nicht die übertragenen Bytes.

mfg.

class ZentrierteSchrift
{
    private:
        std::string Text; /* Text */
        GLfloat x; /* Hälfte der Breite des Textes (negativ) */
        GLfloat y; /* Viertel der "Hoehe des Textes" */
    public:
        ZentrierteSchrift() {} /* Zeiger mit 0 initialisieren */
        ZentrierteSchrift(const char *was)
        {
            Aktualisiere(was);
        }
        ~ZentrierteSchrift() { } /* Speicher freigeben */
        void Aktualisiere(const char *was) /* Welcher Text soll eigentlich gezeichnet werden? */
        {
            Text = was;
            SIZE Groesse; /* Lege SIZE Struktur für WinAPI Funktion an */
            GetTextExtentPoint32(hDC,Text.c_str(),Text.size(),&Groesse); /* Groesse ermitteln */
            x = static_cast<GLfloat>(Groesse.cx) / -2; /* Hälfte und negativ */
            y = static_cast<GLfloat>(Groesse.cy) / 4; /* Viertel, da Windows auch noch Zeilenabstand mit einberechnet */
        }
        void Ausgabe()
        {
            Schrift2.AusgabeEx(Text.c_str(),x,y); /* Schrift zentriert ausgeben */
        }
};

Das klappt auch. Falls trotzdem jemand den Fehler sieht, würde ich mich darüber freuen, denn ich seh ihn immer noch nicht.

mfg.

Ich hatte jetzt kein Bock mehr auf den Kack, und hab jetzt alles durch std::strings ersetzt:

Warum nicht gleich so

Any schrieb:

Machs doch einfach so:

Text[ strncpy(Text, was, strlen(was)) ] = '\0';

Der Rückgabe wert von strncpy ist glaub ich ein Zeiger auf den String und nicht die übertragenen Bytes.

mfg.

Hast recht, man müsste es so machen: strncpy(Text, was, strlen(was)+1) dann wird die 0 mitkopiert

joomoo dein Code ist ja voll gefrickelt.

Das heißt?

mfg.

joomoo schrieb:

Ich hatte jetzt kein Bock mehr auf den Kack, und hab jetzt alles durch std::strings ersetzt:

Warum nicht gleich so

Jetzt muss jeden Frame std::string::c_str() aufgerufen werden. Ist das resourcenfressend?

mfg.

Allerdings: Ist std::string referenzgezählt und ist der String 'gesharet', so muss vor dem Aufruf von c_str eine Kopie angefertigt werden.

du brauchst für die klasse einen copy-construktor, der nicht nur den Zeiger Text, sondern auch dessen Inhalt mitkopiert. Ansonsten hast du, im vector, lauter "ZentrierteSchrift"en, deren Text-Pointer alle auf den selben speicher-inhalt zeigen.