Как организовать обработку исключений set_exception_handler() для их красивого вывода пользователю?

Question

[bedolazhka]

set_exception_handler([$this, 'errorHandler']);
public function errorHandler(Error $error) {
    $this->error = $error->getMessage();
    debug($this->error); // своя функция дебага
}

public function signin()
{
    $data = $this->pdo('select id, password_hash from users where login = ? limit 1', [
        $this->login
    ])->fetch();
    if (is_object($data) and password_verify($this->password, $data->password_hash)) {
        if (!is_null($_SESSION['uid'] = $data->id)) {
            return true;
        }
        throw new Error('Что-то пошло не так'); // раньше было return $this->error = 'Что-то пошло не так';
    }
    throw new Error('Данные введены неверно'); // раньше было return $this->error = 'Данные введены неверно';
}

public function signinAction()
{
    if (filter_has_var(INPUT_POST, 'do')) {
        if ($this->mObj->signin() === true) {
            $this->alertRedirect('success', 'Добро пожаловать');
        }
        $this->alertRedirect('danger', $this->mObj->error);
    }
}

Имеем белый экран с надписью, например, "Данные введены неверно".
Согласно докам после вызова обработчика выполнение будет остановлено.
Можно ли как-то воплотить задуманное в жизнь (продолжить выполнение) или долбить везде try/catch/finally?

Answer 1

[OnlyMyQuestion]

Нет нельзя. set_exception_handler вызывается после того как происходит необработанное исключение, контекст, естественно, прерывается.

Answer 2

[bedolazhka]

Долбить везде try/catch/finally.

Answer 3

[FanatPHP]

"красиво выводить пользователю" надо не исключения, а абстрактный текст "что-то пошло не так". А само исключение должно логироваться. А так же выводиться для программиста в режиме разработки.
Если сообщение надо вывести пользоваелю, то это не исключение. Вот пример нормального (хотя и с минимальной функциональностью PHP exception handler-a

Другое дело что код здесь какой-то странный, и непонятно, как он вообще работает. Почему this? Это вызов внутри метода класса? Что за странная идея устанавливать обработчик ошибок внутри класса? Почему вообще обработчик ошибок устанавливается в каком-то пользовательском классе? А если будет класс Товар - у него тоже будет свой глобальный обработчик ошибок? Зачем бросать исключение если данные введены неверно?
А, главное, какой смысл пихать ошибку в переменную? Что с ней потом делать? Солить на зиму? И да, как правильно написал OnlyMyQuestion, это не будет работать.

Ох. Не посмотрел на юзернейм сначала. Все зря значит писал. Не дойдет.
А жаль. Вопрос-то очень хороший, если его с умом задать.

Comments

[bedolazhka]

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

зачем смотреть на код $this->alertRedirect('danger', $this->mObj->error), лучше же задать вопрос, для чего переменная $this->error

И да, как правильно написал OnlyMyQuestion, это не будет работать

А зачем читать вопрос в котором автор уже про это написал

Согласно докам после вызова обработчика выполнение будет остановлено.

хотя да, нужно было мне сначала посмотреть на ник автора высера

[FanatPHP]

Ох, не трогал бы ты ооп пока.
Ничего хорошего из этого не выйдет.

[bedolazhka]

FanatPHP,

public function signin()
{
    $data = $this->pdo('select id, password_hash from users where login = ? limit 1', [
        $this->login
    ])->fetch();
    try {
        if (is_object($data) and password_verify($this->password, $data->password_hash)) {
            if (!is_null($_SESSION['uid'] = $data->id)) {
                return true;
            }
            throw new Error('Что-то пошло не так'); // чтобы не делать возврат строки return $this->error = 'Что-то пошло не так';
        }
        throw new Error('Данные введены неверно'); // чтобы не делать возврат строки return $this->error = 'Данные введены неверно';
    } catch (Error $error) {
        $this->error = $error->getMessage();
        return false;
    }
}

в комментариях ответы на Ваш следующий вопрос

[FanatPHP]

Убрать весь этот ужас, оставить только

public function signin()
{
    $data = $this->pdo('select id, password_hash from users where login = ? limit 1', [
            $this->login
    ])->fetch();
    if ($data and password_verify($this->password, $data->password_hash)) {
        $_SESSION['uid'] = $data->id;
        return true;
    }
}

а для вывода "данные введены неверно" проверять результат вызова, т.е.
if (!$user->signin()) // сообщаем юзеру.
Хотя по-хорошему и работу с сессией надо из этого метода выкинуть.

[FanatPHP]

по хорошему это должен быть метод модели,

public function auth()
{
    $data = $this->pdo('select id, password_hash from users where login = ? limit 1', [
            $this->login
    ])->fetch();
    return $data and password_verify($this->password, $data->password_hash);
}

А работа с сессией и общение с юзером должны происходить в контроллере.

[bedolazhka]

FanatPHP, смысл всего действа: избавиться от возврата строки вместо false и возвращать именно false

Вы же сами писали, зачем куча инклудов конфига перед созданием подключения

тогда зачем мне возвращать объект для проверки на якобы true?

return data and ......

я уже сделал, как мне нужно

[FanatPHP]

return data and ...... врзвращает булево значение, дурачок.
В том что ты сделал, я не сомневаюсь. мне просто тебя жалко.

[bedolazhka]

FanatPHP, return data вернёт объект, умничка, если данные есть

fetch() - либо объект либо false

ах да забыл все же дебилы кругом

[FanatPHP]

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

[bedolazhka]

FanatPHP, мне не надо возвращать в данном случае объект - это точно, я его неплохо на именно объект и так могу проверить, нужно вернуть true или false явно

[FanatPHP]

ну да, мы снова уперлись в то что ты русски язык не понимаешь :(
Я тебе уже два раза написал открытым текстом что мой код возвращает булево значение. и даже пример привёл.
А ты все долдонишь "мне не нужен объект".
ладно, делай по-своему, все равно через час прибежишь "дядиньки у меня ничиво ниработаиииит!"

[bedolazhka]

FanatPHP, нет, нельзя проверить $data на if ($data === true) после fetch(), вот и всё

[FanatPHP]

а кто где проверяет $data на if ($data === true) после fetch()? и зачем?

[bedolazhka]

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

[FanatPHP]

а тебе никто не предлагает возвращать объект. код, который я написал, возвращает булево значение. Булево значение - это true или false

[bedolazhka]

FanatPHP, не вернёт fetch true, я делаю метод, который возвращает либо true либо false, уже писал же смысл: избавится от возврата строки вместо false и вернуть именно false (и чтобы в $this->error было сообщение для вывода юзеру)

[FanatPHP]

при чем здесь fetch(), господи?
Я тебе написал метод, который возвращает либо true либо false
без всего того ада, который ты понаписал

а никакого $this->error вообще не должно быть в классе модели. и уж тем более дял вывода юзеру. С юзером общается контроллер, а не модель

[bedolazhka]

FanatPHP, return data не вернёт true, если есть данные, мне не нужно возвращать объект для проверки на не false (якобы true), в модели есть валидатор у него есть свойство для сообщений об ошибках, вот его и заполняем, чтобы в контроллере не было логики по выводу сообщений, а был только их вывод в вид

валидация может содержать до 20-ти проверок, мне в контроллере не нужна логика когда какое выводить

[FanatPHP]

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

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

[bedolazhka]

FanatPHP, он не вернёт true

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

супер

[bedolazhka]

но это было интересно: вернуть ОБЪЕКТ для проверки на не false )))

