Problème de requêtes [TITRE MODERE CAR PEU EXPLICITE]

Question

j'ai concu un site pour une agence immobiliere,
en cherchant un bien sur ma base de donnee en multi-critere. je fait ce qui suit :
j'ai un formulaire de plusieur champ, ce sont les critere de recherche :

type de bien : (appartement ou villa ou ....)
lieu : (paris ou  lion ou ....)
prix minimal : 1 ou 2 ...
prix maximal : 9999999 ou 9999999999
suprefacie : 100m ou 200m ou ....

le problem que j'ai et quant un champ est vide.

alors je cherche en testant tout les champ :
si le champ type est remplee et lieu et prix min et prix max et superfacie son vide
alors je accede a la requete qui chere que le champ type 

et 

si le champ lieu est remplee et type et prix min et prix max et superfacie son vide
alors je accede a la requete qui chere que le champ lieu

je fait ces test et ca fait des enormes ligne de code.

je veux savoir svp es que cette methode est exacte ou il y a une methode oplimal qui peut m'eviter toutes ces lignes de code.
merci .

malalam · Answer

Hello,

le mieux aurait &#233;t&#233; de faire une fonction.
Tu mets tes champs &#224; value="null" par d&#233;faut.
A la soumission, tu appelles la fonction qui cr&#233;e la requ&#234;te, en lui passant en param&#232;tre tes diff&#233;rents $_POST.

Dans ta fonction, tu construits tes clauses WHERE (et &#233;ventuellement FROM) en fonction de la valeur des post.
Exemple &#224; prendre avec des pincettes parce que c'est la copie de la m&#233;thode d'une des classes de mon projet actuel...C'est donc juste pour l'exemple :
 function getList($comtId null, $entId null, $contId = null, $artId = null)
        {
        $whereClauseArr = array ();
        if ($comtId !== null) {
            $whereClauseArr[] = ' AND (sk.comt_id = '.$comtId.')';
        }
        if ($entId !== null) {
                    $whereClauseArr[] = ' AND (sk.ent_id = '.$entId.')';
        }
        if ($contId !== null) {
                    $whereClauseArr[] = ' AND (sk.cont_id = '.$contId.')';
        }
        if ($artId !== null) {
                    $whereClauseArr[] = ' AND (sk.art_id = '.$artId.')';
        }
                $sQuery = 'SELECT
                            sk.sk_id, rg.rg_libelle, zone.zone_libelle, etg.etg_libelle
                        FROM
                            stocks sk, sk_rangees rg, sk_zones zone, sk_etages etg
                        WHERE
                            (sk.rg_id = rg.rg_id) AND
                            (sk.zone_id = zone.zone_id) AND
                            (sk.etg_id = etg.etg_id)
                        ';

        foreach ($whereClauseArr as $qry) {
            $sQuery .= $qry;
        }
        $sQuery .= ' ORDER BY
                             sk.rg_id, sk.zone_id, sk.etg_id';
                $aTmp = array();
                $this->oDB->query($sQuery);

                while($a = $this->oDB->fetch_assoc() ) {
                        $aTmp[$a['sk_id']] = $a['rg_libelle'].$a['zone_libelle'].$a['etg_libelle'];
                }
                return $aTmp;
        }

cs_skiso · Answer

Rechercher un bien par critère, ---- Type:, , ---- Ville:, , ---- Nbr de pièces: , , ---- Surface (m²) : , ----, mini maxi

Prix :	mini , maxi

svp c tu veut me rendre service, par ce que je suis debutant en php et en sql peut tu traduire ce que tu a ecrit au par avant (en haut) sur ce formulaire. le nom des champ de haut en bas : - type - ville - nbr pieces - surfmin - surrmax - prixmin - prixmax merci pour ta reponce

FhX · Answer

Nan mais tu veux pas qu'on te le chie non plus ?

cs_EagleEye · Answer

Lol, oui c'est un peu abusé je trouve, tu as juste a remplacer ohh :s

FhX · Answer

Juste pour toi Malalam au passage :)
"if ($comtId !null) {" Que c'est bien mocheuh :) En plus, tu ne testes m&#234;me pas le type de ta variable si c'est diff&#233;rent de NULL ! Bon, j'en d&#233;duis que c'est tout sauf un objet, une ressource, ou alors un type NULL... donc de ce fait, pour tester une bonne fois pour toute> is_scalar() !

Donc :  if ( is_scalar($comtId) ) { ... }
etc... Au moins comme ca, tu es assur&#233; au pire :)

Pareil, pourquoi avoir fait ca sous forme de tableau pour tes requ&#232;tes avec AND ?

