gpt4 book ai didi

php - 请批评我在 PHP 中对 MVC 的第一次尝试

转载 作者:IT王子 更新时间:2023-10-29 00:20:36 25 4
gpt4 key购买 nike

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

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

我还买了一个域来做一些测试,所以如果你想看到它的实际效果,你可以去 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 ( Controller )

<?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 ( View )

<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 ( View )

<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>

最佳答案

首先,您在保持事物分离方面做得很好。将来会有返回,所以不要放弃。

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

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

您的 Controller 令人困惑(它更像是页面 Controller 而不是应用程序 Controller )。这可能是因为 MVC 架构中有一个隐藏的部分。它被称为Front ControllerDispatcher。 Dispatcher 负责解析输入、实例化 Controller (如在应用程序 Controller 中)和调用请求的方法。如果您想继续使用自定义 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 block 的内容移动到 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";
}

}

这样您就可以拥有多个 Controller ,每个 Controller 都有多个操作。

最后的评论。

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

  • 您可以考虑返回 HTML 而不是在 Controller 中回显它。如果你想实现拦截过滤器,这将给你很大的灵 active 来包装 Controller ,拦截它的输出并对它进行预处理或后处理。拥有一个自动附加 HTML 页眉和页脚的过滤器非常方便。

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

祝一切顺利

关于php - 请批评我在 PHP 中对 MVC 的第一次尝试,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/446502/

25 4 0
Copyright 2021 - 2024 cfsdn All Rights Reserved 蜀ICP备2022000587号
广告合作:1813099741@qq.com 6ren.com