C++ error: passing 'A' as 'this' argument of 'B' discards qualifiers [-fpermissive] について

Question

#include "Complex.h"

int main(){
	const Complex c0(4.0, 5.5);
	const Complex c1(1.1, 2.2);
	Complex result0 = c0+c1;
	Complex result1 = c0*c1;
	result0.print();
	result1.print();
}

operatorを使った複素数の加算・乗算のソースコードです
これを実行したところ、

error: passing 'const Complex' as 'this' argument of 'Complex Complex::operator+(const Complex&)' discards qualifiers [-fpermissive]

といったエラーが出ました。
原因が分からないので詳しい方がいらっしゃいましたら教えていただきたいです。

"Complex.h"の中身は以下のようになります。

#ifndef COMPLEX_H_
#define COMPLEX_H_

#include <cmath>
#include <climits>
#include <string>
#include <iostream>
#include <stdio.h>
#include <sstream>

class Complex {
private:
	double real; // 実部
	double imag; // 虚部
public:
	/*
	Complex(double re, double im); // 引数付コンストラクタ
	Complex(){real = 0; imag = 0;} //デフォルトコンストラクタ
	Complex(const Complex& orig); // コピーコンストラクタ
	Complex(double re){real = re; imag = 0;} //変換コンストラクタ

	//後置増分演算子
	Complex operator+() { return *this; } // 単項演算子+c
	Complex operator-() { return Complex(-real, -imag); } // 単項演算子-c
	Complex operator+(const Complex& arg); // 加算
	// Complex operator-(const Complex& arg); // 減算は不要
	Complex operator*(const Complex& arg); // 乗算

	// 前置増分演算子
	Complex& operator+=(const Complex& arg); // 複合演算子
	Complex& operator-=(const Complex& arg); // 複合演算子
	Complex& operator*=(const Complex& arg); // 複合演算子

	bool operator==(const Complex& arg); // 等号
	bool operator!=(const Complex& arg); // 不等号

	double modulus(); // 絶対値
	std::string print(); // 3+4i のような表示文字列
	*/

	Complex(double re, double im){ // 引数付コンストラクタ
		real = re; imag = im;
	}

	Complex(){real = 0; imag = 0;} //デフォルトコンストラクタ

	Complex(const Complex& orig){ // コピーコンストラクタ
		real = orig.real; imag = orig.imag;
	}

	Complex(double re){real = re; imag = 0;} //変換コンストラクタ

	// 後置増分演算子
	Complex operator+() { return *this; } // 単項演算子+c

	Complex operator-() { return Complex(-real, -imag); } // 単項演算子-c

	Complex operator+(const Complex& arg){ // 加算
		return arg;
	}

	Complex operator*(const Complex& arg){ // 乗算
		return Complex(-arg.real, -arg.imag);
	}

	// 前置増分演算子
	Complex& operator+=(const Complex& arg){
		if(real < UINT_MAX) real += arg.real;
		return *this;
	}

	Complex& operator-=(const Complex& arg){
		if(real < UINT_MAX) real -= arg.real;
		return *this;
	}

	Complex& operator*=(const Complex& arg){
		if(real < UINT_MAX) real *= arg.real;
		return *this;
	}

	bool operator==(const Complex& arg){
		return *this == arg;
	}

	bool operator!=(const Complex& arg){
		return *this != arg;
	}

	double modulus(){
		return sqrt(real*real + imag*imag);
	}

	std::string print(){
		std::ostringstream oss;
		oss << real << "+" << imag << "i";
		printf("%s\n",oss.str().c_str());
		return 0;
		//std::string r = std::to_string(real);
		//std::string i = std::to_string(imag);
		//return r + "+" + i + "i";
	}
};

#endif /* COMPLEX_H_ */

Answer 1

constオブジェクトを介しては、const修飾されたメンバ関数しか呼べません。
これは知らない内にメンバが書き換えられるのを避ける為です。

そして、const修飾するべき箇所はエラーメッセージに記載されているとおりです。
二項加算演算を担うメンバ関数ですね。

error: passing 'const Complex' as 'this' argument of 'Complex Complex::operator+(const Complex&)' discards qualifiers [-fpermissive]

その他変更が生じ得ないメンバ関数も、const修飾した方が分かり良いでしょう。

---

両オペランドに影響を与えないのであれば、
非メンバ関数でオーバーロードを実現するのが素直であるように思います。

Complex operator+(const Complex& left, const Complex& right) {
    ...
}

フレンド関数にしてやれば、Complexクラスのprivateメンバにもアクセス可能です。

Comments

[amber_snob]

ありがとうございます！無事解決しました

Answer 2

メンバ関数 print() が const じゃないからじゃない?