C++ ＝のオーバーロード、自己代入のコード

Question

C++の＝演算子のオーバーロードができません。

どなたか解答と解説をお願いできませんか？

該当箇所（間違ってると思われる所）に印をつけておきました。

以下にコードを記載させていただきます。

＝＝＝＝＝ここからヘッダファイル＝＝＝＝＝

#ifndef __Pol__
#define __Pol__

class Polynomial {
private:
    int Size;
public:
    double* Element;
    Polynomial(void);
    Polynomial(int size);
    int size(void)const;
    std::ostream& print(std::ostream &lhs)const;
    void del(void);
    void copy_Pol(Polynomial &result);
    const Polynomial& operator=(const Polynomial &rhs);
};

std::ostream& operator<<(std::ostream &lhs,const Polynomial &rhs);

#endif

＝＝＝＝＝ここから関数ファイル＝＝＝＝＝

#include<iostream>
#include<new>
#include"4.h"

Polynomial::Polynomial(void):Size(0),Element(NULL){}

Polynomial::Polynomial(int size)try:Size(size),Element(new double[Size]){}

catch(std::bad_alloc){
    std::cerr<<"Polynomial::Polynomial(int size):Out of Memory"<<std::endl;
    throw;
}

int Polynomial::size(void)const{
    return Size;
}
    
void Polynomial::del(void){
    delete []Element;
}
    
std::ostream& Polynomial::print(std::ostream &lhs)const{
    for(int i=0;i<Size;i++)
        lhs << "(" << Element[i] << ")x^" << i << "+,";
    return lhs;
}
    
std::ostream& operator<<(std::ostream &lhs,const Polynomial &rhs){
    return rhs.print(lhs);
}


//＊＊＊＊＊ここから下に間違いがあると思います＊＊＊＊＊
void Polynomial::copy_Pol(Polynomial &result){
    int i;
    result.Size=Size;
    if(result.Element != NULL){
        delete []result.Element;
    }
    try{
        result.Element=new double[Size];
    }
    catch(std::bad_alloc){
        std::cout<<"copy_Pol:Out of Memory"<<std::endl;
        exit(1);
    }
    for(i=0;i<result.Size;i++)
        result.Element[i] = Element[i];
}
 
 

    
const Polynomial& Polynomial::operator=(const Polynomial &rhs){
    //std::cout<<"read\n";
    if (this != &rhs) {
        copy_Pol(rhs);
        //std::cout"!="<<std::endl;
    }
    return *this;
}

//＊＊＊＊＊ここまでの間に間違いがあると思われます＊＊＊＊＊

＝＝＝＝＝ここからメイン関数のファイル＝＝＝＝＝

#include<cstdlib>
#include<iostream>
#include"4.h"

int main(void){
    try{
        Polynomial x(4),y(3);
        int i;
        
        for(i=0;i<x.size();i++)
            x.Element[i]=(double)(i+1);
        for(i=0;i<y.size();i++)
            y.Element[i]=(double)(i+2);
        
        std::cout<<"初期値"<<std::endl;
        std::cout<<"x="<<x<<"\n"<<"y="<<y<<std::endl;
        std::cout<<std::endl;
        
        std::cout<<"代入：y=x"<<std::endl;
        y=x;
        std::cout<<"x="<<x<<"\n"<<"y="<<y<<std::endl;
        std::cout<<std::endl;
        
        std::cout<<"自己代入"<<std::endl;
        x=x;
        std::cout<<"x="<<x<<"\n"<<"y="<<y<<std::endl;
        std::cout<<std::endl;
        
        x.del();
        y.del();
    }
    catch(std::exception& e){
        std::cerr<<"main:"<<e.what()<<std::endl;
        exit(1);
    }
    
    return 0;
}

＝＝＝＝＝このようなエラーが出ます＝＝＝＝＝

polynomial.cxx: In member function ‘const Polynomial& Polynomial::operator=(const Polynomial&)’:
polynomial.cxx:55: error: no matching function for call to ‘Polynomial::copy_Pol(const Polynomial&)’
polynomial.cxx:33: note: candidates are: void Polynomial::copy_Pol(Polynomial&)

Comments

[y_waiwai]

このままではコードが見づらいので、質門を編集し、<code>ボタンで、出てくる’’’の枠の中にコードを貼り付けてください

[Tosshi]

変更致しました。

Answer 1

メンバ関数Polynomial::copy_Pol(Polynomial &result)の引数型が「非constなPolynomial&」になっていることが原因です。

コピーコンストラクタPolynomial::operator=(const Polynomial&)はconst Polynomial&を引数にとりますが、この実引数をcopy_Polメンバ関数に渡そうとすると「const Polynomial&からPolynomial&への変換」が必要になってしまいます。C++ではconst修飾を外す変換は（非常に危険なため）自動的では行いませんから、質問中にある通りコンパイルエラーとなります。

---

（コードレビュー的観点になりますが）copy_Polメンバ関数の処理内容と、コピーコンストラクタの処理がかみ合っていないようです。copy_Polでは引数resultに計算結果が格納されるようですが、コピーコンストラクタの右辺rhsを渡そうとしています。これでは a = b; の b 側が変更されてしまうのでは？

Comments

[Tosshi]

おっしゃる通りでした。

解決しました。ありがとうございます

Answer 2

こんにちは。

copy_Pol()がイキナリ return ; しているからではないでしょうか？

Comments

[Tosshi]

見落としてました。
修正しましたが、まだエラーが消えないです