【问题标题】:Please critique my first attempt at MVC in PHP [closed]请批评我在 PHP 中第一次尝试 MVC [关闭]
【发布时间】:2010-10-01 13:59:28
【问题描述】:

好吧,我不是一个庞大的框架专家,但我一直很喜欢我所听到的关于整个 MVC 运动的内容,所以我想我会尝试用我选择的语言 (PHP) 创建一个简单的应用程序

所以我想问题是:我哪里出错了?我知道有很多关于控制器/模型应该多胖的争论,所以希望我们可以避免这种情况,但是我特别好奇你对我如何适应数据层的想法。

我还买了一个域名来做一些测试,所以如果你想看看它的实际效果,你可以去www.omgmvc.com

首先,这是我的数据库架构:

CREATE TABLE `movies` (
  `id` int(11) NOT NULL auto_increment,
  `movie_name` varchar(255) NOT NULL,
  `release_date` date NOT NULL,
  `directors_name` varchar(255) NOT NULL,
  PRIMARY KEY  (`id`)
);

INSERT INTO `movies` VALUES (1,'Star Wars', '1977-05-25', 'George Lucas');
INSERT INTO `movies` VALUES (2,'The Godfather', '1972-03-24', 'Francis Ford Coppola');
INSERT INTO `movies` VALUES (3,'The Dark Knight', '2008-07-18', 'Christopher Nolan');

这是文件:

index.php (控制器)

<?php

include('datatier.php');
include('models/m_movie.php');

if (isset($_GET['movie']) && is_numeric($_GET['movie']))
{
    $movie = new Movie($_GET['movie']);

    if ($movie->id > 0)
    {
        include('views/v_movie.php');
    }
    else
    {
        echo 'Movie Not Found';
    }
}
else
{
    $movies = Movie::get_all();

    include('views/v_list.php');
}

?>

datatier.php (数据层)

<?php

class DataTier
{
    private $database;

    function __construct()
    {
        $this->connect();
    }

    function __destruct()
    {
        $this->disconnect();
    }

    function connect()
    {
        $this->database = new PDO('mysql:host=localhost;dbname=dbname','username','password');
    }

    function disconnect()
    {
        $this->database = null;
    }

    function get_all_from_database($type)
    {
        $database = new PDO('mysql:host=localhost;dbname=dbname','username','password');

        switch ($type)
        {
            case 'movie':
                $query = 'SELECT id FROM movies';
                break;
        }

        $movies = array();

        foreach ($database->query($query) as $results)
        {
            $movies[sizeof($movies)] = new Movie($results['id']);
        }

        $database = null;

        return $movies;
    }

    function get_from_database($type,$id)
    {
        switch ($type)
        {
            case 'movie':
                $query = 'SELECT movie_name,release_date,directors_name FROM movies WHERE id=?';
                break;
        }

        $database_call = $this->database->prepare($query);
        $database_call->execute(array($id));

        if ($database_call->rowCount() > 0)
        {
            return $database_call->fetch();
        }
        else
        {
            return array();
        }
    }
}

?>

models/m_movie.php (模型)

<?php

class Movie extends DataTier
{
    public $id;
    public $movie_name;
    public $release_date;
    public $directors_name;

    function __construct($id)
    {
        parent::connect();

        $results = parent::get_from_database('movie',$id);

        if ($results == array())
        {
            $this->id = 0;
        }
        else
        {
            $this->id = $id;
            $this->movie_name = $results['movie_name'];
            $this->release_date = $results['release_date'];
            $this->directors_name = $results['directors_name'];
        }
    }

    function __destruct()
    {
        parent::disconnect();
    }

    static function get_all()
    {
        $results = parent::get_all_from_database('movie');

        return $results;
    }
}

?>

views/v_list.php (视图)

<html>
    <head>
        <title>Movie List</title>
    </head>
    <body>
        <table border="1" cellpadding="5" cellspacing="5">
            <thead>
                <tr>
                    <th>Movie Name</th>
                    <th>Directors Name</th>
                    <th>Release Date</th>
                </tr>
            </thead>
            <tbody>
<?php foreach ($movies as $movie) { ?>
                <tr>
                    <td><a href="/?movie=<?php echo $movie->id; ?>"><?php echo $movie->movie_name; ?></a></td>
                    <td><?php echo $movie->directors_name; ?></td>
                    <td><?php echo $movie->release_date; ?></td>
                </tr>
<?php } ?>
            </tbody>
        </table>
    </body>
</html>

views/v_movie.php (视图)

<html>
    <head>
        <title><?php echo $movie->movie_name; ?></title>
    </head>
    <body>
        <h1><?php echo $movie->movie_name; ?></h1>
        <h2>Directed by <?php echo $movie->directors_name; ?></h2>
        <h3>Released <?php echo $movie->release_date; ?></h3>
    </body>
