如何在使用指针作为函数返回值后删除该指针？

Question

我已经查阅了类似的问题，但它们都假定返回类型也是一个指针。如果我的描述不够准确，请见谅，因为我对C++还不太熟悉。

我有以下这个函数：

// 将字符串转换为用于对话框的LPCTSTR
LPCTSTR CAnotherAttemptDlg::convertToLPCTSTR(std::string conversion)
{
    size_t convertedChars = 0;
    const size_t newsizew = conversion.size() + 1;
    wchar_t* wcstring = new wchar_t[newsizew];
    mbstowcs_s(&convertedChars, wcstring, newsizew, conversion.c_str(), _TRUNCATE);
    return wcstring;
}

您可能已经注意到，我在函数中返回了一个指针作为结果，但是输出结果不应该是指针，因此我无法在函数外部通过删除该指针所指向的内容来释放内存，因为它不是赋值给一个指针变量。那么，最佳处理方法是什么？

注释：以防万一有人问到这个函数的目的，它是将一个std::string转换为LPCTSTR，后者等同于wchar_t*。

编辑：
为了更清楚地说明问题，这是调用此函数的代码块：

BOOL CAnotherAttemptDlg::OnInitDialog()
{
    std::string settingsLine;
    LPCTSTR defStrPath = L""; // = L"C:\Ste\Runtime\TestFiles";
    LPCTSTR defEndPath = L"";

    // 从FileTrans.ini读取默认起始和结束路径，并将其解析到合适的变量中
    std::ifstream settingsStream("FileTrans.ini");

    while (getline(settingsStream, settingsLine)) {
        std::string delimiter = "=";
        std::string tokenOne = settingsLine.substr(0, settingsLine.find(delimiter));
        std::string tokenTwo = settingsLine.substr(settingsLine.find(delimiter) + 1, '10');

        if (tokenOne == "startPath") {
            defStrPath = convertToLPCTSTR(tokenTwo);
        }
        else if (tokenOne == "endPath") {
            defEndPath = convertToLPCTSTR(tokenTwo);
        }
    }

    settingsStream.close();



    CDialogEx::OnInitDialog();

    // 设置窗口样式
    SetWindowLong(this->m_hWnd, GWL_STYLE, GetWindowLong(this->m_hWnd, GWL_STYLE) | WS_SYSMENU);

    // 设置对话框图标（框架会在应用程序主窗口不是对话框时自动完成这一操作）
    SetIcon(m_hIcon, TRUE);         // 设置大图标
    SetIcon(m_hIcon, FALSE);        // 设置小图标

    // 在此处添加额外的初始化代码
    
    SetDlgItemText(IDC_MFCEDITBROWSE1, defStrPath);
    SetDlgItemText(IDC_MFCEDITBROWSE2, defEndPath);

    return TRUE;  // 除非你将焦点设置到某个控件上，否则返回TRUE

    delete[] defStrPath;
    delete[] defEndPath;
}

Pablo Santa Cruz · Answer

在一些人的帮助下，我已经使程序运行起来且没有内存泄漏。他们建议使用wstring代替LPCTSTR，因为wstring会自动释放内存。

以下是当前的代码：

