Comment utiliser SELECT * FROM ORDER BY pour plusieurs pages

J’ai 5 pages PHP qui utilise exactement le même code, mais seulement un MOT qui change, c’est le nom de la table $query = "SELECT * FROM MOT ORDER BY id"; Comment faire pour utiliser un seul code avec les 5 tables pour les utiliser sur différentes pages ?

Salut !

Tu peux imaginer un code générique inclus dans chacune des pages et auquel tu passes le nom de ladite table.
Je souhaite rappeler pour ceux qui n’y penseraient plus : il n’est pas possible de passer un nom de table (ni quoi que ce soit qui représente un élément de structure de base de données) en paramètre dans une requête préparée.

Un petit "Bonjour !" au début de ton message n’aurait pas été dédaigné 

Bien le bonjour à toi et merci pour ta réponse rapide ! Désolé pour cette impolitesse, ce n’est pas de mon habitude de ne pas saluer les gens. Donc imaginer un code générique ? Une fonction peut-être dans le début de chaque codes, pour la réutiliser dans le même code ? Je vais essayer ça je te redis.

Avec require, tu n’aurais même pas besoin de copier le code de la fonction, juste l’instruction qui inclurait le fichier où se trouverait ladite fonction.

D’accord mais, je comprends pas trop ce que je devrais faire en faite. Je dois écrire un code au début de chaque page de code ? Faire une page de code pour les 5 pages différentes ?

Ce que je propose, c’est :

1. un fichier qui contient la fonction avec le code générique ;
2. inclure ce fichier partout où tu as besoin de cette fonction, avec l’aide de l’instruction require par exemple.

D’accord, je comprends maintenant, mais du coup dans la fonction comment écrire mon code avec toutes les tables dans une même fonction ? Je rappel que mon but c’est d’éviter d’écrire 5 fois la même chose avec juste la table qui changerai à chaque fois.

Tu pourrais passer la table en paramètre de la fonction, non ?

Après, pour savoir quoi passer en paramètre, étant donné que tu vas bien devoir appeler cette fonction (il manque ce point à ma liste précédente, s’il faut faire ça de manière très académique), l’endroit où tu l’appelles implique que tu sais quelle table utiliser.

Ce qui donnerai ça pour la fonction ? Et je devrais ensuite l’appeler dans les différentes pages avec à la place de $name le nom de la table ?

function table($name){

include('database_connection.php');

$query = "SELECT * FROM $name ORDER BY id";
$statement = $connect->prepare($query);
if($statement->execute())
{
  while($row = $statement->fetch(PDO::FETCH_ASSOC))
  {
    $data[] = $row;
  }

  echo json_encode($data);
}
}

Oui, c’est l’idée. J’imagine volontiers ce script dans un fichier nommé un peu comme select_all.php à inclure avec require dans ta page.

Vu que tu inclus database_connection.php dans ta fonction, je te conseille d’y remplacer include par require_once, pour une question de confort :

• si le fichier, pour une raison quelconque, n’est pas trouvé, le script s’arrêtera là plutôt que de tenter de continuer alors que la connexion à la base n’a pas été effectuée ;
• si tu as déjà inclus database_connection.php dans le script qui inclus select_all.php, cela t’évite des soucis parce que database_connection.php serait exécuté deux fois.

A ce dernier propos, je conseillerais plutôt de ne pas inclure la connexion à la base de donnée dans la fonction, mais au même endroit que là où tu inclurais select_all.php.

J’ai un fichier function.php pour toutes mes fonctions, que j’inclus évidemment dans mes pages. j’ai changé include par require_once, et je l’ai mis en dehors de la fonction, mais cette fois la page ne trouve pas les tables… J’ai oublié de préciser, que j’utilise la fonction dans du javascript, c’est peut-être là le soucis ?

MON CODE

$scope.fetchData = function(){
        $http.get(table("user")).success(function(data){
            $scope.namesData = data;
        });
    };

Mmm, attention, ne mélange pas JavaScript et PHP. Le PHP va générer ce que le serveur envoie, et le client va interpréter ce que le serveur a envoyé.
Le client ne devrait jamais voir de PHP, et le serveur de JavaScript — du moins avec cet environnement.

Qu’avais-tu à la place de table("user") à la ligne 2 du code ?

Ah… J’avais 'fetch_data.php' à la place de table("user") et à l’intérieur de fetch_data.php j’avais ce code

<?php
//fetch_data.php

include('database_connection.php');

$query = "SELECT * FROM user ORDER BY id";
$statement = $connect->prepare($query);
if($statement->execute())
{
    while($row = $statement->fetch(PDO::FETCH_ASSOC))
    {
        $data[] = $row;
    }

    echo json_encode($data);
}

?>

Ta page est censée pouvoir afficher des données de différentes tables, ou c’est une page par table ?

En fait, j’ai 5 pages qui commence chacune par fetch_data….php (ex:fetchdata_user.php, fetch_data_room.php etc…) Et dans chacune de ces pages j’ai le nom d’une table qui correspond à la page, donc pour fetch_datauser.php j’ai "SELECT FROM user ORDER BY id" pour room "SELECT FROM room ORDER BY id" etc… Et ensuite j’utilise cette page fetchdata….php dans mes différentes pages user.php, room.php etc… Du coup, oui c’est une page par table.

D’accord, je vais mettre ça, mais pour bien comprendre le but, c’est le fait de mettre switch case qui sécurise mon code ?

Ce qui sécurise ta requête SQL c’est de savoir exactement quelle valeur va prendre $name via une condition (switch case).

Du coup j’ai fait tout ce que tu m’a dit mais mon problème continue, ça m’affiche toujours rien…

Je crois mon soucis est du coté JS PHP, je sais pas comment faire…

Parfait, merci pour l’idée, dernière question, avec le $_GET, je ne risque rien niveau injection ? Il faudra faire quelque chose pour sécurisé le code ? C’est mon plus gros Default, et ma plus grande peur