MyStringクラスの比較演算子をオーバーロードしています

Question

bool operator == (const MyString& left, const MyString& right) 
{ 
    if(left.value == right.value) 
     return true; 
    else return false; 
} 
bool operator != (const MyString& left, const MyString& right) 
{ 
    if(left == right) 
     return false; 
    else 
     return true; 
} 
bool operator < (const MyString& left, const MyString& right) 
{ 
    if(strcmp(left.value, right.value) == -1) 
     return true; 
    else 
     return false; 
} 
bool operator > (const MyString& left, const MyString& right) 
{ 
    if(strcmp(left.value, right.value) == 1) 
     return true; 
    else 
     return false; 
} 
bool operator <= (const MyString& left, const MyString& right) 
{ 
    if(strcmp(left.value, right.value) == -1 || strcmp(left.value, right.value) == 0) 
     return true; 
    else 
     return false; 
} 
bool operator >= (const MyString& left, const MyString& right) 
{ 
    if(strcmp(left.value, right.value) == 1 || strcmp(left.value, right.value) == 0) 
     return true; 
    else 
     return false; 
} 

これらは私のMyStringクラスの比較演算子ですが、私の教授が私に与えたテストプログラムには失敗しています。何が問題ですか？

Answer 1

：

bool operator == (const MyString& left, const MyString& right) { 
    return strcmp(left.value, right.value) == 0; 
} 
bool operator < (const MyString& left, const MyString& right) { 
    return strcmp(left.value, right.value) < 0; 
} 

bool operator != (const MyString& left, const MyString& right) { 
    return !(left == right); 
} 
bool operator > (const MyString& left, const MyString& right) { 
    return !(left == right) && !(left < right); 
} 
bool operator <= (const MyString& left, const MyString& right) { 
    return left < right || left == right; 
} 
bool operator >= (const MyString& left, const MyString& right) { 
    return !(left < right); 
} 

Answer 2

私はstrcmpが非等価の場合にゼロ以外の値を返すと思います。したがって、1と-1を使用する代わりに、0より大きく0より小さいケースを試してみてください。

Answer 3

通常、operator==とoperator<を実装するだけです。他のすべての関係演算子を委譲して、コードの再利用を保証することができます。あなたのためにこれを行うことができるブーストにはCRTPミックスインさえあると私は信じている。

Answer 4

問題はstrcmpにあります。文書hereを見ることができます。

あなたはさらにようなものにコードを減らすことができます：あなたはちょうど2つのそれらの各コンパレータを表現することができ

bool operator < (const MyString& left, const MyString& right) 
{ 
    return (strcmp(left.value, right.value) < 0); 
}