Как понять такое описание?

Question

[QNA-1976]

Не надо располагать try/catch и throw на одном уровне — в этом случае проще написать if:

try {
    ... 
    throw new Exception(...);
    ...
} catch (Exception $e) {
    ...
}

Поясните, пожалуйста:

в этом случае проще написать if

, если можно с примером.

Answer 1

[McBernar]

Это значит, что throw должно кидаться из вложенного уровня.

В try идет, например, вызов функции, которая может выкинуть эксепшен. В catch этот эксепшен ловится. Эксепшены поднимаются наверх.

В вашем случае throw кидается непосредственно из try, что теряет всякий смысл. Это имеет смысл только в том случае, если этот блок try-catch сам находится в функции и хочет пробросить ошибку на уровень выше, но тогда здесь не нужен catch.

Comments

[QNA-1976]

а if как работает в исключениях?

Не надо располагать try/catch и throw на одном уровне — в этом случае проще написать if

[McBernar]

QNA-1976, в этом конкретном случае вам вообще не нужны исключения. Об этом вам и говорят - просто замените try-catch на if-else.

if (true) {
  doSmth();
} else {
  doOther();
}

[QNA-1976]

вот у меня метод внутри try и он в случае ошибки выбрасывает исключение
а если он не выбросил исключение и мне теперь нужно сообщение об удаче, мне где его расположить отдельно после try-catch или выбросить внутри try и дополнительным catch обработать?

[Дмитрий]

QNA-1976, в своих классах не нужен try/catch просто бросать исключения, ловить ловить уже при использовании, например шторм сообщает что метод кидает исключение

[QNA-1976]

Дмитрий, без try-catch у меня исключения улетают в логи еррор-хендлером, мне нужно сообщение об удаче вывести: мне внутри try выбросить и дополнительным SuccessException обработать?

или просто вывести сообщение об удаче и где оно должно идти: внутри try после кода или после try-catch

[Дмитрий]

QNA-1976, ну можно и там и там, хотя после наверное правильнее

[Дмитрий]

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

[QNA-1976]

Дмитрий, просто в моём понятии исключение - это ошибка, поэтому я для удачной обработки как-то не думал их использовать: просто сообщения выводил в таком случае, ок попробую

[FanatPHP]

У тебя уже совсем какие-то бредовые идеи пошли.
Тебе правильно сказали, не нужен здесь try/catch
Надо вывесьи сообщение об успехе? НУ ТАК ВЫВЕДИ. Просто выведи.
Если было исклюбчение, то оно не выведется. Если не было - ито выведется. Все просто как 2 копейки.
А ты уже какие-то SuccessException начал выдумывать - совсем уже бред пошел

[QNA-1976]

FanatPHP, я про такую схему

try {
    $mObj->signinHandler(); // мне тут в этом методе выбрасывать и Danger и Success???
} catch (DangerAlertException $danger) {
    $this->alertRedirect('danger', $danger->getMessage()); // ошибка
} catch (SuccessAlertException $success) {
    $this->alertRedirect('success', $success->getMessage()); // Всё ништяк :)
}

try {
    $mObj->signinHandler(); // или мне тут ТОЛЬКО Danger выбрасывать
    throw new SuccessAlertException('Всё ништяк'); // и потом тут Success
} catch (DangerAlertException $danger) {
    $this->alertRedirect('danger', $danger->getMessage());
} catch (SuccessAlertException $success) {
    $this->alertRedirect('success', $success->getMessage());
}


try {
    $mObj->signinHandler(); // или мне тут ТОЛЬКО Danger выбрасывать
} catch (DangerAlertException $danger) {
    $this->alertRedirect('danger', $danger->getMessage());
}
$this->alertRedirect('success', 'Всё ништяк'); // и потом вот так?

[FanatPHP]

Делай по второму варианту. Чем больше ада, тем лучше.

[FanatPHP]

Да, кстати.

В вашем случае throw кидается непосредственно из try, что теряет всякий смысл.

Тут товарищ McBernar полный бред написал, пытаясь натянуть сову на глобус.

Answer 2

[DevMan]

исключения не выбрасываются просто так, они выбрасываются при каких–то условиях.

try {
	if(! file_exists($file)) {
		throw new Exception('file not exist');
	}
} catch (Exception $e) {
	echo $e->getMessage();
  exit;
}

не имеет смысла, поскольку спокойно сокращается до

if(! file_exists($file)) {
	exit('file not exist');
}

Comments

[QNA-1976]

вот у меня метод внутри try и он в случае ошибки выбрасывает исключение
а если он не выбросил исключение и мне теперь нужно сообщение об удаче, мне где его расположить отдельно после try-catch или выбросить внутри try и дополнительным catch обработать?

[kafkiansky]

Хороший совет exit вызывать, особенно если это веб, а не консоль.

[QNA-1976]

kafkiansky, спс, ну у меня все сообщения пользователю сессиями выводятся после редиректа, там exit уже есть в методе вывода сообщений )

[kafkiansky]

QNA-1976, это был сарказм

[DevMan]

kafkiansky, с чего вы решили, что это совет?

[kafkiansky]

DevMan, по приведенному коду.

[DevMan]

kafkiansky, это пример, а не совет.

Answer 3

[FanatPHP]

Никак не надо понимать.
Автор сам плохо понимает, о чем пишет.

Располагать на одном уровне абсолютно нормально, у него выше куча таких примеров.

Исключение работает как локальный die. Или даже скорее return. То есть код после него исполняться не будет. То есть вместо кучи вложенных иф можно делать кучу throw. Ну в общем получить такой аналог goto :catch.

Но проще всего конечно спросить у самого автора, что он имел в виду, благо это гитхаб, а не бумажная книжка. Создать issue это два клика

Comments

[QNA-1976]

код после него исполняться не будет

finally

[QNA-1976]

Располагать на одном уровне абсолютно нормально

ну вот у меня метод внутри try и он в случае ошибки выбрасывает исключение
а если он не выбросил исключение и мне теперь нужно сообщение об удаче, мне где его расположить отдельно после try-catch или выбросить внутри try и дополнительным catch обработать?

[FanatPHP]

В общем, если ты сам не понимаешь идиотизм этого вопроса, то любые объяснения бесполезны.