Fonction avec variable dans la requête

Version imprimable

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");

26/01/2020, 13h45
Avinetor

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.
26/01/2020, 13h47
Toufik83

as-tu copier/coller le corps de la fonction comme je l'ai mis dans mon post plus haut ?
26/01/2020, 14h31
Avinetor

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;	                    
}

26/01/2020, 16h16
Avinetor

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.
26/01/2020, 18h38
Avinetor

Bon, ça ne fonctionne pas mieux pour autant...
26/01/2020, 18h45
Toufik83
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.
26/01/2020, 19h01
Invité
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!

27/01/2020, 16h53
Avinetor

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.

28/01/2020, 13h30
Invité
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];
28/01/2020, 14h00
Avinetor
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
28/01/2020, 14h24
Invité
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 ! ... }

28/01/2020, 15h23
Avinetor

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
28/01/2020, 15h32
Invité

:cristal: Je ne peux pas deviner ton code...

Pour aider ta boule de cristal :D

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
function alert_exist($idses, $alert_type) {
     global $bdd;
 
     $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));
     $row = $req_alert->fetch();
     $alert = $row[$alert_type];
}    
 
$alert = alert_exist(1, 'mail');

$bdd est bien défini AVANT ?
Comment ?

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
function alert_exist($idses, $alert_type) {
     global $bdd;
 
     $sql_alert = "SELECT ".$alert_type." 
		FROM gdpm_setup 
		WHERE id_user = :idses";
     $req_alert = $bdd->prepare($sql_alert);
     $req_alert->execute(array(
		':idses' => $idses));
     $row = $req_alert->fetch();
     $alert = $row[$alert_type];
}    
 
$alert = alert_exist(1, 'mail');

Il n'y a toujours pas de try... catch(...).

Citation:

Envoyé par jreaux62

1- $bdd est bien défini AVANT ?
Comment ?

Oui oui, avec :
Code:

1 2 3 4 5 6 7 8 9 try { $bdd = new PDO('mysql:host=XXXXXXXX.mysql.db;dbname=YYYYYYYYYY;charset=utf8', 'ZZZZZZZZZZ', '00000000000', array(PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION)); } catch(Exception $e) { die('Erreur : '.$e->getMessage()); }
Il est là le try... catch :D
Citation:
Envoyé par jreaux62
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 function alert_exist($idses, $alert_type) { global $bdd; $sql_alert = "SELECT ".$alert_type." FROM gdpm_setup WHERE id_user = :idses"; $req_alert = $bdd->prepare($sql_alert); $req_alert->execute(array( ':idses' => $idses)); $row = $req_alert->fetch(); $alert = $row[$alert_type]; } $alert = alert_exist(1, 'mail');
Ca marche!!!!!!

Citation:

Envoyé par jreaux62

3- Il n'y a toujours pas de try... catch(...).

Du coup, toujours pas de try... catch! ou c'est mieux d'en mettre un pour être sur?

En tout cas, merci beaucoup pour ton aide!

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
function alert_exist($idses, $alert_type) {
     global $bdd;
 
     $sql_alert = "SELECT ".$alert_type." 
		FROM gdpm_setup 
		WHERE id_user = :idses";
try
{
     $req_alert = $bdd->prepare($sql_alert);
     $req_alert->execute(array(
		':idses' => $idses));
}
	catch(Exception $e)
{
	die('Erreur : '.$e->getMessage());
}
 
     $row = $req_alert->fetch();
     $alert = $row[$alert_type];
}    
 
$alert = alert_exist(1, 'mail');

Le try... catch(...) est surtout utile en phase de TEST (développement).
Ça permet de déboguer les erreurs de syntaxe SQL.

28/01/2020, 17h25
Avinetor

Merci beaucoup!
au plaisir!

Fuseau horaire GMT +2. Il est actuellement 05h59.