-2

我几乎完成了我的智能指针（我知道......）所以我将它上传到我大学的网站，该网站对我的代码运行了许多自动化测试。许多测试有两个问题：

  • 超出内存或时间限制
  • 内存访问问题（即空指针）

问题是我不知道正在执行什么样的测试。我能够阅读我所做的自动化测试的标准输出，这是写在那里的：

In instantiation of ‘my_pointer<T>::my_pointer() [with T = tester]’:
error: no matching function for call to ‘tester::tester()’
note: candidates are:
tester::tester(my_pointer<tester>)
candidate expects 1 argument, 0 provided
tester::tester(const tester&)
candidate expects 1 argument, 0 provided

所以我猜出于某种奇怪的原因它不会调用我的 my_pointer() 构造函数？这是我的智能指针类：

template<class T>
class my_pointer {
    T* raw_pointer;

public:
    my_pointer() {
        raw_pointer = new T();
        raw_pointer->incRefCnt();
    }

    my_pointer(T *obj) : raw_pointer(obj) {
        if(raw_pointer != NULL) raw_pointer->incRefCnt();
    }

    my_pointer(const my_pointer<T>& smart_pointer) : raw_pointer(smart_pointer.raw_pointer) {
        if(raw_pointer != NULL) raw_pointer->incRefCnt();
    }

    T& operator*() {
        return *raw_pointer;
    }

    T* operator->() {
        return raw_pointer;
    }

    operator T*() {
        return raw_pointer;
    }

    my_pointer<T> &operator=(const my_pointer<T> &smart_pointer) {
        if(this != &smart_pointer && raw_pointer != NULL) {
            /** if this was the last reference to the given memory address */
            if (raw_pointer->decRefCnt() == 0) {
            delete raw_pointer;                    
            }

            raw_pointer = smart_pointer.raw_pointer;
            raw_pointer->incRefCnt();
        }

        return *this;
    }

    bool operator== (const T* pointer) {
        return raw_pointer == pointer;
    }

    bool operator!= (const T* pointer) {
        return raw_pointer != pointer;
    }

    bool operator== (const my_pointer<T> &smart_pointer) {
        return raw_pointer == smart_pointer.raw_pointer;
    }

    bool operator!= (const my_pointer<T> &smart_pointer) {
        return raw_pointer != smart_pointer.raw_pointer;
    }

    ~my_pointer() {
       if(raw_pointer->decRefCnt() == 0 && raw_pointer != NULL) {
           delete raw_pointer;
        }
    }
};

这是一个可以计算引用的类：

class refcounted {
private:
    int count;
public:
    refcounted() : count(0) { }

    int incRefCnt() {
        return ++count;
    }

    int decRefCnt() {
        return --count;
    }
};

你能看出代码有什么问题吗？提前致谢！

4

2 回答 2

3

您正在无条件地删除析构函数中的原始指针。这是错误的。您必须减少引用计数，并且只有在它变为零时才删除。

您还调用raw_pointer->incRefCnt();了几个地方而不检查是否raw_pointeris NULL。

于 2015-11-01T16:02:13.487 回答
0

我在您的代码中看到两个问题：

  1. 在 operator=(): 你if(raw_pointer != NULL)已经取消引用后检查raw_pointer（在上面的行）。你应该撤销支票。

  2. 在析构函数中：您不应该先调用decRefCnt()，并且仅raw_pointer在引用计数降至零时才删除？

由于您在两者中几乎都需要相同的代码，operator=()并且~my_pointer()在需要时减少引用计数并删除 raw_pointer (AFAICS)，因此将该功能提取到一个新方法并从两个地方调用它似乎更实用。

于 2015-11-01T16:52:11.433 回答