这是将 URI 与 MVC 的 PHP 中的类/方法匹配的好方法吗

Question

我是 MVC 的新手，所以这是我的第一次尝试，我相信你们可以给我改进，感谢任何提示或帮助！

下面是我为我正在开发的个人框架设计的路由器/调度程序系统，这是我第一次尝试使用 MVC 模式。

第一个代码块只是我的 .htaccess 文件，它通过我的 index.php 文件路由所有请求。

第二个代码块是我的“路由”数组，它将告诉路由器 object、class 和调用方法以及任何 ID 或寻呼号码（如果存在）。

第三块代码是路由器 class。

第四个块正在运行 class

因此，路由器 class 必须使用正则表达式将 URI 与路由 map 中的路由相匹配，理论上，当有 50 多个路由列表时，这听起来像正则表达式必须运行的路由列表时性能不佳，我应该这样做吗这不一样？ 我使用正则表达式的主要原因是匹配路线中存在的页码和 ID 号。

另外请不要只是告诉我使用框架，我这样做是为了更好地学习它，我通过这种方式学习得更好，只是现在不喜欢使用现有的框架，我研究了所有主要的框架和一些不太常见的框架已经有想法了。

1）所以主要问题是，有什么看起来不对吗？

2）有没有更好的方法来检测 URI 中的内容，而不是像我正在做的那样在数组上使用正则表达式，在高流量站点上考虑它？

3）由于所有内容都通过 index.php 文件进行路由，我将如何处理 AJAX 请求？

对不起，如果这令人困惑，我自己有点困惑！

---

.htaccess文件

RewriteEngine On
RewriteCond %{REQUEST_FILENAME} !-d
RewriteCond %{REQUEST_FILENAME} !-f
RewriteRule ^(.*)$   index.php?uri=$1    [NC,L,QSA]

Map 阵列（）

/**
 * Map URI to class/method and ID and Page numbers
 * Must be an array
 */
$uri_route_map = array( 
    //forums
    'forums/' => array(
        'controller' => 'forums',
        'method' => 'index',
        'id_number' => '',
        'page_number' => ''),

    'forums/viewforum/(?<id_number>\d+)' =>  array(
        'controller' => 'forums',
        'method' => 'viewforum',
        'id_number' => isset($id_number),
        'page_number' => ''),  

    'forums/viewthread/(?<id_number>\d+)' =>  array(
        'controller' => 'forums',
        'method' => 'viewthread',
        'id_number' => isset($id_number),
        'page_number' => ''),

    'forums/viewthread/(?<id_number>\d+)/page-(?<page_number>\d+)' =>  array(
        'controller' => 'forums',
        'method' => 'viewthread',
        'id_number' => isset($id_number),
        'page_number' => isset($page_number)),

    // user routes
    // account routes
    // blog routes 
    // mail routes
    // various other routes
);

路由器 class 读取并匹配上面的 Map 数组

/**
 * Run URI against our Map array to get class/method/id-page numbers
 */
 class Router
{
    private $_controller = '';
    private $_method = '';
    public $page_number = '';
    public $id_number = '';

    public function __construct($uri, array $uri_route_map)
    {
        foreach ($uri_route_map as $rUri => $rRoute)
        {
            if (preg_match("#^{$rUri}$#Ui", $uri, $uri_digits))
            {
                //if page number and ID number in uri then set it locally
                $this->page_number = (isset($uri_digits['page_number']) ? $uri_digits['page_number'] : null);
                $this->id_number = (isset($uri_digits['id_number']) ? $uri_digits['id_number'] : null);
                $this->_controller = $rRoute['controller'];
                $this->_method = $rRoute['method'];

                // just for debug and testing while working on it / will be removed from final code
                echo '<hr> $page_number = ' . $this->page_number . '<br><br>';
                echo '<hr> $id_number = ' . $this->id_number . '<br><br>';
                echo '<hr> $controller = ' . $this->_controller . '<br><br>';
                echo '<hr> $method = ' . $this->_method . '<br><br>';
                break;
            }else{
                $this->page_number = '';
                $this->id_number = '';
                $this->_controller = '404';
                $this->_method = '404';
            }
        }
    }

    public function getController()
    {
        return $this->_controller;
    }

    public function getMethod()
    {
        return $this->_method;
    }

    public function getPageNumber()
    {
        return $this->page_number;
    }

    public function getIDNumber()
    {
        return $this->id_number;
    }

