c++ - 对 std::set 使用基于范围的 for> 删除功能

Question

我正在尝试将基于范围的迭代器与一组unique_ptr实例一起使用，但出现以下编译错误：

C2280: 'std::unique_ptr<Component,std::default_delete<_Ty>>::unique_ptr(const std::unique_ptr<_Ty,std::default_delete<_Ty>> &)' : attempting to reference a deleted function

基本代码如下：

#include <set>
#include <memory>

std::set<std::unique_ptr<Component>>* m_components;

class Component
{
    void DoSomething(){};
};

void ProcessComponents()
{
    for (auto componentsIterator : *m_components)

    {
        componentsIterator->DoSomething();
        componentsIterator++;
    }
}

知道为什么这会是一个问题或如何解决它吗？

Answer 1

for (auto componentsIterator : *m_components)

这auto扩展为std::unique_ptr<Component>，这意味着您正在尝试获取每个元素的副本。IOW，那个循环实际上是：

for(auto it=m_components->begin(); it!=m_components->end(); ++it)
{
    std::unique_ptr<Component> componentsIterator=*it;
    componentsIterator->DoSomething();
    componentsIterator++;
}

如您所见，您正在调用std::unique_ptr<Component>复制构造函数，但复制构造函数unique_ptr被删除（因为它违反unique_ptr语义）。

用来auto &代替参考。

（顺便说一句，componentsIterator没有一个合理的名称，因为它不是迭代器，它是实际元素）