CLASS DE PAGINATION

Commentaire de malalam le 28/12/2006 14:43:42 administrateur CS

Merci de remplir les mots clefs!

Commentaire de Naixn le 29/12/2006 11:17:29

Juste une petite remarque, mais qui n'a rien de "tu devrais faire comme ça", c'est juste que je dis comment j'aurais vu la chose :

Pour moi, le constructeur n'est pas celui qui doit faire toutes les assignations etc.
Je préfère que le constructeur appelle plusieurs petites fonctions qui feront leur petit truc dans leur coin, mais globalement, le constructeur ne FAIT pas. Il organise et appelle, c'est tout.

Encore une fois, je parle juste de ma façon de voir les choses :)

Commentaire de malalam le 29/12/2006 13:27:40 administrateur CS

Hello,

d'autant plus qu'un constructeur est censé construire l'objet, selon la sémantique. Donc pour moi aussi ça parait logique d'initialiser les propriétés membres qui doivent l'être dans le constructeur. Et effectivement, pourquoi passer par une méthode tierce alors que l'objet a accès directement à ses propriétés ? (même quand on passe par __set () d'ailleurs : dans l'objet, on ne passe par par __set, me semble t il).
Bref j'avoue être surpris aussi par ta préférence. Mais bon, ça reste une préférence :-)

Commentaire de FhX le 29/12/2006 13:51:10

Alors on va faire un petit peu d'optimisation de ta class "Machin" :p

class Machin {
public function __construct($file, $nb) {

  // Calcule le hashage SHA-1 du fichier transmis via $file.
  $this->_sha1 = sha1_file($file);

  // Récupération du nombre totale de page. (à faire dans le constructeur, et seulement par lui *SAUF* si tu comptes rajouter des pages entre temps ! (m'étonnerait m'enfin bon :p) )
  $this->_pages = ceil (...);

  // Initialisation de la bdd.
  $this->_initBDD('...'); // Ca tu ne dois pas AVOIR ! Aucune méthode pour initialiser une base de donnée dans une classe autre qu'une classe faite EXCLUSIVEMENT pour ca !
  // Mauvaise utilisation de l'objet dans ce cas ci !

  // Ne sachant pas ce que ca fait, je laisse.
  $this->_launch('script'); // J'en déduis que ca a une fonction spécial, alors on sait jamais :p

}

}

Un peu de commentaire n'a jamais tué personne. Je t'ai sauvé 2 appels de méthodes, donc au moins 10 lignes de codes en moins :)
Surtout la méthode sha1_file(), pas besoin de la réécrire elle existe déja :)

La clarification, c'est le commentaire ! Pas le nom de la méthode :p

Commentaire de malalam le 29/12/2006 14:25:00 administrateur CS

Bah pourquoi pas. Disons, si tu as des initialisations qui nécessitent des méthodes, car ce sont de véritables "fonctions" complexes, ok, évidemment.
Mais s'il s'agit de faire :
$this -> maChaine = monTrim ($maChaine);

avec
private function monTrim ($sChaine) {
  return trim ($sChaine);
}

c'est clairement un problème pour l'optimisation.
Mais s'il s'agit de faire appel à une méthode complexe, why not, évidemment.
Et plus encore s'il s'agit d'une initialisation qui peut être réutilisée plus tard.

Je pense que c'est dans le sens de mon exemple que FhX l'entendait.

Commentaire de Naixn le 29/12/2006 14:31:48

Je vois qu'on est tous d'accord.
Faire une méthode pour un return d'appel d'une fonction, c'est idiot... :)

"Mais je comprend ton point de vue, il est vrai que quelques fois, ca rend le code plus lisible. (je le sais, je le fais moi même :p )"
>> C'est pour ça que je disais qu'ici, ça n'était pas vraiment nécessaire, mais quand j'ai vu la taille du construct, même si raisonnable, ça m'est venu à l'idée, donc bon, j'ai dis. XD