BOOL CAnotherAttemptDlg::OnInitDialog()
{
    std::string settingsLine;
    std::wstring defStrPath = L""; // 初始化为空字符串，相当于 "C:\Ste\Runtime\TestFiles"
    std::wstring defEndPath = L"";

    // 从FileTrams.ini获取默认起始和结束路径，并将其解析到相应变量中
    std::ifstream settingsStream("FileTrans.ini");

    while (getline(settingsStream, settingsLine)) {
        std::string delimiter = "=";
        std::string tokenOne = settingsLine.substr(0, settingsLine.find(delimiter)); 
        std::string tokenTwo = settingsLine.substr(settingsLine.find(delimiter) + 1, '10');

        if (tokenOne == "startPath") {
            defStrPath = convertToLPCTSTR(tokenTwo);
        }
        else if (tokenOne == "endPath") {
            defEndPath = convertToLPCTSTR(tokenTwo);
        }
    }

    settingsStream.close();

    CDialogEx::OnInitDialog();

    // 设置窗口样式
    SetWindowLong(this->m_hWnd, GWL_STYLE, GetWindowLong(this->m_hWnd, GWL_STYLE) | WS_SYSMENU);

    // 设置对话框图标（当应用程序主窗口不是对话框时，框架会自动执行此操作）
    SetIcon(m_hIcon, TRUE);         // 设置大图标
    SetIcon(m_hIcon, FALSE);        // 设置小图标

    // 在此处添加额外的初始化代码
    
    SetDlgItemText(IDC_MFCEDITBROWSE1, defStrPath.c_str());
    SetDlgItemText(IDC_MFCEDITBROWSE2, defEndPath.c_str());

    return TRUE;  // 如果未将焦点设置到某个控件，则返回TRUE
}

std::wstring CAnotherAttemptDlg::convertToLPCTSTR(std::string conversion)
{
    std::wstring temp = L"";

    size_t convertedChars = 0;
    const size_t newsizew = conversion.size() + 1;
    wchar_t* wcstring = new wchar_t[newsizew];
    mbstowcs_s(&convertedChars, wcstring, newsizew, conversion.c_str(), _TRUNCATE);
    temp = wcstring;
    delete[] wcstring;
    return temp;
}

注意：这里提到一个问题，wstring在使用nullptr时可能会出现问题，因此暂时初始化为空字符串，尽管这可能不是最佳做法，但现在能正常工作。

StriplingWarrior · Answer

我的回答分为两个方面：

技术上讲：你可以直接使用 delete[]

正如@someprogrammerdude和其他人所指出的，你的方法返回的是一个指向新分配的wchar_t字符序列的指针，即LPCWSTR，即指向常量宽字符“字符串”的（长）指针。你可以对这个方法调用的结果应用delete[]操作符来释放内存。

规范上讲：你应该避免这样做

有多个原因说明你不应该编写这样的函数或方法：

应当尽量避免在非必要情况下使用原始的 new 和 delete；这是C++核心准则 R.11 。主要原因在于，很容易忘记对使用原始new分配的内存执行delete操作，而且没有自动化机制确保你一定会这么做。
应当避免使用代表所有权的原始指针；这是C++核心准则 R.3 。特别是，你的函数或方法签名不应通过返回指针转移所有权（而你的函数恰恰这么做了）。相反地，
应当采用RAII（资源获取即初始化）自动管理内存；这是C++核心准则 R.1 。通常这意味着类会在构造时分配资源并在析构时释放资源。对于字符串来说，
如果你需要用于字符串字符的灵活缓冲区而不是std::string——可以使用适当的缓冲区类如std::vector，或者根据上下文甚至可以使用std::stringstream。

但实际上——你的对话框类无需承担字符串转换的任务！毕竟，conversion参数并不是类实例关心的任何特定字符串。如果你想将char类型的字符串转换为wchar_t类型的字符串，这当然是可行的。参考StackOverflow上的这个问题：

C++ Convert string (or char*) to wstring (or wchar_t*)

你会找到一个简单的答案（最初来源于这里）：

   std::wstring utf8_string_to_wstring(const std::string& str) {
       std::wstring_convert> converter;
       return converter.from_bytes(str);
   }

这个方法适用于UTF-8文本，尤其是ASCII文本，但自C++17起已被弃用。我自己仍会使用它，但另一种替代方案是微软推荐的方法，如下所示（来自这里）：

   std::wstring utf8_string_to_wstring(const std::string& str) {
       int count = MultiByteToWideChar(CP_UTF8, 0, str.c_str(),
           str.length(), nullptr, 0);
       std::wstring wstr(count, 0);
       MultiByteToWideChar(CP_UTF8, 0, str.c_str(), str.length(),
           &wstr[0], count);
       return wstr;
   }