Stampa SQL con dentro array

[Luca1317]

Dovrei avere un risultato del genere:

sql = "SELECT * FROM colori where colore<>'bianco' and colore<>'rosso' "

"bianco" e "rosso" sono 2 valori di esempio di un array che potrebbe essere anche di 10 elementi:

dovrei rendere tutto dinamico ma mi sono incartato:

$sql = "SELECT * FROM colori where ";

foreach( $arrayColore as $ValoreColore)
{
$sqlColore1 = $sql . "colore<>'" . $ValoreColore;
}

echo $sqlColore1;

[Alhazred]

Ti mancano gli "and" nel generare la query e fa anche attenzione a mettere gli spazi nelle stringhe per evitare cose tipo

"where colore<>'bianco'andcolore<>'rosso'..."
in cui andcolore ti darebbe errore di sintassi.

Altra cosa, scritta così, ad ogni ciclo $sqlColore1 viene sovrascritta, quindi alla fine avresti solo l'ultimo valore, i precedenti vanno persi, devi concatenare i vari pezzi, e ti basta usare la variabile $sql per farlo.

[Luca1317]

Si, ma non è un problema di spazi ma proprio che mi sono incartato, ti spiego meglio

in questo modo se stampassi l'sql a video sarebbe:

SELECT * FROM colori where colore<>'rosso'

invece di:

SELECT * FROM colori where colore<>'bianco' and colore<>'rosso'

ovvero mi stampa solo l'ultimo valore dell'array.

[Luca1317]

Ok, credo di essere riuscito a fare progressi:

$sql ="SELECT * FROM colori where ";

foreach( $arrayColore as $ValoreColore)
{
$sqlColore1 = $sqlColore1 . " colore<>'" . $ValoreColore . " and ";
}
$sqlColoreCompleto = $sql . $sqlColore1;

l'unico problema è che mi lascia l'and finale:

SELECT * FROM colori where colore<>'bianco' and colore<>'rosso' and

[Alhazred]

Per esempio

$sql ="SELECT * FROM colori where ";

foreach( $arrayColore as $ValoreColore)
{
    $sql .= " colore<>'" . $ValoreColore . "' and ";
}

$sql = substr($sql, 0, -4);

echo $sql; 

[Luca1317]

ah ok, ancora non conoscevo substr; quindi elimini gli ultimi 4 caratteri (-4) per non far comparire l'ultmo and.

Grazie mille.

[lucavizzi]

Per esempio

$sql ="SELECT * FROM colori where ";

foreach( $arrayColore as $ValoreColore)
{
    $sql .= " colore<>'" . $ValoreColore . "' and ";
}

$sql = substr($sql, 0, -4);

echo $sql; 

Questa query andrà in errore se $arrayColore è vuoto. Se poi $arrayColore non è mai vuoto, puoi anche usarla.



Un'alternativa, che non richiede substr(), potrebbe essere:

$sql ="SELECT * FROM colori where 1=1 ";

foreach( $arrayColore as $ValoreColore)
{
    $sql .= " and colore!='" . addslashes($ValoreColore) . "' ";
}

echo $sql; 

[Luca1317]

In questo modo però avrò l'and all'inizio che non mi fa funzionare la query, giusto?

così:

SELECT * FROM colori where and colore<>'rosso'...

[lucavizzi]

In questo modo però avrò l'and all'inizio che non mi fa funzionare la query, giusto?

così:

SELECT * FROM colori where and colore<>'rosso'...

Non proprio.

Avrai:

SELECT * FROM colori where 1=1 and colore<>'rosso'...

E se l'array dei colori sarà vuoto, perché magari in taluni casi non vuoi escludere nessun colore, la query diventerà:

SELECT * FROM colori where 1=1

Che resta una query sintatticamente corretta, a differenza della query:

SELECT * FROM colori where

Che invece genererebbe un errore.

[Luca1317]

Ah giusto, non avevo visto il where 1=1

Ho da poco cominciato a programmare in PHP dopo anni di ASP e, a differenza di quando iniziai dove il motto era "basta che funzioni", sto cercando di fare le cose per bene già dall'inizio.

Quindi, non è per criticare eh, se qualcuno vedesse il codice potrebbe essere oggetto di critica?