Contruction d'une requete personnalisable

Grafiteur -  
 Grafiteur -
Bonjour,
je voudrais construire une requête SQL personnalisé via PHP

voilà ce que j'ai fais :
function rechercheJeu($nomJeu, $pegi, $date1, $date2, $choix) {
    $Requete_recherche = 'SELECT *
                        FROM JEU ';

    if (!($nomJeu == null)) {
        $Requete_recherche += 'WHERE nomJeu = ' . $nomJeu . ' ';
        $nomJeu = null;
    } elseif (!($pegi == null)) {
        $Requete_recherche += 'WHERE pegi = ' . $pegi . ' ';
        $pegi = null;
    } elseif (!($date1 == null) && !($date2 == null)) {
        $Requete_recherche += 'WHERE dateSortie BETWEEN ' . $date1 . ' AND ' . $date2 . ' ';
        $date1 = null;
        $date2 = null;
    } else return;
    if (!($pegi == null)) $Requete_recherche += 'AND pegi <' . $pegi . ' ';
    if (!($date1 == null) && !($date2 == null)) $Requete_recherche += 'AND dateSortie BETWEEN ' . $date1 . ' AND ' . $date2 . ' ';
    if (!($choix == null)) $Requete_recherche += 'ORDER BY ' . $choix; // choix colonne + ASC ou DESC
    $Requete_recherche += ';';
    return $Requete_recherche;
};


C'est la première fois que j'utilise le potentiel des elseif, du coup je me demandais si mon code était ok.

Sachant que la problématique c'est de savoir si on commence par 'WHERE' ou 'AND' selon si on commence ou non. Je pouvais faire très simple, mais le but reste d'optimiser le code, donc je suis à le recherche d'idées afin de progresser.

En même temps je peux m'en servir comme modèle car j'ai quelques autres recherche dans ce genre à produire

Merci pour votre aide :)

2 réponses

  1. NHenry Messages postés 15235 Date d'inscription   Statut Modérateur Dernière intervention   387
     
    Si tu veux gérer plusieurs conditions de manière variable tu peux faire un truc tu genre :
    $Conditions=array();
    if(...)
        $Conditions[]='NomJeu=...';
    if(...)
        $Conditions[]='....';
    
    if(count($Conditions)>0)
        $Sql.='WHERE '.implode(' AND ',$Conditions)

    0
  2. jordane45 Messages postés 30426 Date d'inscription   Statut Modérateur Dernière intervention   4 830
     
    Bonjour,

    En complément de la réponse de Nhenry
    if (!($nomJeu == null)) {
    
    

    revient plus simplement à
    if($nomJeu){
    

    Idem pour les autres...

    Et les différents IF proposés par NHENRY peuvent s'écrire plus proprement via l'écriture ternaire :
    $Conditions=array();
    
    $Conditions[]= $nomJeu ? "NomJeu='$nomJeu'" : '';
    $Conditions[]= $pegi? "pegi='$pegi'" : '';
    //etc...
    


    NB: Attention à l'utilisation des QUOTES pour entourer les "string".

    NB² : La création d'une requête de cette façons ne te permettra pas d'utiliser les requêtes préparées de PDO (ce qui est fort dommage vu ce que ça amène en terme de sécurisation des données...) mais c'est toi qui vois.

    Cordialement, 
    Jordane                                                                 
    0
    1. Grafiteur
       
      Mais si je fais via écriture ternaire, le tableau conditions va se remplire même si ma valeur est null, je risque d'avoir une requete comme ça :

          $nomJeu= null;
          $pegi = 1;
          $Conditions=array();
          $Conditions[]= $nomJeu ? "NomJeu='$nomJeu'" : '';
          $Conditions[]= $pegi? "pegi='$pegi'" : '';
          $Sql = 'SELECT * FROM JEU ';
          
          if(count($Conditions)>0)
              $Sql.='WHERE '.implode(' AND ',$Conditions);
          return $Sql;
      
      
          => SELECT * FROM JEU WHERE  AND pegi='1'
      
      

      Il va pas y avoir un probléme avec l'exécution de la requête ?
      0
      1. jordane45 Messages postés 30426 Date d'inscription   Statut Modérateur Dernière intervention   4 830 > Grafiteur
         
        Au temps pour moi... j'ai oublié d'indiqué qu'avant de faire le IMPLODE tu peux faire un array_filter pour supprimer les "vides"
        https://www.php.net/array_filter
        0
    2. Grafiteur
       
      merci beaucoup pour l'aide :D
      0