處理重載運算符時多項式類崩潰=

我正在從一個函數類派生Polynomial類的程序。我差不多完成了，但我遇到了最後一個問題。處理重載運算符時多項式類崩潰=

我在調用「=」運算符。在main函數中，它看起來像這樣：

f3 = f2 * 2;

f3和f2都是多項式數據類型。乘法運算符也被重載，並且已經過測試。當我的程序達到這一點時，它會崩潰。這是墜機的回溯：

Program received signal SIGABRT, Aborted. 
0x00000000 in ??() 
(gdb) bt 
#0 0x00000000 in ??() 
#1 0x77cbf8c1 in ntdll!RtlUpdateClonedSRWLock() 
    from /cygdrive/c/Windows/system32/ntdll.dll 
#2 0x75ab0816 in WaitForSingleObjectEx() 
    from /cygdrive/c/Windows/syswow64/KERNELBASE.dll 
#3 0x000000ec in ??() 
#4 0x00000000 in ??()

不幸的是，這並沒有提供任何提示。

最後，這裏是重載的operator =：

Polynomial& Polynomial::operator=(const Polynomial& rhs) { 
if(this->a) 
    delete [] this->a; 
this->setName(rhs.getName()); 
this->degree = rhs.degree; 
a = new double[this->degree]; 
for(int i = 0; i < this->degree; i++) 
{ 
    this->a[i] = rhs.a[i]; 
} 
return *this; 
}

而且該類的頭文件。

class Polynomial : public Function 
{ 
    public: 
        Polynomial (const string& name = "unnamed"); 
        Polynomial (const Polynomial& rhs); 
     void  setCoefficients (int degree, ...); 
     int   getDegree() const { return degree; } 
     double  operator[] (int i) const; 
     double  operator() (double x) const; 
     Polynomial operator* (double c) const; 
     Polynomial& operator*= (double c); 
     Polynomial& operator=(const Polynomial& rhs); 
     void  write(ostream& outfile) const; 

    private: 
     double* a; 
     int  degree; 
}; 
Polynomial operator* (double c, const Polynomial& p);

構造函數：

(Polynomial::Polynomial (const string& name) { 
    a = new double[1]; 
    a[0] = 0; 
    degree = 0; 
    return; 
    } 

Polynomial::Polynomial (const Polynomial& rhs) { 

    //Null a, and then pass over to the overloaded = operator. 
    if(this->a) 
     delete [] this->a; 
    *this = rhs; 
    return; 
    }

我願意爲需要共享大量代碼。感謝您的任何幫助！

來源

2012-11-30 mr0il

除其他事項外，你可能要檢查'這個==＆rhs'在'運營商='否則，你可以這樣做'OBJ = obj'這將導致段錯誤。 –

我明白檢查以確保它是相同的對象，但我可以保證不會發生.. – mr0il

爲什麼你不使用'std :: vector'，它會照顧整個內存管理業務？ –

你有構造函數和複製構造函數的問題。複製構造函數是從沒有作爲普通構造函數構造對象（但它然後從其他對象複製相應的值），所以你不應該檢查那裏的值，只是創建它。空白返回操作符不需要。你也有一個0和1尺寸的不確定性問題。該代碼會更常見：

Polynomial::Polynomial (const string& name) : Function(name) { 
    a = 0; 
    degree = 0; 
} 

Polynomial::Polynomial (const Polynomial& rhs) { 

    setName(rhs.getName()); 
    degree = rhs.degree; 
    if(degree) 
    { 
     a = new double[degree]; 
     for(int i = 0; i < degree; i++) 
      a[i] = rhs.a[i]; 
    } 
    else 
     a = 0; 
} 

const Polynomial& Polynomial::operator=(const Polynomial& rhs) { 
    if(this != &rhs) 
    { 
     if(a) 
      delete [] a; 
     setName(rhs.getName()); 
     degree = rhs.degree; 
     a = new double[degree]; 
     for(int i = 0; i < degree; i++) 
      a[i] = rhs.a[i]; 
    } 
    return *this; 
}

來源

2012-11-30 05:23:59

'delete []'無論如何執行'NULL'檢查; '如果（a）'是不必要的。 – Yuushi

我認爲這個問題是在你的拷貝構造函數，在你的拷貝構造函數，你必須：

//Null a, and then pass over to the overloaded = operator. 
if(this->a) 
    delete [] this->a;

但a這裏沒有初始化，所以你不能刪除它，所以將其替換爲：

Polynomial::Polynomial(Polynomial const& other) : a(NULL) {...}

來源

2012-11-30 05:55:08 BigBoss

處理重載運算符時多項式類崩潰=

回答

相關問題