Konwersja std :: wstepna na char * z wcstombs_s

Question

Mam ciągi wejściowe, które zawierają tylko cyfry (tylko zwykłe łacińskie, 0-9, więc na przykład "0123"), przechowywane jako std :: wstring i potrzebuję każdy jako znak *. Jaki jest najlepszy sposób, aby to zrobić? To jest moje pierwsze podejście:

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()); 
} 

EDYCJA: Poniższy kod działa. Dziękuje wszystkim!

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

nie przeznaczają wystarczającej ilości pamięci do bufora:

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

Powinny być

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

powodu kończącego NUL znaków.

Och, a także, zgodnie z komentarzem pm100: sizeof(outputString) podaje rozmiar wskaźnika. Powinieneś użyć input.length() + 1, ponieważ jest to rozmiar bufora.

Answer 2

W kodzie jest kilka błędów. Po pierwsze, nie przydzielasz wystarczającej ilości miejsca w buforze docelowym dla znaku NULL. Musisz nadać co najmniej input.length() + 1 znaków, aby funkcja odniosła sukces.

Po drugie, nie przekazujesz tej funkcji prawidłowego rozmiaru bufora wyjściowego. sizeof(outputString) zwraca rozmiar samego outputString, czyli char *, a nie liczbę bajtów wskazywanych przez ten wskaźnik.

Więc funkcja powinna wyglądać następująco:

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

W C++ nigdy nie używać czystych wskazówek: używać vector jeśli tablica char potrzebne w kupie! Czy chcesz skopiować ciąg źródłowy? Jeśli nie, należy podać odniesienie do stałej dla input. wcstombs_s jest używany tylko w systemie Windows, więc dlaczego nie używać po prostu WideCharToMultiByte? Czy sukces konwersji? Zwróć wartość.

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; 
} 

Zastosowanie wektora do zewnętrznego 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()); 
}