shared_ptr 被提前删除[重复]

Question

我修改了@Timo 的answer 中使用的代码，试图了解shared_ptr 和自定义删除器的工作原理。

这是新代码的link，或者就在这里：

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

class TopicPointer
{
    public:
     TopicPointer(int x) : _x(std::move(x))
     {

     }

     ~TopicPointer(){
         std::cout << "Deleting " << _x <<  std::endl;
     }
     int GetX()
     {
         return _x;
     }
     private:
        int _x;

};

class Topic
{
    std::string name;
    std::shared_ptr<TopicPointer> _topicPointer;

    public:
        Topic(std::string name,std::shared_ptr<TopicPointer> topicPointer) : name(move(name)), _topicPointer(std::move(topicPointer)) {}

        ~Topic(){
            std::cout << "Deleting " << name << std::endl;
        }
};

struct Deleter
{
    public:
     void operator()(TopicPointer* ptr)
     {
        std::cout << "deleting topic " << ptr->GetX() << '\n';
     }
};

class TopicsCache 
{
public:
    std::unique_ptr<Topic>&& createTopic(std::string name, int y)    
    {
        auto topicPtr = new TopicPointer(y);
        return std::move(std::unique_ptr<Topic>(new Topic(move(name),std::shared_ptr<TopicPointer>(topicPtr, Deleter()))));
    }


};

class Subject
{
    public:
     Subject(std::vector<std::unique_ptr<Topic>> &&topics)  : _topics (std::move(topics))
     {

     }

    private:
      std::vector<std::unique_ptr<Topic>> _topics;
};

TopicsCache cache;

Subject BuildSubject()
{
    std::vector<std::unique_ptr<Topic>> topics;
    std::cout << "Creating topic 1\n";
    topics.emplace_back(std::move(cache.createTopic("a",1)));    
    std::cout << "Created topic 1\n";
    std::cout << "Creating topic 2\n";
    topics.emplace_back(std::move(cache.createTopic("b",2)));
    std::cout << "Created topic 2\n";
    topics.emplace_back(std::move(cache.createTopic("c",3)));
    topics.emplace_back(std::move(cache.createTopic("d",4)));
    return  Subject(std::move(topics));
}

int main()
{
    Subject subject = BuildSubject();

    std::cout << "Done";

}

如您所见，从BuildSubject() 的输出： 创建主题 1

删除一个

删除主题 1

初始化温度 1

创建主题 1

创建主题 2

删除 b

删除主题 2

创建主题 2

删除 c

删除主题 3

删除 d

shared_ptr 在初始化为temp 变量之前被删除。

我认为当 shared_ptr 被复制时，引用计数会更新？ std::move 也不保留引用计数吗？

如何阻止 shared_ptr 被提前释放？

谢谢

Answer 1

我在这里看到了问题：

std::unique_ptr<Topic>&& createTopic(std::string name, int y)    
{
    auto topicPtr = new TopicPointer(y);
    return std::move(std::unique_ptr<Topic>(new Topic(move(name),std::shared_ptr<TopicPointer>(topicPtr, Deleter()))));
}

您正在返回对在您使用时无效的临时对象的引用：

topics.emplace_back(std::move(cache.createTopic("a",1)));

你应该返回一个值：

std::unique_ptr<Topic> createTopic(std::string name, int y)    
{
    auto topicPtr = new TopicPointer(y);
    return std::unique_ptr<Topic>(new Topic(move(name),std::shared_ptr<TopicPointer>(topicPtr, Deleter())));
}

总体而言，您的代码中存在很多问题。移动整数？有不删除的删除器？具有右值引用类型的参数（接收器模式）？创建一个原始指针并将其传递给下一行的std::shared_ptr 构造函数？你忽略了 C++ 的所有习语，你很快就会射中你的腿。