qq_41083830
qq_41083830
2018-08-25 08:56
采纳率: 50%
浏览 492

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

图片说明

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

  • 点赞
  • 写回答
  • 关注问题
  • 收藏
  • 邀请回答

3条回答 默认 最新

  • hnfeng2014
    hnfeng2014 2018-08-25 15:28
    已采纳

    个人认为两个错误应该是如下两个:
    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币用,谢谢啦~~~

    点赞 评论
  • devmiao
    devmiao 2018-08-25 11:44

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

    点赞 评论
  • hnfeng2014
    hnfeng2014 2018-08-25 15:33
     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;
    }
    
    点赞 评论

相关推荐