Fonction avec variable dans la requête

Bonjour à tous,

J'essaie désespérément de faire une fonction contenant une requête SQL avec des variables à l'intérieur afin de savoir si une ligne existe dans ma BDD dans un premier temps et également connaitre la valeur du booléen.

Cependant, je ne sais pas pourquoi mais elle ne fonctionne pas.

Est-ce qu'une âme charitable pourrait m'aider, s'il vous plait?
Voici mon code :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

function alert_exist($idses, $alert_type) { $req_alert = $bdd->query(' SELECT "$alert_type" FROM gdpm_setup WHERE id_user = "$idses" AND "$alert_type" = TRUE'); $alert = $req_alert->fetch(); $alert = $alert[$alert_type]; return $alert; }   $alert = alert_exist(1, mail); echo $alert;

---

Je vous remercie!

Salut,

Modifies ta requête comme ça :

Code:

---

1 2 3 4

$req_alert = $bdd->query("SELECT '$alert_type' FROM gdpm_setup WHERE id_user = '$idses' AND '$alert_type' = TRUE");

---

Mais avant, il faut t'assurer que la requête retourne bien une ligne, pour ça tu peux utiliser num_rows.
En gros ta fonction doit être comme suite :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14

function alert_exist($idses, $alert_type) { $req_alert = $bdd->query("SELECT '$alert_type' FROM gdpm_setup WHERE id_user = '$idses' AND '$alert_type' = TRUE"); $nbrLignes=$req_alert->num_rows; if($nbrLignes>0){ $alert = $req_alert->fetch(); $alert = $alert[$alert_type]; } else $alert="Aucune ligne trouvée !"; return $alert; }

---

et quand tu veux appeler la fonction, il faut mettre le champ mail entre double quotes "" :

Code:

---

$alert = alert_exist(1, "mail");

---

Bonjour Toufik82,

Merci beaucoup pour ta réponse.

J'ai bien effectué les modifications, cependant, j'ai le même problème, lorsque j'appelle la fonction
avec la ligne :

Code:

---

$alert = alert_exist(1, "mail");

---

La page ne s'affiche plus.

as-tu copier/coller le corps de la fonction comme je l'ai mis dans mon post plus haut ?

Oui, la page ne s'affiche pas

Si la page ne s'affiche pas et qu'il n'y a aucune erreur affichée, cela veut dire qu'il y'a un truc anormal.

commençons par voir les exceptions (s'il y'en a), englobe le corps de la fonction dans un bloc try catch :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18

function alert_exist($idses, $alert_type) { try{ $req_alert = $bdd->query("SELECT '$alert_type' FROM gdpm_setup WHERE id_user = '$idses' AND '$alert_type' = TRUE"); $nbrLignes=$req_alert->num_rows; if($nbrLignes>0){ $alert = $req_alert->fetch(); $alert = $alert[$alert_type]; } else $alert="Aucune ligne trouvée !"; }catch(Exception $e){ $alert=$e->getMessage(); } return $alert; }

---

La page ne s'affiche toujours pas...

Après quelques recherche, je viens de trouver la fonction ->rowCount(); pour remplacer ->num_rows;.

Je fais quelques tests et reviens après.

Bon, ça ne fonctionne pas mieux pour autant...

Je ne suis pas un fan de mysqli mais je pense que le souci est dans la ligne :

Code:

---

1 2	$req_alert->fetch();

---

Essaies d'utiliser un fetch_assoc du style orienté objet dans une boucle while.

Bonjour,

:alerte: Attention aux guillemets utilisés !

