mySQL mag mich mal wieder nicht...

[GodfatherDeluxe]

Hallo Leute - wiedermal

Ich hab ein Formular (im Anhang zu sehen) mit dem ich eine Datenbank folgender Struktur durchsuchen möchte:

Tabelle Objekte:
id
preisklasse
region
typ
vermietung
schlafzimmer
badezimmer
zimmer
toiletten
garten
pool
flaeche
wohnflaeche
garage
referenznr
regdate

Tabelle description:
id
objekt_id
lang
name
con

und zwar mit folgendem code:

PHP-Code:

if (isset($_POST['search']))
{
    $nw = $_POST['nw'];
    $no = $_POST['no'];
    $sw = $_POST['sw'];
    $so = $_POST['so'];

    $villa = $_POST['villa'];
    $finca = $_POST['finca'];
    $apartement = $_POST['apartement'];
    $haus = $_POST['haus'];
    $grundstueck = $_POST['grundstueck'];
    $gewerbeobjekt = $_POST['gewerbeobjekt'];
    $mietobjekt = $_POST['mietobjekt'];

    $preiskl_1 = $_POST['preiskl_1'];
    $preiskl_2 = $_POST['preiskl_2'];
    $preiskl_3 = $_POST['preiskl_3'];
    $preiskl_4 = $_POST['preiskl_4'];
    $preiskl_5 = $_POST['preiskl_5'];

    $sql = "select o.preisklasse, o.region, o.typ, o.vermietung, o.schlafzimmer, o.badezimmer, o.zimmer, o.toiletten, o.garten, o.pool, o.flaeche, 
            o.wohnflaeche, o.garage, o.referenznr, o.regdate, d.name, d.con from objekte o, description d where o.id=d.objekt_id and d.lang='$lang'";

    if ($nw == true)
    {
        $sql .= " and o.region='nw'";

        if ($no == true)
            $sql .= " or o.region='no'";
        if ($sw == true)
            $sql .= " or o.region='sw'";
        if ($so == true)
            $sql .= " or o.region='so'";
        
    }
    elseif ($no == true && $nw==false)
    {
        $sql .= " and o.region='no'";
        if ($sw == true)
            $sql .= " or o.region='sw'";
        if ($so == true)
            $sql .= " or o.region='so'";
    }
    elseif ($sw == true && $no == false && $nw == false)
    {
        $sql .= " and o.region='sw'";
        if ($so == true)
            $sql .= "or o.region='so'";
    }
    elseif ($so == true && $no == false && $nw == false && $sw == false)
    {
        $sql .= " and o.region='so'";
    }


    if ($villa == true)
    {
        $sql .= " and o.typ='villa'";

        if ($apartement == true)
            $sql .= " or o.typ='apartement'";
        if ($finca == true)
            $sql .= " or o.typ='finca'";
        if ($haus == true)
            $sql .= " or o.typ='haus'";
        if ($grundstueck == true)
            $sql .= " or o.typ='grundstueck'";
        if ($gewerbeobjekt == true)
            $sql .= " or o.typ='gewerbeobjekt'";
    }
    elseif ($finca == true)
    {
        $sql .= " and o.typ='finca'";

        if ($apartement == true)
            $sql .= " or o.typ='apartement'";
        if ($haus == true)
            $sql .= " or o.typ='haus'";
        if ($grundstueck == true)
            $sql .= " or o.typ='grundstueck'";
        if ($gewerbeobjekt == true)
            $sql .= " or o.typ='gewerbeobjekt'";
    }
    elseif ($apartement == true)
    {
        $sql .= " and o.typ='apartement'";

        if ($haus == true)
            $sql .= " or o.typ='haus'";
        if ($grundstueck == true)
            $sql .= " or o.typ='grundstueck'";
        if ($gewerbeobjekt == true)
            $sql .= " or o.typ='gewerbeobjekt'";
    }
    elseif ($haus == true)
    {
        $sql .= " and o.typ='haus'";

        if ($grundstueck == true)
            $sql .= " or o.typ='grundstueck'";
        if ($gewerbeobjekt == true)
            $sql .= " or o.typ='gewerbeobjekt'";
    }
    elseif ($grundstueck == true)
    {
        $sql .= " and o.typ='apartement'";
        if ($gewerbeobjekt == true)
            $sql .= " or o.typ='gewerbeobjekt'";
    }
    elseif ($gewerbeobjekt == true)
        $sql .= " and o.typ='gewerbeobjekt'";

    if ($preiskl_1 == true)
    {
        $sql .= " and o.preisklasse='1'";
        if ($preiskl_2 == true)
            $sql .= " or o.preisklasse='2'";
        if ($preiskl_3 == true)
            $sql .= " or o.preisklasse='3'";
        if ($preiskl_4 == true)
            $sql .= " or o.preisklasse='4'";
        if ($preiskl_5 == true)
            $sql .= " or o.preisklasse='5'";
    }
    elseif ($preiskl_2 == true)
    {
        $sql .= " and o.preisklasse='2'";
        
        if ($preiskl_3 == true)
            $sql .= " or o.preisklasse='3'";
        if ($preiskl_4 == true)
            $sql .= " or o.preisklasse='4'";
        if ($preiskl_5 == true)
            $sql .= " or o.preisklasse='5'";
    }
    elseif ($preiskl_3 == true)
    {
        $sql .= " and o.preisklasse='3'";
        
        if ($preiskl_4 == true)
            $sql .= " or o.preisklasse='4'";
        if ($preiskl_5 == true)
            $sql .= " or o.preisklasse='5'";
    }
    elseif ($preiskl_4 == true)
    {
        $sql .= " and o.preisklasse='4'";
        
        if ($preiskl_5 == true)
            $sql .= " or o.preisklasse='5'";
    }
    elseif ($preiskl_5 == true)
    {
        $sql .= " and o.preisklasse='5'";
    }
if ($mietobjekt == true)
    $sql .= " and o.vermietung='1'";


//Database-Instanz erzeugen
$db = new Database('usr_web179_1');

$objects = $db->db_select($sql);
$objects_num = $db->db_num_rows($sql);

$template->assign('objects',$objects);
$template->assign('objects_num',$objects_num);
$template->assign('sql',$sql);

} //end search 


