Правильно ли я использую контроллер или данный код должен быть в моделях?

а если я сделал так в контроллере

public  function  buyProduct($id){
        $product = Product::find($id);
        $seller = User::find($product->seller_user_id) ;
        $user = Auth::user();
        
        return $product->changeOwner($product,$seller,$user);
    }

и так в моделе

public function changeOwner($product,$seller,$user)
    {
        if($user->id == $product->owner_user_id){

            return 'You cant buy your product';
        }
        elseif($product->status_id == SOLD){
            return 'This product is sold';
        }
        elseif ($user->balance < $product->price) {
            return 'Not enough money';
        }
        else{
            $product->owner_user_id = $user->id;
            $product->buyer_user_id = $user->id;
            $product->status_id = SOLD;
            $user->balance -= $product->price;
            $seller->balance += $product->price;

            $user->save();
            $seller->save();
            $product->save();
        }

        return $product->load(['category:id,title']);
    }

уже лучше?

Правильно ли я использую контроллер или данный код должен быть в моделях?

tukreb, спасибо за книгу

Правильно ли я использую контроллер или данный код должен быть в моделях?

TheAndrey7, дублирование

Правильно ли я использую контроллер или данный код должен быть в моделях?

TheAndrey7,

public  function  buyProduct($id){
        $product = Product::find($id);
        $user = Auth::user();

        if($user->id == $product->owner_user_id){
            return 'You cant buy your product';
        }
        elseif($product->status_id == SOLD){
            return 'This product is sold';
        }
        elseif ($user->balance < $product->price) {
            return 'Not enough money';
        }
        else{
            //some code
        }
    }

Вот так правильнее?

Как правильно использовать интерфейс в Vuejs?

Aetae, хорошо, а почему тогда интерфейс не работает, то есть

tasks: [{
        name: 'Create app',
        description1: 'Use smth to do this Use smth to do this Use smth to do this Use smth to do this Use smth to do this Use smth to do this Use smth to do this Use smth to do this',
        time: '02.12.2021'
      },
      {
        name: 'Fix bugs',
        description1: 'Fix all bugs',
        time: '02.12.2021'
      },
      ] as TaskInterface[]

тут я могу писать и не стринговые значения, ошибки не вылетают
при этом я удалил метод setTask()

Как правильно использовать интерфейс в Vuejs?

Aetae, извини,я наверное не правильно понял тему интерфейса, я так понял что я просто указываю какие типы должны быть и все, а сaми данные передаю в task{}

Как правильно использовать интерфейс в Vuejs?

Aetae,
получается ,что ничего не выводит, то есть я не правильно его использую?

Как поменять стиль в компоненте в Vue.js при нажатии на элемент в другом компоненте?

Нужно передать этот клик из aside to asd то нужно provide/inject?

Нельзя добавить один и тот же продукт два раза (проверка только по ссылке объекта)?

ThunderCat, стыдно,исправлюсь(

Нельзя добавить один и тот же продукт два раза (проверка только по ссылке объекта)?

Антон Р., по условию,xD

Нельзя добавить один и тот же продукт два раза (проверка только по ссылке объекта)?

FanatPHP, типа домашка в универе...

Нельзя добавить один и тот же продукт два раза (проверка только по ссылке объекта)?

FanatPHP, хорошо,есть класс Product, метод registerProduct добавляет объекты в массив $products[],нужно чтобы объекты с одинаковыми properties добавлялись только в том случае ,если объекты названы по разному:

$a = new Processor("Intel",100,"Max",24);
$a->registerProduct($a);

$a1 = new Processor("Intel",100,"Max",24);
$a1->registerProduct($a1);

,
а в таком случае второй раз не добавлялось:

$a = new Processor("Intel",100,"Max",24);
$a->registerProduct($a);

$a = new Processor("Intel",100,"Max",24);
$a->registerProduct($a);

Нельзя добавить один и тот же продукт два раза (проверка только по ссылке объекта)?

FanatPHP, и сам не могу понять(ссылка не у класса а у обьекта, обьект лежит в памяти а доступ к нему происходит по ссылке, тоесть если в переменную положить обьект то в нее попадает не сам обьект а только ссылка)

Как реализовать метод givemoney?

Алексей Уколов, не могу понять как в одной функции у одного пользователя забрать "деньги",а другому отдать.

Возможна ли работа с несколькими пользователями не используя БД?

понял,проблема в том,что программа должна быть организована только в 1 файле(index.php) :(

Возможна ли работа с несколькими пользователями не используя БД?

У меня есть куки для имени,фамилии,возраста,даты последнего визита и их количество,получается я не могу сделать так,что для каждого пользователя будет свой счетчик,который не обновляется при смене пользователя?
Это как работает у меня : я ввел Иванов и он три раза вошел с одного компьютера,потом за этот комп сел Петров и у него теперь 1 вход,а если потом опять сядет Иванов у него будет 1 вход,возможно ли что бы у Иванова счетчик не обнулялся?

Как сделать так чтобы при последующих входах на сайт все данные сохранялись(имя,фамилия,возраст),не используя БД?

Я в этом примере и сохраняю в куки,но через пару перезагрузок сайта выводится сообщение "Вы
__и вы родились в __.".То есть без данных которые я вводил в форму(