• Pour les noms de colonnes, ce n'est pas ', mais `.
• Pour les valeurs, on met des ' pour des chaines. Pas pour des numériques (id_user)

Code:

---

1 2 3 4

$req_alert = $bdd->query("SELECT `$alert_type` FROM gdpm_setup WHERE id_user = $idses AND `$alert_type` = TRUE");

---

Cela dit, il faudrait au moins :

• vérifier que $alert_type est bien un nom de colonne valide
• sécuriser $idses avec mysqli_real_escape_string()

En suivant les exemples, voici où j'en suis arrivé :

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

$mysqli = new mysqli("localhost", "my_user", "my_password");   /* Vérification de la connexion */ if (mysqli_connect_errno()) { printf("Échec de la connexion : %s\n", $mysqli->connect_error); exit(); }   $alert_type = "mail"; $query = "SELECT '.$alert_type.' FROM gdpm_setup WHERE id_user = .$idses";   if($result = $mysqli->query($query)) {   /* Récupère un tableau associatif */ while ($row = $result->fetch_assoc()) { printf ("%s (%s)\n", $row[$alert_type]; }   /* Libération des résultats */ $result->free(); }   /* Fermeture de la connexion */ $mysqli->close();

---

Cependant, cela ne fonctionne pas.
J'ai enlevé la fonction afin d'essayer de cibler le problème.

Il y a également un truc de bizarre avec la requête PDO :

Code:

---

$alert = $alert[$alert_type];

---

Pour moi, la ligne ci-dessus devrait renvoyé un booléen mais elle renvoie "mail" : le nom de la colonne et non pas la valeur...

Bonjour jreaux62,

Merci pour ta réponse

Je suis passé en "prepare", normalement il ne devrait plus y avoir de problème de ce coté là :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18

$alert_type = "mail"; $req_alert = $bdd->prepare(" SELECT :alert_type FROM gdpm_setup WHERE id_user = :idses"); $req_alert->execute(array( 'alert_type' => $alert_type, 'idses' => $idses));   $nbrLignes=$req_alert->rowCount();   if($nbrLignes>0) { $alert = $req_alert->fetch(); $alert1 = $alert['alert_mail_post']; } else { $alert="Aucune ligne trouvée !"; }

---

Citation:

---

Envoyé par jreaux62

Cela dit, il faudrait au moins :

• vérifier que $alert_type est bien un nom de colonne valide

---

C'est tout bon pour ça

Citation:

---

Envoyé par jreaux62

• sécuriser $idses avec mysqli_real_escape_string()

---

Je ne connais pas du tout, je vais essayer de me pencher dessus.

Merci beaucoup!

Citation:

---

Envoyé par jreaux62

[*]sécuriser $idses avec mysqli_real_escape_string()

---

Jreaux62, ça marche avec PDO ou uniquement avec mysqli?

LOL... mysqli.

Mais si tu utilises PDO, c'est MIEUX !

Code:

---

1 2 3 4 5 6 7 8 9

$alert_type = "mail"; $sql_alert = "SELECT ".$alert_type." FROM gdpm_setup WHERE id_user = :idses";   $req_alert = $bdd->prepare($sql_alert); $req_alert->execute(array( 'idses' => $idses ));

---

:alerte: Remarque importante :

• un nom de la colonne en paramètre ($alert_type) NE PEUT PAS être "préparé" *.
• SEULES les valeurs peuvent l'être (voir PDO::prepare).

* C'est pourquoi j'ai préciser qu'il fallait BIEN VÉRIFIER que $alert_type est bien un nom de colonne valide (présent dans la table gdpm_setup).
On peut/doit utiliser PDO::quote (à confirmer... car non testé)
[EDIT] OUBLIE $bdd->quote() ici : il ne peut pas être utilisé sur un nom de colonne.

Après plusieurs essais, cela ne fonctionne toujours pas.

Il y a deux points qui me semblent bizarre:
- La requête "fonctionne" si elle n'est pas dans la fonction. Mais une fois dans la fonction, le script s'arrête au niveau de la désignation de la variable $sql_alert.
- Hors de la fonction, lorsque que j’exécute la requête suivante

Code:

---

1 2 3 4 5 6 7 8

$alert_type = "mail"; $sql_alert = "SELECT ".$bdd->quote($alert_type)." FROM gdpm_setup WHERE id_user = :idses"; $req_alert = $bdd->prepare($sql_alert); $req_alert->execute(array( 'idses' => $idses)); $donnees = $req_alert->fetch();

---

La variable $donnees['mail'] me retourne "mail" alors que je devrais avoir un booléen 1 ou 0...

Avez-vous d'autres pistes?

Merci pour votre aide.

Bonjour,

'mail' est une chaine :

Code:

---

$alert = alert_exist(1, 'mail');

---

Ensuite, dans la fonction :

Code:

---

1 2	$row = $req_alert->fetch(); $donnees = $row[$alert_type];

---

Merci jreaux62 pour ton temps passé,

La fonction bloque toujours au niveau suivant :

Code:

---

1 2 3

$sql_alert = "SELECT ".$bdd->quote($alert_type)." FROM gdpm_setup WHERE id_user = :idses";

---

Cordialement,

Gauthier

1- On ne peut pas tester/dégoguer à ta place.

2- OUBLIE $bdd->quote() (il ne peut pas être utilisé sur un nom de colonne)

3- Mets un try catch :

• PDO une soupe et au lit !

Citation:

---

Bon sang, mais c'est bien sûr !!...

---

Si tu es dans une fonction, il faut y définir la connexion !


Si $bdd est déjà définit en amont :

Code:

---

1 2 3 4

function alert_exist($idses, $alert_type) { global $bdd; ... }

---

Sinon :

Code:

---

1 2 3 4

function alert_exist($idses, $alert_type) { $bdd = .......; // ou include('connect.php') ou...-> il faut définir la variable de connexion ! ... }

---

Cool! Merci!
ça a déjà débloqué une bonne partie.

Il ne reste plus que le fait que la variable $donnees vaut 'mail' alors qu'elle devrait valoir 1 ou 0, non?

Merci beaucoup en tout cas pour ce déblocage