[Araik]

bedolazhka,
Эта конструкция возвращает true или false

return $data and password_verify($this->password, $data->password_hash);

Читается по русски как:

вернуть true, если $data == true И password_verify($this->password, $data->password_hash) == true

https://www.php.net/manual/ru/language.operators.l...

$a and $b
TRUE, если и $a, и $b TRUE.

[bedolazhka]

Araik, нет, fetch() не вернёт true

[bedolazhka]

ещё один пришёл объект возвращать )))

[Araik]

bedolazhka, а вы где-то увидели fetch в моем сообщении?

[bedolazhka]

Araik, что у Вас в $data?

[Araik]

bedolazhka, там же не return $data, там

return ($data and password_verify($this->password, $data->password_hash));

(я скобки расставил, чтобы было понятнее)

[bedolazhka]

Araik, ну так что в $data?

[Araik]

За столько часов можно было бы и проверить код, даже если в $data объект, конструкция return ($data and password_verify('','')) будет возвращать не объект, а булевое значение (true/false) т.к. это будет результатом логической операции.

[bedolazhka]

Araik, и то что вернёт конструкция можно проверить на === true? ))) ну-ну, объект приводить к типу это даже интересно становится, fetch() не возвращает true у меня, либо объект либо false, я не собираюсь даже возвращать объект, я из метода возвращаю либо true либо false

[Araik]

bedolazhka, Ты похоже перегрелся)) но я все равно отвечу, да, конструкция вернет булевое значение, либо true, либо false и соответственно, можно проверить на === true; это же проверить легко, сделай var_dump() и убедись.

Вот это:
return ($data and password_verify('',''))

Не тоже самое, что и
return $data

По твоей логике, что следующий код?

return (4 + 6)

Сумму двух чисел или 4 ?))

[bedolazhka]

можно проверить на === true

$a = (object)[];
if ($a === true) {
    // заранее даже ничего сюда не писал
} else {
    echo 'Иди на хер';
}

результат: Иди на хер

[Araik]

bedolazhka, вот так попробуй:

$a = (object)[];
$b = 'belolazhka тупой';

if ($a and $b) {
echo 'тупой грубиян идет на хер'
} else {
echo "тупой грубиян не хочет учится и идет на хер";
}

Результат: Parse error: syntax error, unexpected end of file, expecting ',' or ';'
Почему догадаешься сам.

[bedolazhka]

Araik, то есть всё-таки будете объект из базы тащить и возвращать в контроллер для проверки на true))), рад за вас

[FanatPHP]

Araik, у меня реально только один вопрос. Как он с таким своеобразным стилем мышления до ООП добрался :)

[Araik]

bedolazhka,

<?php
function func_name () {
    $a = new stdClass();
    $a->message = 'не получишь ты доступ к этому свойству, вне функции, потому что функция не вернет этот объект';
    $b = true;
    
    return $a and $b;
} 

$result = func_name();
var_dump($result);

// Чтобы прям точно был уверен
var_dump($result->message);

Результат выполнения:

bool(true)

Notice:  Trying to get property 'message' of non-object in [...][...] on line 14
NULL

sandbox.onlinephpfunctions.com/code/a146b37a46cc69...
(чтобы выполнить код по ссылке, нужно нажать на Execute code)

FanatPHP, я вообще начал думать, что это пранк какой-то))

[bedolazhka]

FanatPHP, мне не нужно объект возвращать, чтобы проверить его на не false, я true возвращаю

[bedolazhka]

Araik, значит всё-таки объект ретёрним? )))

[Araik]

bedolazhka, Я пытался указать на ошибку и помочь разобраться, если тебе это не нужно, то и я свое время тратить не намерен. Успехов в учебе.

[bedolazhka]

Araik, ясен пень не нужно, return true же сделано уже давно без возврата объектов