Где правильнее использовать ActiveRecord, в контроллере или модели?

Question

[Дмитрий Б]

Добрый вечер. Подскажите пожалуйста, где правильнее будет использовать ActiveRecord (Yii framework): прямо в контроллере или в контроллере использовать модель, которая будет оперировать объектом ActiveRecord?
Второй вариант позволит использовать один и тот же метод в разных частях модуля, но нужно будет писать метод-велосипед для получения данных.

Answer 1

[Паша]

В модели.

Answer 2

[Сергей Протько]

модель - данные, которыми оперирует ваша бизнес-логика.

контроллер - нечто обрабатывающее пользовательский ввод

толстые контроллеры - когда вся логика заключена в контроллерах

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

сервисы - просто какие-то классы инкапсулирующие в себя какую-то работу.

Принцип единой ответственности - класс должен делать что-то одно и хорошо. Если у вас один и тот же класс отвечает за две разные вещи (валидация и сохранение данных в базе) - это нарушение этого принципа.

active record - один из шаблонов работы с базой данных который нарушает принцип единой ответственности и еще кучи других в угоду быстрой разработки. В контексте языков типа ruby или python вреда от этого не сильно много, так как все динамичненько и не скручено в кашу. Но в php, java (с партиал классами возможно и не так уж все и плохо) и т.д. почти все реализации ужасны. Но люди мирятся с этим в угоду быстрой разработке.

Вывод, правильнее будет использовать active record в сервисах, так как при active record у вас перемешаны сервисы валидации, хранения данных и сама модель. Контроллеры же должны оперировать только сервисами.

Comments

[Дмитрий Б]

Спасибо большое ) Проблема решена )

[Сергей Протько]

@PiloTeZ сомневаюсь.

[Дмитрий Б]

@Fesor Почему? Теперь создам модель для работы с определённым элементом, сделаю класс и буду его везде использовать:
class Tasks
{
function getTasks()...
function deleteTask()...
function saveTask()...
function countTasks()...
}

[Сергей Протько]

class Task extends yii/ar/ActiveRecord { // или как там
}
class TaskManager
{
    function addTask(Task $task) {
        // Task::save($task) или как оно там у вас.
    }
    function getTasks();
    // etc
}
class TaskController{
    function actionAddTask() {
         $task = new Task();
         // не вкурсе что там накрутили с DiC в Yii2
         // посему предположим что зависимость у нас уже есть.
         $this->taskManager->addTask($task);
    }
}

суть такая.

[Дмитрий Б]

@Fesor так понимаю сразу плюсов этого подхода не увидеть, его преимущества будут видны при расширении в дальнейшем?
Это какой-то паттерн проектирования?

[Сергей Протько]

@PiloTeZ плюсы - можно покрыть сервисы тестами, контроллеры тестами никто особо не покрывает (только интеграционными, хотя по сути в большинстве случаев именно ими и стоит покрывать, оставив юнит тесты именно на библиотеки). Ну и да, ощутимо поможет вам расширять систему.

Почитайте про низкую связанность и высокое зацепление.

[Сергей Протько]

@PiloTeZ и да, без тестов рефакторинг и последующая поддержка проектов будет адом а код быстро превратится в мессиво.

[Дмитрий Б]

@Fesor спасибо!