Php 从其他方法返回db查询和附加信息的最佳方法是什么

我是CodeIgniter的新手，并试图在最佳实践的基础上继续学习。 我正在尝试从多个来源返回数据——一个来源是mysql db查询，另一个来源是从另一个方法返回数据，该方法将从facebook获取数据

以下是我到目前为止的情况：

型号：

function get_users() {
  $this->db->select('id');
  $this->db->from('users');
  $query = $this->db->get();
  return $query->results;
}
function attach_cover($user_list) {
  foreach($user_list as $user){
    $user->cover = get_cover_value($user->id);
  }
  return $user_list;
}

控制器：

$this->load->model('user_model', '', TRUE);
$data['users'] = $this->user_model->get_users();
$data['user_cover'] = $this->user_model->attach_cover($data['users']);
$this->load->view('index', $data);

视图：

问题：

---

这是正确的做法吗？还是有更好的方法

这是使用CodeDignister的正确方法

只要不在视图中放置任何逻辑，就不要在任何地方使用任何与数据库相关的逻辑，而只使用模型，并使用控制器纯粹地将模型和视图组合起来。。。你不应该犯太大的错误

---

CodeIgniter不是一个非常严格的框架，不幸的是，它太容易将东西放在错误的位置，例如控制器中的sql。

可能是一个更好的问题。（确保阅读他们关于主题的规则等）我建议观看一些关于codeigniter MVC的php学院教程，尽管它过时了，但它提供了最好的方法。Codeigniter 2.2.1最新版本，但我认为Codeigniter 3.0即将发布测试版

foreach($data as $item) {
  echo $item->id;
  echo $item->cover;
}