如题,一共两个错误,求解答。
感谢!
3条回答
- 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币用,谢谢啦~~~
本回答被题主选为最佳回答 , 对您是否有帮助呢?解决 无用评论 打赏 举报
悬赏问题
- ¥100 求数学坐标画圆以及直线的算法
- ¥35 平滑拟合曲线该如何生成
- ¥100 c语言,请帮蒟蒻写一个题的范例作参考
- ¥15 名为“Product”的列已属于此 DataTable
- ¥15 安卓adb backup备份应用数据失败
- ¥15 eclipse运行项目时遇到的问题
- ¥15 关于#c##的问题:最近需要用CAT工具Trados进行一些开发
- ¥15 南大pa1 小游戏没有界面,并且报了如下错误,尝试过换显卡驱动,但是好像不行
- ¥15 自己瞎改改,结果现在又运行不了了
- ¥15 链式存储应该如何解决