query di select

[fabiovergotti]

Salve a tutti,
Volevo chiedere un chiarimento ed una spiegazione

io ho una pagina con un form con 15 campi, ma di questi 15 solo alcuni vengono compilati (è una ricerca per utente di conseguenza nella pagina troveremo tutte le colonne del database ma in realtà solo una o due vengono usate per una selezione)

io volevo creare una query di select che relevi quali cambi sono pieni e quali vuoti e di conseguenza applichi la voce WHERE xxx='".$yyy."' solo dove il vaolere $_GET['yyy'] esiste

come posso fare?

[mzanella]

Nella sua forma più semplice, costruendo la query osservando quali sono i campi validi (definiti, non nulli, non vuoti, col tipo corretto, ed eventualmente altri criteri che riterrai opportuni) del vettore $_GET ed i relativi valori:

$clauses = array();
foreach($_GET as $field => $value) {
    if (!empty($value)) {
        $clauses[] = "$field = $value";
    }
}


$query = "SELECT * FROM my_table";
if (!empty($clauses)) {
    $query .= "WHERE " . implode(" AND ", $clauses);
} 

Questa è la versione base, per darti l'idea. Per essere considerata almeno accettabile va raffinata, ad esempio controllando che i $field siano effettivamente nomi di campi, convalidando i valori $value (se sono stringhe usando real_escape_string e simili, ecc) e se poi intendi usare prepared statements (cosa da me personalmente caldeggiata), occorre apportare qualche modifica minimale.

Ultima osservazione: nel codice proposto si assume che i criteri di ricerca siano fissati e di uguaglianza.
Se sono almeno fissati, è abbastanza semplice esprimere la nuova clausola in funzione del campo: per quei campi in cui il criterio è di uguaglianza, si genera una stringa "$field = $value", se sono di disuguaglianza "$field <= $value" e così via.
Se i criteri non sono fissati (ovvero: se è l'utente a scegliere una ricerca per uguaglianza, disuguaglianza, ecc.) la questione è un po' più articolata, bisogna leggere da $_GET, per ogni clausola, anche il criterio da utilizzare.

[fabiovergotti]

provo subito
grazie mille

[fabiovergotti]

ok funziona però non riesco a farlo interagire con lo script sucessivo che mi estrapola i valori dal database
ovvero:

if (!$result || !mysql_num_rows($result)) {
    // header('Location: login.php');

    
}
while ($row = mysql_fetch_array($result)) {
    
    // Your code here
    $nome=mysql_result($result,0,'nome');
    $cognome=mysql_result($result,0,'cognome');
    $email=mysql_result($result,0,'email');
    $n_telefono=mysql_result($result,0,'n_telefono');
    $password=mysql_result($result,0,'password');
    $tipo=mysql_result($result,0,'tipo');

} 

[mzanella]

Dovresti usare mysqli o PDO al posto di mysql, quest'ultimo è deprecato.

Il problema qual'è esattamente? Errori durante l'esecuzione? Oppure il risultato ottenuto non corrisponde a quello atteso?
Una volta lanciata la query, il modo in cui questa è costruita non influenza il modo in cui recuperi i dati. Se non l'hai già fatto, controlla che la query venga costruita nel modo corretto, perlomeno osservandone alcuni esempi.
Se durante l'esecuzione della query ci sono errori, stampali anziché effettuare un re-indirizzamento, in modo da sapere cosa è successo.

Ad ogni modo, dato che stai usando fetch_array, probabilmente ti risulterà più comodo leggere i dati in questo modo:

$nome = $row['nome'];
$cognome = $row['cognome'];
$email = $row['email'];
$n_telefono = $row['n_telefono'];
$password = $row['password'];
$tipo = $row['tipo']; 

[fabiovergotti]

l'errore generato è il seguente:

Warning: mysql_fetch_array() expects parameter 1 to be resource, boolean given in /web/htdocs/www.easycomunicazione.it/home/crm/cerca_utente.php on line 292 

la riga 292 corrisponde a:

while ($row = mysql_fetch_array($result)) { 

L'intero codice lo ho adattato così:

$clauses = array();
foreach($_GET as $field => $value) {
    if (!empty($value)) {
        $clauses[] = "$field = $value";
    }
}


$query = "SELECT * FROM CRM_clienti";
if (!empty($clauses)) {
    $query .= "WHERE " . implode(" AND ", $clauses);
} 





$result = mysql_query("'".$query."'");


if (!$result || !mysql_num_rows($result)) {
    // header('Location: login.php');

    
}
while ($row = mysql_fetch_array($result)) {
    
    // Your code here
    $nome=mysql_result($result,0,'nome');
    $cognome=mysql_result($result,0,'cognome');
    $email=mysql_result($result,0,'email');
    $n_telefono=mysql_result($result,0,'n_telefono');
    $password=mysql_result($result,0,'password');
    $tipo=mysql_result($result,0,'tipo');

} 

funziona correttamente ed i dati vengono estrapolati dal db in maniera corretta ma viene stampato a video anche quell'errore

[mzanella]

Direi che se viene stampato un errore forse non funziona poi tanto correttamente...!

L'errore è causato dal fatto che $result = false, ovvero la $query ha fallito nell'estrarre i dati dalla base di dati. Dovresti stampare il valore restituito da mysql_error() dopo la query per avere qualche indicazione sul problema (credo che anche stampare la query possa essere d'aiuto).

Inoltre:

$result = mysql_query("'".$query."'");

query è già una stringa, non va racchiusa tra apici:

$result = mysql_query($query); 

[fabiovergotti]

eseguendo una echo di mysql_error(); mi esce questo:

You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '= dfgh' at line 1

tuttavia tale dicitura non è presente nella mia pagina 0.o

altra cosa strana se tento di fare una echo di $result mi da un output vuoto

[mzanella]

tuttavia tale dicitura non è presente nella mia pagina 0.o

Ciò che è stato stampato è un pezzo della query. Questo suggerisce che la query non è stata costruita correttamente: dovresti stamparla dopo mysql_error() per capire meglio cos'è successo.

altra cosa strana se tento di fare una echo di $result mi da un output vuoto

Il che non è affatto strano, dato che l'interrogazione è fallita e quindi $result vale false, e stampare un valore booleano produce la stringa vuota. Se vuoi usare questo tipo di ispezione, usa var_dump($result):

echo "<pre>"; var_dump($result); echo "</pre>"; 

.

[fabiovergotti]

usato il var_dump effettivamente restituisce: bool(false)

[fabiovergotti]

ok ho risolto
c'era un errore nella quary che mi mandasti tu all'inizio

$clauses = array();
foreach($_GET as $field => $value) {
    if (!empty($value)) {
        $clauses[] = "$field = '".$value."'";
    }
} 

invece di

$clauses = array();
foreach($_GET as $field => $value) {
    if (!empty($value)) {
        $clauses[] = "$field = $value";
    }
} 

Grazie mille dell'aiuto

[mzanella]

L'errore sta nel non aver convalidato i dati come suggerito: trattandosi di stringhe occorre proteggere i caratteri speciali (real_escape_string) e racchiuderle tra apici (o usare prepared statement).

Aggiungendo sempre gli apici hai il problema opposto: i valori numerici verranno sempre convertiti in stringhe e MySQL dovrà provare a riconvertirli.

[fabiovergotti]

ok grazie mille risolto