C++ - 연산자가 없습니다.

Question

사용자 정의 유형의 값으로 채워진 벡터가 있는데, find() 알고리즘은 값 비교에 적합한 == 연산자를 찾을 수 없다는 불만을 제기합니다. 나는 이런 식으로 구현했습니다

bool Ship::operator==(const Ship& source) { 
    return (_type == source._type && 
      _damagedSquares == source._damagedSquares && 
      _orientation == source._orientation && _state == source._state); 
} 

나는 또한 "친구"방법의 접근 방식을 시도했지만 그 중 하나가 작동하지 않습니다. 클래스 자체는 다음과 같이 구성되어있다 :

class Ship { 
private: 
    ShipType _type; 
    int _damagedSquares; 
    ShipOrientation _orientation; 
    ShipState _state; 

public: 
    Ship(); 
    Ship(ShipType type); 
    ~Ship(); 

    bool operator==(const Ship& source); 
}; 

은 내가 잘못 여기서 뭐하는 거지?

추가 정보 :

std::vector<Ship> remainingShips; 
MultiArray& squares = opponentGridCopy.GetSquares(); 
for (RowIterator rowIterator = squares.begin(); rowIterator != squares.end(); 
    ++rowIterator) { 
    for (ColumnIterator columnIterator = rowIterator->begin(); 
     columnIterator != rowIterator->end(); ++columnIterator) { 
     Square* current = &(*columnIterator); 
     SquareState currentState = current->GetState(); 
     if (currentState != SquareState::Hit) 
      current->SetState(SquareState::Vacant); 
     Ship* potentialShip = current->GetOwner(); 
     if (potentialShip != nullptr) { 
      int damagedSquares = potentialShip->GetDamagedSquares(); 
      if (!damagedSquares) { 
       current->SetState(SquareState::Populated); 
       break; 
      } 
      if (remainingShips.empty() || 
       std::find(remainingShips.begin(), remainingShips.end(), 
          potentialShip) == 
        remainingShips.end()) // should be *potentialShip 
       remainingShips.push_back(*potentialShip); 
     } 
    } 
} 
return remainingShips; 

나는 비교 값으로 포인터를 전달했다 ... 단순히 그것을 참조를 취소하고 찾을 수는() 작동합니다.

Answer 1

Ship* potentialShip = ... 
std::find(remainingShips.begin(), remainingShips.end(), potentialShip) 

당신이 포인터를 찾기 위해 노력하고 검색이

std::vector<Ship> remainingShips; 

당신이 선박 객체에 대한 포인터를 비교하고 있으며 따라서 귀하의 비교가 잘못 정의되어 수행되는 벡터 동안

bool Ship::operator==(const Ship& source) // Accepts a Ship reference, not a pointer 

수정하려면 포인터를 역 참조하거나 비교 기능을 변경하십시오.

Answer 2

지금처럼 비교 연산자를 선언 :

bool Ship::operator==(const Ship &source) const 

참고는 const 후행.

Answer 3

뿐만 아니라

귀하의

bool operator==(const Ship& source); 

이어야한다 CONST, 즉

bool operator==(const Ship& source) const; 

그러나 실제로, 나는 대칭 연산자를 선호, 멤버 방법으로하지. 다음을 고려하십시오.

Class Ship 
{ 
private: 
    ShipType _type; 
    int _damagedSquares; 
    ShipOrientation _orientation; 
    ShipState _state; 

public: 
    Ship(); 
    Ship(ShipType type); 
    ~Ship(); 

    static bool eq(const Ship& s0, const Ship& s1) 
    { 
     return (s0._type == s1._type && 
     s0.damagedSquares == s1._damagedSquares && 
     s0._orientation == s1._orientation && 
     s0._state == s1._state); 
    } 

}; 

inline bool operator==(const Ship& s0, const Ship& s1) 
{ 
    return Ship::eq(s0, s1); 
}