C++新手上路,求解这段代码的两个错误及原因

图片说明

如题,一共两个错误,求解答。
感谢!

3个回答

个人认为两个错误应该是如下两个:
1、函数一进来应该立马判断 source 和 this 是否相等(因为重载赋值运算符的时候必须要考虑自己给自己赋值的场景),如果二者相等则直接返回*this. 不必再做申请内存和赋值的动作。
2、在 执行 m_data=newArray之前应该要先判断m_data是否为NULL, 如果m_data不是NULL要先讲m_data所指向的内存释放掉,避免内存泄露。
改正后的代码应该是这样的:
template
Array& Array::operator = (const Array& source)
{
if(this != &source)
{
T* newArray = new T[source.size];
if(m_data != NULL)
{
delete[] m_data;
}
Size = source.size;
m_data = newArray;

    for(int i=0; i< std::min(Size, source.size), ++i)
        m_data[i] = source.m_data[i];

}
return *this;

}

楼主要是对这个回答感到满意的话,记得采纳哦,急着等C币用,谢谢啦~~~

qq_41083830
qq_41083830 想了一下觉得应该是这个问题,感谢啦!
一年多之前 回复
 template <class T>
Array<T>& Array<T>::operator = (const Array<T>& source)
{
    if(this != &source)
    {
        T* newArray = new T[source.size];
        if(m_data != NULL)
        {
            delete[] m_data;
        }
        Size = source.size;
        m_data = newArray;

        for(int i=0; i< std::min(Size, source.size), ++i)
            m_data[i] = source.m_data[i];

    }
    return *this;
}

m_data没有定义
m_data没有释放,造成内存泄漏

hnfeng2014
hnfeng2014 从这个代码片段推测,m_data应该是类Array<T>的成员变量,= 应该是Array<T>中重载的运算符,也就是成员函数,因此m_data不需要在 = 这个函数中定义和释放,而是应该在类Array<T>的构造函数和析构函数中定义和释放。而= 作为成员函数,直接使用 m_data 就好。
一年多之前 回复
qq_41083830
qq_41083830 回复weixin_43064401: 谢谢啦~
一年多之前 回复
weixin_43064401
weixin_43064401 回复qq_41083830: 应该是
一年多之前 回复
qq_41083830
qq_41083830 请问new和delete必须成对使用吗?
一年多之前 回复
Csdn user default icon
上传中...
上传图片
插入图片
抄袭、复制答案,以达到刷声望分或其他目的的行为,在CSDN问答是严格禁止的,一经发现立刻封号。是时候展现真正的技术了!
立即提问