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

Question

[Hfnas]

Доброго всем дня.

Подскажите, пожалуйста, как передать ошибку из 1 класса другому классу b, который вызвал 1класс? мне нужно вывести в попап ошибку из класса b .

try{
$cnt=0;
if ($cnt<0){
throw new Exception("нулевое количество");
}
} catch(Exception $e){
$this->errors=$e->getMessage();
            return array("errors"=>$this->errors);

}

Дебаг почему-то останавливается на строчке throw new Exception("нулевое количество"); -а дальше не пошел, а также `finally` не помог.

Наверное, придется плодить if else....

Comments

[hack504]

псс.. У вас фигурная скобка не лишняя?

[Hfnas]

hack504, Да , убрала... спасибо...

Все равно проблему не решает...
Может, вы знаете, как передавать пользователю в попап сообщение об ошибке?
Мне нужно, чтобы в тот момент, когда возникла ошибка в классе 1 (не выполняя дальше код), передать классу 2(запись в переменную-> вывод ошибки на экран c помощью этой переменной).

на просторах сети советуют использовать Exception, но я не знаю, как их применять правильно в моем случае.

[hack504]

try{
    if (cnt<0){
        throw new Exception("нулевое количество");
    }
} catch(Exception $e){
    $this->errors=$e->getMessage();
}
//кэтч отработал и выполнение идет дальше
    return array("errors"=>$this->errors);

[Hfnas]

Лентюй, это кол-во.(я часть кода привела...)

[Daria Motorina]

Hfnas, покажите больше подробностей в коде - где первый класс, который ловит ошибку, и где второй, который должен ее передать в попап.
На текущий момент могу помочь так. Скажем класс с этим try-catch - это класс А, а класс Б - тот, который дает респонс для попапа.
В классе А есть метод, который делает return с ошибкой из try-catch (кстати return array("errors"=>$this->errors); не пишут, пишут
return ["errors" => $this->errors];.
В классе Б надо вызывать метод из класса A, в котором этот try-catch лежит, сохранять его ответ и передавать эту ошибку в теле респонса "для попапа"

[Hfnas]

Лентюй, обновила ответ, я привела как пример.

[hack504]

Daria Motorina,

(кстати return array("errors"=>$this->errors); не пишут, пишут
return ["errors" => $this->errors];.

смотря какую версию php нужно поддерживать

[Daria Motorina]

Лентюй, извиняюсь, давно не работала с php 5.3 ^_^

Answer 1

[Михаил Ливач]

ПО-моему, исключения обычно используют по-другому: в одном месте их кидают, а в другом - ловят. Вот вам пример:

class A {

    /**
     * A constructor.
     * @throws Exception
     */
    public function __construct()
    {
        // $cnt откуда-то берется...
        $cnt = 0;
        if ( $cnt < 1 ) {
            throw new Exception("нулевое количество");
        }
    }
}

class B {

    private $errors = array();

    public function run()
    {
        try {
            $a = new A();
        } catch (Exception $e) {
            $this->errors[] = $e->getMessage();
        }

        return array("errors"=>$this->errors);
    }
}

Если по какой-то причине нельзя кидать исключение из конструктора ( уже не помню, но может быть для старых версий PHP это было плохо), то просто выносите код инициализации объекта в другой метод, а конструктор делаете пустым:

class A {
    /**
     * A constructor.
     */
    public function __construct()
    {
    }

    /**
     * @throws Exception
     */
    public function init()
    {
        // $cnt откуда-то берется...
        $cnt = 0;
        if ( $cnt < 1 ) {
            throw new Exception("нулевое количество");
        }
    }
}

class B {

    private $errors = array();

    public function run()
    {
        try {
            $a = new A();
            $a->init();
        } catch (Exception $e) {
            $this->errors[] = $e->getMessage();
        }

        return array("errors"=>$this->errors);
    }
}

Comments

[Hfnas]

Благодарю за столь развернутый ответ.

Еще момент, у меня не получалось из-за того, что у меня на сайте отладка была запрещена. ( возможно, php.ini - нужно было включить обработку ошибок).

Кому пригодится, возможно...

Answer 2

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

при использовании механизма исключений никаких return не должно быть
отучайтесь мыслить функциями и их возможностями

Comments

[Hfnas]

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

Как быть, тогда?

[hack504]

Hfnas,

try{
    if (cnt<0){
        throw new Exception("нулевое количество");
    }
} catch(Exception $e){
    $this->errors=$e->getMessage();
}
//кэтч отработал и выполнение идет дальше
    return array("errors"=>$this->errors);

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

Hfnas, я не говорил про передачу конечного результата. Я говорил про return ошибки, как у вас.

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

Answer 3

[Ninazu]

Можно ловить все ошибки в одном месте

<?php

class Response {

	public function sendResponse(array $data) {
		echo json_encode($data);

		return true;
	}

	public function sendError(string $message) {
		echo json_encode(['error' => $message]);

		return false;
	}
}

class ErrorHandler {

	private $response;

	public function __construct(Response $response) {
		$this->response = $response;
	}

	public function handlerError($error_code, $message, $file = null, $line = null) {
		if ($error_code) {
			throw new Exception($message, $error_code);
		}

		return true;
	}

	public function handlerShutdown() {
		$error = error_get_last();

		if ($error) {
			$this->handlerException(new Error($error['message'], $error['type']));
		}

		exit(0);
	}

	public function handlerException(Throwable $exception) {
		if (ob_get_length()) {
			ob_end_clean();
		}

		if (!headers_sent()) {
			return $this->response->sendError($exception->getMessage());
		}

		return true;
	}
}

$response = new Response();
$handler = new ErrorHandler($response);
register_shutdown_function([$handler, 'handlerShutdown']);
set_error_handler([$handler, 'handlerError']);
set_exception_handler([$handler, 'handlerException']);

#region Logic

$cnt = 0;

if ($cnt === 0) {
	throw new Exception("Нулевое количество");
}

#endregion

$response->sendResponse([
	'message' => "Всё окей"
]);

Comments

[Hfnas]

Благодарю за подробный ответ.
Я так понимаю, это создаем свой обработчик обработки ошибок.
Так , наверное, во фреймфорках применяют?

Кстати, с Днем программиста!

[Ninazu]

Hfnas, Да.