Das ganze funktioniert auch, solange ich im Formular nur eine Checkbox auswähle...sonst aber nicht - wenn ich zB. alle auswähle erhalte ich 64 Suchergebnisse, obwohl nur 4 Objekte in der DB eingetragen sind.
Der generierte Query sieht dann zB. so aus:

select o.preisklasse, o.region, o.typ, o.vermietung, o.schlafzimmer, o.badezimmer, o.zimmer, o.toiletten, o.garten, o.pool, o.flaeche, o.wohnflaeche, o.garage, o.referenznr, o.regdate, d.name, d.con from objekte o, description d where o.id=d.objekt_id and d.lang='de' and o.region='nw' or o.region='no' or o.region='sw' or o.region='so' and o.typ='villa' or o.typ='apartement' or o.typ='finca' or o.typ='haus' or o.typ='grundstueck' or o.typ='gewerbeobjekt' and o.preisklasse='1' or o.preisklasse='2' or o.preisklasse='3' or o.preisklasse='4' or o.preisklasse='5' and o.vermietung='1'

Und ich hab eigentlich keinen Plan wieso mir SQL dann 64 Ergebnisse (alles 4x und in allen Sprachen) liefert...
Ich hoffe sehr dass ihr mir helfen könnt, bin nämlich nicht gerade (wie man sieht ) der SQL-Meister...
Danke schonmal im Voraus
GodfatherDeluxe

[Adromir]

Das liegt eher daran, daß die IF-Abfragen nicht den gewünschten Effekt liefern.

Alle Avriabeln die im Script vorkommen, vergleichst du mit true, zB.
$preiskl_4 == true

