Konvertieren von std :: wsting in char * mit wcstombs_s

Question

Ich habe Eingabezeichenfolgen, die nur Ziffern (nur die einfachen lateinischen, 0-9, so zum Beispiel "0123"), gespeichert als std :: wstring, und ich brauche jedes als char *. Was ist der beste Weg für mich, dies zu tun? Dies ist mein erster Ansatz:

void type::convertWStringToCharPtr(_In_ std::wstring input, _Out_ char * outputString) 
{ 
    outputString = new char[outputSize]; 
    size_t charsConverted = 0; 
    const wchar_t * inputW = input.c_str(); 
    wcstombs_s(&charsConverted, outputString, sizeof(outputString), inputW, input.length()); 
} 

EDIT: Der folgende Code funktioniert. Danke allen!

void type::convertWStringToCharPtr(_In_ std::wstring input, _Out_ char * outputString) 
{ 
    size_t outputSize = input.length() + 1; // +1 for null terminator 
    outputString = new char[outputSize]; 
    size_t charsConverted = 0; 
    const wchar_t * inputW = input.c_str(); 
    wcstombs_s(&charsConverted, outputString, outputSize, inputW, input.length()); 
} 

Answer 1

Du Zuweisung für Ihren Puffer nicht genügend Arbeitsspeicher:

char * outputString = new char[input.length()]; 

wegen Beendigung NUL-Zeichen

char * outputString = new char[input.length() + 1]; 

sein sollte.

Oh, und auch, wie pm100 's Kommentar: sizeof(outputString) gibt Ihnen die Größe des Zeigers. Sie sollten input.length() + 1 verwenden, da dies die Größe des Puffers ist.

Answer 2

Es gibt ein paar Fehler in Ihrem Code. Erstens weisen Sie nicht genügend Speicherplatz im Zielpuffer für das NULL-Zeichen zu. Sie müssen mindestens input.length() + 1 Zeichen zuweisen, damit die Funktion erfolgreich ist.

Zweitens übergeben Sie nicht die richtige Größe des Ausgabepuffers an die Funktion. sizeof(outputString) gibt die Größe outputString selbst zurück, eine char *, und nicht die Anzahl der Bytes, auf die der Zeiger zeigt.

So sollte Ihre Funktion wie folgt aussehen:

void CoverageTileManager::convertWStringToCharPtr(_In_ std::wstring input, _Out_ char * outputString) 
{ 
    size_t outputSize = input.length() + 1; 
    outputString = new char[outputSize]; 
    size_t charsConverted = 0; 
    wcstombs_s(&charsConverted, outputString, outputSize, input.c_str(), input.length()); 
    // TODO verify charsConverted = outputSize 
} 

Answer 3

In C++ Ich würde nie reine Zeiger verwenden: vector verwenden, wenn eine char-Array im Heap benötigt! Möchten Sie die Quellzeichenfolge kopieren? Wenn nicht, sollte const Referenz für input verwendet werden. wcstombs_s wird nur in Windows verwendet, also warum nicht einfach WideCharToMultiByte verwenden? War der Umstellungserfolg? Rückgabewert.

bool CoverageTileManager::convertWStringToCharPtr(const std::wstring& input, std::vector<char>& outputString) 
{ 
    if (input.empty()) { 
     return false; 
    } 
    int size = WideCharToMultiByte(CP_ACP,0,input.c_str(),input.size(),NULL,0,NULL,NULL); 
    if (size <= 0) { 
     return false; 
    } 
    outputString.resize(size+1); 
    if (WideCharToMultiByte(CP_ACP,0,input.c_str(),input.size(),&outputString[0],size,NULL,NULL) <= 0) { 
     outputString.clear(); 
     return false; 
    } 
    outputString[size] = '\0'; 
    return true; 
} 

Verwenden Vektor externe C++ lib:

extern void call(const char*, size_t); 
std::vector<char> buffer; 
std::wstring input; 
... 
if (convertWStringToCharPtr(input,buffer)) { 
    call(&buffer[0],buffer.size()); 
}