[英]Is it necessary to avoid memory leak when returning pointer by using shared_ptr?
我有兩個函數用於將 char 數組從 gb2321 轉換為 utf-8,例如,
#include <windows.h>
#include "memory.h"
#include <wchar.h>
#include <iostream>
using namespace std;
//GB2312 to UTF-8
char* G2U(const char* gb2312)
{
int len = MultiByteToWideChar(CP_ACP, 0, gb2312, -1, NULL, 0);
wchar_t* wstr = new wchar_t[len + 1];
memset(wstr, 0, len + 1);
MultiByteToWideChar(CP_ACP, 0, gb2312, -1, wstr, len);
len = WideCharToMultiByte(CP_UTF8, 0, wstr, -1, NULL, 0, NULL, NULL);
char* str = new char[len + 1];
//what about using shared_ptr to release resource but how to modify following code?
//shared_ptr<char> str(new char[len + 1], default_delete<char[]>());
memset(str, 0, len + 1);
WideCharToMultiByte(CP_UTF8, 0, wstr, -1, str, len, NULL, NULL);
if (wstr) delete[] wstr;
return str;
}
我在網站上閱讀了此代碼部分,我想知道是否應該替換char* str = new char[len + 1];
與shared_ptr
。 如果是這樣,如何修改以下代碼memset
和WideCharToMultiByte
? PS, C
不是CPP
使用了一些方法嗎?
您可以簡單地使用std::vector
,而不是管理原始指針,甚至是shared_ptr
來管理char
指針。
它有一個接受大小和值的構造函數(因此您不需要memset
)。 您可以使用std::vector::data
訪問底層數據緩沖區。
您可以在下面看到str
的示例。 類似的解決方案可以應用於wstr
。
#include <vector>
std::vector<char> str(len + 1, 0); // allocate size and set all elements to 0
//----------------------------------------vvvvvvvvvv
WideCharToMultiByte(CP_UTF8, 0, wstr, -1, str.data(), len, NULL, NULL);
筆記:
一般來說,使用std::string
可能更直接。 但是直到 C++17 std::string::data
返回一個const char*
,它與用於寫入數據的 Win32 API 指針不兼容。 您仍然可以使用以下命令從G2U
function 返回std::string
:
return std::string(str.begin(), str.end());
如果您可以使用 C++17 您可以使用std::string
而不是std::vector<char>
然后您將能夠直接返回str
。
更新:
關於問題末尾的“PS”-據我所知, c語言沒有自動堆 memory 管理的機制。
聲明:本站的技術帖子網頁,遵循CC BY-SA 4.0協議,如果您需要轉載,請注明本站網址或者原文地址。任何問題請咨詢:yoyou2525@163.com.