Discussion :

[benbax]

Bonjour a toutes et à tous.

Voici un formulaire qui me permet de rechercher dans ma base de données des maisons.

On peut (enfin, je voudrai qu'on puisse...) rechercher par des encadrement de valeur (Prix ou surface). J'ai donc fait un switch/Case.

Il ne marche pas...
Voici le code du switch :

Code :

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

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
 
switch($TSurfMin)
  	{
  		case $TSurfMin == "":
    		break;
  		case $TSurfMin != "" :
    		$sql = $sql . " AND surface >= ". $TSurfMin;
    		break;
  	}
	switch($TSurfMax)
  	{
  		case $TSurfMax == "":
    		break;
  		case $TSurfMax != "" :
    		$sql = $sql . " AND surface <= ". $TSurfMax;
    		break;
  	}
 
 
	switch($TPrixMin)
  	{
  		case $TPrixMin == "":
    		break;
  		case $TPrixMin != "" :
    		$sql = $sql . " AND prix >= ". $TPrixMin;
    		break;
  	}
	switch($TPrixMax)
  	{
  		case $TPrixMax == "":
    		break;
  		case $TPrixMax != "" :
    		$sql = $sql . " AND prix <= ". $TPrixMax;
    		break;
  	}

Et voici la requete envoyée...

Code :

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

1
2
 
SELECT * FROM ic_bienavendre WHERE type <> 'toutes' AND surface >= 100 AND surface <= AND prix >= AND prix <= ORDER BY prix LIMIT 0, 100

Quelqu'un arriverai à voir quel est le probleme?

[bluemartini]

essaie avec la fonction strcasecmp pour la comparaison de tes chaines. Par exemple :

Code :

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

1
2
 
case $TSurfMin == "":

replacé par :

Code :

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

1
2
 
case (strcasecmp($TSurfMin,"")==0):

[benbax]

Merci de ta réponse, je vais tenter ma chance, je vous tiens au courant...

[EDIT]
Malheureusement, le résultat est identique...
[/EDIT]

[ThomasR]

Code :

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

1
2
3
4
5
6
7
8
switch($TSurfMin)
  	{
  		case $TSurfMin == "":
    		break;
  		case $TSurfMin != "" :
    		$sql = $sql . " AND surface >= ". $TSurfMin;
    		break;
  	}

Ca n'a aucun intérêt.

Pourquoi ne fais-tu pas :

Code :

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

1
2
3
4
if($TSurfMin!="")
    {
    $sql = $sql . " AND surface >= ". $TSurfMin;
    }

Pas la peine de se compliquer la vie hein :p

[benbax]

J4avais essayé cette structure une époque, mais quelque chose déconnaéis, je ne me souviens plus quoi, et j'avais été obligé de passer par ce double test. Ces quelques lignes semblent sure, je les testent de suite...

[ThomasR]

Utilise le switch quand tu as une multitude de cas à gérer

[benbax]

Ca se présente un peu mieux, mais le résultat n'est pas celui escompté. J'ai des résultats hors de ma fourchette "imposée"...

Voici la requête crée par la fourchette:

Code :

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

SELECT * FROM ic_bienavendre WHERE type <> 'toutes' AND prix >= 192000 AND surface <= 194000 ORDER BY prix LIMIT 0, 100

Créée par ce code :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
if($TSurfMin!="")
    {
    	$sql = $sql . " AND surface >= ". $TSurfMin;
    }
	if($TSurfMax!="")
    {
    	$sql = $sql . " AND surface <= ". $TSurfMax;
    }
 
	if($TPrixMin!="")
    {
    	$sql = $sql . " AND prix >= ". $TPrixMin;
    }
	if($TPrixMax!="")
    {
    	$sql = $sql . " AND surface <= ". $TPrixMax;
    }