如何确保模型方法仅执行一次以避免重复数据库查询-php教程-PHP中文网

如何确保模型方法仅执行一次以避免重复数据库查询

在 Yii2 应用中，若模型方法（如 getRating()）被多次调用，会导致相同 SQL 查询重复执行，显著降低性能；通过缓存方法返回值或使用惰性加载机制，可确保数据库仅查询一次。

在 yii2 应用中，若模型方法（如 `getrating()`）被多次调用，会导致相同 sql 查询重复执行，显著降低性能；通过缓存方法返回值或使用惰性加载机制，可确保数据库仅查询一次。

在您提供的代码中，getRating() 方法被直接用于视图渲染：

<?php echo $singleUser->getRating()['rate']; ?>

问题在于：该方法可能在视图中被隐式调用多次（例如，同一模板中其他位置也调用了 $singleUser->getRating()），或因调试、条件判断等逻辑导致重复执行——每次调用都会触发全新数据库查询，造成不必要的性能开销。

✅ 推荐解决方案：单次查询 + 变量缓存

最简单且高效的方式是在控制器或视图逻辑层主动缓存结果，而非反复调用方法：

// 在控制器 action 或视图顶部（推荐在控制器中处理）
$rating = $singleUser->getRating();
// 验证结果有效性，避免空指针异常
if (is_array($rating) && isset($rating['rate'])) {
    $userRate = $rating['rate'];
} else {
    $userRate = 0; // 或 null / 默认占位值
}

然后在视图中安全复用：

<?php echo $userRate; ?>

⚠️ 注意事项与进阶优化建议

SQL 注入风险：原始代码中直接拼接 $_GET['id'] 到 SQL 字符串，存在严重 SQL 注入漏洞！必须改用参数化查询：

public function getRating()
{
    $id = \Yii::$app->request->get('id', 0);
    if (!$id || !is_numeric($id)) {
        return null;
    }
    return \Yii::$app->db->createCommand(
        "SELECT * FROM (
            SELECT *, (@position := @position + 1) AS rate FROM (
                SELECT executor_id, SUM(rate) / COUNT(rate) AS pts
                FROM user_replies, (SELECT @position := 0) AS a
                GROUP BY executor_id ORDER BY pts DESC
            ) AS subselect
        ) AS general WHERE executor_id = :id"
    )->bindValue(':id', $id, \PDO::PARAM_INT)->queryOne();
}

更优雅的封装方式（推荐）：在模型中添加私有属性 + 惰性加载逻辑，实现「首次调用查库，后续直接返回」：

private $_rating;
public function getRating()
{
    if ($this->_rating === null) {
        $id = \Yii::$app->request->get('id', 0);
        if ($id && is_numeric($id)) {
            $this->_rating = \Yii::$app->db->createCommand(/* ... 参数化SQL ... */)
                ->bindValue(':id', $id, \PDO::PARAM_INT)
                ->queryOne();
        }
    }
    return $this->_rating;
}