Проблема с MVC и сетью заключается в том, что вы никогда не сможете четко разграничить вид и контроллер. Это просто внутренняя природа сети. Я скорее парень из CakePHP, но принципы те же. Когда я пишу свой код, я всегда задаю себе несколько вопросов, чтобы узнать, где лучше всего разместить свой код.
- Я получаю, храню или
манипулировать данными? (если так, это
модель)
- Отображаю ли я данные или представляю
данные для конечного пользователя? (если так, это
вид).
- Все остальное входит в мой
контроллер.
При этом, просто взглянув на ваш код, кажется, что вы комбинируете код модели и вид кода в контроллере, и поэтому вам это неловко. Вот простая логика, которая может вам помочь:
Переместите весь код, который строит список дат в модель. Назовите это что-то вроде:
function create_date_list() {
// code here
}
Это создаст только массив дат, которые вы хотите отобразить. Имейте в виду, что вы также можете передавать в массив такие вещи, как дата или нет, текущий день, выходные и т. Д. Это поможет вам определить, как форматировать дату, не добавляя код в представление для Сделай так. Например, у вас может быть массив, возвращенный из модели, который выглядит следующим образом:
[dates] {
[22] {
[day] => [Friday]
[type] => [weekday]
}
[23] {
[day] => [Saturday]
[type] => [Weekend]
}
[24] {
[day] => [Sunday]
[type] => [Weekend]
[Holiday] => [Easter]
}
}
Это может быть излишним или нет. Но я использовал этот пример, чтобы показать, что МОДЕЛЬ - это место, где вы выполняете всю эту тяжелую работу, а не контроллер или представление.
Далее создайте представление. Я не уверен, что это в CodeIgniter, но в торт они называются элементы. Это фрагменты кода представления, которые доступны для любого представления (повторно используемые элементы представления). Поэтому создайте повторно используемый элемент представления, который будет принимать массив дат, проходить по нему и записывать вывод в HTML. Обратите внимание, как я могу использовать массив дат для форматирования моего представления.
<ul>
<?php foreach($dates as $date): ?>
<li class="date <?php echo $dates[$date]['type']; ?>><?php echo $date; ?></li>
<?php endforeach; ?>
</ul>
Красиво, чисто и лаконично. Преимущество в том, что если вы хотите изменить внешний вид, вам не нужно прикасаться к контроллеру. ;)
Теперь для контроллера. Поместите код в контроллер, который вызывает массив дат из модели и передает его в представление. Идея - модель FAT, КОЖНЫЙ вид.
function my_function() {
// get code from model
// set code for view
// render view
}
Хотя я на самом деле не "очистил" ваш код. Я надеюсь, что это даст вам руководство, которое вы искали, о том, как перемещать код вокруг себя так, чтобы: а) это имело больше смысла для вас, и б) вы могли очистить код таким образом, который все еще будет соответствовать архитектуре MVC, которую вы стремимся к.
Удачи и счастливого кодирования!