Рефакторинг кода контроллера MVC модели?

Question

[Александр Шаповалов]

Здравствуйте товарищи! Я долго пытался понять как избавится от обилия условных операторов в коде, они изрядно заставляют глаза побегать и снижают читаемость кода. Подскажите пожалуйста, как без создания новых методов класса избавится от обилия условных операторов, ниже пример кода метода класса контроллера:

//вывод запрашиваемой статьи из базы
	function getAction()
	{
		$fc=FrontController::get();
		$url=$fc->getParams();
		$view=new View();
		//возваращаем запрашиваемую статью из базы в виде ассоциативного массива
		$view->content=Articles::get()->getArticle((int)$url['id']);
		//выборка комментариев к статье
		$view->comments=Articles::get()->getComments((int)$url['id']);
		
		if($_SERVER['REQUEST_METHOD']=='POST' and isset($_POST['addcomment']))
		{
			if($_SESSION['captcha']==crypt($_POST['captcha'],'x)p_q1'))
			{
				if(!empty($_POST['comment_txt']) and !empty($_POST['name']))
				{
					if($_POST['article_id']==(int)$url['id'])  
					{
						$check=Articles::get()->checkComment($_POST['article_id'], $_POST['comment_txt'], $_POST['name']);
						if($check=='0')
						{
							$view->comments=Articles::get()->addComment($_POST['article_id'], $_POST['comment_txt'], $_POST['name']);						    
							($view->comments) ? $view->comments='Ваш комментарий успешно добавлен!' : $view->comments='Ошибка при вставке комментария!';
						}
						else
							$view->comments='Данный комментарий уже добавлен!';
					}
					
					else 
						$view->comments='Вы пытаетесь вставить комментарий не в ту статью!';
				}
				else
					$view->comments='Поля не должны содержать пустых значений!';
			}
			else
			{
				$view->comments='Вы ввели символы с картинки некорректно!';
			}
			
		}
		else
		{
			//проверка на наличие пришедших данных
			($view->comments==false) ? $view->comments='Комментариев к статье еще нет!': $view->comments;
		}
		
		(empty($view->content['title'])) ? $view->title='Такой статьи нет': $view->title=$view->content['title'];
		(empty($view->content['meta_key'])) ? $view->keywords='' : $view->keywords=$view->content['meta_key'];
		(empty($view->content['meta_description'])) ? $view->description='' : $view->description=$view->content['meta_description'];
		
		$result=$view->render('../views/article.php');
		$fc->setBody($result);
	}

Заранее благодарен! не отрубайте мне руки пожалуйста :)

Answer 1

[anitspam]

Использовать модель?

<?php
//вывод запрашиваемой статьи из базы
    function getAction()
    {
        $fc=FrontController::get();
        $url=$fc->getParams();
        $model = new ArticleModel();
        $model->setParams($fc->getParams());
        $model->validate();
        $view=new View();
        //возваращаем запрашиваемую статью из базы в виде ассоциативного массива
        $view->content=$model->getContent();
        //выборка комментариев к статье
        $view->comments=$model->getComments();
        
        $view->title=$model->getTitle();
...
        
        $result=$view->render('../views/article.php');
        $fc->setBody($result);
    }

ПС. Почему-то не включилась подсветка кода.

Comments

[Александр Шаповалов]

Вы предлагаете проверять данные на уровне модели, очень разумное решение, это позволит сделать валидатор более универсальным! Благодарю Вас за подсказку, как только допишу веб-приложение приступлю к его рефакторингу )

[Борис Сёмов]

Собственно, обычно так и делается. =)
В контроллере обрабатывается запрос и собираются в кучу необходимые данные, в модели ввод/вывод/валидация данных, во view шаблонизация добытых из модели/моделей и подготовленных в контроллере данных.

[Александр Шаповалов]

Ну я обычно использую модели как средства доступа к базе данных и немного логики иногда к ним добавляю, контроллер для обработки запросов и валидации, ну а в шаблонах генерирую формы по пришедшим данным. Но теперь я всю основную логику в моделях буду прятать, разгружу контроллеры!
Как только доделаю дипломку и произведу рефакторинг кода, выложу готовый вариант на github и представлю проект на суд хабравчан. Я за open source!

Answer 2

[Ivan]

Использовать валидатор форм.

Answer 3

[Алексей Журбицкий]

Как один из вариантов

<?php
function getAction()
{
    $fc   = FrontController::get();
    $url  = $fc->getParams();
    $view = new View();
    //возваращаем запрашиваемую статью из базы в виде ассоциативного массива
    $view->content = Articles::get()->getArticle((int)$url['id']);
    //выборка комментариев к статье
    $view->comments = Articles::get()->getComments((int)$url['id']);

    if ($_SERVER['REQUEST_METHOD'] != 'POST' || !isset($_POST['addcomment'])) {
        $view->comments == false ? $view->comments = 'Комментариев к статье еще нет!' : $view->comments;

        return _renderView($view);
    }

    if ($_SESSION['captcha'] != crypt($_POST['captcha'], 'x)p_q1')) {
        $view->comments = 'Вы ввели символы с картинки некорректно!';

        return _renderView($view);
    }

    if (empty($_POST['comment_txt']) || empty($_POST['name'])) {
        $view->comments = 'Поля не должны содержать пустых значений!';

        return _renderView($view);
    }

    if ($_POST['article_id'] != (int)$url['id']) {
        $view->comments = 'Вы пытаетесь вставить комментарий не в ту статью!';

        return _renderView($view);
    }

    $check = Articles::get()->checkComment($_POST['article_id'], $_POST['comment_txt'], $_POST['name']);

    if ($check != '0') {
        $view->comments = 'Данный комментарий уже добавлен!';

        return _renderView($view);
    }

    $view->comments = Articles::get()->addComment($_POST['article_id'], $_POST['comment_txt'], $_POST['name']);
    $view->comments ? $view->comments = 'Ваш комментарий успешно добавлен!' : $view->comments = 'Ошибка при вставке комментария!';

    return _renderView($view);
}

function _renderView($view)
{
    $fc = FrontController::get();

    (empty($view->content['title'])) ? $view->title = 'Такой статьи нет' : $view->title = $view->content['title'];
    (empty($view->content['meta_key'])) ? $view->keywords = '' : $view->keywords = $view->content['meta_key'];
    (empty($view->content['meta_description'])) ? $view->description = '' : $view->description = $view->content['meta_description'];

    $result = $view->render('../views/article.php');
    $fc->setBody($result);
}

правда с дополнительным методом для удобства.

Comments

[Александр Шаповалов]

Спасибо Вам огромное! Воспользуюсь Вашей удобочитаемой реализацией!

Answer 4

[markoffko]

Зачем принимать id статьи при добавлении комментария если статья уже выбирается и id есть в любом случае?
Почему один экшен обрабатывает выбор статьи и добавление комментариев?
Валидировать данные лучше в отдельном классе формы.

Comments

[Александр Шаповалов]

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

 f($_POST['article_id']==(int)$url['id'])  
                    {
                        $check=Articles::get()->checkComment(...
> 

Кстати, Вы правы на счет валидации отдельной моделью, надо будет попробовать!

[markoffko]

$check=Articles::get()->checkComment($view->content->id, $_POST['comment_txt'], $_POST['name']);

$_POST['article_id']==(int)$url['id'] вообще лишнее

[Александр Шаповалов]

Да вы правы, думаю спамеры найдут другие пути закидывания моего будущего проекта однотипными сообщениями )

[markoffko]

Все же советую разнести на два метода добавление комментария и выбор статьи.