PDO, MySQL: requête paramétrée ne fonctionne pas

Version imprimable

Bonjour,

J'ai la fonction suivante qui fonctionne avec des variables PHP dans la requête mais que je n'arrive pas à paramétrer. J'obtiens une erreur parameter number.
Dans le code ci-dessous, la première variante fonctionne bien mais pas les deux autres.
Pour éviter une question qui m'a déjà été posée, je précise que j'ai déjà couramment réalisé des requêtes en utilisant plusieurs fois un même paramètre et que cela fonctionne très bien.

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
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
function getSpouses($gender, $selection=null, $minGap=null, $maxGap=null, $spouse1=null)
{
	global $db;
 
	$dateFormat = LOCAL_DATE_FORMAT;
	$where = "id_abo=:id_abo AND gender=:gender";
	if (!$selection and isset($minGap, $maxGap, $spouse1))
	{
		$where .= "
			AND (
				(birth_date BETWEEN('$spouse1->birth_date' - INTERVAL $maxGap YEAR) AND ('$spouse1->birth_date' - INTERVAL $minGap YEAR))
			OR
				(birth_date BETWEEN('$spouse1->birth_date' + INTERVAL $minGap YEAR) AND ('$spouse1->birth_date' + INTERVAL $maxGap YEAR))
				)
		";
		/*
		$where .= "
			AND (
				(birth_date BETWEEN(:refBirthdate - INTERVAL $maxGap YEAR) AND (:refBirthdate - INTERVAL $minGap YEAR))
			OR
				(birth_date BETWEEN(:refBirthdate + INTERVAL $minGap YEAR) AND (:refBirthdate + INTERVAL $maxGap YEAR))
				)
		";
		*/
		/*
		$where .= "
			AND (
				(birth_date BETWEEN('$spouse1->birth_date' - INTERVAL :maxGap YEAR) AND ('$spouse1->birth_date' - INTERVAL :minGap YEAR))
			OR
				(birth_date BETWEEN('$spouse1->birth_date' + INTERVAL :minGap YEAR) AND ('$spouse1->birth_date' + INTERVAL :maxGap YEAR))
				)
		";
		*/
	}
	if (empty($selection) and empty($minGap) and empty($maxGap))
	{
		$where.= " AND birth_date IS NULL";
	}
	$query = "
		SELECT
			id, last_name, first_name, middle_name, CONCAT(COALESCE(last_name, ''), ' ', COALESCE(first_name, ''), ', ', COALESCE(middle_name, '')) AS full_name,
			DATE_FORMAT(birth_date, '$dateFormat') AS birth_date, ' - ',
			DATE_FORMAT(death_date, '$dateFormat') AS death_date
		FROM dat_persons
		WHERE $where
	;";
	$result = $db->prepare($query);
	$result->bindParam('id_abo', $_SESSION['user']['id_abo']);
	$result->bindParam('gender', $gender);
	//$result->bindParam('refBirthdate', $spouse1->birth_date);
	//$result->bindParam('minGap', $minGap);
	//$result->bindParam('maxGap', $maxGap);
	$result->execute();
	return $result->fetchAll();
}

24/05/2018, 14h44
Celira

Citation:

Envoyé par moimp

j'ai déjà couramment réalisé des requêtes en utilisant plusieurs fois un même paramètre et que cela fonctionne très bien.

Ou pas, vu ce que dit la Doc PHP :

Citation:

Vous devez inclure un marqueur avec un nom unique pour chaque valeur que vous souhaitez passer dans la requête lorsque vous appelez PDOStatement::execute(). Vous ne pouvez pas utiliser un marqueur avec deux noms identiques dans une requête préparée, à moins que le mode émulation ne soit actif.

Donc, il est normal que ça ne marche pas, sauf si tu configures ta connexion avec $pdo->setAttribute(PDO::ATTR_EMULATE_PREPARES, 1);.

Au passage, il me semblait qu'il faut mettre les deux-points dans l'appel à bindParam $result->bindParam(':gender', $gender);, mais je ne retrouve pas la référence dans la doc.
24/05/2018, 17h39
moimp

