Как в Laravel правильнее инициализировать класс чтоб его методы и переменные были доступны везде?

Question

[Слава]

Создали класс, в котором есть массив - это список кодов ошибок и сообщений а так же метод, который по коду ошибки выдает соощение и возвращает это в виде
return response()->json(
.......
);

и как бы лучше сделать чтобы можно во всех контроллерах вызывать эту строку типа
return $this->responser->getReturnData($ErrorCode);

Можно в каждом контроллере переопределить конструктор и там заводить объект класса при помощи нью

$this->responser = new RectonderClass;

а потом в каждом методе обращаться к объекту класса и его методу. Но есть же более правильные методы?

Comments

[jazzus]

Нужен трейт с методами. хелпер response упаковываем в responser, а его в getResponser (для подстраховки), а для сообщений делаем getTranser - transer в котором хелпер trans, а для кодов getCode(errorCode) со switch case return int. подключаем к контроллеру и вызываем через this.

[JhaoDa]

Предложение «прочитать документацию и сделать как там предлагают», полагаю, будет отклонено...

[Слава]

jazzus, не совсем понял только. запутался. у меня то класс всего лишь из одного массива и одного метода состоит.

private $errors = [
        ''NOUSERID' => ['error_text' => 'ID пользователя не указан', 'status' => 200],
        'NOROLE' => ['error_text' => 'Роль пользователя не указана', 'status' => 200],
        'CANTDELETE' => ['error_text' => 'Невозможно удалить', 'status' => 200],
        'ACCES_DENIED_FOR_ACTIVATION' => ['error_text' => 'Недостаточно права доступа для активации пользователя', 'status' => 200],
        '1401' => ['error_text' => 'Вы не авторизованы или сессия устарела', 'status' => 200],
        'SERVER_STATUS_401' => ['error_text' => 'Вы не авторизованы или сессия устарела', 'status' => 200],
        'SERVER_STATUS_403' => ['error_text' => 'В доступе отказано', 'status' => 200],
        'SERVER_STATUS_404' => ['error_text' => 'Страница не найдена', 'status' => 200],
        'SERVER_STATUS_500' => ['error_text' => 'Внутренняя ошибка сервера', 'status' => 200],
        'SERVER_STATUS_502' => ['error_text' => 'Ошибочный шлюз', 'status' => 200],
        'SERVER_STATUS_503' => ['error_text' => 'Сервис недоступен', 'status' => 200],
        'SERVER_STATUS_504' => ['error_text' => 'Шлюз не отвечает', 'status' => 200],
    ];

    public function getReturnData($code)
    {
        $error = $this->errors[$code];
        return response()->json(
            [
                'error' => $error['error_text']
            ],
            $error['status']
        );
    }

[Слава]

JhaoDa, где именно? с тем же успехом можно было бы сказать "предложение обучиться на программиста и потом написать используя полученные знания" или "изучить php".

но в документации очень плохо объяснены такие вещи. совсем не описано ядро и как используются патерны. Будто отдельные листы из книги вырывали. Это утверждают многие специалисты высокого уровня. В докуменатации можно почитать базовые вещи, но ответ на сложные вопросы там не найти если сам не знаешь соседние фреймворки в совершенстве.

[Антон Антон]

JhaoDa, А "как предлагают в документации" - это про сервис провайдеры или про примеси? Или про фасады? или про DI/хелперы?

[JhaoDa]

Антон Антон, да. И ещё макросы. Несколько вариантов, можно выбрать. Я, конечно, за DI.

[Антон Антон]

JhaoDa, ну вот вопрос, похоже, про то, что выбрать и почему

[JhaoDa]

Антон Антон, учитывая прошлые вопросы автора и вообще его взгляд на происходящее — нет, вопрос не про выбор, он просто ждёт решения.

[jazzus]

Слава, мда..

[Слава]

JhaoDa, ну вот ты и сел в лужу. вопрос как раз про то как лучше это сделать. чтобы правильно подобрать способ решения. а ты по прежнему паразитируешь на этом форуме занимаясь бестолковой деятельностью принося нуль пользы а лишь направляее на ложный след людей.

[Слава]

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

[jazzus]

Слава, в Ларавел уже завернули

Answer 1

[jazzus]

Внезапно пришла абсурдная идея (она автору не понравится), что
return response()->json(trans('errors.500'), 500);
итак прекрасно работает

Comments

[Слава]

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

return $this->responser->getErrorByCode("USER_NOT_FOUND");

вот это идеально. Но мой вопрос был в том какими средствами фреймворка это реализовать. Есть dependency injection , а можно просто статический метод класса использовать. меня интересовало что скажут другие и чем обоснован их выбор. А вместо этого я наткнулся на дискость с отсылом к документации.

[jazzus]

Слава, тут наши полномочия всё

Answer 2

[Aleksey]

Вы можете, например, сделать это в конструкторе файла app/Http/Controllers/Controller.php

class Controller extends BaseController
{
    protected Responser $responser;

    public function __construct(Responser $responser)
    {
        $this->responser = $responser;
    }
}

НО ИМХО то что вы пытаетесь сделать полная фигня. Статусы HTTP которые вы пытаетесь переопределить и переописать уже придуманы и описаны.

'NOUSERID' => ['e...' 200????
// Лававел автоматически вернёт статус 422 и все ошибки валидации
// Вместо этого вы ловите эксепшн валидации и вручную выбираете только первую ошибку?

'SERVER_STATUS_401'...503 
// Без комментариев. Вы опять вручную ловите например ошибку 401 и возвращаете её со статусом 200?
// А на клиенте не понимаете что нужно показать форму входа, ведь статус 200 а сравнивать
// if (error === 'Вы не авторизованы или сессия устарела') звучит бредово. 
// Завтра вы точку в текст добавите и всё сломалось

Вот статусы в читаемом виде, если уж нужно где-то ручное управление https://github.com/symfony/http-foundation/blob/5....