0

我不明白为什么在最后一行return *newP; 进入复制构造函数的方法会导致内存泄漏，因为我无法释放Poly从复制构造函数创建的新对象？

const Poly Poly::operator * (const Poly &p1) const {
Poly* newP= new Poly;
delete newP->headList->mono;
newP->headList->mono=NULL;
delete newP->headList;
newP->headList=NULL;
newP->SetCount(0);
//two pointer to run on the list

Node* tmp,* tmp2;
tmp= this->headList;
tmp2=p1.headList;
Mono* mono= new Mono;
int n,d,p;
while (tmp != NULL)
{
    while (tmp2!=NULL)
    {
        *mono=(*tmp->mono)*(*tmp2->mono);
        n=mono->GetiNom();
        d=mono->GetiDenom();
        p=mono->GetPower();
        newP->Insert(n,d,p);
        tmp2=tmp2->next;
    }   
    tmp=tmp->next;
    tmp2=p1.headList;
}
delete mono;
return *newP;
}
4

1 回答 1

0

您可以使用返回引用，例如*this，这对于方法链接非常有用。如果对象在其他地方被删除，它不会泄漏内存。

好吧，既然你编辑了你的代码，是的，这确实泄漏了......如果你使用的是最近的编译器，你可以使用智能指针来解决这个问题，虽然如果你正在学习，我不推荐它，这很重要很好地理解内存管理。

对于这种特殊情况，函数应该返回一个指针，是的。或者，它可以通过在 return 语句中调用复制构造函数来返回对象的副本，并在主体中使用临时对象，但这不是一个非常干净的解决方案。

于 2013-04-22T22:01:14.007 回答