如何避免SELECT查询代码重复？是否需合并PHP数据库操作方法？

首先，你的代码目前存在一个小问题：把数据查询和HTML渲染的逻辑混在了一起，这才导致了重复代码的隐患。咱们可以通过「职责拆分」来解决这个问题，让每个方法只做一件事，同时复用通用逻辑。

第一步：抽离通用的数据库查询逻辑 #

先把重复的数据库操作（prepare、execute、获取结果）抽成一个私有方法，这样所有需要查询广告数据的方法都能复用它：

private function getAdsData(string $sql, array $params = []): array {
    $stmt = $this->getConnection()->prepare($sql);
    $stmt->execute($params);
    return $stmt->fetchAll(PDO::FETCH_ASSOC);
}

这个方法负责接收SQL语句和可选参数，执行查询后返回结构化的广告数据数组，不用关心后续怎么渲染。

第二步：抽离通用的HTML渲染逻辑 #

再把渲染广告的HTML代码单独抽成一个方法，专门负责把数据转换成页面输出：

private function renderAds(array $ads): void {
    if (empty($ads)) {
        echo "<u><br>Nothing to show yet</u>";
        return;
    }
    
    foreach ($ads as $row) {
        echo '<div class="result">';
        echo "Title: <b>".$row['title']."</b>";
        echo "<h6>Category:".ucfirst($row['category'])."</h6>".$row['content']."<br>". "Date: ".$row['date_added'];
        echo '</div>';
        echo "<br>";
    }
}

这个方法只关心怎么把传入的广告数据变成用户能看到的HTML，不用管数据是怎么来的。

第三步：简化原来的两个业务方法 #

现在selectAd和randomAds只需要专注于自己的业务逻辑（构造对应的SQL），然后调用上面两个通用方法就行，完全不用重复写数据库操作和渲染代码：

public function selectAd() {
    $sql = "SELECT category, title, content, date_added from ads";
    $ads = $this->getAdsData($sql);
    $this->renderAds($ads);
}

public function randomAds() {
    // 建议不要用SELECT *，只查询需要的字段更高效
    $sql = "SELECT category, title, content, date_added FROM ads ORDER BY RAND() LIMIT 5";
    $ads = $this->getAdsData($sql);
    $this->renderAds($ads);
}

关于你的几个问题的明确回答 #

1. 能否避免重复编写SELECT查询代码？
   完全可以！通过上面的拆分，数据库查询的通用逻辑只写了一次，所有业务方法都能复用，以后要修改查询逻辑（比如加参数过滤），只需要改getAdsData就行。

2. randomAds是否需要重复selectAd的操作步骤？
   不需要！现在randomAds只需要写自己的SQL，剩下的查询和渲染都复用通用方法，代码简洁很多。

3. 是否应该将所有逻辑合并到一个函数中？
   不建议合并！selectAd是获取全部广告，randomAds是获取随机5条，它们的业务职责不同。分开保持方法的单一职责，代码更清晰，以后扩展（比如加一个按分类查询的方法）也更方便。

内容的提问来源于stack exchange，提问作者magiczna_sowa