wiedereinmal komischer Fehler? - Tausenderanzeige

TheShadowAE

Ich hab diesmal mal ein kleines Testproblem, ich hab mir gedacht, nachdem ich jemanden gesehen habe, der eine Zahlausgabe mit Tausenderzeichen nicht schafft, das mache ich doch schnell, das funktioniert auch ganz gut, jedoch wundere ich mich, dass das return den string löscht!?

C

#include <Windows.h>
#include <string>
#include <iostream>


using namespace std;




char* tausendzahl(unsigned int zahl)
{
    char ausgabe[21];
    _itoa_s((int)zahl,ausgabe,10);
    string str=ausgabe;
    int len=str.length();
    for (int x=len-3; x>0; x-=3)
    {
        str=str.insert(x,".");
    }
    char *nzahl=const_cast<char*>(str.c_str());//Hier ist noch 23.769.237
    return nzahl;//Hier wird es plötzlich ""
}






int main()
{
    char* nzahl=tausendzahl(23769237);
    cout << nzahl;
    system("pause >nul");
    return 0;
}

Alles anzeigen

Ich frage mich, warum wird der String gelöscht, wenn ich es doch zurückgebe

Marthog

Am Ende der Funktion wird die Variable str gelöscht. Der Destruktor gibt dabei den vorher aloziierten Arbeitsspeicher wieder frei. Das ist ganz praktisch, weil man den Speicher nicht mehr von Hand löschen muss, wie es sonst beim Heap der Fall ist.
Bei deiner Funktion zeigt der Pointer auf den freigegebenen Arbeitsspeicher.

Lösungsmöglichkeiten:
1.

Spoiler anzeigen

Code

std::string tausendzahl(unsigned int zahl)
{
    char ausgabe[21];
    _itoa_s((int)zahl,ausgabe,10);
    string str=ausgabe;
    int len=str.length();


    for (int x=len-3; x>0; x-=3)
        str=str.insert(x,".");


    return str;
}

Alles anzeigen

2.

Spoiler anzeigen

Code

bool tausendzahl(unsigned int zahl, char* buffer, unsigned int size)
{
    char ausgabe[21];
    _itoa_s((int)zahl,ausgabe,10);
    string str=ausgabe;
    int len=str.length();
    for (int x=len-3; x>0; x-=3)
    {
        str=str.insert(x,".");
    }
    for (int i=0; i<str.length() && i<size; ++i)
        buffer[i] = str[i];




    return (str.length()<= size);   //Gibt true zurück, wenn buffer groß genug war
}

Alles anzeigen

TheShadowAE

Thx, ausgehend von Methode 2 habe ich jetzt folgendes:

Code

void tausendzahl(unsigned int zahl, char back[31])
{
	char ausgabe[21];
	_itoa_s(zahl,ausgabe,10);
	string str=ausgabe;
	int len=str.length();
	for (int x=len-3; x>0; x-=3)
	{
		str=str.insert(x,".");
	}
	try
	{
		strcpy(back,const_cast<char*>(str.c_str()));
	}
	catch (char *e)
	{
		strcpy(back,"");
	}
}

Alles anzeigen

EDIT: ging nicht, geändert, aber das try fängt den Stackerror nicht ab wenn das zB ein Array[2] ist... (dh. Zahl wird einfach irgendwo hingeschrieben...)