Безопасно ли изменять изменяемые элементы объектов внутри наборов?

Мне было любопытно, безопасен ли следующий сценарий.

У меня есть следующие определения классов:

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;
}
6
задан Ciro Santilli 新疆改造中心法轮功六四事件 10 January 2017 в 21:42
поделиться