Question

请查看下面的代码。

require_once("initvars.inc.php");
require_once("config.inc.php");

?>
<?php
if($latestads_count)
{
?>


<div class="latestposts">

<div class="head"><?php echo $lang['LATEST_ADS']; ?></div>


<table border="0" cellspacing="0" cellpadding="0"  class="postlisting" width="100%">


<?php
$sql = "SELECT a.*, ct.cityname, UNIX_TIMESTAMP(a.createdon) AS timestamp, feat.adid      AS isfeat,
            COUNT(*) AS piccount, p.picfile AS picfile, scat.subcatname, scat.catid, cat.catname
        FROM $t_ads a
            INNER JOIN $t_cities ct ON a.cityid = ct.cityid
            INNER JOIN $t_subcats scat ON a.subcatid = scat.subcatid
            INNER JOIN $t_cats cat ON scat.catid = cat.catid
            LEFT OUTER JOIN $t_featured feat ON a.adid = feat.adid AND feat.adtype = 'A' AND feat.featuredtill >= NOW()
            LEFT OUTER JOIN $t_adpics p ON a.adid = p.adid AND p.isevent = '0'
        WHERE $visibility_condn
            $loc_condn
        GROUP BY a.adid
        ORDER BY a.createdon DESC
        LIMIT $latestads_count";
$res_latest = mysql_query($sql) or die($sql.mysql_error());

$css_first = "_first";
while($row = mysql_fetch_array($res_latest))
{

    $url = buildURL("showad", array($xcityid, $row['catid'], $row['catname'], 
        $row['subcatid'], $row['subcatname'], $row['adid'], $row['adtitle']));


?>

    <?php 
    if($row['isfeat'])
    {
        //$feat_class = "class=\"featured\"";
        $feat_img = "<img src=\"images/featured.gif\" align=\"absmiddle\">";
    } 
    else 
    { 
        //$feat_class = "";
        $feat_img = "";
    }

    if($row['picfile']) 
    {
        $picfile = $row['picfile'];
        $imgsize = GetThumbnailSize("{$datadir[adpics]}/{$picfile}", $tinythumb_max_width, $tinythumb_max_height);
    }
    else 
    {
        $picfile = "";
    }
    ?>

    <tr>
        <td width="15">
        <img src="images/bullet.gif" align="absmiddle">
        </td>

        <td>
        <b><a href="<?php echo $url; ?>" <?php echo $feat_class; ?>><?php echo $row['adtitle']; ?></a></b> 
        <?php if(0&&$row['picfile']) { ?><img src="images/adwithpic.gif" align="absmiddle"><?php } ?>
        <?php echo $feat_img; ?><br>


        <span class="adcat">



        <?php echo "$row[catname] $path_sep $row[subcatname]"; ?>



        <?php 
        $loc = "";
        if($row['area']) $loc = $row['area'];
        if($xcityid < 0) $loc .= ($loc ? ", " : "") . $row['cityname'];
        if($loc) echo "<br>$loc";
        ?>             

        </span>



        </td>

        <td  align="right" width="<?php echo $tinythumb_max_width; ?>">
        <?php if($picfile) { ?>
        <a href="<?php echo $url; ?>"><img src="<?php echo "{$datadir[adpics]}/{$picfile}"; ?>" border="0" width="<?php echo $imgsize[0]; ?>" height="<?php echo $imgsize[1]; ?>" style="border:1px solid black"></a>
        <?php } ?>
        </td>

    </tr>

这是现有项目的代码文件之一。正如你所看到的，它有混合在一起的html，sql，php ......显然很难维护。

此应用程序中大约有55个类似大小和类型的文件。

我想重构这段代码，以下是我这样做的目标：

1）易于维护。

2）通过添加其他功能，可以轻松扩展。

3）能够将此代码重用于不同但有些类似的应用程序。

基于以上事实，我有很多问题：

1）您认为我们可以将此代码重构为mvc应用程序吗？

2）如果可以的话。对于专家程序员来说，重构整个项目需要花多少时间（55个文件）？

3）我应该重用上面的代码，还是应该从头开始使用现有的mvc框架？

4）我们需要多长时间才能完成整个项目，以防我们选择现有的mvc框架说symfony，zend等？

5）据我所知，这个站点到目前为止只运行Mysql（因为当前的数据库是在mysql中完成的）。我们是否应该允许模型中的数据抽象/数据访问层（假设有一些比Mysql更好的东西，我不确定这个）

7）我们是否可以轻松地重构代码以包含模型（数据抽象/访问），视图（更多模板，视图逻辑等），控制器等的子层，以防我们将来想要这样做或者我们需要从头开始吗？

8）mvc是否可行，或者是否有比它更好的模式/方式（假设该网站适用于数十万用户）？

Answer 1

我认为没有像“mvc应用程序”这样的东西，但mvc模式肯定可以在这个应用程序中使用。您可以构建一些封装各种查询的数据类，并将结果转换为对象或对象数组。控制器类可以获取这些结果并对其进行处理，然后将处理后的数据提供给特定模板。这是一个很容易维护的类似mvc的构造。我也会构建一些辅助类。你不想到处调用mysql_query，因为也许你明年需要使用mysqli，或者甚至可能是一个完全不同的数据库。

所以，是的，使用mvc模式，但不仅如此。重构整个代码。尽可能使用现成的库（例如用于数据库抽象），并且在开始构建之前花了很多时间考虑正确的方法。

Answer 2

是
我会说每个文件大约一个小时，以确保一切顺利。所以1-2周的扎实工作。
如果效果很好，只需重新考虑因素。如果你真的想要新的东西，那就抓住一个框架。
现有框架可能需要更长时间才能复制现有功能（可能是一个月）。但是，从长远来看，您将拥有更稳定的应用程序（最有可能）。
MySQL没有任何问题。如果您因为其他原因希望将来切换，请使用数据抽象层。
没有理由不能重新考虑因素。我最近这样做了，并不是那么困难。
MVC非常理想。也就是说，有许多不同的方法来做MVC。有些好，有些好，有些可怕。

需要来自PHP技术架构师的意见

2 个答案: