Optimizar función: condicionales

[Eleazan]

Al final, he hecho algo así

Código PHP:

  function getPageLine($page, $title = "") { 
        return '<li '.($page == $this->_page ? 'class="active"' : null ).'><a href="#" class="datatable_reload" data-source="'.$this->_source.'" data-id="'.$this->_id.'" data-search="'.$this->_search_query.'" data-page="'.($page).'">'.($title == "" ? $page : $title).'</a></li>'; 
    } 
     
    function getPages() { 
        if(!$this->_pagination_enabled) return ""; 
        if($this->_table_total_rows > 0 && $this->_table_total_rows > $this->_limit) { 
             
            $total_paginas = ceil($this->_table_total_rows / $this->_limit); 
             
            $paginator = '<ul class="pagination pull-right" style="margin: 0px;">'; 
            if($this->_page != 1) $paginator .= $this->getPageLine(1, "&laquo;"); 
             
            $page_lines = array(); 
            for($i = ($this->_page - 2); $i <= ($this->_page + 2); $i++) { 
                $page = $i > 0 ? $i : 5-abs($i); 
                if($page > $total_paginas) $page = $i - 5; 
                $page_lines[$page] = $this->getPageLine($page); 
                             
            } 
            ksort($page_lines); 
                 
            foreach($page_lines as $line) $paginator .= $line; 
                 
            if($this->_page != $total_paginas) $paginator .= $this->getPageLine($total_paginas, "&raquo;"); 
            $paginator .= '</ul>';     
             
            return $paginator; 
        } 
        else return ""; 
         
         
    }  
   

Que creo que es más legible que el anterior.

Voy a darle un vistazo al Decorator :P