Что нужно возвращать в контроллере если коллекция пустая?

Question

[Дмитрий]

Здравствуйте, сайт с админкой на laravel, с фронта через axios идет запрос на получение товара по его slug.
В контроллере в случае если нет товара (ошибка в слаге) возвращается пустой json.

public function show($id)
    {
        $productAttachments = DB::table('product_attachments')->where('product_slug', $id)->select('attachment', 'type', 'id')->get();

        $product = Products::where('slug', $id)->first();

        return response()->json([
            'product' => $product,
            'productAttachments' => $productAttachments,
        ]);
    }

Что нужно возвращать если коллекция пустая? Возвращать пустой json и обрабатывать его на фронте (react)? Либо другой вариант? Спасибо.

Answer 1

[Алексей Николаев]

Пустой JSON - самый корректный вариант. Отсутствие элементов не является исключением. У меня во всех проектах используется унифицированный формат ответа:

{
    "status": "success",
    "response": []
}

Так что просто будет пустой массив. Вам рекомендовал бы тоже прийти к формату наподобие вышеизложенного.

Comments

[Alex Wells]

На самом деле нахрен не нужен status: success, success: true, response и подобные им бесполезные ключи. Никакого профита от слова "совсем" - только лишний оверхед как на беке, так и на фронте.

[Алексей Николаев]

Alex Wells, ответ неверный. Это, во-первых, дает стройность и логичность структуре, дает ей свойство, которое есть всегда. Дает ей единый формат, что крайне важно. А во-вторых, дает возможность не обвешивать фронт бесчисленными проверками контента, а проверять лишь свойство "success". А то будут там всякие утиные типизации, "если в контенте то-то, то показываем это, если то-то, то вот это, если закрякал, то уточка". Это неправильно.

А оверхед в доли килобайта - такая экономия на спичках, что просто смешно.

[Alex Wells]

Алексей Николаев, зачем проверять контент? Проверять нужно HTTP status code - для этого он и был придуман. Есть 204 no content, есть 200/201 - и любой из этих статусов ЭЛЕМЕНТАРНО проверяется любой существующей библиотекой типа axios, вызовом одной функции (или вовсе успешный/неуспешный промис). Нет нужды отправлять и http code, и свой success.

А какой смысл от такого единого формата? В чем толк от ключа response? В чем толк от success? Ни в чем. Буквально. Смысла ровно ноль.