CRelayTracker: Make HasRelay() a const member function

While we're at it, we can make use of HasRelay() internally where
applicable to simplify code a little bit.
This commit is contained in:
Lioncash 2020-03-18 01:03:32 -04:00
parent 6ddbd15cfc
commit cb4884fcef
2 changed files with 13 additions and 7 deletions

View File

@ -23,18 +23,24 @@ CRelayTracker::CRelayTracker(CBitStreamReader& in, const CSaveWorld& saveworld)
}
}
bool CRelayTracker::HasRelay(TEditorId id) {
return std::find(x0_relayStates.begin(), x0_relayStates.end(), id) != x0_relayStates.end();
bool CRelayTracker::HasRelay(TEditorId id) const {
return std::find(x0_relayStates.cbegin(), x0_relayStates.cend(), id) != x0_relayStates.cend();
}
void CRelayTracker::AddRelay(TEditorId id) {
if (std::find(x0_relayStates.begin(), x0_relayStates.end(), id) == x0_relayStates.end())
x0_relayStates.push_back(id);
if (HasRelay(id)) {
return;
}
x0_relayStates.push_back(id);
}
void CRelayTracker::RemoveRelay(TEditorId id) {
if (std::find(x0_relayStates.begin(), x0_relayStates.end(), id) != x0_relayStates.end())
x0_relayStates.erase(std::remove(x0_relayStates.begin(), x0_relayStates.end(), id), x0_relayStates.end());
if (!HasRelay(id)) {
return;
}
x0_relayStates.erase(std::remove(x0_relayStates.begin(), x0_relayStates.end(), id), x0_relayStates.end());
}
void CRelayTracker::SendMsgs(TAreaId areaId, CStateManager& stateMgr) {

View File

@ -31,7 +31,7 @@ public:
CRelayTracker() = default;
CRelayTracker(CBitStreamReader&, const CSaveWorld&);
bool HasRelay(TEditorId);
bool HasRelay(TEditorId) const;
void AddRelay(TEditorId);
void RemoveRelay(TEditorId);
void SendMsgs(TAreaId, CStateManager&);