$query_ext = '';
 if ( is_scalar($comtId) ) {      $query_ext .'AND (sk.comt_id '.$comtId.')';
 }
// etc...
// et &#224; la fin :
if ( !empty($query_ext) ) $sQuery .= $query_ext;

Voila, c'est qu'un conseil que je te donne, mais si c'est plus rapide ta facon, pourquoi pas :)

malalam · Answer

FhX => disons qu'on est 2 &#224; bosser l&#224;-dessus, et que je dois m'adapter... ;-)
le getList est appel&#233; explicitement, &#224; partir de valeurs connues; le but est simplement de dire : si on passe NULL, on ne fait rien. Sinon, c'est qu'on a pass&#233; un entier (c'est epxlicite, comme je te l'ai dit, on sait ce qu'on aura toujours : un entier).
Maintenant, oui, is_scalar () est plus joli :-)
Quant au tableau, je ne d&#233;fends pas : mes portions de code &#233;taient avec une b&#234;te concat&#233;nation, en effet. Mais comme mon coll&#232;gue a plus de mal &#224; s'adapter &#224; mon code, que moi au sien, j'ai laiss&#233; tomber et j'ai calqu&#233; son mode de fonctionnement.
Bref, non, je ne pense pas que ce soit plus rapide, bien au contraire.

malalam · Answer

J'ai remplac&#233; is_scalar (), en tous cas dans mes portions de code...lol.
Je nepense pas qu'il ait des probl&#232;mes pour lire &#231;a...quand m&#234;me ;-)

FhX · Answer

oh quand m&#234;me !! Sinon tu lui dis &#224; quoi ca sert en commentaire :)

Par contre dis moi, si tu ne passes aucun argument dans ta m&#233;thode, quand tu fais un foreach, t'as pas une erreur du fait que ton tableau peut &#234;tre vide ?

FhX · Answer

Enfin moi je te dis ca comme ca, &#233;tant donn&#233;e que c'est une classe, et qu'une classe peut se faire h&#233;riter tr&#232;s vite, autant la faire la plus g&#233;n&#233;rique possible, et surtout la plus strict au niveau des valeurs qui viennent et qui repartent des m&#233;thodes :)

cs_skiso · Answer

en tous les cas ca donne un erreur $ttype = null; $tpices = null; $tville = null; $tsurfmin = null; $tsurfmax = null; $tprixmin = null; $tprixmax = null; if ($HTTP_POST_VARS['typeID'] !== null) { $ttype '(type '.$HTTP_POST_VARS['typeID'].')'; } if ($HTTP_POST_VARS['picesID'] !== null) { $tpices ' AND (pices '.$HTTP_POST_VARS['picesID'].')'; } if ($HTTP_POST_VARS['villeID'] !== null) { $tville ' AND (ville '.$HTTP_POST_VARS['villeID'].')'; } if ($HTTP_POST_VARS['surfmin'] !== null) { $tsurfmin = ' AND (surface >= '.$HTTP_POST_VARS['surfmin'].')'; } if ($HTTP_POST_VARS['surfmax'] !== null) { $tsurfmax = ' AND (surface <= '.$HTTP_POST_VARS['surfmax'].')'; } if ($HTTP_POST_VARS['prixmin'] !== null) { $tprixmin = ' AND (prix >= '.$HTTP_POST_VARS['prixmin'].')'; } if ($HTTP_POST_VARS['prixmax'] !== null) { $tprixmax = ' AND (prix <= '.$HTTP_POST_VARS['prixmax'].')'; } $query_Reff = sprintf("SELECT * FROM bien WHERE $ttype.$tpices.$tville.$tsurfmin.$tsurfmax.$tprixmin.$tprixmax");

malalam · Answer

Elle h&#233;rite d&#233;j&#224;, en fait, d'une classe g&#233;n&#233;rique.
Mais bon, j'ai pas dit que j'appr&#233;ciais cette fa&#231;on de coder lol...seulement moi je suis ing&#233;nieur de d&#233;v, responsable de d&#233;v, certes, mais lui est le responsable informatique et code parce qu'il adore &#231;a. Donc bon...

Ca donne quoi comme erreur ?

cs_skiso · Answer

Erreur de syntaxe pr&#232;s de '. AND (pices = ). AND (ville = ). AND (surface >= ). AND (surfac' &#224; la ligne 1

je pence que le problem est due a la facon dant je concatine.

malalam · Answer

Tes '.' sont DANS ta requ&#234;te, l&#224;.

Problème de requêtes [TITRE MODERE CAR PEU EXPLICITE]

13 réponses

Votre réponse

Discussions similaires