Как объединить (уменьшить код) для обработки исключений?

Question

[QNA-1976]

try {
    $this->mObj->checkOperation();
} catch (DomainException $e) {
    $this->alertRedirect('danger', $e->getMessage(), '/operations');
}
try {
    $this->mObj->checkMission();
} catch (DomainException $e) {
    $this->alertRedirect('danger', $e->getMessage(), "/operation-{$this->mObj->oid}/missions");
}

сразу говорю, что читал и видел, что вложенные как-то можно использовать, но нужен правильный конкретный пример данного случая, покажите, пожалуйста, чтобы не перемудрить

Comments

[QNA-1976]

Hellayas, ну если можно как-то, то почему не сделать, а если ерунда эта затея, то и бог с ним - не смертельно :)

[QNA-1976]

Hellayas, понял Вас, третий параметр не даст, спс

[QNA-1976]

Hellayas, я думал сделать вот так

public function checkOperation()
{
    if (!$this->operation) {
        throw new DomainException('Вам недоступна такая спецоперация', '/operations');
    }
}

public function checkMission()
{
    if (!$this->mission) {
        throw new DomainException('Вам недоступна такая миссия', "/operation-{$this->mObj->oid}/missions");
    }
}

try {
    $this->mObj->checkOperation();
    $this->mObj->checkMission();
} catch (DomainException $e) {
    $this->alertRedirect('danger', $e->getMessage(), $e->getCode());
}

Но уже не помню вроде ругается что превышены размеры поля code, что-то такое :)

[QNA-1976]

Hellayas, вот что выводит такое объединение

Wrong parameters for DomainException([string $message [, long $code [, Throwable $previous = NULL]]])

ну я короче оставил, как в вопросе

[QNA-1976]

Hellayas, мне проще, но я согласен с Вами

[Дмитрий]

QNA-1976, создайте себе эксепшонов, просто свои наследуя стандартные, в блоке catch проверяйте Trowable
Ну и можно в новых версиях объединять эксепшоны
https://www.php.net/manual/ru/language.exceptions.php
Пример #6

[QNA-1976]

Дмитрий, да порешали уже, спс

Answer 1

[Дмитрий Ким]

Так вы выкидывайте разные исключения:

class A {
    public function checkOperation() {
        throw new ADomainException;
    }
}

class B {
    public function checkMission() {
        throw new BDomainException;
    }
}

try {
    $this->mObj->checkOperation();
    $this->mObj->checkMission();
} catch (ADomainException $e) {
    $this->alertRedirect('danger', $e->getMessage(), '/operations');
} catch (BDomainException $e) {
    $this->alertRedirect('danger', $e->getMessage(), "/operation-{$this->mObj->oid}/missions");
}

Comments

[QNA-1976]

а откуда эти классы взялись?

[Дмитрий Ким]

Это для примера. Я ведь не знаю, как называется класс, где вызываются методы checkOperation и checkMission.

[Дмитрий Ким]

class MyObj {
    public function checkOperation() {
        throw new ADomainException;
    }

    public function checkMission() {
        throw new BDomainException;
    }
}

try {
    $this->mObj->checkOperation();
    $this->mObj->checkMission();
} catch (ADomainException $e) {
    $this->alertRedirect('danger', $e->getMessage(), '/operations');
} catch (BDomainException $e) {
    $this->alertRedirect('danger', $e->getMessage(), "/operation-{$this->mObj->oid}/missions");
}

[QNA-1976]

Дмитрий Ким, ADomainException - это откуда?

[Дмитрий Ким]

Да, что ж такое то.
У вас два метода, которые могут выкинуть одно и то же исключение. Но вы хотите по-разному обрабатывать эти исключения. Верно?
Не легче ли сделать так, чтобы эти два метода кидали разные исключения?
Пусть метод checkOperation() кидает исключение OperationDomainException, а метод checkMission кидает исключение MissionDomainException.
Тогда оба метода можно вызвать внутри блока try, а ловить исключения в разных catch.

try {
    $this->mObj->checkOperation();
    $this->mObj->checkMission();
} catch (OperationDomainException $e) {
    $this->alertRedirect('danger', $e->getMessage(), '/operations');
} catch (MissionDomainException $e) {
    $this->alertRedirect('danger', $e->getMessage(), "/operation-{$this->mObj->oid}/missions");
}

Первый блок catch будет ловить только исключения OperationDomainException, а второй только MissionDomainException.

[QNA-1976]

так где эти классы? ))) их же нету у меня MissionDomainException, OperationDomainException

Вы просто пишете мне эти названия?

[Дмитрий Ким]

Ну, так создайте их!

[QNA-1976]

Дмитрий Ким,

Ну, так создайте их!

)))

(уменьшить код)

[Дмитрий Ким]

QNA-1976, если вы место на диске экономите, то рекомендую переименовать методы checkOperation и alertRedirect в более короткие названия.
Не бойтесь создавать такие простые вещи как исключения и интерфейсы. Часто пользы от них больше, чем ожидается. Место на диске они, конечно же, занимают, но практической пользы от них обычно кратно больше.
Если вы хотите по-разному обрабатывать одно и то же исключение, выкидываемое разными методами, то логичнее кидать разные исключения из этих методов, чем писать костыли. Что я вам и предложил.