Commentaire de malalam le 01/01/2007 09:22:16 administrateur CS

Là, on en revient à ce qu'on disait...
Pourquoi ne pas renvoyer directement ta propriété dans la méthode __get () ?? Plutôt que d'appeler dans __get () une autre méthode qui elle va renvoyer la propriété...?

Commentaire de xmag le 03/01/2007 10:47:55

Salut,

Très bien cette source mais est-ce qu'elle fonctionne si tu dois passer des paramètres en POST de page en page (par exemple tu as un formulaire de recherche et il y a plusieurs pages de réponses à ta requêtes multicritères)?
(Désolée, je n'ai pas regardé à fond le code, c'est juste une question pour ceux qui recherchent ce genre de pagination ;-))

Commentaire de guill76 le 03/01/2007 16:03:47

Il manque des parenthèses fermantes lignes 236 et 247.

Commentaire de kankrelune le 05/01/2007 17:53:58

En fait ta class ne devrait pas se soucier de l'origine de la variable "page"... page devrait pouvoir provenir de $_GET, $_POST, $_COOKIE ou de toute autre source (variable php, résultat de requete sql, etc)... c'est au developpeur de la passer en paramètre sa récupération ne concerne pas ta classe... d'une part parce que ça n'est pas son boulot d'autre part parce que ça simplifie son utilisation... .. .

@ tchaOo°

Commentaire de NainPuissant le 08/02/2007 10:44:18

Voilà exemple rajouté :)

Commentaire de notestablished le 03/04/2007 02:06:22

moi je viens de trouver cette source...elle m'a tout de suite plus parce que je cherche une class comme celle-ci depuis très longtemps ( j'avais laissé tombé puis je viens de reprendre en fait)

mon problème c'est que la source ne marche par..je suis chez free ( pê est-ce cà ? ) mais dès la ligne 4 il me dit qu'il y a une erreur de syntaxe...

merci de m'aider si vous pouvez ou avez le temps

Commentaire de Popop56 le 14/05/2007 20:43:31

J'ai un petit problème avec cette classe. Les liens pour aller à la premier page,la dernière page etc (les liens >, >>, <...) ne fonctionne pas. Il me renvoie à la racine du site. J'ai l'url rewriting activé.

Commentaire de Popop56 le 16/05/2007 16:40:43

Parfait NainPuissant. Cela marche maintenant parfaitement ! Merci pour ce beau code :)

Commentaire de hindioumax le 16/11/2007 14:56:03

Excellente source, c'est exactement ce que je recherchais. En url rewriting = false, j'ai du toutefois apporter quelques modif sur la classe. En url rewriting = false, j'avais les même problème d'url (avec seulement le &p= après mon host)

J'ai ajouté aussi un paramètre d'option pour définir une classe css.
Maintenant ca fonctionne impeccable.

Merci encore :)


Pour ceux que ca interresse voici les modifs que j'ai fait (sans url rewriting activé):

dans la classe 'pagination', en haut dans les déclarations :
[code]
* @var définition de la classe css des liens
* @access private
*/ private $classe_css = '';
/**
* @var options disponibles
* @access private
*/ private $options = array(
'nb_datas',
'data_per_page',
'nb_link_per_page',
'url',
'url_rewriting',
'var_page',
'space_before',
'space_after',
'separator',
'classe_css'
);
[/code]

