c++ operator 演算子の練習 class

Question

大幅に書き変えました, すみません.

演算子の練習をしているのですが, 以下のエラーがでてしまいます.
どこを直せば良いですか？

11th.operator.cpp:32:23: error: assigning to 'double' from incompatible type 'A'
        a3.x = (*this)*s;
               ~~~~~~~^~
11th.operator.cpp:45:19: error: member reference base type 'double' is not a structure or
      union
            x += a.x;

#include <iostream>

using namespace std;

class A{
    double x;
public:
    A operator+(A a) // やりたいこと: (*this) + a;
    {
        A a1;
        a1.x = (*this).x + a.x;
        return a1;
    }
    A operator-(A a) // やりたいこと: (*this) - a;
    {
        A a2;
        a2.x = (*this).x - a.x;
        return a2;
    }
    bool operator>(A a) // やりたいこと: (*this) > a;
    {
        return (*this).x > a.x;
    }
    bool operator==(A a){ // やりたいこと: (*this) == a;
        return (*this).x == a.x;
    }
    A operator*(double s) // やりたいこと: (*this)*s; 
    {
        A a3;
        a3.x = (*this)*s;
        return a3;
    }
    double& operator[](int i){ // やりたいこと: (*this).x = some thing; 
        return (*this).x;
      }

      A &operator=(const A &a){ // やりたいこと: (*this) = a; 
        (*this).x = a.x;
        return (*this);
      }
      A& operator<<(double a){ // やりたいこと: (*this) << a;
            
            x += a.x;
            return (*this);
      }

};
// 以下に正しい出力を示しておく．
// 1, 2
// 1 < 2
// 1 != 2
// 3, 2
// 1, 2
// 2, 2
// 4
// 8
// 10
// 16

int main(){
    A a1, a2;

    a1[0] = 1;
    a2[-1] = 2; 

    cout << a1[-9999] << ", " << a2[0] << "\n";

    if(a1 > a2){
        cout  << a1[1651651] << " > " << a2[0] << "\n";
    }else{
        cout << a1[0] << " < " << a2[0] << "\n";
    }

    if(a1 == a2){
        cout << a1[0] << " == " << a2[0] << "\n";
    }else{
        cout << a1[0] << " != " << a2[0] << "\n";
    }

    a1 = a1 + a2;
    cout << a1[0] << ", " << a2[0] << "\n";

    a1 = a1 - a2;
    cout << a1[0] << ", " << a2[0] << "\n";

    a1 = a2;
    cout << a1[0] << ", " << a2[0] << "\n";

    a1 = a1*2;
    cout << a1[0] << "\n";
    a1 = a1 + a2*2; 
    cout << a1[0] << "\n";

    a1 << 2;
    cout << a1[0] << "\n";

    a1 << 1 << 2 << 3;
    cout << a1[0] << "\n";


    return 0;
}

Comments

[kazuma-s]

class A の定義がないと、何も言えません。
operator* でやることは - でいいんですか？

[_._._ami]

すみませんでした, class A追記しました！

[_._._ami]

できました, ありがとうございました！！

[kazuma-s]

元のコードはそのままで、できたコードを質問に「追記」してもらえますか？

Answer 1

おかしなところだらけなので箇条書きとします。

• A aを引数にすると、演算のたびにコピーが行われてしまいます。a + bでbが書き換わるような動作は一般に不適切ですので、const A& aのように、const参照で取るのが一般的です。
• 通常、比較演算などでオブジェクトは変化しませんので、メソッドもconstにしておきましょう。
• 同様に、Aを返り値にするとコピーが発生します。*thisを返すようなoperator =の返り値はA&型が適切です。
• operator>やoperator ==でifが条件を満たさなかった場合の返り値がありません。
• operator*(double s)とdoubleを取っているのに、s.xとプロパティ参照を試みています。
• operator[]はfoo[bar]のような添字アクセスの場合に使われるものです。ドットアクセスはオーバーロードできません。

Comments

[_._._ami]

詳しくありがとうございます...！！
そうなんですね...知らなかったです.
読む前にA aで全部書いてしまったので一旦それを修正として追記しました.
maisumakunさんの回答を参考に書き直してみます！

[_._._ami]

できました, ありがとうございました！！