Aber woher bekommt die Variable den Wert true?
Normalerweise wird true nur von einer Funktion gesetzt.
Kann mich jetzt aber auch irren, dann würde die Beduingung sofort erfüllt sein, sobald die Variable gesertzt wurde, und daß tust du am Anfang des Scriptes..

[GodfatherDeluxe]

Naja, die If-Abfragen liefern ja eben schon den gewünschten Effekt - der Query der generiert wird sieht ja genauso aus wie ich mir das vorstelle - er liefert nur in den meisten Fällen nicht das gewünschte Ergebnis, also wenn ich eben zB. alle Checkboxen anklicke, wird mir folgender Query generiert:

select o.preisklasse, o.region, o.typ, o.vermietung, o.schlafzimmer, o.badezimmer, o.zimmer, o.toiletten, o.garten, o.pool, o.flaeche, o.wohnflaeche, o.garage, o.referenznr, o.regdate, d.name, d.con from objekte o, description d where o.id=d.objekt_id and d.lang='de' and o.region='nw' or o.region='no' or o.region='sw' or o.region='so' and o.typ='villa' or o.typ='apartement' or o.typ='finca' or o.typ='haus' or o.typ='grundstueck' or o.typ='gewerbeobjekt' and o.preisklasse='1' or o.preisklasse='2' or o.preisklasse='3' or o.preisklasse='4' or o.preisklasse='5' and o.vermietung='1'

Meinem Verständnis nach, sollte diese Abfrage ja alle Datensätze liefern auf die zB. d.lang='de' zutrifft - wieso bekomme ich dann auch die Datensätze für alle anderen Sprachen?!

oder ein andres Beispiel:
Diesmal hab ich alle Regionen ausgewählt und als Objekt-Typ 'Villa' und keine Preisklasse:

select o.preisklasse, o.region, o.typ, o.vermietung, o.schlafzimmer, o.badezimmer, o.zimmer, o.toiletten, o.garten, o.pool, o.flaeche, o.wohnflaeche, o.garage, o.referenznr, o.regdate, d.name, d.con from objekte o, description d where o.id=d.objekt_id and d.lang='de' and o.region='nw' or o.region='no' or o.region='sw' or o.region='so' and o.typ='villa'

Das sollte mir doch alle Datensätze liefern, bei denen die Sprache 'de', die Region entweder 'nw', 'no', 'so' oder 'sw' und der typ 'villa' ist, wobei die preisklassen unberücksichtigt bleiben - oder nicht?!
Dennoch bekomme ich 49 Ergebnisse (alle Sprachen und jeder Datensatz 4x)?!

P.S.: Habe bis jetzt immer Checkboxen mit true abgefragt und keine Probleme damit gehabt, nichtsdestotrotz habe ich mal $nw ausgegeben, wenn es ausgewählt wurde enthält es den wert 'on' ansonsten wird nix ausgegeben - Ich hab also jetzt auch die Überprüfung von 'true' auf 'on' geändert, was aber keinen Effekt hat...

Ich bin echt verzweifelt, wenn ich das nicht bis morgen hinkriege, muss ich das mit select-feldern realisieren (da das morgen fertig sein muss) und das wäre nicht grade die eleganteste Lösung (da man sich dann eben auch nicht seine Kriterien individuell zusammenstellen kann) - ich hoffe ihr versteht mein problem...
Ich bin für jede Hilfe dankbar und hoffe ihr könnt mir helfen
ein verzweifelter GodfatherDeluxe

[Adromir]

Mmh. Vieleicht liegt es an der JOIN Verbindung in dem Query.
So ein ähnliches Problem hatte ich auch mal, das lag aber daran, daß die WHERE on ON Bedingungen nicht eindeutig genug waren..

[GodfatherDeluxe]

Heureka

Du hattest recht, die vielen and und or's hat er nicht gepackt, ich hab das ganze jetzt mit Klammern verdeutlicht und es funktioniert perfekt Vielen vielen Dank für deine Hilfe!
-->erledigt
ein glücklicher GodfatherDeluxe