Нормально ли в Yii хранить url-адреса в моделях?

Question

[MegaMufa]

Доброе утро.

Есть система на Yii. Для верстки шаблонизаторы не используются. Делаю стандартным Yii-шным способом: через \CController::render() подключается файл с html разметкой, в которую вставляются данные чрез <?php echo ''; ?>

В представление передается модель AR, чьи свойства и подставляются. Удобнее, чем в контроллере формировать массив параметров и передавать его. Модели построены так, что в представлении не вызывается ни один ее метод, кроме гетеров.

Система использует ЧПУ. Поэтому адреса ссылок вписываются не напрямую, а вормируются через \Yii::app()->createUrl();. Но в таком случае при смене адреса старницы, надо искать во всех представлениях эту ссылку и менять ее.

Я вот думаю сделать в модели методы, которые будут возвращать соответствующие ссылки. И тогда вместо такого кода в представлении:

<a href="<?php echo \Yii::app()->createUrl('users/Settings',['userId'=>$user->id]); ?>">settings</a>

я в модели описываю метод:

public funtion getSettingsLink() 
{
  return \Yii::app()->createUrl('users/settings',['userId'=>$this->id]); 
}

а в представлении:

<a href=<?php echo $user->settingsLink; ?>">settings</a>

Такой вариант мне видится логичным т.к. формирование ссылок инкапсулируется внутри объекта. Но смущает тот факт, что модель содержи методы зависящие от url (грубо говоря работает с представлением).

Подскажите, стоит ли использовать данных подход. Или есть другие пути решения этой задачи?

Answer 1

[enchikiben]

В yii ЧПУ можно настроить в конфиге, и если захотели другой адрес то менять там, а контроллер не трогать, а в представлении можно так писать:

echo CHtml::link('settings',array('users/settings','userId'=>$user->id));

Comments

[MegaMufa]

Про чпу в конфиге знаю и автивно использую.

Ваш вариант с изменениями через конфиг - не прокатит. Если каждый раз, когда я переношу экшен из одного контроллера в другой (при разделении контролера или выделении модуля) - конфиг станет очень здоровым. А куча ненужных привил ротинга - это лишняя нагрузка.

В данной ситуации использование \CHtml не принесет никакой выгоды.

Меня интересует ответ на единственный вопрос: допустимо ли с идеологической точки зрения формирование адресов в модели?

[enchikiben]

@HaruAtari в случае если действительно в дальнейшем будет много переносов, то да лучше места на ссылку не найти, чтобы потом не искать по всему коду, а поменять в одном месте.

Answer 2

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

в моделях нормально хранить URI, url должен генерить маршрутизатор.

Answer 3

[goodnickoff]

В вашем случае скорее всего это не верно. Если вы измените контроллер, и вместо users/settings ссылка превратится в member/settings, то получение из модели users ссылки для membrs выглядит не красиво. То же самое касается экшина. Если вместо users/settings ссылка изменится на users/account то получение ее через $user->settingsLink; то же выглядит не логично. Я бы посоветовал вам уделить больше внимания проектированию что бы в последствии кардинально не менять контроллеры и экшины.

Comments

[MegaMufa]

Работаем по agile. Я бы и рад проектировать заранее, но постоянно приходится вносить изменения, причем зачастую кардинальные.
Единственное, что могу - проектировать как можно более гибкую систему и инкапсулировать все, что можно.

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

При agile проектирование системы намного более важно, ибо нужно спроектировать очень слабо связанную систему. Вообще yii не очень подходит для agile разработки в силу как раз таки очень большой связанности кода.

[MegaMufa]

Ну не сказал бы. Пока все нормально работает. Сильно связанно ядро фреймворка, а сама система разнесена по разным модулям.

Практически вся логика лежит в моделях. И это действительно придает системе гибкости. Контроллер отвечает только за проверку прав, рендеринг и вызов нужных методов моделей.

В данной системе единственный модуль, который взаимодействует с другими - модель пользователей. Остальные полностью изолированны друг от друга.

Единственная проблема в конфигах. И в одном месте хранить неудобно - слишком большие и по модулям/моделям растаскивать не хочется.