Efficacité et performance du script de pagination

Question

Je devais faire un script de pagination où je devais accorder une attention particulière à la performance, à la sécurité et à l'algorithme. Je suis venu avec le code suivant:

<?php 

class Paginador{ 

public $current_page;  // current page 
public $total_pages;  // total 
public $boundaries = 1;   // pages to link in the beggining and in the end 
public $around = 1;    // pages to link before and after the current 

public $pageArray = array(); // array com toda a informação a paginar 


function __construct($current_page, $total_pages, $boundaries = 1, $around = 1){ 
    $this->current_page = $current_page; 
    $this->total_pages = $total_pages; 
    $this->boundaries = $boundaries; 
    $this->around = $around; 
} 

function __destruct(){ 
    $this->current_page = 0; 
    $this->total_pages = 0; 
    $this->boundaries = 0; 
    $this->around = 0; 
} 

function setCurrentPage($newValue){ 
    $this->current_page = $newValue; 
    return $this; 
} 

function setTotalPages($newValue){ 
    $this->total_pages = $newValue; 
    return $this; 
} 

function setBoundaries($newValue){ 
    $this->boundaries = $newValue; 
    return $this; 
} 

function setAround($newValue){ 
    $this->around = $newValue; 
    return $this; 
} 

function generateLeftBoundaries(){ 
    if ($this->boundaries + $this->around > $this->current_page) { 
     $this->pageArray = array_merge($this->pageArray, $this->populateArray(1 , $this->current_page)); 
    } else { 
     if ($this->boundaries > 0) { 
      $this->pageArray = array_merge($this->pageArray, $this->populateArray(1, 1 + ($this->boundaries - 1))); 
     } 

     if ($this->around > 0) { 
      $this->pageArray = array_merge($this->pageArray, $this->populateArray($this->current_page - $this->around, $this->current_page)); 
     } 
    } 
} 

function generateRightBoundaries() { 
    if ($this->boundaries + $this->around + $this->current_page > $this->total_pages) { 
     $this->pageArray = array_merge($this->pageArray, $this->populateArray($this->current_page, $this->total_pages)); 
    } else {  
     if ($this->around > 0) { 
      $this->pageArray = array_merge($this->pageArray, $this->populateArray($this->current_page, $this->current_page + $this->around)); 
     } 
     if ($this->boundaries > 0) { 
      $this->pageArray = array_merge($this->pageArray, $this->populateArray($this->total_pages - ($this->boundaries - 1), $this->total_pages)); 
     } 
    } 
} 

function populateArray($start, $end) { 
    $output = array(); 
    for ($i = $start; $i <= $end; $i++) { 
     $output[] = $i; 
    } 
    return $output; 
} 

function paginate(){ 

    //create the structure 
    $this->pageArray = array(); 
    $this->generateLeftBoundaries(); 
    $this->pageArray[] = $this->current_page; 
    $this->generateRightBoundaries(); 
    $this->pageArray[] = $this->total_pages; 
    $this->pageArray = array_values(array_unique($this->pageArray)); 

    //now print the pagination array 
    $pages = $this->pageArray; 

    $items = count($pages); 
    for ($i = 0; $i < $items; $i++) { 
     $current_page = $pages[ $i ]; 
     echo $current_page . ' '; 
     if (($i + 1) < $items 
       && $pages[ $i + 1 ] != ($current_page + 1)) { 
      echo '...'; 
     } 
    } 

} 

} 

?> 

Ce code fonctionne actuellement comme prévu, ma question porte sur la performance, la lisibilité et la sécurité. Comment puis-je améliorer ce code pour être plus efficace et plus facile à comprendre. Si vous pouvez énoncer les raisons pour lesquelles ce serait génial. Merci d'avance!

Answer 1

Pour certaines performances et la sécurité, veuillez définir la visibilité de vos méthodes. Ainsi, par exemple:

public function setCurrentPage($newValue){ 
    $this->current_page = $newValue; 
    return $this; 
} 

Ou

private function setCurrentPage($newValue){ 
    $this->current_page = $newValue; 
    return $this; 
} 

ou protégé si vous utilisez des paquets.

Vous pouvez vérifier le type de la variable comme dans toutes les autres langues de programmation moderne:

function setBoundaries($newValue){ 
    $this->boundaries = $newValue; 
    return $this; 
} 

Vous pouvez en savoir plus sur typehinting en php http://www.php.net/manual/en/language.oop5.typehinting.php

Utiliser l'encapsulation en vous appartient à la classe: https://en.wikipedia.org/wiki/Encapsulation_%28object-oriented_programming%29 Donc, vous êtes des variables privées.

Et ne renvoyez pas $ this; dans vous êtes les setters. Je ne sais vraiment pas pourquoi tu veux ça.