OOP, MVC - Модели и объекты

Question

Я строю сайт с CodeIgniter, а у меня есть модель под названием Blog_model.

В пределах Blog_model есть способы вытащить список сообщений по определенной теме, например getPopularPosts().

getPopularPosts() запрашивает таблицу сообщений для списка сообщений с topic_id, соответствующих указанному, и сортирует их по популярности. Итак, это один запрос ко всей таблице сообщений (предположим, что это будет очень большой), чтобы найти все сообщения с topic_idx.

Затем foreach результат as индивидуальный идентификатор сообщения, он создает новый объект Post. Класс Post создает сообщение, устанавливая поле id.

Чтобы вернуть содержимое Post, я назначаю $post->getPost();, который снова запрашивает таблицу сообщений, чтобы вернуть всю строку для заданного id.

Эта организация (AFAIK) следует за хорошим объектно-ориентированным принципом. Но теперь, для каждого сообщения (опять же, давайте предположим, что тысячи, миллионы, независимо ...), я должен сначала запросить список id, а затем снова получить содержимое каждого сообщения. Если я возвращаю 30 сообщений, это означает 31 отдельный запрос.

В качестве альтернативы, я мог бы нарушить объектно-ориентированный шаблон и тянуть * для каждого поста в posts, где topic_id = х. Затем у меня есть один запрос, который возвращает все 30 сообщений, но теперь я не чувствую себя объектно-ориентированным.

Что делать?

Answer 1

Нет причин иметь столько запросов. Вы просто ищете 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'; 
     } 

    } 

} 

Там нет никакой выгоды (и на самом деле большая проблема) с генерированием тонны запросов в f ashion, который вы в настоящее время настроили, вместо того, чтобы генерировать один объект из запроса и выполнять итерацию по каждой из строк в контроллере и делать все, что вам нужно с данными.