Pagination en PHP

Version imprimable

Pagination en PHP

Bonjour,
Je souhaite faire une pagination pour mes annonces. Tout marche bien: l'url, l'affichage etc. Cependant, j'ai dû mal à insérer les variables $debut et $nbre_de_pages en LIMIT dans ma fonction getAnnonces() sans que cela crée des erreurs. Voici le code:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
 
...
include 'cmdaffichage.php';
 
 
//On récupère le nombre d'enregistrements
$count= "SELECT COUNT(id) AS cpt FROM annonces";
$stmt= $bdd->prepare($count);
$stmt->execute();
$tstmt=$stmt->fetchAll();
 
 
//Pagination
@$page=$_GET["page"];
if(empty($page)) $page=1;
$nbr_elements_par_page=3;
$nbre_de_pages=ceil($tstmt[0]["cpt"]/$nbr_elements_par_page);
$debut=($page-1)*$nbr_elements_par_page;
 
 
 
 
//On récupére les annonces
 $anouncement= afficherAnnonce();
 $villesm= getVilles();

Page cmdaffichage:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
 
function getAnnnonces(){
  global $bdd;
  $sql = ' SELECT  A.*
                  ,C.nom_'.$sql_lang.' AS CATEGORIE
                  ,V.ville_'.$sql_lang.' AS VILLE
           FROM annonces A
           LEFT JOIN categories C ON C.id = A.id_categorie
           LEFT JOIN villes V ON V.id = A.id_ville
           ORDER BY A.created_date DESC';
  try {
    $req = $bdd->prepare($sql);
    $req->execute();
    $res = $req->fetchAll(PDO::FETCH_ASSOC);
  } catch (Exception $e) {
    echo "Erreur dans la requête " . $sql;
  }
  // le temps des tests pour voir le résultat
  return $res;
}
//...suite
 
function afficherAnnonce(){
 
  $annonces = getAnnnonces();
  $images = getImagesAnnonces();
//...suite

Auriez-vous une idée?

10/01/2022, 21h16
jreaux62

Bonjour,

Code:

getAnnnonces( $page )

Il serait temps que tu apprennes à utiliser les fonctions PHP, non ? :weird:

Bonjour jreaux, merci pour ton aide :) En te suivant, j'ai réussi à le faire en définissant l'équation qui dépend de $page, ie. $début, mais aussi la variable $nbr_elements_par_page = 3. Il fallait les ajouter dans les fonctions getAnnnonces($page) et afficherAnnonce().
La solution:
Page cmdaffichage:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
function getAnnnonces($page){
  global $bdd;
  @$page=$_GET["page"];
  if(empty($page)) $page=1;
  $nbr_elements_par_page=3;
  $debut=($page-1)*$nbr_elements_par_page;
 
  $sql = ' SELECT  A.*
                  ,C.nom_'.$sql_lang.' AS CATEGORIE
                  ,V.ville_'.$sql_lang.' AS VILLE
           FROM annonces A
           LEFT JOIN categories C ON C.id = A.id_categorie
           LEFT JOIN villes V ON V.id = A.id_ville
           ORDER BY A.created_date DESC
           LIMIT '.$debut.','.$nbr_elements_par_page.' ';
  try {
    $req = $bdd->prepare($sql);
    $req->execute();
    $res = $req->fetchAll(PDO::FETCH_ASSOC);
  } catch (Exception $e) {
    echo "Erreur dans la requête " . $sql;
  }
  // le temps des tests pour voir le résultat
  return $res;
}
 
//....suite
 
function afficherAnnonce(){
  @$page=$_GET["page"];
  if(empty($page)) $page=1;
  $nbr_elements_par_page=3;
  $debut=($page-1)*$nbr_elements_par_page;
 
  $annonces = getAnnnonces($page);
  $images = getImagesAnnonces();
 
 
//...suite
}

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 
include 'cmdaffichage.php'; 
 
//On récupère le nombre d'enregistrements
$count= "SELECT COUNT(id) AS cpt FROM annonces";
$stmt= $bdd->prepare($count);
$stmt->execute();
$tstmt=$stmt->fetchAll();
 
 
//Pagination
@$page=$_GET["page"];
if(empty($page)) $page=1;
$nbr_elements_par_page=3;
$nbre_de_pages=ceil($tstmt[0]["cpt"]/$nbr_elements_par_page);
$debut=($page-1)*$nbr_elements_par_page;
 
 
//On récupére les annonces
 $anouncement= afficherAnnonce();
 $villesm= getVilles();