Citation:

Envoyé par Celira

Ou pas, vu ce que dit la Doc PHP :

Il s'agit bien de la même variable avec la même valeur. Lorsque je lis la doc, je comprends qu'il s'agit de bindValue et non pas de bindParam. Lorsque j'ajoute le paramètre PDO::ATTR_EMULATE_PREPARES, l'erreur persiste. Si je mets un nom différent, cela ne marche pas non plus.

Citation:

Envoyé par Celira

Au passage, il me semblait qu'il faut mettre les deux-points dans l'appel à bindParam $result->bindParam(':gender', $gender);, mais je ne retrouve pas la référence dans la doc.

Parce qu'on peut très bien faire les deux: Je l'ai lu quelque part et je le fais toujours, même si comme on me l'a déjà dit ce n'est pas une très bonne pratique.

1- Affiche $query.

2- ça vous arrive de lire la DOC (et les exemples qui vont avec) ?

bindValue : "Associe une valeur à un paramètre"
bindParam : "Lie un paramètre à un nom de variable spécifique"

3- en l'occurrence, c'est bindValue .
Code:

1 2 3 4 5 6 $sth = $dbh->prepare('SELECT nom, couleur, calories FROM fruit WHERE calories < :calories AND couleur = :couleur'); $sth->bindValue(':calories', $calories, PDO::PARAM_INT); $sth->bindValue(':couleur', $couleur, PDO::PARAM_STR); $sth->execute();

25/05/2018, 10h08
Celira
Citation:
Envoyé par jreaux62

ça vous arrive de lire la DOC (et les exemples qui vont avec) ?

bindValue : "Associe une valeur à un paramètre"
bindParam : "Lie un paramètre à un nom de variable spécifique"

3- en l'occurrence, c'est bindValue .
Pour moi, la principale différence entre bindValue et bindParam, c'est que bindValue lie au paramètre la valeur de la variable à l'instant t alors que bindParam lie une référence et évalue la valeur au moment de l'exécution. Autrement dit, si la valeur de la variable liée change entre l'appel à la fonction bind et l'exécution, bindValue utilise l'ancienne valeur et bindParam utilise la nouvelle valeur.
Du coup, si on enchaine les bind et l'exécution, je ne vois pas bien la différence entre utiliser l'une ou l'autre fonction. :koi:

@Celira

tu as parfaitement résumé la situation.
bindValue et bindParam, c'est kif kif... dans le cas d'UNE SEULE exécution.

Par contre, si on a besoin de PLUSIEURS exécutions d'une même requête préparée, c'est là qu'on peut voir la différence de fonctionnement.

1- bindValue

Code:

1
2
3
4
5
6
7
8
9
10
$sql = "INSERT INTO t_fruits (nom_fruit, couleur_fruit) VALUES ( :nom, :couleur )";
$sth = $dbh->prepare( $sql );
 
$fruits = array( ['pomme','rouge'], ['banane','jaune'], ['orange','orange'] );
foreach( $fruits as $fruit )
{
   $sth->bindValue( ':nom', $fruit[0], PDO::PARAM_STR); // on lie la valeur de la variable
   $sth->bindValue( ':couleur', $fruit[1], PDO::PARAM_STR); // on lie la valeur de la variable
   $sth->execute();
}

2- bindParam

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
$sql = "INSERT INTO t_fruits (nom_fruit, couleur_fruit) VALUES ( :nom, :couleur )";
$sth = $dbh->prepare( $sql );
 
$fruit_nom = '';
$fruit_couleur = '';
$sth->bindParam( ':nom', $fruit_nom, PDO::PARAM_STR); 
$sth->bindParam( ':couleur', $fruit_couleur, PDO::PARAM_STR);
 
$fruits = array( ['pomme','rouge'], ['banane','jaune'], ['orange','orange'] );
foreach( $fruits as $fruit )
{
   $fruit_nom = $fruit[0];
   $fruit_couleur = $fruit[1];
   $sth->execute();
}

(mais je peux me tromper...) à tester donc.

N.B. Perso, j'utilise toujours bindValue ! :aie:

Salut jreaux62

pour bindParam, tu dois déclarer tes références aux variables avant de les utiliser :

Code:

1
2
3
$sth = $dbh->prepare( $sql );
$sth->bindParam( ':nom', $fruit_nom, PDO::PARAM_STR); // on lie le nom de la variable (la valeur de $fruit_nom n'est pas encore définie)
$sth->bindParam( ':couleur', $fruit_couleur, PDO::PARAM_STR); // on lie le nom de la variable (la valeur de $fruit_couleur n'est pas encore définie)

doit être :

Code:

1
2
3
4
5
$fruit_nom = '';
$fruit_couleur = '';
$sth = $dbh->prepare( $sql );
$sth->bindParam( ':nom', $fruit_nom, PDO::PARAM_STR); 
$sth->bindParam( ':couleur', $fruit_couleur, PDO::PARAM_STR);

Tout simplement parce que bindParam travaille en interne par référence équivalent à &$fruit_nom

25/05/2018, 14h41
Celira

@rawsrc : ah, bon à savoir ça.
@jreaux : moi j'utilise execute avec un tableau de paramètres :aie:

Bonjour moimp !
Bonjour l'équipe de modérateurs ! :D
Je vois que la polygamie ou la polyandrie attire les foules ! :mouarf:

Bon, blague à part, et au risque de me faire engueuler (spa grave, j'ai un peu de cuir), je mets mon avis :
1) je ne trouve pas que c'est une bonne pratique d'utiliser direct une variable de session dans une fonction. Sauf si tu stipules une variable globale, comme ton $db, je trouve que tu devrais passer $user en paramètres sans préjuger de ce que sera $user au moment de l'appel de la fonction.
2) $spouse1 est un objet de la classe Spouse ? Autant le dire dès la fonction.
3) J'ai supposé, peut être à tort, que le genre du paramètre, était le genre du user... modifie si j'ai mal compris.
4) Enfin (et c'est surtout là que je vais énerver mes amis haut placés de la modération, j'aime po les marqueurs nommés. Et là, je trouve que les marqueurs anonymes allègent le code... Bon à tester hein, c'est peut être bourré de petits bugs...
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 41 42 43 <?php function getSpouses(array $user, $selection=null, $minGap=null, $maxGap=null,Spouse $spouse1=null){ global $db; $dateFormat = LOCAL_DATE_FORMAT; $query = "SELECT id, last_name, first_name, middle_name, CONCAT(COALESCE(last_name, ''), ' ', COALESCE(first_name, ''), ', ', COALESCE(middle_name, '')) AS full_name, DATE_FORMAT(birth_date, ?) AS birth_date, ' - ', DATE_FORMAT(death_date, ?) AS death_date FROM dat_persons"; $where = " WHERE id_abo=? AND gender=? "; if (!$selection && ! empty($minGap) && ! empty($maxGap) && ! empty($spouse1) && ! empty($user['id_abo']) && ! empty($user['gender']) ) $minGap=(int) $minGap; $maxGap=(int) $maxGap; $where .= "AND ( (birth_date BETWEEN(? - INTERVAL ? YEAR) AND (? - INTERVAL ? YEAR)) OR (birth_date BETWEEN(? + INTERVAL ? YEAR) AND (? + INTERVAL ? YEAR)))';"; $result = $db->prepare($query.$where); $result->execute(array($user['id_abo'],$user['gender'],$spouse1->birth_date,$maxGap,$spouse1->birth_date,$minGap,$spouse1->birth_date,$minGap,$spouse1->birth_date,$maxGap)); } elseif (! empty($user['id_abo']) && ! empty($user['gender']) && empty($selection) && empty($minGap) && empty($maxGap)){ $where .= "AND birth_date IS NULL;"; $result = $db->prepare($query.$where); $result->execute(array($user['id_abo'],$user['gender'])); } else{ return false; } return $result->fetchAll(); } //version courte //$spouses=getSpouses($_SESSION['user']); //version longue $spouses=getSpouses($_SESSION['user'], true, 1, 3, $spouse1); if($spouses!==false){ echo '<pre>'; print_r($spouses); echo '</pre>'; } else{ echo 'Erreur dans les paramètres passés !<br/>'; }?>
edit : je ne comprends pas pourquoi tu passes sélection en paramètres s'il doit être toujours à false ?

