我意识到这个话题已经被反复询问和解决，尽管我已经阅读了无数类似的问题并阅读了无数文章，但我仍然未能捕获一些关键问题......我正在尝试构建我自己的 MVC 框架用于学习目的并更好地熟悉 OOP。这是供个人私有(private)使用的，并不是说这是偷懒的借口，而是我不太关心拥有更强大框架的所有功能。

我的目录结构如下:

public
- index.php
private
- framework
  - controllers
  - models
  - views
  - FrontController.php
  - ModelFactory.php
  - Router.php
  - View.php
- bootstrap.php

我有一个 .htaccess 文件，将所有请求定向到 index.php，该文件包含基本配置设置，例如时区和全局常量，然后加载 bootstrap.php 文件。 Bootstrap 包含我的类自动加载器、启动 session 、定义要在整个项目中使用的全局函数，然后调用路由器。路由器从 URL 中分离请求，使用 ReflectionClass 对其进行验证，并以 example.com/controller/method/params 的形式执行请求。

我所有的 Controller 都扩展了 FrontController.php:

<?php
namespace framework;

class FrontController
{
    public $model;
    public $view;
    public $data = [];

    function __construct()
    {
        $this->model = new ModelFactory();
        $this->view = new View();
    }

    // validate user input
    public function validate() {}

    // determines whether or not a form is being submitted
    public function formSubmit() {}

    // check $_SESSION for preserved input errors
    public function formError() {}
}

这个前端 Controller 加载 ModelFactory:

<?php
namespace framework;

class ModelFactory
{
    private $db       = null;
    private $host     = 'localhost';
    private $username = 'dev';
    private $password = '********';
    private $database = 'test';

    // connect to database
    public function connect() {}

    // instantiate a model with an optional database connection
    public function build($model, $database = false) {}
}

和基础 View :

<?php
namespace framework;

class View
{
    public function load($view, array $data = [])
    {
        // calls sanitize method for output
        // loads header, view, and footer
    }

    // sanitize output
    public function sanitize($output) {}

    // outputs a success message or list of errors
    // returns an array of failed input fields
    public function formStatus() {}
}

最后，这是一个示例 Controller ，用于演示当前如何处理请求:

<?php
namespace framework\controllers;

use framework\FrontController,
    framework\Router;

class IndexController extends FrontController implements InterfaceController
{
    public function contact()
    {
        // process form if submitted
        if ($this->formSubmit()) {
            // validate input
            $name = isset($_POST['name']) && $this->validate($_POST['name'], 'raw') ? $_POST['name'] : null;
            $email = isset($_POST['email']) && $this->validate($_POST['email'], 'email') ? $_POST['email'] : null;
            $comments = isset($_POST['comments']) && $this->validate($_POST['comments'], 'raw') ? $_POST['comments'] : null;

            // proceed if required fields were validated
            if (isset($name, $email, $comments)) {
                // send message
                $mail = $this->model->build('mail');
                $to = WEBMASTER;
                $from = $email;
                $subject = $_SERVER['SERVER_NAME'] . ' - Contact Form';
                $body = $comments . '<br /><br />' . "\r\n\r\n";
                $body .= '-' . $name;

                if ($mail->send($to, $from, $subject, $body)) {
                    // status update
                    $_SESSION['success'] = 'Your message was sent successfully.';
                }
            } else {
                // preserve input
                $_SESSION['preserve'] = $_POST;

                // highlight errors
                if (!isset($name)) {
                    $_SESSION['failed']['name'] = 'Please enter your name.';
                }
                if (!isset($email)) {
                    $_SESSION['failed']['email'] = 'Please enter a valid e-mail address.';
                }
                if (!isset($comments)) {
                    $_SESSION['failed']['comments'] = 'Please enter your comments.';
                }
            }
            Router::redirect('contact');
        }

        // check for preserved input
        $this->data = $this->formError();

        $this->view->load('contact', $this->data);
    }
}

据我所知，我的逻辑是错误的，原因如下:

• 验证应该在模型而不是 Controller 中完成。但是，模型不应该访问 $_POST 变量，所以我不完全确定我是否正确地完成了这一部分？我觉得这就是他们所说的“胖 Controller ”，这很糟糕，但我不确定需要更改什么...
• Controller 不应该向 View 发送数据；相反，View 应该有权访问 Model 以请求其自己的数据。那么，将 $data 属性从 FrontController 移到 ModelFactory 中，然后在不传递数据的情况下从 Controller 调用 View 是否可以解决这个问题？从技术上讲，它会遵循 MVC 流程图，但建议的解决方案似乎是一个微不足道甚至微不足道的细节，假设它很简单，但它可能不是......
• 让我质疑我的整个实现的部分是我有一个用户对象，它是用用户相应的角色和权限实例化的，我一直在试图弄清楚如何或更具体地在哪里创建 isAllowed() 方法，可以从 Controller 和 View 调用。那么将此方法放在模型中是否有意义，因为 Controller 和 View 都应该可以访问模型？

总的来说，我是否在正确的轨道上，或者我需要解决哪些明显的问题才能走上正确的轨道？我真的希望得到针对我的示例的个人回应，而不是“去读这个”。我感谢任何诚实的反馈和帮助。

最佳答案

• $_POST 超全局变量应该由请求实例抽象，如 this post 中所述。 .

• 输入验证不是 Controller 的责任。相反，它应该由 domain objects 处理在模型层内。

• 模型工厂不是模型。

• 将类参数可见性定义为 public 会破坏对象的封装。

• HTTP 位置 header (重定向)是一种响应形式。因此，它应该由 View 实例处理。

• 在目前的形式中，您的 Controller 直接操纵超全局变量。这导致与全局状态的紧密耦合。

• 应执行授权检查 outside controller .不在里面。

• 您的“模型工厂”应该是一个服务工厂，它被注入(inject)到 Controller 和 View 中。它将确保每个服务仅实例化一次，从而让您的 Controller 使用相同模型层的状态。

关于php - 理解/改进准系统 MVC 框架，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/16659279/