Page d'accueil.php:

<<Il serait temps que tu apprennes à utiliser les fonctions PHP, non ? >>
==>J'avais du mal à fusionner les fonctions contenant des requêtes et les simples requêtes, mais grâce à toi, ça commence à venir :zoubi: , Tu y participes :D

C'est bien ce que je disais :
Tu n'as pas compris l'utilisation des paramètres d'une fonction...
Code:

1 2 3 4 function getAnnnonces($page){ // c'est l'un... global $bdd; @$page=$_GET["page"]; // ...ou l'autre, mais pas les 2. ...

11/01/2022, 20h07
mouls22
J'ai choisi celle là:
Code:

1 2 3 4 function getAnnnonces(){ // global $bdd; @$page=$_GET["page"]; //
.
Elle marche aussi. Merci :)
11/01/2022, 20h39
Séb.

Pas de chance, c'était le mauvais choix ;)

Bonjour Séb, pourquoi? En suivant les instructions de jreaux ça a marché:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
 
function getAnnnonces(){
  global $bdd;
  @$page=$_GET["page"];
  if(empty($page)) $page=1;
  $nbr_elements_par_page=3;
  $debut=($page-1)*$nbr_elements_par_page;
 
  $sql = ' SELECT  A.*
                  ,C.nom_'.$sql_lang.' AS CATEGORIE
                  ,V.ville_'.$sql_lang.' AS VILLE
           FROM annonces A
           LEFT JOIN categories C ON C.id = A.id_categorie
           LEFT JOIN villes V ON V.id = A.id_ville
           ORDER BY A.created_date DESC
           LIMIT '.$debut.','.$nbr_elements_par_page.' ';
  try {
    $req = $bdd->prepare($sql);
    $req->execute();
    $res = $req->fetchAll(PDO::FETCH_ASSOC);
  } catch (Exception $e) {
    echo "Erreur dans la requête " . $sql;
  }
  // le temps des tests pour voir le résultat
  return $res;
}
// suite...

11/01/2022, 21h22
Séb.

Parce que tu créées une forte dépendance entre le contexte local de ta fonction et le contexte global, alors que normalement ces 2 contextes sont isolés.

C'est considéré comme une mauvaise pratique, comme l'utilisation de @ d'ailleurs, et du goto dont heureusement peu de personnes connaissent l'implémentation PHP :mouarf:
11/01/2022, 21h51
mouls22

Mais si je ne mets pas le @, j'ai des erreurs comme "UNDEFINED INDEX PAGE";
<<C'est considéré comme une mauvaise pratique, comme l'utilisation de @ d'ailleurs, et du goto dont heureusement peu de personnes connaissent l'implémentation PH >>
Quelle type de surprise pourrait y avoir?
11/01/2022, 22h13
jreaux62
Citation:

Envoyé par mouls22

Mais si je ne mets pas le @, j'ai des erreurs comme "UNDEFINED INDEX

Tu vois donc bien que ça MASQUE une erreur !
Or, les éventuelles erreurs, il faut les TRAITER.

ICI :
Code:

1 2 @$page=$_GET["page"]; if(empty($page)) $page=1;
sera avantageusement remplacé par :

Code:

$page = ( !empty($_GET["page"]) )? intval($_GET["page"]) : 1;
11/01/2022, 22h20
mouls22

D'accord, c'est vrai qu'avec ce code:

Code:

$page = ( !empty($_GET["page"]) )? intval($_GET["page"]) : 1;

c'est plus simple, et ça n'affiche pas d'erreurs. Remierciements jreaux!
Bien à vous :)
12/01/2022, 07h47
Séb.

Citation:

Quelle type de surprise pourrait y avoir?

Que ça ne fonctionne pas sans que tu saches pourquoi

Attention à empty() qui est un faux-ami (la chaîne de longueur 1 "0" serait empty), perso je ne l'utilise qu'avec les tableaux
Pour déterminer si une variable est initialisée il y a plutôt isset()

Et comme pour $db, il aurait mieux valu passer la page en paramètre de ta fonction
et valider une bonne fois pour toute $_GET['page'] en début de script car là, potentiellement, tu dois refaire tes vérifs avant chaque utilisation