将 std::make_unique 与 GetProfileBinary 函数调用一起使用

Question

我已经看到了这个答案 (Advantages of using std::make_unique over new operator)，它指出：

如果您需要自定义删除器或采用其他地方的原始指针，请不要使用make_unique。

这是我的代码：

void CAutomaticBackupSettingsPage::GetLastBackupDate(COleDateTime& rBackupDate)
{
    DATE* pDatTime = nullptr;
    UINT uSize;

    theApp.GetProfileBinary(_T("Options"), _T("BackupLastBackupDate"), pointer_cast<LPBYTE*>(&pDatTime), &uSize);
    if (uSize == sizeof(DATE))
        rBackupDate = *pDatTime;
    else
        rBackupDate = COleDateTime::GetCurrentTime();

    delete[] pDatTime;
    pDatTime = nullptr;
}

代码分析给了我两个警告：

和

后一个警告建议我使用std::make_unique，但由于我的指针数据是从GetProfileBinary 调用返回的，并且鉴于相关问题中的陈述，这是否意味着我不应该使用std::make_unique？我承认这是我以前没有做过的事情。

---

GetProfileBinary的用法明确指出：

GetProfileBinary 分配一个缓冲区并在*ppData 中返回其地址。 调用者负责使用delete[]释放缓冲区。

Answer 1

pDateTime 应该是nullptr，GetProfileBinary 处理分配。代码分析误以为你忘记了分配。

在调用delete[] 之前确实需要检查是否成功。我们不能使用delete[]pDatTime，因为pDatTime 不是一个数组。但是GetProfileBinary 分配使用new BYTE[size]，所以我们需要转换回BYTE。

您还可以在阅读pDatTime 之前添加NULL 检查，这可能会使代码分析变得愉快。

if (pDatTime && uSize == sizeof(DATE))
    rBackupDate = *pDatTime;
else
    rBackupDate = COleDateTime::GetCurrentTime();
if(pDatTime) delete[](BYTE*)pDatTime;

您可以使用std::unique_ptr<BYTE[]> cleanup((BYTE*)pDatTime) 进行删除，但这必须在调用GetProfileBinary 之后。

例子：

DATE* pDatTime = nullptr;
GetProfileBinary(_T("Options"), _T("BackupLastBackupDate"), (LPBYTE*)(&pDatTime), &uSize);
std::unique_ptr<BYTE[]> cleanup((BYTE*)pDatTime); //automatic delete

if (pDatTime && uSize == sizeof(DATE))
    rBackupDate = *pDatTime;
else
    rBackupDate = COleDateTime::GetCurrentTime();

//pDatTime = NULL; <- Error when used with unique_ptr
...
//pDatTime is deleted later, when `cleanup` goes out of scope