Les fonctions modifiées qui prenne en compte la variable 'classe_css' + correction URL pour la navigation des pages:
[code]
/**
* Génère les liens
*/ private function generateLink()
{
if ($this->url_rewriting)
{
$this->link = $this->url;
$this->next_page = sprintf($this->url, $this->current_page + 1);
$this->previous_page = sprintf($this->url, $this->current_page - 1);
$this->first_page = sprintf($this->url, 1);
$this->last_page = sprintf($this->url, $this->nb_pages);
}
else
{
$this->link = strstr($this->url, '?') ? '&' : '?';
$this->link .= $this->var_page . '=';
      $this->next_page     = $this->url . $this->link . ($this->current_page + 1);
      $this->previous_page = $this->url . $this->link . ($this->current_page - 1);
$this->first_page    = $this->url . $this->link . 1;
      $this->last_page     = $this->url . $this->link . $this->nb_pages;
}
}
/**
* Retourne le lien de la première page si la page courante est supérieur à 3 et celui de la page précédente
*
* @return string
*/ public function getFirstLinks()
{
$display = '';
if ($this->current_page != 1)
{
if ($this->current_page >= 3)
{
$display .= ' <a class="' . $this->classe_css . '" href="' . $this->first_page . '"><<</a> ';
}
$display .= ' <a class="' . $this->classe_css . '" href="' . $this->previous_page . '"><</a> ';
return $display;
}
}
/**
* Retourne le lien de la page suivante et celui de la dernière page si la page courante n'est pas égale au nombre de pages
*
* @return string
*/ public function getLastLinks()
{
$display = '';
if ($this->current_page != $this->nb_pages)
{
$display .= ' <a  class="' . $this->classe_css . '" href="' . $this->next_page . '">></a> ';
if ($this->current_page <= $this->nb_pages - 2)
{
$display .= ' <a class="' . $this->classe_css . '" href="' . $this->last_page . '">>></a> ';
}
return $display;
}
}
/**
* Retourne la liste des liens avec le séparateur si il n'est pas vide : 0 | 1 | 2 | 3 | 4
*
* @return string
*/ public function getLinks()
{
$display = '';
for ($i = 1; $i <= $this->nb_pages; $i++)
{
if ($i - 1 < ($this->current_page + $this->nb_link_per_page) && $i + 1 > ($this->current_page - $this->nb_link_per_page))
{
if ($i == $this->current_page)
{
$display .= '<span>' . $i . '</span>';
}
else
{
if ($this->url_rewriting)
{
$url = sprintf($this->link, $i);
}
else
{
$url = $this->url . $this->link . $i;
}
$display .= '<a class="' . $this->classe_css . '" href="'. $url . '">' . $i . '</a>';
}
if ($i != $this->current_page + $this->nb_link_per_page && $i != $this->nb_pages)
{
for ($j = 0; $j <= $this->space_before; $j++)
{
$display .= ' ';
}
$display .= $this->separator;
for ($j = 0; $j <= $this->space_after; $j++)
{
$display .= ' ';
}
}
}
}
return $display;
}
[/code]

Construction de l'objet :
[code]
$options = array(
'nb_datas' => $count, // Avec une base de donnée
// 'nb_datas' => count($datas), // Avec un array
'data_per_page' => 1,
'nb_link_per_page' => 10,
'url' => 'index.php?p='.$id_page, // Sans url rewriting
// 'url' => 'index-%d.html', // Avec url rewriting
// 'url_rewriting' => true, // Avec url rewriting
'var_page' => 'display',
'classe_css' => 'pagination'
);

$pagination = new Pagination($options);
$pagination->separator = '-';

// etc ...
[/code]

:)

Commentaire de _klesk le 29/02/2008 17:22:20

@hindioumax :
Si tu est en php4 c'est foutu pour toi ^^, enfin pas sans modif en tous cas.

@NainPuissant

J'ai voulu essayer le space_besore et space_after mais ça ne marche pas, car il faut mettre
for ($j = 0; $j <= $this->space_before; $j++)
{
  $display .= '&nbsp;';
}

$display .= $this->separator;
for ($j = 0; $j <= $this->space_after; $j++)
{
  $display .= '&nbsp;';
}

car pas défaut il ne prend qu'un espace si on ne lui indique pas le non breakable space, (enfin le navigateur).

voila.

mis à part ça j'ai du aussi modifier de façon à pourvoir utiliser mes variable déjà en $_GET car,
soit :

Ça na pas été gérer

Soit :

J'ai pas vu et/ou compris comment faire.

:)