Question

所以我一直在研究和构建我自己的MVC框架，但是继续运行6种不同的实现方式，我想知道我到目前为止是否在正确的轨道上。是的，我知道我可以使用Zend或其他类似的人，但我真的想学习框架是如何工作的，而不仅仅是使用别人的。

以下是我的索引文件的简单版本：

if(isset($_GET['url']))
{
    $url = strtolower($_GET['url']);
}
else
{
    $url = 'home';
}

switch($url)  // Select the controller based on the GET var in the url
{
    case 'home': include(ROOT_DIR . 'app/controllers/homeCon.php'); // This page has the link to the DB test page on it
        break;
    case 'dbtest': include(ROOT_DIR . 'app/controllers/dbTestCon.php');
        break;
    default: include(ROOT_DIR . 'app/views/error404View.php');
}

这是我的dbTestCon.php控制器的简单版本：

if(isset($_POST['dbSubBtn']))
{
    $model = new DbTestModel();

    if($_POST['firstName'] != '' && $_POST['lastName'] != '')
    {
        $model->submitToDb($_POST['firstName'], $_POST['lastName'])
        $model->displayPage('goodToGo');
    }  
    else
    {
        $model->displayPage('noInput');
    }
}
else
{
    $model->displayPage('normal');
}

这是我的DbTestModel.php：

class DbTestModel
{
    public function displayPage($version)
    {
        $title = "DB Test Page";
        $themeStylesheetPath = 'public/css/cssStyles.css';

        include(ROOT_DIR . 'app/views/headerView.php');
        include(ROOT_DIR . 'app/views/dbTestView.php');

        switch($version)
        {
            case 'goodToGo':
                include(ROOT_DIR . 'app/views/dbTestSuccessView.php');
                break;
            case 'noInput':
                include(ROOT_DIR . 'app/views/noInputView.php');
                break;
        }
        include(ROOT_DIR . 'app/views/footerView.php');
    }

    public function submitToDb($firstName, $lastName)
    {
        try 
        {
            $db = new PDO("mysql:host=" . DB_HOST . ";dbname=" . DB_NAME, DB_USER, DB_PASS); 
            $db->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION); 

            $sql = $db->prepare('insert into dbtest(firstName, lastName) values(:firstName, :lastName)');
            $sql->bindParam(':firstName', $firstName);
            $sql->bindParam(':lastName', $lastName);
            $sql->execute();
            $db = null;
        }
        catch(PDOException $e)
        {
            echo "It seems there was an error.  Please refresh your browser and try again. " . $e->getMessage();
        }
    }
}

这是我的dbTestView.php：

<form name="dbTestForm" id="dbTestForm" method="POST" action="dbtest">
    <label for="firstName">First Name</label>
    <input type="text" name="firstName" id="firstName" />
    <label for="lastName">Last Name</label>
    <input type="text" name="lastName" id="lastName" />
    <input type="submit" name="dbSubBtn" id="dbSubBtn" value="Submit to DB" />
</form>

这个简单的例子适用于我的测试环境，但是我担心我会在下一个项目中开始使用它，并意识到我的框架存在一些根本性错误并且必须重新开始。感谢您提供任何帮助或建议。

Answer 1

不，这不是正确的方法！

路由机制完全不足：

在您当前的代码库中，您必须手动注册每个控制器，这显然会使引导阶段（ index.php 文件）容易混淆且容易出错
您有“模型”渲染模板

即使在最原始的MVC设计模式解释中，这也是错误的。视图应该是包含表示逻辑的实例，而不是由您称为“模型”的内容呈现的模板。
模型是一个层，而不是任何单个类

MVC设计模式由两层组成：表示层和模型层。模型层包含所有域业务逻辑，并通过某种形式的抽象与存储交互。
每次都不要重新初始化数据库连接。

每次需要使用数据库时，您的“模型”都会初始化新的PDO实例。相反，您应该只创建一次连接实例，并通过构造函数将其传递给每个对象。
停止使用隐藏警告和错误消息编写代码！

Answer 2

在同一个地方进行数据存储（例如模型逻辑）和输出（例如视图逻辑）通常被视为不良形式，这在DbTestModel中发生。

你可以做一个绝对吨的重构，但我不会为你拼出来，因为我最终会写段落。

我恳请您阅读Fabien Potiencer的博客系列Create your own framework... on top of the Symfony2 Components。

即使您不想使用他的组件来帮助构建任何东西，它也应该为您提供一个好主意的堆。

这是建立MVC框架的正确方法吗？

2 个答案:

不，这不是正确的方法！