这些中的任何一个都会使我的程序崩溃吗？ [关闭]答案

【问题标题】：Will any of these crash my program? [closed]这些中的任何一个都会使我的程序崩溃吗？ [关闭]
【发布时间】：2015-11-02 00:00:27
【问题描述】：

我在大学接到了一个任务，就是写一个智能指针。我需要将代码上传到他们的服务器上运行一些自动化测试，我只有一个错误：内存访问问题（空指针等）。现在我怀疑我的 *, -> 运算符重载，我认为如果我的 raw_pointer 为 NULL 并且我已经尝试 NULL 保护它们，我认为它可能会崩溃，但是如果是这种情况，我不知道要返回什么。如果不是这样，那我在这种情况下真的很无奈。

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

public:
    my_pointer() : raw_pointer(NULL) { }

    my_pointer(T *pointer) : raw_pointer(pointer) {
        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 (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 != NULL && raw_pointer->decRefCnt() == 0) {
         delete raw_pointer;
     }
}

};

当我做这样的事情时：

// T = refcounted
my_pointer<refcounted> obj1;    
refcounted p = *obj1;

我的程序以退出代码 -1073741819 (0xC0000005) 退出。提前致谢！

【问题讨论】：

您似乎通过编写 *obj1 来取消引用空指针。让它先管理一个对象？
这是一种实现智能指针的奇怪方法。
你的“智能指针”到底有什么要求？
它需要支持解引用，->，隐式转换为原始指针，需要支持==，!=运算符，引用计数由被引用的对象完成，而不是由被引用的对象完成智能指针，需要多个构造函数
T* operator->() { return raw_pointer; } 我认为这里有错误。但这不是崩溃的原因，这是由@M.M 给出的

标签： c++ pointers smart-pointers null-pointer

【解决方案1】：

类实现看起来不错，但您在第二个代码块中有错误。

obj1 使用 NULL 指针初始化，并在第二行取消引用它并复制到新变量。所以它崩溃了。

您必须在取消引用之前为 obj1 分配一些东西，或者检查它是否为 NULL。

比如这样

// T = refcounted
refcounted p0;
my_pointer<refcounted> obj1(&p0);    
refcounted p = *obj1;

或者像这样

// T = refcounted
my_pointer<refcounted> obj1;
refcounted p;
if( obj1 != NULL )
{
    p = *obj1;
}

【讨论】：