高效优雅地返回安置unique_ptr
Efficiently and elegantly returning emplaced unique_ptr
我发现(感谢StackOverflow注释)我的代码中的一个安全漏洞:
std::vector<std::unique_ptr<Item>> items;
template<class... TS> Item& create(TS&&... mArgs)
{
auto item(new Item(std::forward<TS>(mArgs)...);
items.emplace_back(item); // Possible exception and memory leak
return *item;
}
基本上,如果emplace_back
引发,使用原始new
分配Item
可能会泄漏内存。
解决方案绝不是使用原始new
,而是在方法主体中使用std::unique_ptr
。
std::vector<std::unique_ptr<Item>> items;
template<class... TS> Item& create(TS&&... mArgs)
{
auto item(std::make_unique<Item>(std::forward<TS>(mArgs)...);
items.emplace_back(std::move(item));
return *item; // `item` was moved, this is invalid!
}
如您所见,返回item
是无效的,因为我必须使用std::move
将其放入items
容器中item
移动。
我想不出需要将item
的地址存储在附加变量中的解决方案。然而,原始(错误)解决方案非常简洁且易于阅读。
有没有更优雅的方式来返回被移动以放置在容器中的std::unique_ptr
?
你可以
这样写:
template<class... TS>
Item& create(TS&&... mArgs)
{
items.emplace_back(std::make_unique<Item>(std::forward<TS>(mArgs)...));
return *items.back();
}
在 emplace 之前缓存引用是一个更普遍适用的选项(例如,对于非矢量容器):
template<class... TS> Item& create(TS&&... mArgs)
{
auto item = std::make_unique<Item>(std::forward<TS>(mArgs)...);
auto& foo = *item;
items.emplace_back(std::move(item));
return foo; // This *is* valid.
}
您的问题被标记为C++11,而其他答案表明make_unique
没有提到它是C++14功能。 我相信这种C++11方法也可以解决泄漏问题。
#include <vector>
#include <memory>
#include <utility>
#include <iostream>
struct Item
{
int a, b;
Item(int aa, int bb) : a{aa}, b{bb} { }
};
static std::vector<std::unique_ptr<Item>> items;
template <class... Ts> Item& create(Ts&&... args)
{
items.emplace_back(std::unique_ptr<Item>{new Item(std::forward<Ts>(args)...)});
return *items.back();
}
int main()
{
Item& x = create(1, 2);
std::cout << "( " << x.a << ", " << x.b << " )" << std::endl;
}
这应该是安全的,因为在构建unique_ptr<Item>
之前无法调用emplace_back()
,因此即使emplace_back()
确实抛出,您的Item
也已经由unique_ptr
管理。
相关文章:
- 为什么 std::unique 不调用 std::sort?
- CLANG 编译器 说:变量"PTR"可能未初始化
- 在以唯一ptr为值的C++映射中,动态内存何时会被销毁
- 将 ptr 传递给 ptr 到 A 作为参数传递给 A 的函数是不好的做法吗?
- 为共享 ptr 向量实现复制 c'tor?
- 字符和整数中 **(ptr+1) 的值差异
- C++:在不中断共享的情况下通过引用传递共享 PTR?
- 生成"unique"矩阵
- 如何将派生类从基 ptr 分配给 nlohmann::json
- 引用 std::shared:ptr 以避免引用计数
- 我对 std::unique(算法)C++有问题
- 为什么我不能在不进行任何转换的情况下将浮点数放入任何类型的 ptr 中?
- 在调用函数时,ptr** 和 ptr*& 之间是否有区别,或者首选C++?
- 另一种类型的智能ptr,比如具有弱refs的unique_ptr
- 尝试打印出 *ptr++ 的值,以了解它是如何工作的
- 如何控制共享 ptr 引用计数?
- std::shared_ptr::unique(),复制和线程安全
- 如何在C++03中用自定义谓词调用std::unique
- C++中的指针否定 (!ptr == NULL)
- C++14 unique_ptr并使用已删除的函数'std::unique-ptr' unique_ptr错误