std::string in char*

tntnet

groovemaster schrieb:

camper schrieb:

was meinst du? dass der standard das garantiert, oder das jede existierende implmentation sich so verhält?

Dass jede existierende Implementation sich so verhält. Eine Garantie ist mir seitens des Standards jedenfalls nicht bekannt.

Bei der Programmierung sollte man sich darauf beschränken, was der Standard definiert. Daß die existierenden Impelentationen eine gewisse Eigenschaft hat (habt ihr auch alle exisitiernden C++-Compiler getestet?), garantiert nicht, daß die nächste Version dieses Verhalten zeigt.

Im übrigen sollte man beachten, daß ein std::string häufig Referenzgezählt ist und daher eine Veränderung der Daten nicht zu empfehlen ist. Es hat schon seinen Grund, warum die data()-Methode einen const-Zeiger liefert.

Im übrigen hat die Originallösung noch einen Fehler. std::string::size() liefert die Länge des Strings. Ein strcpy des c_str()-Zeigers kopiert aber ein Zeichen mehr. Ihr habt den Nullterminator vergessen, welcher von c_str() garantiert wird, aber nicht zur Stringlänge gehört.

Tntnet

.filmor

template <typename CharT>
std::auto_ptr<CharT> string_to_array (const std::basic_string<CharT>& str)
{
    CharT* cstr = new char[str.size () + 1];
    std::copy (str.begin (), str.end (), cstr);
    cstr[str.size ()] = 0;
    return std::auto_ptr<CharT> (cstr);
}

Das erspart das (möglicherweise ineffiziente) Gefrickel mit c_str oder data.

Apollon

Du hast da ein +1 vergessen.

LordJaxom

Das solltest Du nochmal überdenken...

tntnet

Statt

std::copy (str.begin (), str.end (), cstr);

geht auch

str.copy(cstr, str.size());

Tntnet

camper

.filmor schrieb:

template <typename CharT>
std::auto_ptr<CharT> string_to_array (const std::basic_string<CharT>& str)
{
    CharT* cstr = new char[str.size () + 1];
    std::copy (str.begin (), str.end (), cstr);
    cstr[str.size ()] = 0;
    return std::auto_ptr<CharT> (cstr);
}

Das erspart das (möglicherweise ineffiziente) Gefrickel mit c_str oder data.

array new und auto_ptr? keine gute idee.

groovemaster

tntnet schrieb:

Bei der Programmierung sollte man sich darauf beschränken, was der Standard definiert.

Naja, mit dieser Einstellung wirst du in der Praxis aber schnell an Grenzen stossen. Es geht ja nicht darum, dass irgendetwas Verbotenes gemacht werden soll.

tntnet schrieb:

Im übrigen sollte man beachten, daß ein std::string häufig Referenzgezählt ist und daher eine Veränderung der Daten nicht zu empfehlen ist.

Was meinst du mit "eine Veränderung der Daten nicht zu empfehlen ist"? Du kannst jedes String Objekt ganz normal per op [] ändern.

LordJaxom

groovemaster schrieb:

Was meinst du mit "eine Veränderung der Daten nicht zu empfehlen ist"? Du kannst jedes String Objekt ganz normal per op [] ändern.

Er meinte höchstwahrscheinlich die Änderung der Daten in (char*)&str[0] oder const_cast<char*>(str.data()), also das worum es in diesem Thread die meiste Zeit ging

groovemaster

Wenn es um das const wegcasten ging, dann würde aber das "nicht zu empfehlen" nicht passen. Denn dann wäre es schlichtweg falsch.

.filmor

camper schrieb:

array new und auto_ptr? keine gute idee.

Izz, stimmt. Dann halt mit normalem Zeiger:

template <typename CharT>
CharT* string_to_array (const std::basic_string<CharT>& str)
{
    CharT* cstr = new char[str.size () + 1];
    std::copy (str.begin (), str.end (), cstr);
    cstr[str.size ()] = 0;
    return cstr;
}

Ich wollte dem Anwender Arbeit abnehmen.
BTW, beim Test hatts wohl nur deshalb nicht gekracht, weil die Benutzung so aussah:

std::cout << string_to_array (std::string ("asdf")).release () << std::endl;