Как сделать удобный метод выборки данных (Yii)?

Question

[Дмитрий Б]

Здравствуйте. Скажите пожалуйста, как сделать метод для выбора данных в стиле MVC, что бы его бы легко удобно применять и просто расширять.
Мой вариант:

/**
 * Менеджер подзадач
 */
class SubtasksManager
{
        /**
          * Получение подзадач
          */
	public function getSubtasks($params = [])
	{
		$subtasksCriteria = new CDbCriteria;

		/*
		 * Фильтрация по предопределённым критериям
		 */
		// Получение задач ожидающих выполнения
		if (!empty($params['requires_performing'])) {
			// Только выполняемые задачи (есть ещё связующие)
			$params['type'] = '1';
			// Задачи ожидающие выполнения
			$params['performing_status'] = '1';
		}

		// Получение только задач в стадии подготовки
		if (!empty($params['preparing'])) {
			$params['performing_status'] = '0';
		}

		/*
		 * Фильтрация по определённым полям
		 */
		// Получение по идентификатору родительской задачи
		if (!empty($params['task_id'])) {
			$subtasksCriteria->addColumnCondition(['task_id' => $params['task_id']]);
		}

		// Получение по типу
		if (!empty($params['type'])) {
			$subtasksCriteria->addColumnCondition(['type' => $params['type']]);
		}

		// Получение по статусу выполнения
		if (!empty($params['performing_status'])) {
			$subtasksCriteria->addColumnCondition(['performing_status' => $params['performing_status']]);
		}

		return SubtasksAr::model()->findAll($subtasksCriteria);
	}

В этом методе есть возможность получать как по предопределённым критериям, так и по определённым полям таблицы.
Вопросы:
1. Правильный ли это подход или лучше убрать получение по определённым полям таблицы из вне, ведь теряется гибкость скрипта, вдруг изменится название какого-то поля, тогда придётся менять его название во всей системе, в каждом файле.

2. Если например мне нужны получить подзадачи по определённому родителю, то в параметрах метода лучше получать только идентификатор родительской задачи или целый объект со всеми его данными? Второй вариант кажется более гибким.

Хочу узнать, правильно ли идёт ход моих мыслей или это очередной быдлокод

Answer 1

[Андрей Ежгуров]

В MVC выборка дынных производится в моделях. Соответственно, всё, что ты написал в "менеджере подзадач", должно быть размещено в соответствующей модели. Если тебе нужен дополнительный механизм выборки данных - создаешь новый метод в модели. Именно модели должны отображать структуру данных и связей.

В Yii достаточно мощная Active Record, позволяющая гибко выбирать данные из дочерних таблиц без дополнительных ухищрений. При правильной настройке связей в моделях подзадачи заданного родителя вполне можно получить так: Task::model()->findByPk($id)->Subtask.

Comments

[Дмитрий Б]

Это и есть модель - сервис

[Дмитрий Б]

Или я не правильно понял назначение менедежера? Он у меня также имеет методы saveSubtask и deleteSubtask. Есть ещё сущность Subtask, которая в себе хранит только геттеры и сеттеры для подзадач. За обращения к базе (activerecord) отвечает класс SubtasksAr, который используется в SubtasksManager

[Дмитрий Б]

public function saveSubtask(Subtask $subtask)
{
return $subtask->save();
}

[Андрей Ежгуров]

Модель - это то, что наследуется от класса Model.

И когда ты удаляешь подзадачу, ты делаешь Subtask::model()->delete(), а никак не самописный метод самописного класса. Никаких saveSubtask и deleteSubtask быть не должно.

[Дмитрий Б]

@eandr_67 в Yii от класса CModel не наследуюутся

[Дмитрий Б]

@eandr_67 то есть вы предлогаете использовать SubtasksAr прямо в контроллере? Это не правильно

[Андрей Ежгуров]

Если же тебе при удалении нужно выполнить какие-то действия, то ты в классе Subtask (который является наследником класса cModel - в прошлом комментарии я немного ошибся) переопределяешь метод метод delete.

[Дмитрий Б]

@eandr_67 с получением как быть?

[Андрей Ежгуров]

Посмотри, как именно сделаны демонстрационные примеры Yii, какой именно код генерирует gii. А уже потом утверждай, что правильно, а что нет.

Я говорю о том, что контроллер должен только вызвать метод delete модели, а уже модель должна решать, что и как удалять. Впихивать дополнительные классы между контроллером и моделью не имеет смысла.

[Андрей Ежгуров]

А для получения данных у каждой модели есть множество стандартных методов для простых случаев и придётся писать собственные методы для случаев нестандартный.

[Дмитрий Б]

@eandr_67 Gii былокод генерирует вообще-то. Хотя бы потому что там на прямую используется $_POST и $_GET

[Александр Зеленин]

@PiloTeZ не слушайте @eandr_67 - он следует канонам yii. Вы же хотите более правильно по php сделать.

[Дмитрий Б]

@zelenin спасибо, я тоже так думаю