c++ - vector 中重复的 unique_ptr 项的所有权错误

Question

考虑到以下代码，我该如何解决我面临的所有权问题？

我需要在不复制 Item 的情况下处理这种情况，因为我做不到。

Item 和 ItemContainer 是用户不应该担心重复的 API 的一部分，这意味着如果用户调用它，我需要默默地添加一个指针两次同一个指针两次。

我以为我的删除器会解决它，但它没有。

例如，我不能使用 unordered_set，我应该像现在这样处理这种情况。

元素容器.h:

typedef std::unique_ptr<Item, std::function<void(Item *)>> ItemDeleter;
std::vector<ItemDeleter> items_;

元素容器.cpp:

void ItemContainer::addItem(Item *item)
{
  ItemDeleter uniqPtrItem(item, [](Item *p) {
    if (p != nullptr) { 
      delete p; //it's crashing here, obviously
      p = nullptr;
    }
  });
  items_.push_back(std::move(uniqPtrTask));
}

主要.cpp

int main() {
  Item *item = new Item();
  ItemContainer itemContainer;
  itemContainer.addItem(item);
  itemContainer.addItem(item);
}

Answer 1

如果您不能使用 std::shared_ptr(这是显而易见且最自然的解决方案)，那么您可以引用计数您的 Item。下面是一些简单的演示代码，展示了如何做到这一点:

#include <vector>
#include <memory>
#include <iostream>

class RefCount
{
public:
    void Retain ()
    {
        ++refcount;
    }

    void Release ()
    {
        if (--refcount == 0)
            delete this;
    }

protected:
    virtual ~RefCount () {}

private:    
    int refcount = 1;
};

class Item : public RefCount
{
public:
    Item () { std::cout << "Item constructor\n"; }
private:
    ~Item () { std::cout << "Item destructor\n"; }
};

typedef std::unique_ptr <Item, void (*) (Item *)> ItemDeleter;

void addItem (std::vector <ItemDeleter> &items, Item *item)
{
    item->Retain ();
    ItemDeleter uniqPtrItem (item, [] (Item *p) { p->Release (); });
    items.push_back (std::move (uniqPtrItem));
}

int main()
{
    std::vector <ItemDeleter> items;
    Item *item = new Item;
    addItem (items, item);
    addItem (items, item);
    item->Release ();
}

输出:

Item constructor
Item destructor

Live demo