</html>

【问题讨论】:

  • 好问题,但我建议这确实属于 codereview。

标签: php model-view-controller


【解决方案1】:

首先,您在将事物分开方面做得很好。它会在未来得到回报,所以不要放弃。

数据库布局(甚至数据库本身)与 MVC 的本质无关。在大多数情况下,它恰好是关系数据库,但是 MVC 并不明确要求它(您也可以使用 XML 存储或一些网格/云)。对 MVC 来说至关重要的是让 Model 与其他部分分开,而您已经这样做了。

您的视图也明显与其他视图分开。类似于 MVC 的 M 部分,视图不仅可以呈现 HTML,还可以呈现任何文本可表示的输出(XML、XML+XSL、RSS、纯文本甚至电子邮件消息),视图可以通过多种方式实现:PHP 包括像你的一样、模板(即 Smarty)或可序列化为文本的成熟对象。我无法判断哪种策略是最好的,这是个人编码风格和项目要求的问题。

您的控制器令人困惑(它更像是页面控制器而不是应用程序控制器)。这可能是因为 MVC 架构中有一个隐藏的部分。它被称为 Front ControllerDispatcher。由 Dispatcher 解析输入、实例化控制器(如在应用程序控制器中)和调用请求的方法。如果您想继续使用自定义 MVC 实现,我建议您使用一些在 URL 中传递 Controller 类和方法名称的常用方法,即

index.php/Movies/list
index.php/Movies/details/35

然后在新的 index.php 中解析 $_SERVER['PATH_INFO'],实例化类 Movies 并调用它的 list 方法,即

$args = explode('/', ltrim($_SERVER['PATH_INFO'], '/'));
$className = array_shift($args);
$method = array_shift($args);
require "$className.php";
call_user_func_array(array(new $className(), $method), $args);

然后,您只需将 if-else 块的内容移动到 Movies 类中的两个单独的方法中。

class Movies { // may extend generic Controller class if you wish

    public function list() {
        $movies = Movie::get_all();
        include 'views/v_list.php';
    }

    public function details($movieId) {
        $movie = new Movie($movieId);
        if ($movie->id > 0) {
            include 'views/v_movie.php';
        } else {
            echo "Movie Not Found";
    }

}

这样你就可以拥有多个控制器,每个控制器都有多个动作。

最后的评论。

  • 在数据库方面,使用现有的 ORM 框架之一会很方便。它们将为您节省数天的工作时间,并且可能比手工制作的数据库层表现更好。我还建议处理 PDO 实例的实例化,因为在每个 Model 对象中实例化 PDO 并不是最干净的方式。像DBFactory::getConnection 这样的东西就可以了。

  • 您可以考虑返回 HTML 而不是在控制器中回显它。如果您想实现 Intercepting Filters 来包装控制器、拦截其输出并对其进行预处理或后处理,这将为您提供极大的灵活性。拥有一个自动附加 HTML 页眉和页脚的过滤器非常方便。

  • 创建自定义框架是非常有趣和宝贵的教育经验,但我建议使用现有框架之一来完成更严肃的任务。

一切顺利。

【讨论】:

  • +1 用于使用现有的 ORM - 尝试自己将类映射到关系模型是一种痛苦
  • 感谢您的好帖子
【解决方案2】:

你做得很好,但我有一些建议:

  1. 既然你用的是php5,别忘了__autoload函数。
  2. 最好将您的数据层命名为模型。
  3. get_all_from_database 未声明为静态但您正在静态调用它,这会生成 E_STRICT 级别警告。设置错误报告(E_ALL | E_STRICT);你应该会看到警告。
  4. get_all() 静态函数应该在 Model 类中(在您的情况下为 Datatier),这样您就不必为每个其他模型重写它。您需要对该函数进行的唯一更改是替换该行:

    $results = parent::get_all_from_database('movie');

$results = $this->get_all_from_database(get_class($this));

这假定模型的名称必须与数据库中表的名称匹配

【讨论】:

  • 很多优点。谢谢乔伊!
  • 我很惊讶没有其他人支持,我发现这是一个非常好的答案
【解决方案3】:

唯一让我感到“奇怪”的事情(尚未提及)是您使用两个 PDO 实例与同一个数据库通信。本身并不算太糟糕,但您还要存储用户名、密码和 dsn 的其余部分两次。

这可能只是因为这是发布到网络上的示例代码。

【讨论】:

    猜你喜欢
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 2013-07-01
    • 1970-01-01
    • 1970-01-01
    • 2017-12-23
    相关资源
    最近更新 更多