Как в laravel построить сложную логику?

Question

[Сергей Бурдужа]

Привет всем.
Есть request для сохранения данных

public function rules()
  {
    return [
      'category_id' => 'exists:App\Category,id',
      'client_id' => 'exists:App\Client,id',
      'book_id' => 'exists:App\Book,id',
      'days_issued' => 'nullable|string',
      'is_returned' => 'nullable|boolean',
      'return_date' => 'nullable|date',
    ];
  }

Это заказ клиента.
Клиент может заказать только 2 книги, и чтобы они были разные и чтобы поле is_returned было false.
Это поле будет позже менятся на странице списка заказов.

Я понимаю, что могу написать эту логику прямо в контроллере, но тогда прийдется избавиться от StoreRequest, и использовать обычный Request.

Хотя можно сделать первую валидацию на существование полей в StoreRequest, а потом в контроллере дописть логику?

Заранее благодарен.

Comments

[Дмитрий]

Ну а что мешает написать свое правило проверки на что угодно?

Answer 1

[Sanes]

В StoreRequest валидируйте. Логику, если удобно, в контроллере. В чём проблема?
На уникальность книги надо раньше проверять. На этапе корзины. Если существует, то добавляем кол-во.

Comments

[Сергей Бурдужа]

Я так сделал, просто у меня мало опыта в ларавель.
В контроллере.

public function store(StoreIssueBook $request)
  {
    $validated = $request->validated();
    $count_not_returned = IssueBook::where('client_id', $validated['client_id'])->where('is_returned', $validated['is_returned'])->count();
    if ($count_not_returned >= 2) {
      return response()->json(['status' => 0, 'message' => 'Client has already issued a 2 books'], 400);
    }
    $count_unique = IssueBook::where('client_id', $validated['client_id'])->where('book_id', $validated['book_id'])->count();
    if ($count_unique) {
      return response()->json(['status' => 0, 'message' => 'Book is already issued to this client'], 400);
    }

    $issueBook = IssueBook::create($validated);
    return response()->json(['data' => $issueBook], 201);
  }

[Sanes]

Главное, чтобы работало, как задумали.

[vism]

Это самое фиговое решение проблемы... :)

[Сергей Бурдужа]

vism, Я написал, что новичок, и пока что не ищу сложных путей.

[Дмитрий]

Сергей Бурдужа, это не вопрос сложности пути. Представьте что вы сдаете на права или только учитесь. Вы садитесь в машину и на слова экзаменатора или учителя - говорите "я новичок, я пока не ищу сложных путей - не буду пристегиваться". Реакцию сидящего рядом представили?

[vism]

Сергей Бурдужа, Дмитрий, Это скорее, как по кольцу проехать не направо, а налево. Или по встречке проехать на пустой дороге:)

[Дмитрий]

vism, в любом случае я хочу выйти.

Answer 2

[Дмитрий]

Бизнес-логике - не место в контроллере.

AppServiceProvider
public function boot(){
Validator::extend('count_books', function($attribute, $value, $parameters) {
        $count = IssueBook::where('client_id', $value)->where('is_returned', false)->count()
        return $count < 2;
});
}

StoreRequest
public function rules()
  {
    return [
      'category_id' => 'exists:App\Category,id',
      'client_id' => 'exists:App\Client,id|count_books',
      'book_id' => 'exists:App\Book,id',
      'days_issued' => 'nullable|string',
      'is_returned' => 'nullable|boolean',
      'return_date' => 'nullable|date',
    ];
  }

Comments

[Sanes]

А если этим сервисом будет пользоваться один метод из контроллера?

[Дмитрий]

Sanes, каким конкретно сервисом? и вопрос то в чем? ну будет один, значит будет один. будет два значит будет два - что это меняет.

[Sanes]

Дмитрий, сервис-провайдером, который предложил добавить. Про код в boot() ты же не серьезно?

[vism]

типичный тостер.
Нормальный ответ внизу, тупой ответ отмечен решением.
И ещё нормальный ответ хейтят.

[Sanes]

vism, валидатор в AppServiceProvider это по-твоему нормально?

[Дмитрий]

Sanes, ну если немного пошуршать по доке ларки, то можно обнаружить что Validador::extend они вечно запихивают в boot от сервиса провайдера: 1, 2, 3, 4, 5. Происходит это ровно до того момента когда они начинают двигать народ в сторону написания своих правил валидаций через классы или clousure - c 8 версии. Но что люди с сайта ларавел понимаю в том что и куда надо запихивать, они лошки, ведь да?

Ну а по существу даже если его запихать в index.php это будет лучше нежели хранить бизнес логику в контролерах. Потому что максимум что будет - это мне придется вспоминать в какую задницу я его запихал, и искать. А случае с логикой в контроллере у вас может возникнуть ситуация что когда к вам придет бизнес и скажет - нам срочно нужно апи для приложения или еще чего - а у вас 1000 методов 50 контроллеров и все они с бизнес логикой и надо сидеть и думать как мы это будем костылировать

З.Ы. Твою ж мать, уже на каждом заборе написано - тонкие контроллеры, нет бизнес-логике в контроллерах, в контроллерах максимум что должно быть - это. Но пока сами лбом не впаяются - нихрена не запомнят

[Sanes]

Дмитрий, до 1000 методов и 50 контроллеров начинающий разработчик никогда не дойдет.
В противном случае, он уже не начинающий и знает, как лучше.

[Дмитрий]

Sanes, ну возьмите статью про MVC для самых маленьких - первая ссылка из гугла https://ruseller.com/lessons.php?id=666, ешкин кот - логика должна быть в моделях.
То что бизнес логики не должно быть в контроллерах это базис для начинающих, продолжающих и уже заканчивающих. Паттерн MVC так закостылен. И если кому то кажется что лучше таки в контроллере - надо встать перед стеной и биться головой ровно до тех пор пока казаться так не перестанет.

[Sanes]

Дмитрий, если там логики на 3 строки и её никак не переиспользовать в другом месте, какой смысл её куда-то выносить? Ну можно в Actions. Чем принципиально это будет лучше?
Холивар бесполезный.

[Дмитрий]

Sanes, нет никакого холивара. Есть набор правил которое айтисообщество сформировало через кровь пот и страдания. И если их не соблюдать - будут ровно кровь, пот и страдание. За мелким исключением - если проект создать а потом смыться в закат так далеко что те кто его будут поддерживать - вас не найдут. Тогда да, кровь, пот и страдания будут у них.

[Дмитрий]

Sanes, холивар - это где хранить логику в сервисах или моделях вот то холивар. А идея хранить ее в контроллерах это не холивар - это глупость.

[vism]

Дмитрий, Слышал, что с глупым спорить бесполезно? Твои доводы разобьются о стену непонимания:)

[Дмитрий]

vism, да я не спорю - я пытаюсь обьяснить.

[vism]

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

[Дмитрий]

vism,

Среди людей, знающих об эффекте Даннинга — Крюгера, часто имеет место не вполне верное понимание данного психологического феномена. Первое заблуждение состоит в том, что эффект Даннинга — Крюгера относится только к «некомпетентным» людям. В действительности же, описываемое когнитивное искажение присуще каждому человеку.

Все мы косячим.

Answer 3

[jazzus]

Данная проверка никакого отношения к валидации полей формы не имеет. Смотри авторизацию с возвратом сообщений.