Какую проверку лучше использовать?

Question

[QNA-1976]

В БД при выполнении миссии жёстко задано exp_mission = exp_mission + 10
Сейчас проверяю так:

if ($this->mission->exp_mission == 100) {
    throw new DomainException('Вы уже выполнили эту миссию полностью');
}
// тут запрос вида ... update missions set exp_mission = exp_mission + 10 where id = ? and id_mission = ? limit 1 ...

А может быть надо так?

if ($this->mission->exp_mission >= 100) {
    if ($this->mission->exp_mission > 100) {
        // тут запрос вида ... update missions set exp_mission = 100 where id = ? and id_mission = ? limit 1 ...
    }
    throw new DomainException('Вы уже выполнили эту миссию полностью');
}
// тут запрос вида ... update missions set exp_mission = exp_mission + 10 where id = ? and id_mission = ? limit 1 ...

Страхуем exp_mission = exp_mission + 10 или не нужно?

Answer 1

[Максим]

class Controller 
{
  private $repo;

  pulic function __construct(MissionRepository $repo) 
  {
    $this->repo = $repo;
  }

  pulic function excuteMission($id)
  {
    $mission = $this->repo->find($id);
    $mission->next();
    $mission->save();
  }
}

class Mission
{
  private $progress;

  public function next(): void
  {
    $progress = $this->progress + 10;

    if($progress > 100) {
      throw new DomainExeption('Миссия не может быть выполнена.')
    }
    if($progress = 100) {
      throw new DomainExeption('Миссия уже выполнена.')
    }
  }
}

А вообще используйте тесты и вы сами поймёте что нужно, что не нужно.

Comments

[QNA-1976]

Ну Вы просто описали, то что я сделал)

[Максим]

QNA-1976, не совсем так, но почти. А что мне нужно было описать соседний вопрос?)

Я склоняюсь ко второму варианту, но только без двойных if и два написанных запроса

А вообще используйте тесты и вы сами поймёте что нужно, что не нужно.

[QNA-1976]

Максим, во втором варианте не будет второй запрос отработан, если >= 100, там выброс идёт и запрос перед выбросом только если > 100

[QNA-1976]

Максим, всё я понял Ваш смыл: сначала добавляем свойству +10 а потом проверка и запись, ок

[Максим]

QNA-1976, я вижу. Но вы его напишите. Это дублирование кода. Лучше присваивать переменную и её заносить в базу.

[QNA-1976]

Максим, выше коммент)))

[Одиночка Айс]

Зачем всюду пихать ООП?

[Дмитрий]

В условии присваивание