Такое можно в сервисы? Не, вроде ничего криминального, но

я думал, что в сервисы просто выносится какая-то логика в виде методов, чтобы не загрязнять контроллеры. А тут целый класс со своим конструктором и полями, который, вообще является самостоятельной единицей логической, а не просто набором методов, которым контроллер делегирует выполнение задач.

<?php

namespace App\Cart;

use App\Models\Book;

class Cart
{
private $oldCart;

public function __construct(
public $total = 0,
public $items = [],
)
{
$this->oldCart = session()->get('cart');

if ($this->oldCart instanceof Cart){
$this->total = $this->oldCart->total;
$this->items = $this->oldCart->items;
unset($this->oldCart); // Чтобы не копились вложенности
}
}

public function AddToCart(int $id)
{
$book = Book::findOrFail($id);

if (isset($this->items[$id])) {
$this->items[$id]['quantity']++;
} else {
$this->items[$id] = [
'id' => $id,
'name'=> $book->name,
'quantity' => 1,
'price' => $book->price,
'path' => $book->path,
];
}
$this->recountTotal();
}

public function updateItemQuantity($id, $quantity){
if (isset($this->items[$id])) {
$this->items[$id]['quantity'] = $quantity;
}
$this->recountTotal();
}

public function removeItem($id){
if (isset($this->items[$id])) {
unset($this->items[$id]);
}
$this->recountTotal();
}


public function recountTotal(): void
{
$this->total = 0;
foreach ($this->items as $item){
$this->total += $item['quantity']*$item['price'];
}
}

}

Egor (@PAPRIKA_1994), мы обнаружили что Ваше сообщение больше чем наполовину состоит из кода. Рекомендуем сохранять примеры кода на внешних источниках, например, https://paste.laravel.io

разве это будет работать? ```

public function AddToCart(int $id) почему метод с большой буквы, почему нет типизации аргумента

потому что метод позиционирует себя как клас

findOrFail - ну такое себе, у тебя сервис по хорошему должен сам обработать такую ситуацию, что нет книги, иначе тебе 404ую кинет лара.

$this->oldCart = session()->get('cart'); это вообще забей. используй БД для хранения корзины.

у меня флешбеки. У нас phpcs в пайплане проверяет код, но он срабатывает только на измененные файлы. Правишь одну строчку в старом коде - все, линтер сходит с ума