Нужна помощь в MVC PHP. Это сложно так или я дибил?

Question

[Алексей Ярков]

Есть какрас простенькой CMS на основе MVC. С добавлением страниц я разобрался, bootstrap шаблон натянул, БД MySQL прикрутил. Теперь вот напоролся на подводный камень. Сейчас структура такая:
site.com/blog - открывается страница и на ней 5 последних записей
http://site.com/blog/ID_категории/ID_статьи - сейчас, естественно, открывается страница 404, а надо чтоб открывалась статья.

Помогите добавить просмотр статей, ребята!

Я залил архив с исходниками. Посмотрите кому не жалко времени, пожалуйста.

freelancer-blog.ru.zip

Comments

[Сергей Зеленский]

пару минут пробежался по коду, вот пару советов:
1) используйте PDO или mysqli для работы с бд там есть замечательная штука prepared staments
2) у вас есть файлик functions.php и в нем функции для запроса из бд, они должны быть в моделях а не в отдельном файле
где вы вообще взяли этот движок?

[Алексей Ярков]

Сергей Зеленский: на хабре накопал. А functions.php взял с другого моего сайта (http://случайноеимя.рф). Мне удобнее когда функции в отдельном файле.

Answer 1

[Vitaliy Orlov]

Хоть как и писали выше, нет привычного роутинга, параметры можно легко добавить:

Надо внести такие изменения в класс route.php:

<?php

/*
Класс-маршрутизатор для определения запрашиваемой страницы.
> цепляет классы контроллеров и моделей;
> создает экземпляры контролеров страниц и вызывает действия этих контроллеров.
*/
class Route
{

	static function start()
	{
		// контроллер и действие по умолчанию
		$controller_name = 'Main';
		$action_name = 'index';
// --------------------------------------------------

$action_params = array();
		
$routes = explode('/', parse_url($_SERVER['REQUEST_URI'], PHP_URL_PATH));

// --^^------------------------------------------------
		// получаем имя контроллера
		if ( !empty($routes[1]) )
		{	
			$controller_name = $routes[1];
		}
		
		// получаем имя экшена
		if ( !empty($routes[2]) )
		{
			$action_name = $routes[2];
		}
// --------------------------------------------------		

if (count($routes)>2) {
	$action_params = array_slice($routes, 2);
}

// --^^------------------------------------------------
		// добавляем префиксы
		$model_name = 'model_'.$controller_name;
		$controller_name = 'controller_'.$controller_name;
		$action_name = 'action_'.$action_name;

		/*
		echo "Model: $model_name <br>";
		echo "Controller: $controller_name <br>";
		echo "Action: $action_name <br>";
		*/

		// подцепляем файл с классом модели (файла модели может и не быть)

		$model_file = strtolower($model_name).'.php';
		$model_path = "application/models/".$model_file;
		if(file_exists($model_path))
		{
			include "application/models/".$model_file;
		}

		// подцепляем файл с классом контроллера
		$controller_file = strtolower($controller_name).'.php';
		$controller_path = "application/controllers/".$controller_file;
		if(file_exists($controller_path))
		{
			include "application/controllers/".$controller_file;
		}
		else
		{
			/*
			правильно было бы кинуть здесь исключение,
			но для упрощения сразу сделаем редирект на страницу 404
			*/
			Route::ErrorPage404();
		}
		
		// создаем контроллер
		$controller = new $controller_name;
		$action = $action_name;
		
		if(method_exists($controller, $action))
		{
			// вызываем действие контроллера
// --------------------------------------------------

call_user_func_array(array($controller, $action), $action_params)

// --^^------------------------------------------------
		}
		else
		{
			// здесь также разумнее было бы кинуть исключение
			Route::ErrorPage404();
		}
	
	}

	function ErrorPage404()
	{
        $host = 'http://'.$_SERVER['HTTP_HOST'].'/';
        header('HTTP/1.1 404 Not Found');
		header("Status: 404 Not Found");
		header('Location:'.$host.'404');
    }
    
}

после этого можно строить любые роуты, согласно схемы:
http:// site.com / [controller] / [action] ( / параметр_1 ( / параметр_2 ( / .. ( / параметр_n))))

для того, чтобы использовать указанные параметры, надо в методе action-а, соответствующего контроллера указать их в сигнатуре, т.е. вот так:

class controller {
  public function action($param_1, $param_2,...,$param_n) {
  
  }
}

P.S. Код я не проверял, так что если будут ошибки, пиши их сюда, коллективный разум исправит :)

