Problem z selectem Opcje

[di@blo]

mam takie zapytanie

[SQL] pobierz, plaintext 
$wykonaj = mysql_query("SELECT * FROM sprzet WHERE (nazwa LIKE '%$nazwa%') AND (opis LIKE '%$opis%') AND (firma LIKE '%$firma%') AND (kolor='$kolor') ");
[SQL] pobierz, plaintext 

gdy sie poda w formularzu wszystko tj nazwa, opis, firma, kolor to wszystko dziala ok ale jesli ktores z pol zostaje niewypelnione (a użytkownik musi miec taka mozliwosc) to mimo ze pozostale wartosci znajduja sie w bazie nie zwraca nic Jesli ktos wie jak to rozwiazac to bylbym wdzieczny Interesuja mnie rozwiazana w samym zapytaniu mysql oraz przez php

Ten post edytował batman 2.09.2008, 18:12:04

Powód edycji: nobody expects spanish inquisition

[Fixus]

problem zapewne wynika z tego, że kiedy UŻYTKOWNIK (pisownia polska nie jest taka trudna;P ) nie poda jakiejś wartości to zapytanie wyszukuje pól z pustymi ciągami znaków...przez to, że masz wszędzie AND to wygląda na tej zasadzie, że szuka pól gdzie opis czy tam firma jest pustym ciągiem znaków, a takowych nie znajduje bo jak sam napisałeś pozostałe wartości są w tabeli. Dlatego moja rada:
1. albo zamiast AND daj OR wtedy powinno działać
2. albo zrób sprawdzanie tego co jest zaznaczone i konstruuj w locie zapytanie
np:

Kod

$sql = "SELECT * FROM sprzet WHERE ";
if($_POST['nazwa'] != '') {
  $sql .= "nazwa LIKE '%nazwa%' AND";
}
if($_POST['opis'] != '') {
  $sql .= "opis LIKE '%opis%'";
.... // i tak dalej

[di@blo]

nie jest trudna ale kazdemu sie zdazaja bledy ;p ale nie o tym tutaj Dzieki za rozwiazanie ale nie chodzi o to zeby je skopiowac tylko rozumiec o co chodzi.
W tym rozwiazaniu chodzi o to zeby posklejac zapytanie a pozniej dopiero go uzyc?

[SQL] pobierz, plaintext 
$sql = "SELECT * FROM sprzet WHERE ";
IF($_POST['nazwa'] != '') { $sql .= "nazwa LIKE '%nazwa%' AND"; }
IF($_POST['opis'] != '') {  $sql .= "opis LIKE '%opis%'"; }
 
mysql_query("$sql");
[SQL] pobierz, plaintext 

PS
nie do konca dziala bo jak mam

[SQL] pobierz, plaintext 
nazwa LIKE '%nazwa%' AND";
[SQL] pobierz, plaintext 

i pozostale zmiene sa puste to przyczepia sie do 'AND' ale pracuje nad tym jak by ktos mial proste rozwiazanie to moze sie podzielic bo nie chce za bardzo komplikowac kodu

Ten post edytował di@blo 2.09.2008, 20:21:31

[Fixus]

tak chodzi o to żebyś posklejał zapytanie i go potem użył.
Też się kiedyś spotkałem z tym problemem co opisujesz. Możesz to zrobić na dwa sposoby. Jeden jest dość skomplikowany bo wymaga od ciebie szeregu instrukcji warunkowych które uwzględnią wszystkie możliwe przypadki.
drugi sposób jest taki

Kod

...
if($zmienna1 != '') { $sql .= "cos LIKE '%cos%' AND "; }
if($zmienna2 != '') { $sql .= "cos2 LIKE '%cos2%' AND "; }
$sql .= "jakies_pole != ''"; // może tu też być np: $sql .= "jakies_pole LIKE '$''$'";

może to nie jest zbyt eleganckie i nie powinno się tak robić, ale ja tak rozwiązałem ten problem kiedyś przy czymś tam pracując. Ogólnie rzecz polega na tym, że dajesz jakiś absurdalny warunek który będzie spełniony zawsze. Np. pole z identyfikatorem różne od pustego ciągu znaków. W ten sposób zamykasz zapytanie i nie zostaje ci to brzydkie AND na końcu wydajności to nie obniża w żaden dostrzegalny dla przeciętnego śmiertelnika sposób. Kodu AŻ tak bardzo nie bruździ a spełnia swoje zadanie

mam nadzieje, że nikt się nie zgorszy tym może trochę mało wyrafinowanym sposobem....on po prostu spełnia swoją rolę i nikomu nie wadzi

[likemandrake]

Zawsze możesz to zrobić tak:

[PHP] pobierz, plaintext 
<?php
$sQuery = 'SELECT * FROM tabela';
$aWhereStmt = array();
if($zmienna1) {
	$aWhereStmt[] = "warunek1 = '" . mysql_real_escape_string($zmienna1) . "'";
}
if($zmienna2) {
	$aWhereStmt[] = "warunek2 = '" . mysql_real_escape_string($zmienna2) . "'";
}
if(sizeof($aWhereStmt) > 0) {
	$sQuery .= ' WHERE ' . join(' AND ', $aWhereStmt);
}
$sQuery .= ';';
?>
[PHP] pobierz, plaintext 

nie do konca dziala bo jak mam

[SQL] pobierz, plaintext 
nazwa LIKE '%nazwa%' AND";
[SQL] pobierz, plaintext 

i pozostale zmiene sa puste to przyczepia sie do 'AND'

Najszybszy sposób na pozbycie się tego 'AND' w zlepionym zapytaniu, to skorzystanie z funkcji substr(). Można to zrobić tak:

[PHP] pobierz, plaintext 
<?php
$sQuery = substr($sQuery, 0, -4);
?>
[PHP] pobierz, plaintext 

Ten skrawek kodu wykonujesz, gdy zapytanie jest już praktycznie gotowe

[di@blo]

dzieki za pomoc myslalem nad rozwiazaniami ale za bardzo komplikowaly kod

[SQL] pobierz, plaintext 
$sQuery = substr($sQuery, 0, -4);
[SQL] pobierz, plaintext 

to w zupełności mi wystarczy