函数内char*转wchar_t*返回后字符串丢失问题排查

兄弟你踩了C++里一个经典的指针参数陷阱——指针默认是值传递！你在char_str_to_wchar_t_str里操作的dst，只是外部main里那个未初始化指针的一个副本。当你在函数里写dst = new wchar_t[newsize];时，只是把这个副本的指向改成了新分配的内存，外面的dst指针压根没变化，还是原来的野指针状态，这就导致你在main里用它时必然出现乱码、内存访问违规这些问题。

几种修复方案，按需选择： #

方案1：传指针的指针（wchar_t**） #

通过指向指针的指针，直接修改外部指针的指向：

int main(int argc, char* argv[]) {
    wchar_t* dst = nullptr; // 先初始化，避免野指针
    for (int i = 0; i < argc; i++) {
        char_str_to_wchar_t_str(&dst, argv[i]);
        // 这里可以正常使用dst了
        std::wcout << dst << std::endl;
        delete[] dst; // 用完必须释放，防止内存泄漏
        dst = nullptr;
    }
}

static size_t char_str_to_wchar_t_str(wchar_t** dst, char* src) {
    size_t newsize = strlen(src) + 1;
    *dst = new wchar_t[newsize]; // 修改外部指针的指向
    size_t convertedChars = 0;
    mbstowcs_s(&convertedChars, *dst, newsize, src, _TRUNCATE);
    return convertedChars;
}

方案2：传指针的引用（wchar_t*&） #

C++的引用语法更直观，直接绑定外部指针：

int main(int argc, char* argv[]) {
    wchar_t* dst = nullptr;
    for (int i = 0; i < argc; i++) {
        char_str_to_wchar_t_str(dst, argv[i]);
        std::wcout << dst << std::endl;
        delete[] dst;
        dst = nullptr;
    }
}

static size_t char_str_to_wchar_t_str(wchar_t*& dst, char* src) {
    size_t newsize = strlen(src) + 1;
    // 先释放之前的内存（如果有的话），防止泄漏
    if (dst != nullptr) {
        delete[] dst;
        dst = nullptr;
    }
    dst = new wchar_t[newsize];
    size_t convertedChars = 0;
    mbstowcs_s(&convertedChars, dst, newsize, src, _TRUNCATE);
    return convertedChars;
}

方案3：直接返回分配好的指针 #

让函数负责分配内存，调用者只管释放，逻辑更清晰：

int main(int argc, char* argv[]) {
    for (int i = 0; i < argc; i++) {
        wchar_t* dst = char_str_to_wchar_t_str(argv[i]);
        if (dst != nullptr) {
            std::wcout << dst << std::endl;
            delete[] dst;
        }
    }
}

static wchar_t* char_str_to_wchar_t_str(char* src) {
    size_t newsize = strlen(src) + 1;
    wchar_t* dst = new wchar_t[newsize];
    size_t convertedChars = 0;
    mbstowcs_s(&convertedChars, dst, newsize, src, _TRUNCATE);
    return dst;
}

重要提醒：内存别忘清理 #

不管用哪种方案，用new[]分配的内存必须用delete[]释放，否则会造成内存泄漏。另外，指针初始化时最好设为nullptr，避免野指针引发意外问题。

关于std::wstring的方案 #

你说的没错，用C++标准库的字符串类确实更省心，它自动管理内存，不用手动处理new/delete，代码也更简洁：

#include <string>
#include <iostream>

int main(int argc, char* argv[]) {
    for (int i = 0; i < argc; i++) {
        std::wstring dst;
        size_t newsize = strlen(argv[i]) + 1;
        dst.resize(newsize);
        size_t convertedChars = 0;
        mbstowcs_s(&convertedChars, &dst[0], newsize, argv[i], _TRUNCATE);
        dst.resize(convertedChars); // 调整到实际转换后的长度
        std::wcout << dst << std::endl;
    }
}

内容的提问来源于stack exchange，提问作者apierion