Как правильно завершить работу скрипта PHP?

Question

[Даниил]

Здравствуйте! Есть консольное приложение, которое на входе принимает некоторые параметры и выводит результат. Если параметр введён некорректно, то перехватывается ошибка, выводится пользовательское сообщение и завершается работа скрипта через die() / exit(), так как дальнейшее его выполнение не имеет смысла.

Код просто для примера:

class App
    {
        public static function checkEven ($number)
        {
            try {
                if ($number % 2 !== 0) {
                    throw new Exception("Odd number");
                } 
                return $number . ' even' . PHP_EOL;
            } catch (Exception $error) {
                print_r($error->getMessage() . PHP_EOL);
                
                die(); // @codeCoverageIgnore
                print_r("Too late");
            }
        }
    }
    
    print_r(App::checkEven(9));

При тестировании UNIT-тестами с некорректными входными данными завершается работа приложения вместе с самим тестированием (что не удивительно). Запись с // @codeCoverageIgnore не помогла.

Существуют ли другие способы пропустить выполнение die() или исключения следует обрабатывать иначе? Если да, то как?

Comments

[v3shin]

throw и проверять исключение в тестах.

[FanatPHP]

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

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

[Vitsliputsli]

В дополнение, то что вы хотите вывести что-то на экран, нисколько не отменяет, что нужно бросить Exception. Т.е. выводите что нужно и снова бросаете тот же Exception, без всяких die(). А так как вы скорее всего хотите обрабатывать так, все исключения, то все это нужно вынести в общий обработчик исключений.

[FanatPHP]

в целом же надо пересмотреть структуру приложения.
я не вижу здесь ни одного повода использовать исключения.
исключение должно возникать в ислючительном случае, а не штатном.

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

[FanatPHP]

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

[Vitsliputsli]

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

Answer 1

[Decadal]

>Существуют ли другие способы пропустить выполнение die() или исключения следует обрабатывать иначе? Если да, то как?

замените die на return и поменяйте его местами с print_r. Вам на самом деле не нужно насильно обрывать работу php - как только код выполнится, обработка запроса завершится сама.
Если вы опасаетесь что где-то ниже по коду выполнится какое-то действие, которое должно выполниться только в случае успешного завершения - добавьте туда дополнительное условие которое будет проверять результат, либо перехватывайте исключение именно там, а в catch оставьте пустоту. И всё будет хорошо.

Comments

[FanatPHP]

а в catch оставьте пустоту.

чего?

[Decadal]

>чего?

что чего? игнорирование ошибок так делается. По какой ещё причине стоит die, если не для того чтобы не мусорить аутпут?

[FanatPHP]

о господи
откуда у вас у всех такая каша в голове? Что ни сам вопрос, ни даже свой ответ понять не можете, не говоря уже о теме, на которую взялись рассуждать.

Где у автора желание "игнорировать ошибки"? ему наоборот надо ошибку вывести.

[Decadal]

>Где у автора желание "игнорировать ошибки"? ему наоборот надо ошибку вывести.

я ему сказал всё что нужно. Если захочет, задаст вопрос.

Answer 2

[Северное Сияние]

https://github.com/nategood/commando