具有引用计数器的共享指针的 C++ 实现函数

Question

getPointer() 返回此实例的原始指针。 getData() 返回实例的原始指针指向的数据。这个返回的数据/类型应该是可修改的。 一次最多允许 3 个引用。

我正在做一些练习任务以更好地理解 c++ :-) 已经做了大约 2 个月了，现在开始对各种其他事情的基础知识很慢，现在我发现这个任务是“重新发明 shared_ptr 的轮子”，我在尝试实现这些功能时卡住了，我决定寻求帮助。

我已将 cpp wiki 用于 shared_ptr operator* 和 -> 并尝试使用函数实现它，但我得到表达式必须具有指针类型 // 表达式必须具有类类型并且没有运算符“

1. 所以我认为这些功能没有返回它们应该提供的功能:-) 所以我想我搞砸了。我尝试使用 this->pData 和 pData.get() 但仍然出现相同的错误:-(

这是我的 main.cpp

#include "sharedPtr.hpp"
#include "Cat.hpp"
#include <iostream>

    int main(void)
    {
        sharedPtr<Cat> newCat(new Cat(0, "Ferrari"));

        // should print "Ferrari"
        std::cout << newCat.getPointer()->getName() << std::endl;

        // should also print "Ferrari"
        std::cout << newCat.getData().getName() << std::endl;

        newCat.getData().score = 50;

        // should printf "50", as it was just assigned before
        std::cout << newCat.getPointer()->score << std::endl;

        // make some copies
        sharedPtr<Cat> copyOfnewCat = newCat;
        sharedPtr<Cat> copyOfnewCat2 = newCat;

        // this copy should fail
        sharedPtr<Cat> copyOfnewCat3 = newCat;

        // should be nullptr
        std::cout << copyOfnewCat3.getPointer() << std::endl;

        // should be something other than 0 and equal
        std::cout << "copy2 pointer: " << copyOfnewCat2.getPointer() << " copy1 pointer: " << copyOfnewCat.getPointer() << std::endl;
        copyOfnewCat2.getData().score = 40;

        // should be 40 now 
        std::cout << newCat.getPointer()->score << std::endl;
        sharedPtr<Cat> copyOfnewCat4(newCat);

        // should be nullptr
        std::cout << copyOfnewCat4.getPointer() << std::endl;
    }

我的猫.cpp

#include <string>

class Cat
{
public:
    Cat(unsigned int w_score, const std::string& w_name) :
        score(w_score),
        name(w_name)
    {}
    std::string getName()
    {
        return name;
    }

    unsigned int score;
private:
    std::string name;
};

更新确切的错误信息

Severity    Code    Description Project File    Line    Suppression State
Error (active)  E0349   no operator "<<" matches these operands main.cpp    31  
Error (active)  E0044   expression must have pointer type   main.cpp    10  
Error (active)  E0153   expression must have class type main.cpp    13  
Error (active)  E0153   expression must have class type main.cpp    15  
Error (active)  E0044   expression must have pointer type   main.cpp    18  
Error (active)  E0349   no operator "<<" matches these operands main.cpp    28  

这些错误指的是 main.cpp 中的这一行

// should also print "Ferrari"
    std::cout << newCat.getPointer()->getName() << std::endl;

// should also print "Ferrari"
    std::cout << newCat.getData().getName() << std::endl;

问题

嗯，我想我注意到我应该在我的函数开头有我的类名，所以它应该是 sharedptr& getData 和 sharedptr* getPointer 吗？

Answer 1

Werner Henze 已经在他的 cmets 中说了所有相关的内容，所以这只不过是把它放在正确的位置。

该错误是由getData() 返回一个指针并在main 中用作引用引起的。由于getPointer() 已经返回指针，解决方法是更改​​getData() 以返回引用：

T& getData() {
    return *pData;
}

但您的代码仍有一些可能的改进：

• 您没有实现对 3 个引用的限制，因此您的 cmets 不一致
• 计算指向 NULL 的指针（默认构造函数）的引用并没有实际意义，因为在任何对象上都没有任何引用
• operator = 的常见实现是 复制和交换 习惯用法。它通常会导致更健壮和更简单的代码

除了第一条评论和修复错误之后，您可能会要求在 Code Review 上对您的代码进行彻底审查 - 请注意，损坏的代码或仍未编写的代码明显不在主题范围内......