我试图在不使用像 Smarty 这样的模板引擎的情况下将表示和逻辑分开。到目前为止，我所拥有的是有效的，但我不确定如何在不将比我想要的更多的 PHP 放入我的演示文稿的情况下做某些事情。例如，现在我有这样的东西:

产品列表.php

    try {
        $query = $conn->prepare("SELECT p.id, p.name, p.description, IFNULL(url, title) AS url, GROUP_CONCAT(c.category SEPARATOR ', ') AS category,
            FROM products p
            LEFT JOIN product_categories pc ON p.id = pc.product_id
            LEFT JOIN categories c ON pc.category_id = c.id
            WHERE p.active = 1
            GROUP BY p.id");
        $query->execute();
        $result = $query->fetchAll(PDO::FETCH_ASSOC);
    }
    catch (PDOException $e) {
            echo $e->getMessage();
    }
include('templates/product_list_tpl.php');

product_list_tpl.php

<div class="card">
    <div class="product-list with-header">
        <div class="product-list-header center-align">
            <h2><?= $header_title; ?></h2>
        </div>
        <?php foreach ($result as $row): ?>
            <!-- Some Product Info -->
            Category:&nbsp;<?= $row['category']; ?>
        <?php endforeach; ?>
    </div>
</div>

在上面的例子中，有些产品有一个类别，有些有多个。它们以逗号分隔的列表很好地显示，但我想将类别名称放入链接中。我知道我可以做类似下面的事情，但对我来说它看起来很乱。

<div class="card">
    <div class="product-list with-header">
        <div class="product-list-header center-align">
            <h2><?= $div_title; ?></h2>
        </div>
        <?php foreach ($result as $row): ?>
            <?php $categories = explode(', ', $row['category']); ?>
            <div class="product-list-item avatar">
                <img src="img/product/<?= $row['id']; ?>.jpg" alt="<?= $row['title']; ?>" class="square">
                <a href="product/<?= generate_link($row['url']); ?>" class="title bold"><?= $row['title']; ?></a>
                <p class="caption"><?= $row['caption']; ?></p>
                <div class="item-bottom">
                    <span class="responsive"><?= $row['description']; ?></span>
                    <p>
                        Category:&nbsp;
                        <?php foreach ($categories as $key => $category): ?>
                            <a href="category/<?= strtolower($category); ?>"><?= $category; ?></a>
                            <?= (sizeof($categories) > 1 && $key == end($categories)) ? ',&nbsp;' : ''; ?>
                        <?php endforeach; ?>
                    </p>
                    <p>
                        <span>Rating:&nbsp;<?= $row['rating']; ?></span>
                        <span class="right">Rated&nbsp;<?= $row['rated']; echo ($row['rated'] == 1) ? '&nbsp;time' : '&nbsp;times'; ?></span>
                    </p>
                </div>
            </div>
        <?php endforeach; ?>
    </div>
</div>

提前感谢您的任何建议。此外，如果有人对我在示例代码中使用的通用分隔格式有任何意见，我很乐意听到。在中断 8 年后，我刚刚重新开始编码。

编辑:添加了缺失的 endforeach 并根据 @Devon 在评论中的建议改进了第三个代码块的缩进。

编辑:我更新了第三个代码块以包含我之前遗漏的 HTML，并添加了实现我正在寻找的输出所需的所有 PHP 功能。它有效，但 IMO 这样做消除了我的小分离。我现在基本上只有一个文件包含我的数据库调用，而另一个文件包含此乱七八糟的文件。我觉得我没有朝着适当的业务逻辑/表示逻辑分离的正确方向前进。

我哪里错了？

最佳答案

冗长、复杂的逻辑，例如:
(sizeof($categories) > 1 && $key == end($categories)) ? ', ' : '';
不应该打扰前端开发人员。 “这行可怕的代码是什么？ 它有什么作用？为什么后端开发人员不给我一些更容易使用的东西？” MVC 的部分力量不仅在于关注点分离，而且 还将后端和前端开发人员的工作分开。

代码类似于<?php foreach ($result as $row): ?>不包含关于它正在使用什么的指示。 DIV、P 和 SPAN 也失控了。
这就是我喜欢 View 助手的原因。

我建议:

product_list_tpl.php

<div class="card">
    <div class="product-list with-header">
        <div class="product-list-header center-align">
            <h2><?= $div_title; ?></h2>
        </div>
        <?= displayItems($items); ?>
    </div>
</div>

上面使用的 View 助手:

function displayItems($items)
{
    foreach ($items as $item)
    {
        $categories = explode(', ', $item['category']);
        $id = $item['id'];
        $title = $item['title'];
        $url = $item['url'];
        $caption = $item['caption'];
        $description = $item['description'];
        $rating = $item['rating'];
        $rated = $item['rated'];
        include('product_list_item_tpl.php');
    }
}

product_list_item_tpl.php

<div class="product-list-item avatar">
  <img src="img/product/<?= $id; ?>.jpg" alt="<?= $title; ?>" class="square">
  <a href="product/<?= generate_link($url); ?>" class="title bold"><?= $title; ?></a>
  <p class="caption"><?= $caption; ?></p>
  <div class="item-bottom">
    <span class="responsive"><?= $description; ?></span>
    <p>
      Category:&nbsp;
      <?php displayCategories($categories); ?>
    </p>
    <p>
      <span>Rating:&nbsp;<?= $rating; ?></span>
      <span class="right">Rated&nbsp;<?= $rated; ?>&nbsp;<?= isPluarl($rated)?'times':'time'; ?></span>
    </p>
  </div>
</div>

上面使用的 View 助手:

function isPlural($number)
{
    return $number != 1;
}

function displayCategories($categories)
{
    $last = end($categories);
    $count = sizeof($categories);
    foreach ($categories as $key => $category)
    {
        $cat = strtolower($category);
        $isLast = $category == $last;
        include('product_list_category_tpl.php');
    }
}

product_list_category_tpl.php

<a href="category/<?= $cat; ?>"><?= $category; ?></a>
<?= ($count > 1 && !$isLast) ? ',&nbsp;' : ''; ?>

注意我倒转了$key == end($categories)与您之前使用的部分分开 !$isLast并交换了 $key至 $category .这个逻辑仍然让人觉得很脏，因为两个类别可能有相同的名称。可能最好只使用 count($categories)连同$i++决定它是否是最后一个循环。

编辑:
这很好地工作并避免了前面提到的问题，因为它依赖于键而不是值:

function displayCategories($categories)
{
    end($categories);
    $last = key($categories);
    $count = sizeof($categories);
    foreach ($categories as $key => $category)
    {
        $cat = strtolower($category);
        $isLast = $key == $last;
        include('product_list_category_tpl.php');
    }
}

关于php - 保持表示 (HTML) 和逻辑 (PHP) 分离，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/37873405/