    /**
     * Call our class and method from values in the URI
     */
    public function dispatch()
    {
        if (file_exists('controller' . $this->_controller . '.php'))
        {
            include ('controller' . $this->_controller . '.php');
            $controllerName = 'Controller' . $this->_controller;
            $controller = new $controllerName($this->getIDNumber(),$this->getPageNumber());
            $method = $this->_method;
            if (method_exists($this->_controller, $this->_method))
            {
                return $controller->$method();
            } else {
                // method does not exist
            }
        } else {
            // Controller does not exist
        }
    }

}

运行

/**
 * Testing the class
 */
$uri = isset($_GET['uri']) ? $_GET['uri'] : null;
$router = new Router($uri, $uri_route_map);
$router->dispatch();

?>

Answer 1

1）对我来说没问题。 代码看起来有点乱。

2）是的，有更好的方法。 您正在执行正则表达式，因为您想匹配您不知道的 URL 部分。 为什么不这样做$parts = explode("/", $uri)然后看看你是否能找到你正在寻找的页面？ 您将需要为每个页面定义您期望的参数数量，否则您将不知道是选择带有参数array("viewform", 123)的forums还是带有参数array(123)的forums/viewforum 。

explode比正则表达式感觉负载更好。 它还增加了改进错误处理的好处。 如果传递给viewforum的参数不是数字怎么办？ 当然，您可以做得比"404"更好；）

3) 制作一个单独的 ajax 处理程序。 Ajax 无论如何都是隐藏的，因此您无需费心提供语义 URL。

例子：

function find_route($parts) {
    foreach ($uri_route_map as $route => $route_data) {
        $route_check = implode("/", array_slice($parts, 0, count($parts) - $route_data['num_arguments']));
        if ($route_check === $route) {
            return $route_data;
        }
    }
    throw new Exception("404?");
}

$uri = "forum/viewforum/522";

$parts = explode("/", $uri);
$route = find_route($parts);
$arguments = array_slice($parts, count($parts) - $route['num_arguments']);

$controller = $rRoute['controller'];
$method = $rRoute['method'];
$controller_instance = new $controller();
call_user_func_array(array($controller_instance, $method), $arguments);

（未经测试）

插件

由于 $uri_route_map 您不能“动态”注册更多插件或页面或“路线”。 我会添加一个 function 以动态添加更多路由到Router 。

此外，您可以考虑一些自动发现方案，例如，将检查文件夹plugins/以查找具有名为“manifest.php”的文件的文件夹，当调用该文件时，将有选择地向路由器添加更多路由。

Answer 2

1),2) 我不认为将 id_number 和 page_number 放在路由器中是个好主意，因为将来您可能会遇到 url 的许多其他参数。 最好只使用 controller 和方法并在 controller 中定义如何处理其他参数或创建其他处理请求信息的 class 请求。

3) 对于 ajax 使用 url 就像 ajax/module/action。 并创建 ajax controller 来做基本的 ajax 安全工作，比如检查 XSRF，然后决定运行哪些控制器并调用操作。

Answer 3

1) & 2) 我不会说，这是不对的，但为什么不使用默认路由呢？ 大多数时候像这样的路线

控制器/动作/param1/param2

对于您的大部分页面来说已经足够了。

您可能可以执行类似的操作来定义默认路由：

$this->controller = 'index';
$this->action = 'index';

private function getDefaultRoutes()
{
    $url = $_SERVER['REQUEST_URI'];
    $tabUrl = explode('/',$url);


    if(!empty($tabUrl))
    {
        $this->controller = array_shift($tabUrl);
        $this->action = array_shift($tabUrl);
        $this->params = $tabUrl;
    }
}

然后，如果您需要更具体的路线，您可以在数组或您想要的任何内容中定义它们。 在您的路由器中，您只需检查当前 URI 是否匹配特定路由或默认路由。 通过这样做，您将减少要匹配的路由数量并提高路由器的速度。

3）你的路由器可能是由你的索引实例化的，没有索引就没有根，所以不幸的是你可能无法避免使用它。 这就是为什么避免在索引中执行昂贵的操作非常重要的原因。 如果您的所有页面都不需要它，通常不要在索引中初始化您的数据库连接。

另外请不要只告诉我使用框架

不要忘记下载一些著名的框架并查看他们的代码。 这是更好的学习方式。 通过这样做，您可能会找到很多好的做法和答案。

Answer 4

