E
CreateUserAction - класс ответственнен за одно действие
Size: a a a
2020 February 27
И
он копипастить научился чтоли?
вредничает. Про солиды какие-то тут ругается))))))
E
тебе 1 тест написать
Д
Товарищи, подскажите как это правильно делается?
$task = new Task();
$letter = Letter:where("id",25)->get();
$task->letter()->save($letter);
Модель Letter
$task = new Task();
$letter = Letter:where("id",25)->get();
$task->letter()->save($letter);
Модель Letter
public function task()Модель Task :
{
return $this->hasOne('App\Task');
}
public function letter()Может я что то спутал?
{
return $this->belongsTo('App\Letter');
}
A
теперь рефакторим твой код в actions:
/Actions
CreateUserAction
теперь в контроллере
RegisterController
public function register(CreateUserAction $action, Request $request)
$action->execute($request);
/Actions
CreateUserAction
теперь в контроллере
RegisterController
public function register(CreateUserAction $action, Request $request)
$action->execute($request);
если доменная логика расположена в сущностях, то все действия сервисного класса очень простые и одинаковые. можно и в одном классе собрать
E
это не очень
И
CreateUserAction - класс ответственнен за одно действие
да знаю я про такой подход. Но это на любителя
E
теперь напиши мне тест для твоего сервиса
AI
Дмитрий
Товарищи, подскажите как это правильно делается?
$task = new Task();
$letter = Letter:where("id",25)->get();
$task->letter()->save($letter);
Модель Letter
$task = new Task();
$letter = Letter:where("id",25)->get();
$task->letter()->save($letter);
Модель Letter
public function task()Модель Task :
{
return $this->hasOne('App\Task');
}
public function letter()Может я что то спутал?
{
return $this->belongsTo('App\Letter');
}
Вроде все ок
A
а вот если ты там всё делаешь. и с базой работа и с очередями и бизнес-логика. тогда да. правильнее наверно по классу на действие
AI
Разве что можно Letter::find(25)
ИЛ
Дмитрий
Товарищи, подскажите как это правильно делается?
$task = new Task();
$letter = Letter:where("id",25)->get();
$task->letter()->save($letter);
Модель Letter
$task = new Task();
$letter = Letter:where("id",25)->get();
$task->letter()->save($letter);
Модель Letter
public function task()Модель Task :
{
return $this->hasOne('App\Task');
}
public function letter()Может я что то спутал?
{
return $this->belongsTo('App\Letter');
}
Task::create()->letter()->associate($letter)
И
а вот если ты там всё делаешь. и с базой работа и с очередями и бизнес-логика. тогда да. правильнее наверно по классу на действие
да чтож такое - опять переучиваться чтоли
E
я помню эти сервис классы были популярные, я тогда на Zend Framework писал =/
A
да чтож такое - опять переучиваться чтоли
я же даже в книжке про это писал
ИЛ
а вот если ты там всё делаешь. и с базой работа и с очередями и бизнес-логика. тогда да. правильнее наверно по классу на действие
И все равно не правильно
И
я помню эти сервис классы были популярные, я тогда на Zend Framework писал =/
и на Yii к ним приобщают, потому что новички пишут все в одном контроллере)
A
И все равно не правильно
ага :)
И
я же даже в книжке про это писал
я помню!
ИЛ
Ведь даже после разбивки по действию на класс, в одном классе может быть куча связанных зависимостей