select count avec implode

[le nOoB]

Bonjour a tous,
je sèche sur une erreur dont je ne trouve pas l'origine.
si quelqu'un pourrait m'éclairer

erreur : Warning: PDO::query(): SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'departement=1 OR departement=2 OR departement=3 OR departement=4 OR departement='

code :

Code :

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
 
$departs = $_POST['depart'];
 
		$departs = implode(" OR departement=", $departs);
		$departs = "departement=".$departs;
 
		// Post de la requete
		if($departs == NULL)
			{
			$requete = $departs;
			$smarty->assign('requete',$requete);
			}
			else
				{
				 $requete = $departs;
				 $smarty->assign('requete',$requete);
				}
 
		// nombre d'offres
		$offremotclef = $connexion -> query("SELECT count(*) AS id_offre FROM offres WHERE valide=1 ".$requete." ")->fetchColumn();
		$smarty->assign('offremotclef',$offremotclef);

[vorace]

il te manque un opérateur entre "valide=1" et "departement=1" :

Code :

"SELECT count(*) AS id_offre FROM offres WHERE valide=1 ".$requete." "

tu devrais faire :

Code :

$departs = "and departement=".$departs;

[Celira]

Attention AND est prioritaire sur OR, donc il faut ajouter des parenthèses autour de ta série de OR, ou utiliser une condition IN.

Code :

1
2
$departs = implode(", ", $departs);
$departs = ' AND departement in ('.$departs.') ';

Dans ce genre de cas, une bonne idée est de décomposer l'exécution et de faire un affichage de la requête histoire de vérifier ce qu'il ya dedans et éventuellement de la faire tourner manuellement dans PhpMyAdmin (ou assimilé)

Code :

1
2
3
4
$query = "SELECT count(*) AS id_offre FROM offres WHERE valide=1 ".$requete." ";
echo 'Requete : '.$query;
$result = $connexion->query($query);
$offremotclef = $result->fetchColumn();

[CinePhil]

Envoyé par Celira

Dans ce genre de cas, une bonne idée est de décomposer l'exécution et de faire un affichage de la requête histoire de vérifier ce qu'il ya dedans et éventuellement de la faire tourner manuellement dans PhpMyAdmin (ou assimilé)

Je dirais même que c'est TOUJOURS ce qu'il faut faire dès qu'une requête contient un paramètre !
Bien entendu, on enlève le test quand on est sûr de la requête générée.