LISTING DE RÉPERTOIRE AVEC AFFICHAGE PAGE PAR PAGE

Commentaire de kiki2sirom le 12/07/2007 12:01:52

je pense qu'ici un peu d'opérateur ternaire ne ferait pas de mal
mais ce n'est que mon avis personnel.
vu que tu as beaucoup de if, else, du style :

if($num_page > 1) {
echo '<a href="'.$current.'?page='.($num_page - 1).'">Page précédente</a> - ';
} else {
echo 'Page précédente - ';
}

pourrait devenir simplement :
echo ($num_page > 1?'<a href="'.$current.'?page='.($num_page - 1).'">Page précédente</a> - ':'Page précédente - ');

tout comme :
if(10 * $num_page < $compteur){
echo ' - <a href="'.$current.'?page='.($num_page+1).'">Page suivante</a>';
} else {
echo ' - Page suivante';
}

pourrait devenir :
echo (10 * $num_page < $compteur?' - <a href="'.$current.'?page='.($num_page+1).'">Page suivante</a>':' - Page suivante');

* de plus:
if($div == $compteur / 10){
$total_page = $div;
} else {
$total_page = $div + 1;
}
for($b = 1;$b <= $total_page;$b++){
pourrait aisément devenir :
if ($div != $compteur / 10)
$div++;
for($b = 1;$b <= $div;$b++){

ton $total_page n'est pas très utile ici !

* de plus:

if(is_numeric($_GET['page'])){
$num_page = $_GET['page'];
} else {
$num_page = 1;
}

deviendrait :
$num_page=(is_numeric($_GET['page'])?$_GET['page']:1);

* tout comme :
for($b = 1;$b <= $div;$b++){
if($b == $num_page){
echo $b.' ';
} else {
echo '<a href="'.$current.'?page='.$b.'">'.$b.'</a> ';
}
}

deviendrait :
for($b = 1;$b <= $div;$b++){
        echo ($b == $num_page?$b.' ':'<a href="'.$current.'?page='.$b.'">'.$b.'</a> ');
}


j'aime beaucoup l'opérateur ternaire, quand ça reste dans le domaine du lisible, soit if / else ou if / elseif / else

47 lignes au final devient 27 lignes :

<?php
$path = '.';
$current = $_SERVER['PHP_SELF'];
$num_page=(is_numeric($_GET['page'])?$_GET['page']:1);
$dh = opendir($path);
$i = 0;$compteur = 0;
$debut = ($num_page - 1) * 10;
$fin = $debut + 9;
while (($file = readdir($dh)) !== false) {
if($file != '.' && $file != '..' && $file != $current) {
if ($i >= $debut && $i <= $fin) {
printf('%s. <a href="%s/%s">%s</a><br />', $i+1, $path, $file, $file);
}
$i++;$compteur++;
}
}
closedir($dh);
echo '<p />';
echo ($num_page > 1?'<a href="'.$current.'?page='.($num_page - 1).'">Page précédente</a> - ':'Page précédente - ');
$div = floor($compteur / 10);
if ($div != $compteur / 10)
$div++;
for($b = 1;$b <= $div;$b++)
echo ($b == $num_page?$b.' ':'<a href="'.$current.'?page='.$b.'">'.$b.'</a> ');

echo (10 * $num_page < $compteur?' - <a href="'.$current.'?page='.($num_page+1).'">Page suivante</a>':' - Page suivante');
?>

c'est quand même intéressant, non ?

allez
kiki

Commentaire de masterhill le 12/07/2007 13:37:33

Salut !
Bon code, j'me suis juste permis de rajouter un pti '@' à la ligne 4 parce qu'au lancement de page sans le $_get['page'], on a le droit à un pti avertissement.
Mais sinon tout est impec 9.5/10  ;-)

Commentaire de tozman2 le 16/07/2007 11:36:31

Les if...else sont plus rapide à executer par PHP... donc l'interêt de les changer reste limité. d'autant que pour modification du code un bloc if...else est bien plus explicite, surtout pour les débutants.

Commentaire de simonviei le 16/07/2007 12:50:55

une fonction tres interressrante pour lister un dossier : glob()

Commentaire de Eliays le 20/07/2007 15:21:09

coucou, c'est possible  de modifier ce code pour qu'il parcour aussi les dossier/sous dossier/ ect? ;)

Commentaire de jlmcreation le 24/11/2007 17:07:04

BONJOUR
BIEN COOL CE SCRIPT , mais est ce possible de lire que les fichiers .html ou .htm ,ne pas voir les photo gif,jpeg et autres

merci

Commentaire de jlmcreation le 25/11/2007 12:00:58

voila ce que ca affiche quand j'ouvre la page

Parse error: syntax error, unexpected T_BOOLEAN_AND in liste.php on line 14

Commentaire de sparko_dutos le 25/11/2007 12:30:11

Il suffit de de ne pas fermer la parenthse du if avant le && mais après cet élément !!

if($file != '.' && $file != '..' && $file != $current && in_array(array('.htm','.html'),strrchr($file,'.')))

ca devrait mieux marcher !!