演算子のオーバーロードでコンパイルエラーが出ます

質問

いつもお世話になっております。よろしくお願いします。C++のプログラミング暦、半月です。

構造体Sのvector、Vを考えます。
struct S{
　　string　　　strA;
　　string　　　strB;
　　int　　　　intData;
　　double　　　doubleData;
};
vector<S>　V;

ここで、「Vを、intDataの昇順（第１キー）とdoubleDataの昇順（第２キー）にソートして、最初の要素のstrAとstrBを返す。」処理をしたいです。
それで、イテレータの中でintDataを比較する演算子を以下の通り定義しました。
bool operator < ( const S& left, const S& right ){
　　return S.intData < S.intData;
}

ですが、次のエラーがでました。引数を２つとる例も見たことがあるのですが、上記定義のどこがおかしいのかがわかりません。
error: `bool XXX::operator>(const S&, const S&)' must take exactly one argument

ちなみにLinux(RedHatEnterprise系)で、コンパイラはgcc3.4.3です。

回答No.2

colder · ベストアンサー

bool operator < ( const S& left, const S& right )をXXXのメンバ関数として定義していませんか?
bool operator < ( const S& left, const S& right )は非メンバ関数として定義する必要があります。

お礼

早速のご回答、ありがとうございました。ご指摘通り、XXXのメンバ関数としていました。これを、クラスの外で定義すればよいということですね。今、質問した時の環境がないので確認できませんが、そうすると構造体Sの定義もクラスの外でしないといけなくなるのでしょうか。

回答No.3

RAPTsong

アクセスする対象が public であれば、グローバルに定義できます。
もしくは、メンバ関数宣言で friend 宣言をつける必要があります。

お礼

御礼が遅くなって、どうもすみません。実機で確認しました。どうもありがとうございました。

回答No.1

koko_u_

とりあえず、S.intData とか書いてあるのは left.intData の typo ですか？
そしてエラーメッセージの演算子の向きが逆になっていますが。

お礼

ご指摘ありがとうございました。補足にて訂正しましたので、よろしくお願いします。

補足

すみません、質問用に表現を修正した時に２箇所間違えてしまいました。

　　return S.intData < S.intData;
は、
　　return left.intData < right.intData;
の間違いでした。

< と > の２つ作って、同じエラーが出たのですが、質問用に転記した時に、エラーメッセージだけ > の方を転記していました。

よろしくお願いします。