Мне было любопытно, безопасен ли следующий сценарий.
У меня есть следующие определения классов:
class ActiveStatusEffect
{
public:
StatusEffect* effect;
mutable int ReminaingTurns;
ActiveStatusEffect() : ReminaingTurns(0)
{
}
//Other unimportant stuff down here
}
Затем я сохраняю их группу внутри std :: set следующим образом:
struct ASECmp
{
bool operator ()(const StatusEffects::ActiveStatusEffect &eff1, const StatusEffects::ActiveStatusEffect &eff2)
{
return eff1.effect->GetPriority() < eff2.effect->GetPriority();
}
};
std::set<StatusEffects::ActiveStatusEffect, ASECmp> ActiveStatusEffects;
Я помечаю RemainingTurns как изменяемый, потому что я хочу иметь возможность изменять его без необходимости постоянно стирать / вставлять в набор. Т.е.
void BaseCharacter::Tick(Battles::BattleField &field, int ticks)
{
for (auto effect = ActiveStatusEffects.begin(); effect != ActiveStatusEffects.end();)// ++index)
{
auto next = effect;
++next;
if (effect->effect->HasFlag(StatusEffects::STATUS_FLAGS::TickEffect) && effect->ReminaingTurns > 0)
{
effect->effect->TickCharacter(*this, field, ticks);
--effect->ReminaingTurns;
}
if (effect->ReminaingTurns == 0)
{
ActiveStatusEffects.erase(effect);
}
effect = next;
}
}
меня беспокоит, потому что это может испортить порядок в наборе, то есть я не могу гарантировать, что набор всегда будет отсортирован по эффекту-> GetPrority ()
Если это правда, то это есть ли безопасный способ (например, не использовать RemainingTurns как часть ключа) для этого помимо копирования, изменения, стирания и вставки того, что мне нужно изменить?
EDIT:
@ildjarn - извините, я не думаю, это имело значение. Он просто возвращает int, хранящийся в StatusEffect. Этот int гарантированно не изменится во время выполнения программы.
int StatusEffect::GetPriority() const
{
return StatusPriority;
}