Правильно ли хранить дополнительные методы в модели?

Question

[Ильяс Галиев]

Есть модели и в них штук по 20 дополнительных статических методов для получения разных данных (по разному отсортированных или отдельные данные, используемые в нескольких местах).
Правильно ли так? Или лучше как-то оформить в виде компонента, поведения или типа такого?
И будет ли лучше сделать их не статическими, а обычными методами?

Answer 1

[Евгений Дождиков]

По мне, так это не правильно.
Для получения из одной модели данных по разному отсортированному, но делать это через разные методы, это большое излишество кода. У вас есть CDbCriteria->order, например, а порядок сортировки тоже не проблема передать в модель.
А получение разных данных, если подразумевается разные поля из одной модели, то тоже лишнее кодирование, на мой взгляд. Можно всегда отдавать дозволенный набор полей.
Если таких пересечений много, то даже для читабельности проще объединить эти методы, хоть банальный search() сделать. Выделять в компонент или поведение, наверно нет. Хотя все это можно вынести в scopes(), где и описать и сортировку и поля в select.
И по мне лучше их сделать не статикой, но это все зависит от контекста модели и ее методов.

Comments

[Ильяс Галиев]

Я говорил немного про другое. Я не говорил выбирать тупо отдельные поля и не о сортировке sql. Допустим, надо выбрать некоторые данные и собрать массив с определенным ключом и т.п. Да, действительно, некоторые методы у меня просто на выборку отдельных полей и другой сортировки. Их я постараюсь вынести в контроллеры, где возможно. Но как в примере выше, если надо перебрать массив, изменить, отфильтровать, выбрать рандомные данные и т.д., то оставить эти методы в модели, или еще куда-то можно?

[Евгений Дождиков]

Ильяс Галиев: Да, не так понял изначальный вопрос, каюсь... В целом, можно сделать и через поведения, т.к. это все таки работа с данными БД, что относится именно к модели, а не к компонентам или т.п. Это если преобразования данных могут быть одинаковыми в разных моделях. А если в каждой модели свои преобразования, отличные от всех других, то я бы оставил все методами модели, только сделал бы их, скорее, не статикой, чтобы внутри модели не было вызовов самой себя.

Answer 2

[Роман Франк]

Я считаю что это правильно. Ведь модель для этого и нужна, для работы с базой. Главное чтоб вы не делали дублирование стандартных методов :)