Comments

[Алексей Ярков]

Уже додумался сам, не дождался вас )) Сделал почти то же самое, только именно для контроллера controller_blog.php. Добавил метод action_view($id=false) и в него передаю из роутера ID статьи. Ссылки теперь вида http://bla-bla.com/blog/view/ID_статьи. И шаблон страницы естественно генерирую для списка или для одной статьи.

А вам огромное спасибо за отклик.

Answer 2

[Константин Б.]

altorouter.com один файл всего. Тебе его думаю за глаза будет

и кстати почитай про автолоад классов.

Comments

[sbh]

сами то ссылку открывали?

[Сергей Зеленский]

убрать https://toster.ru/ и все норм))

[Константин Б.]

Я думаю ты не там ищешь )) тебе сюда freelansim.ru

Answer 3

[Иван Черный]

сам игрался с "Ололошей", решил подобную проблему подключением стороннего вендора www.itlessons.info/php/routing-library

Comments

[Алексей Ярков]

Спасибо за ссылку. Но я начинающий в этом. Не могли бы вы мои исходники глянуть и заменить мой роутер на то что по ссылке? Или хотя бы на пальцах объясните.

[Иван Черный]

"Полную реализацию системы роутинга вы можете посмотреть в Guthub-репозитории. А также скачать исходные коды домонстрации и поэксперементировать самим." цитата из этого линка, там такой же бутстрап есть, по сути создать обьект роутера и заюзать add метод, на крайняк стукнитесь в скайп, дам пример(i.chorniy)

Answer 4

[Сергей Зеленский]

В ваших исходниках нет обработки маршрутов нормальной

/*
Класс-маршрутизатор для определения запрашиваемой страницы.
> цепляет классы контроллеров и моделей;
> создает экземпляры контролеров страниц и вызывает действия этих контроллеров.
*/
class Route
{

	static function start()
	{
		// контроллер и действие по умолчанию
		$controller_name = 'Main';
		$action_name = 'index';
		
		$routes = explode('/', $_SERVER['REQUEST_URI']);

		// получаем имя контроллера
		if ( !empty($routes[1]) )
		{	
			$controller_name = $routes[1];
		}
		
		// получаем имя экшена
		if ( !empty($routes[2]) )
		{
			$action_name = $routes[2];
		}

		// добавляем префиксы
		$model_name = 'model_'.$controller_name;
		$controller_name = 'controller_'.$controller_name;
		$action_name = 'action_'.$action_name;

		/*
		echo "Model: $model_name <br>";
		echo "Controller: $controller_name <br>";
		echo "Action: $action_name <br>";
		*/

		// подцепляем файл с классом модели (файла модели может и не быть)

		$model_file = strtolower($model_name).'.php';
		$model_path = "application/models/".$model_file;
		if(file_exists($model_path))
		{
			include "application/models/".$model_file;
		}

		// подцепляем файл с классом контроллера
		$controller_file = strtolower($controller_name).'.php';
		$controller_path = "application/controllers/".$controller_file;
		if(file_exists($controller_path))
		{
			include "application/controllers/".$controller_file;
		}
		else
		{
			/*
			правильно было бы кинуть здесь исключение,
			но для упрощения сразу сделаем редирект на страницу 404
			*/
			Route::ErrorPage404();
		}
		
		// создаем контроллер
		$controller = new $controller_name;
		$action = $action_name;
		
		if(method_exists($controller, $action))
		{
			// вызываем действие контроллера
			$controller->$action();
		}
		else
		{
			// здесь также разумнее было бы кинуть исключение
			Route::ErrorPage404();
		}
	
	}

	function ErrorPage404()
	{
        $host = 'http://'.$_SERVER['HTTP_HOST'].'/';
        header('HTTP/1.1 404 Not Found');
		header("Status: 404 Not Found");
		header('Location:'.$host.'404');
    }
    
}

вот эта штука будет обрабатывать только запросы вида site.com/controller/action/
то есть ваш url парсится так blog - контроллер, id_категории - action и естественно его не существует
поэтому и выдает 404 ошибку, ищите другой движок, ибо как написать хороший роутинг это целая статья получиться