Requête BETWEEN et conditions if

**djedje37et28** · 09/08/2006, 09h22

bonjour à tous, je suis entrain de faire une requête sur le prix entre un mini et un maxi.
une personne de developpez.com m'avait expliqué comment mettre des conditions dans une requête différente que celle que je vais vous montrer.
j'ai voulu faire pareil mais ca ne fonctionne pas:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
$SQL ='SELECT count(bienid) as total FROM bien WHERE biensurfacehab between';
if(!empty($_GET['mini']))
$SQL .=	' "'.$choix.'" and';
if(!empty($_GET['maxi']))
$SQL .=	' "'.$choix2.'";';

ce que je veux faire c est

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$SQL ='SELECT count(bienid) as total FROM bien WHERE biensurfacehab between "'.$choix.'" and "'.$choix2.'" ';

voyez vous mon erreur car il me renvoie :

Erreur SQL ! select * from bien where biensurfacehab between Erreur de syntaxe près de '' à la ligne 1

merciiiiiiiiiii

**Joe Le Mort** · 09/08/2006, 09h26

y'a un probleme dans ta requete :
Supposons que mini et maxi soit vide, ta requete est :

SELECT count(bienid) as total FROM bien WHERE biensurfacehab between
le between pose probleme vu qu'il n'y a rien apres !

tu pourrais faire cela situ as plusieurs conditions (AND) :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
$SQL ='SELECT count(bienid) as total FROM bien WHERE 1=1';
if(!empty($_GET['mini'] && (!empty($_GET['maxi']))))
$SQL .=	'AND biensurfacehab between "'.$choix.'" and "'.$choix2.'" ';

EDIT : Si tu n'as qu'une condition

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
$SQL ='SELECT count(bienid) as total FROM bien ';
if(!empty($_GET['mini'] && (!empty($_GET['maxi']))))
$SQL .=	'WHERE biensurfacehab between "'.$choix.'" and "'.$choix2.'" ';

**Amara** · 09/08/2006, 09h26

Si tes champs ne sont pas des chaînes de caractères il ne faut pas les entourer de quotes. D'ailleurs au passage tu mets des guillemets mais en SQL je crois que c'est des quotes autour des chaînes, jamais essayé avec des guillemets.

Edit : +1 avec Joe même si dans ce cas il n'y a pas d'intérêt au 1=1 vu qu'il n'y a qu'une seule condition ajoutée, autant ajouter directement si besoin le WHERE

**Joe Le Mort** · 09/08/2006, 09h30

Envoyé par JWhite

Edit : +1 avec Joe même si dans ce cas il n'y a pas d'intérêt au 1=1 vu qu'il n'y a qu'une seule condition ajoutée, autant ajouter directement si besoin le WHERE

j'ai rajouté le code

**djedje37et28** · 09/08/2006, 09h39

merci beaucoup!! vous êtes super réactif ça fait plaisir
effectivement je n'ai qu'une condition. en fait j'ai deux requêtes dans cette page, une qui va compter le nombre de biens pour la pagination et une qui va les afficher.
j'ai donc fait :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
$SQL ='SELECT count(bienid) as total FROM bien';
if((!empty($_GET['mini'])) && (!empty($_GET['maxi'])))
$SQL .=	'where biensurfacehab between "'.$choix.'" and "'.$choix2.'" ';

et pour la deuxième :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
$query ='SELECT * from bien';
if((!empty($_GET['mini'])) && (!empty($_GET['maxi'])))
$query .=	'where biensurfacehab between "'.$choix.'" and "'.$choix2.'" LIMIT $Ndeb,$Nmax' or die ('Probleme lors de la lecture de la base de donnees');

mais il me sort :

Erreur de syntaxe près de 'between "12" and "56"' à la ligne 1

apparemment j'ai un ' de trop
mais dans le code je ne le vois pas!!!

**Joe Le Mort** · 09/08/2006, 09h42

met des quotes a la place des guillemets :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

'12' and '56'

ou sans si c'est des entiers!

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

12 and 56

**Amara** · 09/08/2006, 09h50

Oui +1 avec Joe tu devrais lire tous les messages en entier djedje j'ai déjà fait la remarque dans ma première réponse

**djedje37et28** · 09/08/2006, 09h51

normalement les personnes rentreront 56m², 12...
des valeurs entières.
je dois donc faire ca:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$query .=	'where biensurfacehab between ".$choix." and ".$choix2." LIMIT $Ndeb,$Nmax' or die ('Probleme lors de la lecture de la base de donnees');

????
je suis pas sur d'avoir bien compris!!!

**Amara** · 09/08/2006, 09h52

Tu te mélanges les pinceaux entre les quotes et les guillemets...

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$query .= 'where biensurfacehab between '.$choix.' and '.$choix2.' LIMIT $Ndeb,$Nmax';

Le or die te sers à rien à ce niveau, tu construis juste une chaîne, par contre il est préférable de le mettre quand tu fais le mysql_query...

**djedje37et28** · 09/08/2006, 10h04

j'ai donc fais comme ceci :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
$SQL ='SELECT count(bienid) as total FROM bien';
if((!empty($_GET['mini'])) && (!empty($_GET['maxi'])))
$SQL .=	'where biensurfacehab between '.$choix.' and '.$choix2.' ';

et

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
$query ='SELECT * from bien';
if((!empty($_GET['mini'])) && (!empty($_GET['maxi'])))
$query .=	'where biensurfacehab between '.$choix. ' and '.$choix2.' LIMIT $Ndeb,$Nmax';
$resultat = mysql_query ($query) or die('Erreur SQL ! '.$query.''.mysql_error());

ça me donne

Erreur de syntaxe près de 'between 45 and 57' à la ligne 1

peut être faut-il que je mette plutôt les valeur $_get??

**Joe Le Mort** · 09/08/2006, 10h07

ben en fait $maxi et $mini ne sont pas initialisées :

tu n'as pas fait précédemment :

$mini = $_GET['mini']; ????

fait le et tu verras le résultat !

**djedje37et28** · 09/08/2006, 10h10

si j ai fais

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
$choix=$_GET['mini'];
$choix2=$_GET['maxi'];

**gerald2545** · 09/08/2006, 10h11

si tu fais un echo de ta requête avant exécution, ça donne quoi?
tu la colles ici STP

il me semble qu'entre "bien" et "where" tu n'as pas laissé d'espace

**Yogui** · 09/08/2006, 10h20

Salut

Dans le même style que ce qui t'a été indiqué ici : Requête BETWEEN pour un formulaire (varchar => int)
Il faut que tu fasses un cast de ta valeur provenant du formulaire, sans quoi tu auras une jolie faille d'injection SQL.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
$choix = intval($_GET['mini']);
$choix2 = intval($_GET['maxi']);

@ JWhite : MySQL accepte les guillemets et les apostrophes comme délimiteurs de chaînes. Dans le cas des valeurs numériques, ces délimiteurs ne sont pas obligatoires.

Edit, oubli : http://securite.developpez.com/faq/?..._sql_injection

**djedje37et28** · 09/08/2006, 10h27

tout à fait c'est ça !!
bravo à tous et merci beaucoup

Requête BETWEEN et conditions if [MySQL]

PHP & Base de données

Discussions similaires

Partager

Partager