php - 尝试通过制作可重用功能来减少代码点火器中的代码

Question

在我的主 Controller 中，我有一堆访问同一组代码的函数。我一直在尝试通过创建一个可重用函数来获取我需要的数据，然后将其传递给另一个函数调用的函数，从而减少 Controller 的大小。 _getSales() 函数将在同一 Controller 内的大约 30 个函数中使用。基本上不是将这些行添加到每个函数。完成后，我可能还会在该私有(private)函数中添加大约 5 个其他变量。现在 View 中的销售变量显示为空。

现有 Controller

function _getSales() {
    $sales = $this -> sale_model -> getSalesById();
    $data['sales'] = $sales;
}

function home() {
    $this -> _getSales();
    $data['pageName'] = 'home';
    $this -> load -> view($currentPage, $data);
}

Answer 1

_getSales() 函数中的 data['sales'] 变量无法在 home() 函数中访问。所以试试

function _getSales() {
    $sales = $this -> sale_model -> getSalesById();
    return $sales;
}

function home() {
    $data['sales'] = $this -> _getSales();
    $data['pageName'] = 'home';
    $this -> load -> view($currentPage, $data);
}

此外，模型函数看起来像是通过 id 获取销售额，而且您似乎没有向它传递任何参数