Кастомизировать и оптимизировать ответ (Response)?

Question

[Ilya Loopashko]

Вмем привет. У меня есть API CRUD контроллер, который после успешного выполнения возвращает результат.

Пример метода, удаления

public function destroy(Request $request)
{
    $idle = Idle::find($request->get('id'));
    if ($idle !== null) {
        [$status = "OK", $message = "Запись с таким id удалена"];
        $idle->delete();
    } else {
        [$status = "Error", $message = "Запись с таким id не обнаружена"];
    } 
    return response()->json([
        "status" => $status,
        "message" => $message,
        "data" => $idle,
    ]);
}

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

public function destroy(Request $request, Response $response)
{
$idle = Idle::find($request->get('id'));
$response->data['idle'] = $idle;
if ($idle !== null) {
    $response->status = 'OK';
    $response->message = "Запись с таким id удалена";
    $idle->delete();
} else {
    $response->message = "Запись с таким id не обнаружена";
} 
return $response;
}

Я смотрел вариант с созданем Ресурса или Хранилище(Repositories). Но там не то что мне надо.

Comments

[JhaoDa]

else {
    [$status = "Error", $message = "Запись с таким id не обнаружена"];
}

И что, это даже работает?

[Алексей Уколов]

JhaoDa, это же валидный синтаксис, хоть и кривой.

[JhaoDa]

Алексей Уколов, я в курсе. Полезную работу он какую выполняет?

[Алексей Уколов]

JhaoDa, видимо, польза для автора в экономии строчек заключается, я хз ¯\_(ツ)_/¯

[jazzus]

Но там не то что мне надо.

А здесь советуют не то, что тебе надо (с этим на фриланс), а как правильно. Например, правильно использовать model binding и api resources.

Answer 1

[Ilya Loopashko]

Спасибо всем за ответы, чуть покапался и реализовал через Хранилище(Repositories).

public function destroy(Request $request)
    {
        $idle = Idle::find($request->get('id'));
        if ($idle !== null) {
            $idle->delete();
            return $this->responseRepository->Response($idle, true, 'Запись с таким id удалена');
        }
        return $this->responseRepository->Response('Запись с таким id не обнаружена');
    }

ResponseRepository.php

public static function Response($message = "Ошибка", $data = null, $status = false)
    {
        return response()->json([
            'status' => $status,
            'message' => $message,
            'data' => $data,
        ]);
    }

Comments

[JhaoDa]

Про findOrFail() ещё не узнал, да? Ах да, ты же документацию не читаешь...

[Ilya Loopashko]

JhaoDa, И как мне это может помочь, ведь этот метод выдает либо 404 или исключение. Исключения я не ловлю, 404 мне не нужна.

[JhaoDa]

Ilya Loopashko,

Исключения я не ловлю

А ты лови.

404 мне не нужна

И правда, нафига следовать общепринятым подходам, изобретай свой!

[jazzus]

т.е. ничего не изменилось. лишняя прокладка над понятным json методом

Answer 2

[pLavrenov]

1) Название Response c большой буквы?
2) Response статическое ?! Оо
3) Зачем оно нужно в репозитории?
4) Изучить Dependency Injection в Laravel
5) Изучить REST и роутинг
6) Изучить валидацию в Laravel

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

public function destroy(Request $request, Idle $idle)
{
    // Оставил ее тут но ее надо бы вынести отдельно
    $request->validate([
        'some' => 'required',
    ]);

    return response()->json([
        'success' => $idle->delete(),
    ]);
}

Comments

[jazzus]

Если у тебя Idle $idle то в exists:idles,id смысла нет. И вообще ид не должен быть в реквесте. Да и в целом не должно быть вариантов, когда id не найден.

[pLavrenov]

jazzus, там подписано что валидацию надо вынести и тогда она отрабатывает до выполнения метода.

[pLavrenov]

lavren, но вообще ты прав валидация тут и не нужна

[jazzus]

lavren, ну да странно получать ошибку валидации или непонятный json там где 404. это все потому что тесты не пишутся, поэтому типа "куча вариантов" и свобода творчества)

[Ilya Loopashko]

lavren jazzus Спасибо.