Как я могу упростить этот PHP? - PullRequest
Новая
       31

Как я могу упростить этот PHP?

1 голос
/ 06 декабря 2009

У меня есть следующие функции модели с PHP. Я знаю, что повторяюсь.

Можно ли как-нибудь упростить этот код? 

function getTopMenus(){
     $data[0] = 'root';
     $this->db->where('parentid',0);
     $Q = $this->db->get('menus');
     if ($Q->num_rows() > 0){
       foreach ($Q->result_array() as $row){
         $data[$row['id']] = $row['name'];
       }
    }
    $Q->free_result();  
    return $data; 
 }  

function getheadMenus(){
     $this->db->where('parentid',0);
     $Q = $this->db->get('menus');
     if ($Q->num_rows() > 0){
       foreach ($Q->result_array() as $row){
         $data[] = $row;
       }
    }
    $Q->free_result();  
    return $data; 
 }  
function getrootMenus(){
     $this->db->where('parentid',0);
     $Q = $this->db->get('menus');
     if ($Q->num_rows() > 0){
       foreach ($Q->result_array() as $row){
          $data[$row['id']] = $row['name'];
       }
    }
    $Q->free_result();  
    return $data; 
 }

Ответы [ 3 ]

3 голосов
/ 06 декабря 2009

Я вижу одно упрощение, которое вы можете попробовать, используя переход по ссылке, чтобы преобразовать вещи в функцию: 

function prepareMenu(&$data) {
     $this->db->where('parentid',0);
     $Q = $this->db->get('menus');
     if ($Q->num_rows() > 0){
       foreach ($Q->result_array() as $row){
          $data[$row['id']] = $row['name'];
       }
    }
    $Q->free_result();
}

function getTopMenus() {
    $data[0] = 'root';
    prepareMenus($data);
    return $data;
}

function getRootMenus() {
    prepareMenus($data);
    return $data;
}

Существует также возможность использования функций передачи по ссылке и переменных , чтобы выделить часть в середине. Может уменьшить дублирование, но может или не может считаться «упрощением».

РЕДАКТИРОВАТЬ Вот что я имею в виду. Этот код не проверен. 

function getMenus(&$data, $appendFunc) {
     $this->db->where('parentid',0);
     $Q = $this->db->get('menus');
     if ($Q->num_rows() > 0){
       foreach ($Q->result_array() as $row){
         $appendFunc(&$data, $row);
       }
    }
    $Q->free_result(); 
}

function appendTopMenu(&$data, $row) {
    $data[$row['id']] = $row['name'];
}

function appendHeadMenu(&$data, $row) {
    $data[] = $row;
}

function getTopMenus() {
    $data[0] = 'root';
    getMenus($data, "appendTopMenu");
    return $data; 
}  

function getheadMenus() {
    getMenus($data, "appendHeadMenu");
    return $data; 
 } 

function getrootMenus() {
    getMenus($data, "appendTopMenu");
    return $data;  
}
0 голосов
/ 06 декабря 2009

Я не знаю, как выглядят ваши классы БД и запросов, но я бы начал их улучшать. Добавьте функции «выборки из массива» и «выборки из хеша» в класс запроса: 

 class Query ...

        function as_array() {
           $data = array();
           if($this->num_rows() > 0)
             foreach ($this->result_array() as $row)
                 $data[] = $row;
           $this->free_result();  
           return $data;
         }
        function as_hash($key = 'id') {
           $data = array();
           if($this->num_rows() > 0)
             foreach ($this->result_array() as $row)
                 $data[$row[$key]] = $row;
           $this->free_result();  
           return $data;
         }

Сделайте так, чтобы db-> where () возвращало себя 

     class DB
          function where(...) {
              stuff
              return $this;

Как только вы это сделаете, ваши клиентские функции станут тривиальными: 

function getTopMenus() {
     $data = $this->db->where('parentid',0)->get('menus')->as_hash();
     $data[0] = 'root';
     return $data;
 } 

function getheadMenus() {
     return $this->db->where('parentid',0)->get('menus')->as_array();
 }  

function getrootMenus() {
     return $this->db->where('parentid',0)->get('menus')->as_hash();
}
0 голосов
/ 06 декабря 2009

Почему бы не передать параметры в вашу функцию и поместить их в методы 'where' и 'get'?

...