[英]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.