Маршрутизация в моем PHP MVC Framework

Question

В течение нескольких лет я работаю над своей собственной облегченной средой MVC для PHP. Который я могу в какой-то момент выпустить под лицензией с открытым исходным кодом.

Вот что я использовал для обработки маршрутов:

function routes($routes, $uriPath) {

    // Loop through every route and compare it with the URI
    foreach ($routes as $route => $actualPage) {

        // Create a route with all identifiers replaced with ([^/]+) regex syntax
        // E.g. $route_regex = shop-please/([^/]+)/moo (originally shop-please/:some_identifier/moo)
        $route_regex = preg_replace('@:[^/]+@', '([^/]+)', $route);

        // Check if URI path matches regex pattern, if so create an array of values from the URI
        if(!preg_match('@' . $route_regex . '@', $uriPath, $matches)) continue;

        // Create an array of identifiers from the route
        preg_match('@' . $route_regex . '@', $route, $identifiers);

        // Combine the identifiers with the values
        $this->request->__get = array_combine($identifiers, $matches);
        array_shift($this->request->__get);

        return $actualPage;
    }

    // We didn't find a route match
    return false;
}

$ routs - это переданный массив, отформатированный так:

$routes = array(
    // route => actual page
    'page/:action/:id' => 'actualPage',
    'page/:action' => 'actualPage',
)

$ uriPath - это путь URI без передней косой черты, например страница / обновление / 102

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

echo $this->request->__get['action'];
// update

echo $this->request->__get['id'];
// 102

Мой вопрос по сути "это можно упростить или оптимизировать?" С особым упором на упрощение регулярных выражений и количества вызовов preg_replace и preg_match.

Answer 1

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

Код:

<?php
$routes             = array
(
    // actual path => filter
    'foo'   => array('page', ':action', ':id'),
    'bar'   => array('page', ':action')
);

/**
 * @author Gajus Kuizinas <g.kuizinas@anuary.com>
 * @copyright Anuary Ltd, http://anuary.com
 * @version 1.0.0 (2011 12 06)
 */
function ay_dispatcher($url, $routes)
{
    $final_path         = FALSE;

    $url_path           = explode('/', $url);
    $url_path_length    = count($url_path);

    foreach($routes as $original_path => $filter)
    {
        // reset the parameters every time in case there is partial match
        $parameters     = array();

        // this filter is irrelevent
        if($url_path_length <> count($filter))
        {
            continue;
        }

        foreach($filter as $i => $key)
        {
            if(strpos($key, ':') === 0)
            {
                $parameters[substr($key, 1)]    = $url_path[$i];
            }
            // this filter is irrelevent
            else if($key != $url_path[$i])
            {       
                continue 2;
            }
        }

        $final_path = $original_path;

        break;
    }

    return $final_path ? array('path' => $final_path, 'parameters' => $parameters) : FALSE;
}

die(var_dump( ay_dispatcher('page/edit', $routes), ay_dispatcher('page/edit/12', $routes), ay_dispatcher('random/invalid/url', $routes) ));

Вывод:

array(2) {
  ["path"]=>
  string(3) "bar"
  ["parameters"]=>
  array(1) {
    ["action"]=>
    string(4) "edit"
  }
}
array(2) {
  ["path"]=>
  string(3) "foo"
  ["parameters"]=>
  array(2) {
    ["action"]=>
    string(4) "edit"
    ["id"]=>
    string(2) "12"
  }
}
bool(false)