为什么我收到编译错误“使用已删除的函数'std :: unique_ptr ...”

Question

我的消息收到了很大的编译错误

c:\mingw\include\c++\6.1.0\bits\predefined_ops.h:123:18: error: use of deleted function 'std::unique_ptr<_Tp, _Dp>::unique_ptr(const std::unique_ptr<_Tp, _Dp>&) [with _Tp = Deduction; _Dp = std::default_delete<Deduction>]'
         { return bool(_M_comp(*__it1, *__it2)); }

当我将自定义比较器传递给STL set_difference函数时。

我的代码：

struct Value{
   std::string ded_code;
   float amount;
   Value(std::string code, float amt):ded_code(code), amount(amt){}
};

struct Deduction{
  std::string p_number;
  std::vector<std::unique_ptr<Value>> values;
  Deduction(string pnum, string code, float amt):p_number(pnum){ 
    auto val = std::make_unique<Value>(code, amt);
    values.emplace_back(move(val));
  }
};

class compute{
public:
   vector<unique_ptr<Deduction>> deductions;
   void fillDeductions(){
    // fill deductions
    ...
   }

};

class CompareDiff{
public:
  bool operator()(unique_ptr<Deduction>& ded1, unique_ptr<Deductions>& ded2){
    rPtr1 = ded1.get();
    rPtr2 = ded2.get();
    return ( rPtr1->p_number < rPtr2->p_number);
 }
};

...
int main(){
  ...
  // fill two deduction vectors
  Compute compA = Compute()
  compA.fillDeductions()

  Compute compB = Compute()
  compB.fillDeductions()

  vector<unique_ptr<Deduction>> diffs

  set_difference(compA.begin(), compA.end(),
                 compB.begin(), compB.end(),
             inserter(diffs, diffs.begin()), CompareDiff());

 }

我在Windows 7机器上使用gcc 6.1.0。

我错过了什么？

问候。

PG

Answer 1

std::unqiue_ptr的主要特点是无法复制。 这是设计，名称告诉你。

但是， CompareDiff尝试按值获取其参数。 这需要一份副本。 相反，采用std::unique_ptr<..> const& - 不需要复制。

Answer 2

你仍然得到一个错误的原因：

std :: set_difference在内部复制 ：

将排序范围[first1，last1]中的元素（在排序范围[first2，last2]中找不到）复制到从d_first开始的范围。

http://en.cppreference.com/w/cpp/algorithm/set_difference

如果您希望编译代码，请改用std :: shared_ptr。

请记住，标准库是针对对象优化的，而不是针对指针的。 如果使用指针，则必须自己进行所有权管理。

Answer 3

你不能复制构造unique_ptr因为它是一个已删除的函数 ，你可以移动唯一的指针来转移所有权，但是你需要一个函子来比较你需要通过引用传递那些unique_ptr的东西。

Answer 4

使用unique_ptr<x>表示函数承担x所有权。

使用shared_ptr<x>表示函数是x部分所有者。

如果您确实要传递unique_ptr并转移所有权，则应move智能指针move到函数参数中。

关于传递智能指针的更多注释 ，Herb Sutter在这个CppCon讲话中有一些好的想法。