ich komme mit dem konvertieren einer long Variable in einen char Pointer nicht klar. Ich möchte in C++ die Zeitdifferenz messen(m_deltaT) und in einen char Pointer konvertieren, damit ich sie über SDL ausgeben kann. Die Messung funktioniert korrekt, m_deltaT wird kontinuierlich erhöht.

Mein aktueller Code ist folgender (mit Hilfe http://www.gidforums.com/t-1164.html):

long m_deltaT = 0; (global)

m_deltaT += //Zeitmessung mittels timeval
char* longvar = new char[4];
longvar = reinterpret_cast<char*>(&m_deltaT);
//RenderText...

longvar ist dann der Text, den ich dann in OpenGL mit SDL_ttf schreiben möchte.

Da der obrige Code 100/sec. aufgerufen wird (eingestellte Wiederholrate in OpenGL) dauert es nicht lange bis ein Speicherzugriffsfehler kommt und das Programm abstürzt (Ausgabe funktioniert etwa 3 sec.).

delete[] longvar;

nach der Übergabe an RenderText einzufügen bricht sofort nach dem Start mit free(): invalid pointer: 0x08059014 ab.

Vor das long, bzw. char* kann ich kein unsigned vor setzen, da die Methode TTF_RenderText_Blended von SDL mit const char* arbeitet.

Ich hoffe ihr könnt mir da helfen.
Grüße

Dann versuchst Du, diesen Stack-Speicher per 'delete []' freizuräumen, was nicht klappt. Der von Dir belegte Heap-Speicher wird nie verwendet.

danke erstmal für die Antwort. Nach der Erklärung habe ich den Fehler im Grunde verstanden, nur an der Umsetzung hapert es dennoch
Ist denn die Änderung in

const char* longvar = reinterpret_cast<const char*>(&m_deltaT);

immernoch das selbe Problem? Der Zeiger geht jetzt doch direkt auf die Variablenadresse.

Funktionieren tut es nämlich immernoch nicht.

Hallo,

danke erstmal für die Antwort. Nach der Erklärung habe ich den Fehler im Grunde verstanden, nur an der Umsetzung hapert es dennoch
Ist denn die Änderung in

const char* longvar = reinterpret_cast<const char*>(&m_deltaT);

immernoch das selbe Problem? Der Zeiger geht jetzt doch direkt auf die Variablenadresse.

Funktionieren tut es nämlich immernoch nicht.

Was funktioniert denn nicht? Dieser Code ist so an sich korrekt. Du versuchst doch hoffentlich nicht, diesen Speicher dann mit 'delete' freizugeben? Das ist ja Stack-Speicher und kein Heap-Speicher.

Danke trotzdem.

Dann ist das auch kein Wunder, dass nach kurzer Zeit ein Speicherfehler auftritt, nämlich dann, wenn die Größe der ASCII Tabelle überschritten ist.

Ich möchte doch nur den Wert von m_deltaT augegeben haben.

Ich möchte doch nur den Wert von m_deltaT augegeben haben.

Ah, dann hast Du den Sinn von 'reinterpret_cast' falsch verstanden. Das, was Du willst, ist eine Zahl als String ausgeben? Dann kannst Du einfach folgendes schreiben:

cout << m_deltaT;

Wenn Du den Inhalt der Zahl in einen String speichern musst, dann schau Dir mal die FAQ „Einmal Zahl nach String und zurück“ an.

(statt long, nun int)

char buffer[256];
sprintf(buffer, "%i", m_deltaT/180);

Grüße.

Was ist an der Variante zu C-ig ?

Dass Du 'char*' statt 'std::string' benutzt, und außerdem, dass hier ein Puffer fester Größe verwendet wird. 256 ist zwar eine sichere Größe, aber vom Konzept her ist solch ein Code eine Einladung für Buffer Overflows. Solch ein Code ist einfach konzeptionell unsicher und sollte gemieden werden. Dafür gibt's ja extra die 'std::string'-Klasse.

std::string s("Dein Text");

func(s.c_str());

unsigned char GetLaenge (const unsigned long &nr)
//& kannst du auch weglassen, wir der compiler eh nicht nutzen - schaden tut es aber nicht
{
//entweder du gehst über powerof oder machst es so in etwa:
unsigned char laenge (0);
for (long i (1); i <= nr;i*=10)
 {
  ++laenge;
 }
return (laenge-1);
};

void LongToChar (long nr, char *string)
{
unsigned char laenge (GetLaenge (nr));
delete string;
string = new char [laenge+1];
string[laenge--] = 0;
for (laenge; laenge != 0; --laenge)
 {
  string[laenge] = nr%10 + '0';
  nr /= 10;
 }
string[0] = nr + '0';
};

Sollte so eigtl stimmen ^^
Byebye : )

jz sollte es wirklich funktionieren ^^

Dein Code ist grauenvoll...
- free() sollte man in C++ vermeiden (und erst recht nicht in Kombination mit new einsetzen)
- & ist völlig unnutz bei GetLaenge (und wenn schon dann eine konstante Referenz)
- viel zu C lastig

Fazit: ignorieren -)

3. na und?
wenn er char will, dann is das nu ma der effektivste weg ("Da der obrige Code 100/sec. aufgerufen wird")

//edit: der funktionsname fehlt auch noch seh ich gerade - aber nun ja ^^
@~skilled denk dir halt drecks_c_style_fkt dort hin
und war noch nen fehler im schleifenkopf -.-

long myLong;
std::stringstream myString;
myString << myLong;
myFunction(myString.str().c_str());