1）所以主要问题是，有什么看起来不对吗？

就个人而言，我认为随着您网站的增长，这会变得更加复杂。 正如我所学到的，MVC 框架应该是“设置它并忘记它”——您将请求处理程序（控制器）与数据库查询和业务端（模型）以及显示元素（视图）分开。

[ NB：您可能需要其他核心方面。 我的标准框架包括一些核心元素，它们通过各个部分承载 session，以及处理站点工作的基本方面——例如，虽然模型负责按照 controller 的指示进行正确的数据库调用，但有核心sql.class.php文件中的函数，它为我提供了一组标准化的方法来进行这些调用并根据需要传递或缓存结果。]

您的调度方法是正确的——您从 URI 中提取 controller 的名称（论坛、个人资料等）。 你需要一个 uri map 吗？ 我觉得你正在创造一种不必要的情况，你必须每次都更新这个 map，而不是在需要新功能时简单地创建一个新的 controller，然后在数据库中注册它。 我并不是说你本身就错了，我只是觉得我不会那样做。

2）有没有更好的方法来检测 URI 中的内容，而不是像我正在做的那样在数组上使用正则表达式，在高流量站点上考虑它？

控制结果（没有双关语，因为在这里工作的是 controller）。 考虑这种方法，看看它是如何为您工作的：

您的 index.php 文件（又名“主控制器”）获取 URI 并将沿“/”的值分解为位。 bit[0] 是 controller ID - 这表示“我想使用名为 bit[0]=>value 的 controller”。 这是这样做的：

require_once( dirname( __FILE__ )."/controllers/".$bit[0]."controller.php" ); 

就个人而言，在目录结构方面有点怪异，我使用 bit[0] 来标识 controller.php 所在的目录，因为我可能有子控制器。

这是我用来解析其他位的 controller 文件。 为此，我将使用一个示例：

假设 bit[0] 带有值“forums”。 如果设置了，我可能会将 bit[1] 传递给 switch 语句。 默认情况下，我总是想列出，但我可能会专门将其指向 bit[1] 中的“list”、“view”或“post”。 这将在 controller class 中告诉我调用哪个方法。 例如，如果我需要执行查询和缓存论坛列表，该方法将告诉我调用关联的“论坛”model。

无关的“位”可能会做以下两件事之一：它们可以作为简单的 arguments 传递给从 model 请求什么数据的方法，或者位 [1] 可能足够复杂，足以保证子 Z594C103F2C9FAB6E04C3D8ABE, 和位将被传递给 controller 以确定适当的操作，就像论坛 controller 所做的那样。

正则表达式很慢，应尽可能避免。 由于我们可能有一个/forums/view/102305的 URI，我们可以假设forums controller 会将102305传递给与view参数关联的方法（该方法类似于private function displayPost( $id ) 102305 $id ）。 不需要正则表达式，因为我们可以简单地沿着一个常见的预期分隔符分解这些值。

3）由于所有内容都通过 index.php 文件进行路由，我将如何处理 AJAX 请求？

不是很困难。 如果 controller 设置为 AJAX，您可以重建 URL 并直接访问它。 您可以在 .htaccess 文件（ RewriteRule ^(AJAX)($|/) - [L] ）中编写排除项。 或者（不理想，但一个偷偷摸摸的解决方法）是 append ../到您的 AJAX URI 将 URI 推回根目录 - 它不再尝试访问index.php

编辑

假设我们在您的示例中使用/forums/id-1234/page-4的 URI。 同样，假设如上所述， forums指的是要使用的 controller，并且每隔一个/分隔 arguments（我喜欢称之为“钻探”）。 所以，在我们的论坛 controller 文件（我们称之为forumcontroller.php ，我们可能有这样的（极其简化的）构造函数：

