Controllo di una mia espressione SQL ... ???

[Grejo]

Ciao.

Mi potreste dire se una stringa SQL che vi scrivo è giusta?

Io ho la seguente tabella "attivita" con i seguenti campi:
- Stato (tipo stringa, valori possibili "Rinviata" o "Eseguita");
- CausaleRinvio (tipo stringa, valori possibili "Causa Forza Maggiore" "Sconosciuto" "Motivazioni Legali");
- Controllo (tipo stringa, valori possibili "24 ore OK" "OK 1200" "OK 2400" "OK 3600", in generale "OK xxxx", dove xxxx è un numero di 3 o 4 cifre).

Dovrei eseguire la seguente interrogazione: Controllare/Trovare i dati che hanno stato "Rinviata" e, CausaleRinvio uguale a "Causa Forza Maggiore" o "Motivazioni Legali" e, Controllo uguale a "24 ore OK" o "OK xxxx" dove xxxx è un numero di 3/4 cifre.

Io ho 'tradotto' l'interrogazione sopra in:

SELECT *
FROM attivita
WHERE Stato='Rinviata' AND (CausaleRinvio='Causa Forza Maggiore' OR CausaleRinvio=Motivazioni Legali') AND (Controllo='24 ore OK' OR Controllo LIKE 'OK ____');

E` giusto secondo voi il codice scritto sopra?

Ciao e Grazie in anticipo.

Grejo

[piero.mac]

una query e' giusta quando non da errori e quando estrae i record attesi.

nel merito della tua sara' giusta, ma ottimizzabile:

SELECT *
FROM attivita
WHERE Stato='Rinviata' 
AND (CausaleRinvio = 'Causa Forza Maggiore' OR CausaleRinvio = 'Motivazioni Legali') 
AND (Controllo='24 ore OK' OR Controllo LIKE 'OK ____');

in pratica: il campo controllo lo prendi in ogni caso e quindi puoi ometterlo. I campo causa rinvio ha tre valori possibili... invece di cercarne due escludine uno.

SELECT *
FROM attivita
WHERE Stato = 'Rinviata' 
AND CausaleRinvio != 'Sconosciuto'

fa le stesse cose con meno codice.

[Grejo]

Purtroppo devo fare per forza questi controlli, il motivo è che mi è stato detto che in futuro aumenterano i possibili valori dei 2 campi (CausaleRinvio e Controllo), ma per questa interrogazione/uso rimangono importanti solo questi valori.

Comunque, è giusto il codice così scritto?

Ciao e Grazie.

Ivan

[piero.mac]

Originariamente inviato da Grejo
Purtroppo devo fare per forza questi controlli, il motivo è che mi è stato detto che in futuro aumenterano i possibili valori dei 2 campi (CausaleRinvio e Controllo), ma per questa interrogazione/uso rimangono importanti solo questi valori.

Comunque, è giusto il codice così scritto?

Ciao e Grazie.

Ivan

per il codice vale quanto detto prima.

per l'ottimizzazione della tabella poiche' la CausaleRinvio e' una stringa ripetuta e forse modificabile converrebbe fare una tabella "causali" in modo da scrivere una volta sola ed in modo "certo", cioe' senza errori che ne possano poi inficiare la ricerca, e riportare solo l'id della causale nel record. Per il campo controllo e' inutile scrivere in ogni record "OK" basta il solo numero. Se non e' OK scrivici 0 (zero).

Prima si ottimizzano le tabelle "poi" si fanno le query.

[Grejo]

Ok ... grazie mille per i consigli ... !!!

Ciao e Grazie ancora.

Grejo.