Почему Router странно обрабатывает запрос типа 'article/$1/$2', при этом 'article/$1' верно обрабатывается?

Question

[Асиков Артур]

Router работал без нареканий, до того момента как потребовалось формировать на сайте запросы типа 'article/$1/$2' и 'woking/$1/$2'.

В процессе генерирования страницы верно находится нужный метод, при этом возникает следующая ошибка: при попытке захода на страницу 'working/1/2' ищет несуществующий action1 в контроллере, при попытке захода на страницу 'working/3/2' ищет несуществующий action3.

Warning: call_user_func_array() expects parameter 1 to be a valid callback, class 'WorkingController' does not have a method 'action3' in C:\OS\OSPanel\domains\mysite.ru\components\Router.php on line 65

Помогите разобраться в чем дело.

Файл routes

return array(
    'work' => 'work/view',
    'working/([0-9]+)/([0-9]+)' => 'working/getTestByNumq/$1/$2',
    'job/([0-9]+)' => 'job/getTest/$1',
    'article/([0-9]+)' => 'article/view/$1',
    'kurs' => 'kurs/index',
    '' => 'main/index',
);

Сам Router

<?php 

class Router
{
    private $routes;

    public function __construct()
    {
        $routesPath = ROOT.'/config/routes.php';
        $this->routes = include($routesPath);
    }

    // return request string

    private function getURI()
    {
        if (!empty($_SERVER['REQUEST_URI'])) {
            return trim($_SERVER['REQUEST_URI'], '/');
        }
    }

    public function run()
    {
        //Получить строку запросы
        $uri = $this->getURI();

        //Проверить наличие такого запроса в routes.php
        foreach ($this->routes as $uriPattern => $path) {
        
           // Сравниваем $uriPattern и $uri
            if (preg_match("~$uriPattern~", $uri)) {

                //Получаем внутренний путь из внешнего согласно правилу
                $internalRoute = preg_replace("~^$uriPattern$~", $path, $uri);


                //Определить какой контроллер и action обрабатывает запрос и параметры
                
                $segments = explode('/', $internalRoute);

                $controllerName = array_shift($segments) . 'Controller';
                $controllerName = ucfirst($controllerName);

                $actionName = 'action'.ucfirst(array_shift($segments));

                $parameters = $segments;

                //Подключить файл класса контроллера
                $controllerFile = ROOT . '/controllers/' . 
                $controllerName . '.php';

                if (file_exists($controllerFile)) {
                    include_once($controllerFile);
                } 
                // else {
                //     header('HTTP/1.0 404 Not Found');
                //     readfile('err404.html');
                //     exit();
                // }


                //Создать объект, взывать метод (т.е. action)
                $controllerObject = new $controllerName;                
                try { 
                    $result = call_user_func_array(array($controllerObject, $actionName), $parameters); 
                } catch (Exception $e) {
                    header('HTTP/1.0 404 Not Found');
                    readfile('err404.html');
                    exit();
                } 
                
                if($result != null){
                    break;
                } 
            }
        }
    }
}

Файл WorkingController

include_once ROOT.'/models/Working.php';

class WorkingController{  

    public static function actionGetTestByNumq($sub, $numq){

        require_once (ROOT. '/views/job/job.php');

        return true;
    }
}

Comments

[Arman]

Кажись не работает замена

$internalRoute = preg_replace("~^$uriPattern$~", $path, $uri);

что в этой переменной с таким запросом?

[Асиков Артур]

В переменной с таким запросом
string(17) "working/view/2/32"

Во время экспериментов поменял местами строки в роутах:

return array(
    'working/([0-9]+)/([0-9]+)' => 'working/getTestByNumq/$1/$2',
    'work' => 'work/view',
    'job/([0-9]+)' => 'job/getTest/$1',
    'article/([0-9]+)' => 'article/view/$1',
    'kurs' => 'kurs/index',
    '' => 'main/index',
);

заработало, на удивление..

[Antony Tkachenko]

Асиков Артур, ничего удивительного

preg_match сработает на паттерне work, прежде чем дойдет до working.
Добавьте в match
"~^{$uriPattern}$~"

Символ ^ - начало строки и $ - конец.

[Асиков Артур]

Antony Tkachenko, после такой замены действительно ошибок с 'article/$1/$2' не возникает, но не работают ajax запросы на сайте(

[Асиков Артур]

Antony Tkachenko, при замене в match
"~^{$uriPattern}~"

ошибки в ajax POST запросах нет, но снова возникает ошибка с 'article/$1/$2'

пример ajax запроса, который обращается к методу actionGetThemes в контроллере Work

function getThemes(sub){			
			sub = Number.parseInt(sub);	
			$.ajax({ 
						type: "POST",
						url: "work/getThemes",
						data: {asub : sub},
						success:function (resp) {
							var text = JSON.parse(resp);
							for(i = 0; i < text.length; i++){
								$('.choice_de').find('ul').append('<li><a href="/working/' + text[i]['sub'] + '/' + text[i]['numq'] +  '">' + text[i]['numq'] + '. ' +
								text[i]['title'] + '</li>');
							}
						}
				}).fail(function () {
					return;
				});
			}

[Antony Tkachenko]

url: "/work/getThemes",

Иначе ajax-запросы будут относительно текущего урл

Answer 1

[Randel Kovalsky]

Знакомый код :)

$segments = explode('/', $internalRoute);

Вот тут ставьте бряку и смотрите на дальнейшие действия шаг за шагом.

Прочёл комментарии и вспомнил свой камент массиве маршрутов - регулярки писать от сложных к простым!!!

Answer 2

[Андрей]

preg_match("~^$uriPattern\$~", $uri)