Escludere dalla query SELECT i campi vuoti

[condor_uk]

Buongiorno a tutti voi,
ho realizzato una query che prende dei dati da un database verificando le variabili POST e mostra a video i dati. il mio problema è questo, inserendo i dati nel form e cliccando su cerca mi vengono correttamente passati i dati richiesti, ma, ahime, mi vengono mostrate tutte le righe del database e non solo quelle con i campi richiesti.

questa la query:

$cliente = $_POST['cliente'];
$targa = $_POST['targa'];


$cerca = mysql_query ("SELECT * FROM MiaTabella WHERE `targa` = '$targa' OR `cliente` = '$cliente'");


echo "
    <table border>
    <tr><th colspan='3'>Risultati Ricerca</tr>
    <tr><th>Id<th>Targa<th>Cliente<th>Vettura</tr>
";


while ($riga = mysql_fetch_row($cerca)){
    echo "<tr><td>$riga[0]<td>$riga[2]<td>$riga[5]<td>$riga[1]";
}


echo "</table>";

mi sapreste indicare dove sbaglio ?

grazie

[satifal]

AND al posto di OR nella query.

[condor_uk]

AND al posto di OR nella query.

buongiorno satifal,
ma se metto la clausola AND al posto di OR verifica sono costretto a specificare nel modulo di ricerca
entrambe le condizioni ossia il cliente e la targa, se metto solo una delle due non mi restituisce nulla

[satifal]

La condizione corretta è l'AND è la query che così com'è non va bene. Devi comporla dinamicamente in base alla presenza o meno dei vari parametri:

<?php
$cliente = $_POST['cliente'];
$targa = $_POST['targa'];

$query = "SELECT * FROM MiaTabella WHERE 1";

if (!empty($cliente)) {
    $query .= " AND `cliente` = '$cliente'";
}
if (!empty($targa)) {
    $query .= " AND `targa` = '$targa'";
}

echo $query;

$cerca = mysql_query($query);
...
?>

[condor_uk]

La condizione corretta è l'AND è la query che così com'è non va bene. Devi comporla dinamicamente in base alla presenza o meno dei vari parametri:

<?php
$query = "SELECT * FROM MiaTabella WHERE 1"; <----- non ho capito 
...
?>

buongiorno,
non ho ben capito questa condizione !!!

[satifal]

L'ho inserita esclusivamente per comodità dato che mi permette di risparmiare qualche controllo a codice. Il "WHERE 1" è una condizione sempre vera per cui potrebbe anche essere eliminata. Il WHERE va però inserito nel caso in cui, non sempre vero, sia presente almeno una condizione. Quindi, bisognebbe controllare che vi sia almeno una condizione prima di inserire la clausola WHERE nella query.
Inserendola sempre, a prescindere, tale controllo può essere evitato infatti mi limito ad inserire le condizioni in AND in caso esse siano presenti.

[condor_uk]

L'ho inserita esclusivamente per comodità dato che mi permette di risparmiare qualche controllo a codice. Il "WHERE 1" è una condizione sempre vera per cui potrebbe anche essere eliminata. Il WHERE va però inserito nel caso in cui, non sempre vero, sia presente almeno una condizione. Quindi, bisognebbe controllare che vi sia almeno una condizione prima di inserire la clausola WHERE nella query.
Inserendola sempre, a prescindere, tale controllo può essere evitato infatti mi limito ad inserire le condizioni in AND in caso esse siano presenti.

perfetto, ma se costruisco la query in questo modo :

<?php>
// mi connetto al DBMS
$myconn = mysql_connect($host, $user, $pass) or die('Errore...');


//Mi connetto al database
mysql_select_db($database, $myconn) or die('Errore...');


$cliente = $_POST['cliente'];
$targa = $_POST['targa'];


$query ("SELECT * FROM CommesseOfficina WHERE 1");


if (!empty($cliente)) {
    $query .= " AND `cliente` = '$cliente'";
}
if (!empty($targa)) {
    $query .= " AND `targa` = '$targa'";
}


echo $query;


$cerca = mysql_query($query);


echo "
	<table border>
	<tr><th colspan='3'>Risultati Ricerca</tr>
	<tr><th>Id<th>Targa<th>Cliente<th>Vettura</tr>
";


while ($riga = mysql_fetch_row($cerca)){
	echo "<tr><td>$riga[0]<td>$riga[2]<td>$riga[5]<td>$riga[1]";
}


echo "</table>";


// E chiudiamo la connessione a MySQL
mysql_close();




?>

?>

non mi restituisce nessun risultato...

[satifal]

Beh c'è l'echo $query che te la stampa e puoi verificare se la query è corretta o meno!

[condor_uk]

Beh c'è l'echo $query che te la stampa e puoi verificare se la query è corretta o meno!

$query ("SELECT * FROM CommesseOfficina WHERE 1");

PHP Fatal error: Function name must be a string in on line 17

che è proprio la riga $query

[Alhazred]

Se $query è una variabile, e lo è, allora ti manca un = e devi togliere le parentesi.