25/05/2018, 17h17
Invité
1-

Citation:

Envoyé par Celira

...moi j'utilise execute avec un tableau de paramètres :aie:

Oui, chacun son choix.
Le tout est de le faire en connaissance de cause.
- bindvalue / bindParam permettent de définir le type de données requises (PDO::PARAM_STR, PDO::PARAM_INT,...).
- Ce que ne permet pas execute(array(...))
2- De même pour le choix des placeholders anonymes ou nommés : c'est un choix personnel.
- $where = " WHERE id_abo=? AND gender=? "; et $where = " WHERE gender=? AND id_abo=? "; auront des conséquences sur l'ORDRE des paramètres à fournir.
- $where = " WHERE id_abo=:id_abo AND gender=:gender "; et $where = " WHERE gender=:gender AND id_abo=id_abo "; non.
25/05/2018, 17h50
Willy_k

@rawSrc pas forcément. On peut se rendre compte avec un var_dump($fruit_nom); après le bindParam , PHP aura créé la variable (si elle n'existe pas) avec comme valeur NULL.

Cette discussion me laisse perplexe dans le sens où j'ai mieux compris la doc et les tutos que j'ai pus lire mais où je n'ai toujours pas compris pourquoi ma requête ne fonctionne pas, sachant qu'elle ne fonctionne

ni avec bindValue ni avec bindParam,
ni en gardant un seul paramètre ou une seule valeur (version 1) ni en en mettant plusieurs (version 2), sachant que la requête n'est exécutée qu'une fois, que la valeur ne change pas et que dans ce cas précis je l'ai souvent fait et que ça ne me paraît pas contraire au passage de la doc cité par Celira en #2:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 // Version 1 $where .= " AND ( (birth_date BETWEEN(:refBirthdate - INTERVAL $maxGap YEAR) AND (:refBirthdate - INTERVAL $minGap YEAR)) OR (birth_date BETWEEN(:refBirthdate + INTERVAL $minGap YEAR) AND (:refBirthdate + INTERVAL $maxGap YEAR)) ) "; // Version 2 $where .= " AND ( (birth_date BETWEEN(:refBirthdate1 - INTERVAL $maxGap YEAR) AND (:refBirthdate2 - INTERVAL $minGap YEAR)) OR (birth_date BETWEEN(:refBirthdate3 + INTERVAL $minGap YEAR) AND (:refBirthdate4 + INTERVAL $maxGap YEAR)) ) ";
[*]

Je ne pose cette question que pour comprendre, car j'ai complètement réécrit mon code.

28/05/2018, 10h52
Invité
Bonjour,
1. As-tu testé la requête dans phpMyAdmin avec des valeurs tests ?
2. As-tu mis un try catch pour afficher les erreurs SQL ?
3. As-tu affiché $query pour vérifier sa structure ? (ça donne quoi ?)
4. As-tu vérifié les variables ? ($maxGap, $minGap,.....)
5. ... ?
Et montre-nous au moins la requête complète (et le code PHP associé) !

Bref : TESTE, DEBUGUE et donne-nous assez d'infos pour te répondre.

N.B. ça doit fonctionner en indiquant UN SEUL nom de paramètre :refBirthdate (ta version1)
Evidemment, on doit mettre UNE SEULE FOIS :

Code:

$pdo_select->bindValue( ':refBirthdate', $refBirthdate, PDO::PARAM_STR);
28/05/2018, 14h07
moimp

Comme, j'ai réécrit ce code (et que je ne l'ai pas conservé), je ne peux plus tout vérifier mais je suis sûr de répondre par l'affirmative pour tes questions 1, 3 et 4. Le code complet est en #1. Les arguments 2 à 5 sont optionnels et varient à chaque appel de la fonction.
Faute de pouvoir mieux vous aider à m'aider, je laisse tomber, marque le sujet résolu sans oublier de vous remercier tous.