OOP，MVC - 模型和对象

Question

我正在使用 CodeIgniter 构建一个站点，并且我有一个名为 Blog_model 的模型。

在Blog_model 中，有一些方法可以拉取特定主题的帖子列表，例如getPopularPosts()。

getPopularPosts() 查询posts 表以获取topic_id 与指定的匹配的帖子列表，并按受欢迎程度对其进行排序。因此，这是对整个帖子表的查询（假设最终这将非常大）以查找所有带有 topic_id x 的帖子。

然后，foreach 结果 as 一个单独的帖子 ID，它创建一个新的 Post 对象。 Post 类通过设置字段id 来构造一个帖子。

为了返回Post 的内容，我指定$post->getPost();，它再次查询posts 表以返回给定id 的整行。

这个组织（AFAIK）遵循一个很好的面向对象原则。但是现在，对于每个帖子（再次，假设有数千、数百万，等等），我必须首先查询ids 的列表，然后再次获取每个帖子的内容。如果我返回 30 个帖子，这意味着 31 个单独的查询。

或者，我可以打破面向对象的模式并为posts 中的每个帖子拉*，其中topic_id = x。然后，我有一个查询返回所有 30 个帖子，但现在我感觉不是那么面向对象。

怎么办？

Answer 1

没有理由有这么多的查询。您基本上只是在寻找来自特定主题 ID 的 X 个帖子...您应该将其作为一个对象返回，然后在 PHP 中遍历结果，因为一旦您到达，这样做会明显更快拥有数百万行的意义

你应该这样做：

class blog_model extends CI_Model {

     function __construct(){
         parent::__construct();
     }

     function getPopularPosts($cat_id){
        /* Using method chaining here since you sound like you
           really want to utilize everything OO CI has to offer */
        $posts = $this->db->select('id, title, post_info')
                  ->where('topic_id', $topic_id)
                  ->get('posts');

        if($posts->num_rows() > 0){
             return $posts;
        }else{
             return FALSE;
        }
     }    
}

那么您的控制器将如下所示：

class blog extends CI_Controller {

     function __construct() {
         parent::__construct();
     }

     function blog_posts($popular_post_id) {
         $this->load->model('blog_model');
         $posts = $this->blog_model->getPopularPosts($popular_post_id);

         if(!empty($posts){
             foreach($posts as $post){
                 echo $post->id;
                 echo $post->title;
                 echo $post->post_info;
             }
         }else{
             echo 'There are no posts';
         }

     }

 }

以您当前设置的方式生成大量查询没有任何好处（实际上是一个大问题），而不是从查询中生成一个对象并遍历控制器中的每一行并执行数据随心所欲。