Laravel Service Layer обработка ошибок?

Question

[SM_ST]

Как правильно организовать сервисный слой, чтобы отлавливать ошибки и переиспользовать в дальнейшем?

контроллер

public function store(RetailProductCreateService $productCreateService, RetailProductCreateRequest $request)
    {
        $retail = Retail::getRetailStoreIdByCompanyId(50);

        $productCreateService->store($retail->id, $request->getDto());
    }

сервис

class ProductCreateService
{
    public function __construct(
        private ProductImageService $productImageService,
    ) {
    }

    public function store(ProductCreateDto $dto)
    {
        DB::beginTransaction();

        try {
            $product = new Product();
            $product->category_id = $dto->getCategoryId();
            $product->name = $dto->getName();
            $product->description = $dto->getDescription();
            $product->meta_title = $dto->getMetaTitle();
            $product->meta_description = $dto->getMetaDescription();
            $product->text = $dto->getText();
            $product->slug = $dto->getSlug();
            $product->price = $dto->getPrice();
            $product->old_price = $dto->getOldPrice();
            $product->competitor_prices = $dto->getCompetitorPrices();
            $product->consist = $dto->getConsist();
            $product->settings = $dto->getSettings();
            $product->is_published = $dto->getIsPublished();
            $product->is_accounting = $dto->getIsAccounting();

            $product->saveOrFail();

            $images = $this->productImageService->storeProductImages($product->id, $dto->getImages());

            if ($images) {
                $product->path = $images[0]['path'];
                $product->image = $images[0]['image'];
                $product->saveOrFail();
            }
            DB::commit();

            return true;
        } catch (QueryException $e) {
            DB::rollBack();
        }
    }
}

Думал в catch делать return false, но не смогу намекнуть на тип ошибки, например если такая запись есть (в валидации не вариант делать unique slug )

потом думал сделать return throw new .... , но тоже неправильно и сервиса выпуливать сразу ответ

Answer 1

[pLavrenov]

sentry.io - попробуй этот сервис и забудь про отслеживание ошибок Установка
--------
Исключения вызывают событие и в слушателе этого события можно реализовать свою логику. Не нужно никаких return throw

Comments

[SM_ST]

Да, я знаю про этот сервис и для администрирования сайта я буду использовать, здесь речь о том, как сервис переиспользовать и какое выдавать пользователю сообщение

[FanatPHP]

В огороде бузина, а в киеве дядька.
каким местом этот сервис относится к вопросу?

Answer 2

[FanatPHP]

в catch надо проверить тип ошибки.
если это ошибка "такая запись есть", то кинуть исключение, которое будет понято контроллером, и преобразуется в пользовательскую ошибку
если это любая другая ошибка, то просто throw $e;

} catch (QueryException $e) {
    DB::rollBack();
    if (условие по которому видно что запись уже есть) {
        throw new NonUniqueValueException(); 
    } else {
        throw $e;
    }
}

и тогда потом в контроллере можно будет

try {
    $productCreateService->store($retail->id, $request->getDto());
}catch(NonUniqueValueException $e){
    // пишем юзеру
}

Comments

[SM_ST]

Да, а как из контроллера выводить? Тоже в try обернуть? А если мне нужно будет в другом контроллер проверить если успешно сохранилось то выполнить один код, если не успешно, то другой, сейчас в моей записи сразу идёт прерывание и кидается пользователю Exception

[FanatPHP]

да, обернуть в трай, который ловит только NonUniqueValueException.
ну или ловит все DataValidationExcepton, чтобы превратить их в пользовательские ошибки.

если другой контроллер знает, как обработать NonUniqueValueException, то в нем тоже обернуть в трай
если не знает, то не оборачивать

[SM_ST]

FanatPHP, мне нужно знать тип ошибки, чтобы пользователю самому писать сообщения

[FanatPHP]

тип исключения - это и есть тип ошибки
то есть если мы ловим NonUniqueValueException ты мы в try уже знаем, какой это тип
если ловим DataValidationExcepton, то через instanceof проверяем по очереди

Хотя возможно в ларе всё это уже есть. Я здесь описываю общий принцип.

[SM_ST]

FanatPHP,

то есть сервис остается такой, а контроллер

public function store(RetailProductCreateService $productCreateService, RetailProductCreateRequest $request)
    {
        $retail = Retail::getRetailStoreIdByCompanyId(50);

        try {
        $productCreateService->store($retail->id, $request->getDto());
         }catch(){}
    }

[FanatPHP]

сервис НЕ остаётся такой.
если его оставить таким, то вообще никакого исключения не будет
Как я писал в своем ответе

} catch (QueryException $e) {
    DB::rollBack();
    if (условие по которому видно что запись уже есть) {
        throw new NonUniqueValueException(); 
    } else {
        throw $e;
    }
}
и тогда потом в контроллере можно будет

try {
    $productCreateService->store($retail->id, $request->getDto());
}catch(NonUniqueValueException $e){
    // пишем юзеру
}