Как лучше всего писать и вызывать контроллеры в рамках шаблона MVC?

А каким образом лучше всего произвести вызов? Подключить файл контроллера с помощью require и вызвать в нем определенную функцию? А может написать в файл контроллера целый класс, создать его экземпляр и вызвать метод?

ой.

ixon, файл контроллера должен определяться роутером, он же его и рекуайрит (или инклудит). Как пример:

class Router {
	
	private $routes;
	
	public function __construct(){
		
		$routerPath = ROOT.'/routing/routes.php';
		$this->routes = include($routerPath);
		
	}
	
	private function getURI(){
		
		if(!empty($_SERVER['REQUEST_URI'])){
			return trim($_SERVER['REQUEST_URI'], '/');
		}
		
	}
	
	public function Run(){
		
		$uri = $this->getURI();
		
		if(!array_key_exists($uri, $this->routes)){
			echo "Такой старницы нет!";
			//Перенаправить на 404.
			exit();
		}
		
		foreach($this->routes as $pattern=>$path){
			
			if($pattern == $uri){
				
				$controllerAndAction = explode('/',$path);
				
				//Узнаем имя контроллера и экшена:
				$controllerName = $controllerAndAction[0];
				$actionName = $controllerAndAction[1];
				
				//Назначаем имена файлу контроллера:
				$controllerFileName = $controllerName . 'Controller.php';
				
				//Подключаем файл контроллера:
				include ROOT . '/controllers/'.$controllerFileName;
				
				//Создаем класс контроллера:
				$controllerObject = new $controllerName;
				
				//Вызываем метод класса контроллера:
				$result = $controllerObject->$actionName();
				
				}
			}
	}
}

?>

P.S. Да-да, я знаю что это говнокод, сэкономьте себе и мне время, не пишите это.

Антон Р., почему это говнокод? Не понимаю, все приводя какие-то примеры по части MVC пишут, что у них мл говнокод, а если не пишут, находится с десяток людей, которые прокомментируют код, дополнив его этой пометкой.
Где же не говнокод? Как писать правильно? Писать правильно без модных фреймворков вообще возможно?

Возможно. Но сначала надо понять как они работают.
Водить машину может любой. Сделать новую - далеко не каждый.
И тем более не следует браться за создание автомобиля с нуля, не научившись ездить на готовых.

ixon, не парься, всё говнокод, потому что оценка субъективна.

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

ixon,

как лучше всего реализовать и использовать не готовый контроллер, а свой собственный.

- я тебе показал код простейшего роутера который пишется по памяти в блокноте в перерыве на обед. Там даже параметры экшена не определяются, только сам контроллер и метод.
Суть контроллера - понять что хочет юзер анализируя адресную строку и вызвать нужный метод контроллера в зависимости от запроса юзера.

Антон Р., да, большое спасибо. Полагаю, правильнее сказать что это суть контроллера-роутера, определить какой контроллер требуется пользователю и какой метод этого контроллера следует вызвать, верно?

ixon, все начинается с index.php. В нем создается реквест (не используйте глобальные переменные в ядре своего проекта, просто передавайте реквест туда, где он вам нужен), создается экземпляр контейнера, в котором хранится вся низкоуровневая информация для ваших сервисов (данные базы, пути к папкам кэша и так далее), о которых контейнер знать не может. Дальше из реквеста достается path (урл, по которому юзер сделал запрос), мэтчите его на контроллер (тут в работу вступает роутер, которому вы передаете коллекцию роутов (controller -> route) и текущий path, регулярками или как-либо еще парсите path, достаете маршрут, возвращаете данные о вызываемом контроллере), создаете экземпляр вашего контроллера (это может быть класс и метод, callable, да даже простая ф-ция), пропускаете его конструктор и вызываемый экшен через контейнер, кладете все нужные аргументы, возвращаете респонс (тут и заголовки, и все остальное делаете).

kafkiansky, думаю он тут не многое понял, вы бы пример кода написали.

Антон Р., это примерно то же самое, что вы написали, только ваш роутер занимается созданием контроллера и вызовом экшена. я считаю, это не его обязанность. роутер должен лишь иметь коллекцию роутов, разбирать запрос и возвращать контроллер. к тому же у вас контроллеры голые, методы голые, без параметров, в реальной жизни контроллерам нужны сервисы по работе с базой, нужен реквест, доступ к кукам, сессиям и так далее. передать это все руками тоже нельзя, потому что у разработчика должна быть возможность у одного контроллера вообще не иметь параметры, у другого иметь их штук 5. в общем, тут этим занимается контейнер, у которого есть различные провайдеры, умеющие автовайрить (загружать) аргументы по их типу. ТС может про рефлексию почитать. Ну а уже после заполненного контроллера вызывать его может ядро проекта (в симфони, к примеру, это HttpKernel).