// $registry is a class containing fundamental methods, and is meant to exemplify all 
// classes tied to the main controller "index.php". Keep in mind, I'm assuming we've 
// found the right controller by exploding the URI, and passed the remainder as bits
// to the constructor.
public function __construct( registry $registry ) {
  $this->registry = $registry;  //tying this controller to main controller.
  // For ease and clarity, we're assuming there's no case in which you wouldn't have
  // bits set. Error checking is easy.
  $bits = $this->registry->getURLBits;
  switch( $bits[0] ) {
    case 'view': $this->showForumEntry( $bits[1], (isset( $bits[2] ) ? $bits[2] : '' ); break;
    case 'edit': $this->editForumEntry( $bits[1] ); break;
    case 'post': $this->postForumEntry(); break;
    default: $this->listForumEntries(); break;
  }
}

private function showForumEntry( $thread, $offset ) {
  // Because you wanted to prepend id to the id element, we can use this for
  // cheekiness in the query if our DB is well designed.
  $data = explode('-', $thread);
  // Select all from forums where id = 1234
  $sql = "SELECT * FROM forums WHERE $data[0] = $data[1]";
  if( $offset != '' ) {
    $page = explode('-', $offset);
    $offset = $page[1] * 25; // Or whatever your max per page is. Make it dynamic.
    $max = $offset+25;
    $sql .= " LIMIT $offset, $max";
  }
  // You see where I'm going with this...
}

关键是您可以控制传递的内容以及处理方式。 控制 URI，您可以简化它们的处理。

编辑 2再次通读，我认为有一些概念会对您有所帮助，并且您应该熟悉：

在此处查看“工厂”模式（我的 $registry 本质上是一组工厂）： http://php.net/manual/en/language.oop5.patterns.php

MVC 的一个很好的图形分解： http://best-practice-software-engineering.ifs.tuwien.ac.at/patterns/images/mvc3.jpg

有关工厂方法的更多信息： http://www.devshed.com/c/a/PHP/Design-Patterns-in-PHP-Factory-Method-and-Abstract-Factory/

另一个注意事项，这是在使用 Joomla、Drupal、Wordpress 以及各种不同的企业 CMS 和 BBS 解决方案后的个人观察 - 只为您设计。 当您开始尝试成为“适合所有人的东西”时，您最终会遇到很多不必要的膨胀，每个页面都会加载并使用 100 次。MVC 是一种设计模式，将其用作模式会告诉您摆脱各个方面的过剩，包括 URI。 处理/controller/arg1-Identifier/arg2-offset是不必要的，您可以轻松摆脱/controller/id/offset （例如/forums/1234/4 ）。 如果您想让它对 SEO 友好，请添加主题标题，而不是标识 ID 的标签（例如/forums/1234-This-Is-A-Topic/4 ）。

现在，让我们还指出我在上面的编辑中显而易见的一点——这是一个 controller，专门用于论坛元素。 您网站的每个元素（即论坛、画廊、个人资料等）都应该有自己的 controller。 为什么？ 因为每个人都在其页面上做完全不同的事情。 So make use of that – you don't need to use a URI map so long as you understand that you're directing to the controller, and the controller is delegating responsibilities to the model and any sub controllers that might be needed.

我真的希望这会有所帮助。

Answer 5

1）它有效吗？ 如果是，那么是的。 由于上面的代码只包含数组、正则表达式和验证，我不认为你的代码有问题。 只要它有效。 但如果你问，'那代码是可扩展的吗？ 那么答案将是多种多样的，并且这一切都取决于您的 MVC 框架目的（例如，该框架是否用于一般用途，例如：博客，或者它对于 REST API 提供程序的特殊性。等等......）

2) 是的。 Kohana、Zend、CI 和其他流行的（和高度优化的）PHP 框架使用它（路由器上的数组+正则表达式）。

3）我认为您可以在路由块/部分中给它一个标志，并将该标志作为全局变量提供。 因此，在您的 controller 中，您可以通过检查该标志来决定为不同的请求类型（ajax/非 ajax）发送哪个响应（例如，您可以提供$this->is_ajax作为 Controller 范围内可用的全局方法）。

Answer 6

如果我可以补充几点：

• 从路由器中删除id_number和page_number - 只需传递与 controller 匹配的所有内容，毕竟，处理该数据是 controller 作业，而不是路由器的

• 不要将$uri传递给构造函数，而是将其传递给dispatch() 。

• 为什么$uri_route_map中有那些isset() -s？ 显然它们是false ，因为$uri_route_map是在Router() object 实例化之前定义的。

• 建议在匹配例程中添加更多逻辑 - 在您当前的情况下， sitename/forums将不匹配任何导致 404 的内容（没有尾部斜杠）

• 您还可以在$uri_route_map中定义默认参数，然后将它们与匹配的参数进行array_merge 。 因此，例如，当没有指定页码时page_number将等于 1

• 如果您担心高流量网站的性能，您可以缓存路线。 毕竟， forums/viewforum/100将始终指向同一个控制器/方法。

为什么您担心将 AJAX 请求发送到您的 index.php 文件？ 那有什么问题？