创建包含已分配数组的unique_ptr的正确方法

lau*_*auw 54 c++ linux gcc unique-ptr c++11

创建包含在免费商店中分配的数组的unique_ptr的正确方法是什么?Visual Studio 2013默认支持这个,但是当我在Ubuntu上使用gcc版本4.8.1时,我得到了内存泄漏和未定义的行为.

使用此代码可以重现此问题:

#include <memory>
#include <string.h>

using namespace std;

int main()
{
    unique_ptr<unsigned char> testData(new unsigned char[16000]());

    memset(testData.get(),0x12,0);

    return 0;
}
Run Code Online (Sandbox Code Playgroud)

Valgrind将给出这个输出:

==3894== 1 errors in context 1 of 1:
==3894== Mismatched free() / delete / delete []
==3894==    at 0x4C2BADC: operator delete(void*) (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==3894==    by 0x400AEF: std::default_delete<unsigned char>::operator()(unsigned char*) const (unique_ptr.h:67)
==3894==    by 0x4009D0: std::unique_ptr<unsigned char, std::default_delete<unsigned char> >::~unique_ptr() (unique_ptr.h:184)
==3894==    by 0x4007A9: main (test.cpp:19)
==3894==  Address 0x5a1a040 is 0 bytes inside a block of size 16,000 alloc'd
==3894==    at 0x4C2AFE7: operator new[](unsigned long) (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==3894==    by 0x40075F: main (test.cpp:15)
Run Code Online (Sandbox Code Playgroud)

jua*_*nza 85

使用T[]专业化:

std::unique_ptr<unsigned char[]> testData(new unsigned char[16000]());
Run Code Online (Sandbox Code Playgroud)

请注意,在理想的世界中,您不必明确地使用new实例化a unique_ptr,从而避免潜在的异常安全陷阱.为此,C++ 14为您提供了std::make_unique功能模板.有关详细信息,请参阅此优秀的GOTW.语法是:

auto testData = std::make_unique<unsigned char[]>(16000);
Run Code Online (Sandbox Code Playgroud)

  • 只是为了确保:如上所示创建的`T[]` 的`unique_ptr` 是否对数据调用正确的删除器(即`delete []`)?我在某个地方看到了一个教程,他们指定了一个自定义删除器,所以我想知道这是否(仍然)必要,或者是否所有内容都将被正确删除。 (2认同)
  • @ j00hi是的。这就是拥有专业化的原因。 (2认同)
  • @juanchopanza std::make_unique&lt;T[]&gt; 调用 T 的默认构造函数。对于像上面的 unsigned char 这样的基本数据类型会发生什么?make_unique 0 是否初始化基本数据类型?当我打印“testData”(上面)时,它什么也不打印。 (2认同)
  • @kartiktrivikram _value初始化元素,对于具有默认构造函数的类型而言,这意味着它们是默认构造的;对于内置类型而言,这意味着零初始化。 (2认同)

gal*_*p1n 30

使用阵列版本:

auto testData = std::unique_ptr<unsigned char[]>{ new unsigned char[16000] };
Run Code Online (Sandbox Code Playgroud)

或者使用c ++ 14,一个更好的形式(VS2013已经拥有它):

auto testData = std::make_unique<unsigned char[]>( 16000 );
Run Code Online (Sandbox Code Playgroud)

  • 这两个陈述不相同.第二个值初始化数组,而第一个值创建未初始化的数组.从这个意义上讲,"make_unique"并不总是比构造函数加上"new"更好.这意味着对于不需要值初始化并且要避免使用的情况,因为它对性能至关重要,构造函数版本更好.注意`auto testData = unique_ptr <unsigned char []> {new unsigned char [16000]()];`等同于`make_unique`版本. (10认同)
  • @maxschlepzig C++20 添加了 [`std::make_unique_for_overwrite`](https://en.cppreference.com/w/cpp/memory/unique_ptr/make_unique) 来解决这个问题。 (2认同)

Tem*_*Rex 13

最有可能的更好的办法是使用std::vector<unsigned char>替代

#include <vector>
#include <string>

using namespace std;

int main()
{
    vector<unsigned char> testData(0x12, 0); // replaces your memset
    // bla    
}
Run Code Online (Sandbox Code Playgroud)

这样做的好处是,它更不容易出错,并且可以访问各种功能,例如轻松迭代,插入,达到容量时自动重新分配.

有一点需要注意:如果您要大量移动数据,则需要std::vector花费更多,因为它会跟踪大小和容量,而不仅仅是数据的开头.

注意:您memset没有做任何事情,因为您使用零计数参数调用它.

  • `std::vector` 比手动内存管理更不容易出错。但是如果 OP 足够了解使用 `std::unique_ptr`,我会说 vector 不再是明显的赢家。它的内存占用至少可能是 3 倍多一点。例如,如果您在性能关键代码中按值(移动)传递它,这可以算数。 (2认同)
  • @Angew更可能的情况是OP用于执行原始指针分配,听说过`unique_ptr`,现在想要"升级"到现代C++.您的性能参数值得注意,但除了最关键的代码之外,其他所有内容都无关紧要.`unique_ptr`加上`memset`容易出错,特别是如果这将被推广到非POD类型. (2认同)