ixon,

Где же не говнокод? Как писать правильно? Писать правильно без модных фреймворков вообще возможно?

Все можно, все возможно, идеального кода не существует)
Фреймворки не "модные", они жизнеспособные за счет удобного применения и нормальной документации.

Я пользовался Laravel, пользовался Yii, даже просматривал какую-то часть их исходного кода.

Тут загвоздка, из-за которой начинают придираться умники - программист должен наработать базу знаний по ООП, шаблонам проектирования и стандартным подходам к решению задач, и это решается книгами, своими поделками и вдумчивым просмотром исходного кода фреймворков, на которых работаете. Стандартное PHP приложение не завязано на MVC, оно может брать идеи оттуда, но реально используется куча подходов помимо этого.
TL;DR - пока не вникаете глубоко в архитектуру приложений - не стоит париться над красотой роутера

я бы сделал так:

use FastRoute\Dispatcher;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpFoundation\Response;

//$container - использование любого контейнера

**
 * Routes
 */
$dispatcher = FastRoute\simpleDispatcher(function (FastRoute\RouteCollector $r) {
    $routes = require __DIR__ . '/routes.php';
    foreach ($routes as $route) {
        $r->addRoute($route[0], $route[1], $route[2]);
    }
});
/**
 * Dispatch
 */
$request = Request::createFromGlobals();
$route_info = $dispatcher->dispatch($request->getMethod(), $request->getPathInfo());
switch ($route_info[0]) {
    case Dispatcher::NOT_FOUND:
        Response::create("404 Not Found", Response::HTTP_NOT_FOUND)->send();
        break;
    case Dispatcher::METHOD_NOT_ALLOWED:
        Response::create("405 Method Not Allowed", Response::HTTP_METHOD_NOT_ALLOWED)->send();
        break;
    case Dispatcher::FOUND:
        $class_name = $route_info[1][0];
        $method = $route_info[1][1];
        $vars = $route_info[2];
        $object = $container->get($class_name);
		
        $response = $object->$method($vars);
        if ($response instanceof Response) {
            $response->prepare(Request::createFromGlobals());
            $response->send();
        }
        break;
}

dmitriy, дополню
Вместо использования контейнера напрямую можно взять PHP-DI/Invoker, т.о. вызов контроллера будет примерно таким:

$invoker = new Invoker\Invoker(null, $container);

$callable = $route_info[1];
$vars = $route_info[2];
    
$response = $invoker->call($callable, $vars);

Удобно, если какой-нибудь тяжелый сервис требуется только для одного экшена. Чтобы не инжектить его в весь контроллер. Ну и можно использовать именованные параметры, вместо обычного массива.

dmitriy, вот этот коммент идеально подходит на роль ответа.

BoShurik, я обычно использую автоиньекцию зависимостей

dmitriy, это не решает те задачи, что решает invoker (по крайней мере не все di контейнеры так могут)

class Controller
{
    public function __construct(FooService $fooService)
    {

    }

    public function __invoke(BarService $barService, $id)
    {

    }
}

$invoker->call('Controller::__invoke', [
    'id' => 'id',
]);

Обычно контейнеры могут заинжектить FooService, но не BarService

BoShurik, если я правильно понял то invoke отключает конструктор и инжектит другой объект?
но не пугает тот момент что класс ожидает поведение FooService а ему прилетает BarService ?

а если это просто доп.иньекция зависимости то в контейнере который я использую League Container
есть конструкция

$container
    ->inflector('LoggerAwareInterface')
    ->invokeMethod('setLogger', ['Acme\Logger']) // Acme\Logger will be resolved via the container
;

https://container.thephpleague.com/3.x/inflectors/

из плюсов что мы опираемся на интерфейс а не на конкретику

dmitriy, нет
В приведенном примере произойдет следующее:
- Вытащим из контейнера объект класса Controller (FooService будет заинжектен в него, тут все как обычно)
- Выполним метод __invoke у полученного объекта, передав туда все нужные параметры
-- BarService будет вытащен из контейнера
-- id возьмется из переданных параметров
- Получим результат $response = $invoker->call(/**/)

По вашей ссылке немного не то. Там вы автоматом передаете зависимости через сеттеры. Но не получится таким способом, во-первых, получить результат метода setLogger, во-вторых, передать туда какие-то свои параметры, которые пришли из реквеста.