Query SQL di formattazione del codice

Devo interrompere le query SQL in diverse righe? Ad esempio nel progetto a cui sto lavorando, abbiamo una query che richiede 1600 colonne! 1600 + caratteri tab. Ho scritto domande come questa:

   "SELECT bla , bla2 , bla FROM bla " . 
     "WHERE bla=333 AND bla=2" . 
      "ORDER BY nfdfsd ...";

Ma mi hanno chiesto di metterli in una riga e hanno detto che il mio stile è una formattazione sbagliata. Perché è una cattiva pratica?

Per motivi di controllo del codice sorgente, abbiamo interruzioni di riga dopo ogni clausola where o virgola. Quindi il tuo sopra si trasforma in

SELECT bla 
     , bla2 
     , bla 
FROM   bla 
WHERE  bla=333 
  AND  bla=2
ORDER  BY nfdfsd
        , asdlfk;

(la tabulazione e l'allineamento non hanno standard qui, ma le virgole sono generalmente in testa)

Tuttavia, non fa alcuna differenza nelle prestazioni.

Una query di 1600 colonne suona come se avesse bisogno di una revisione seria da parte di un buon DBA.

Se una query è complessa la avvolgerò. Se è semplice lo lascerò come una riga singola a meno che non sia troppo lungo, quindi ricomincerò a racchiuderlo.

Riguarda la gestibilità e la comprensione di ciò che dovrebbe fare, quindi il wrapping o il non wrapping può essere deciso al volo, a meno che l'organizzazione non abbia alcune regole di formattazione del codice al riguardo.

Ri: è una cattiva pratica di programmazione. Quasi! È un'ottima pratica. Non ci sono buoni motivi che conosco per usare una query così a lungo, e molti buoni motivi per riformattarla. Come ho detto prima, un esperto DBA probabilmente ha bisogno di lavorarci sopra.

L'unico vantaggio delle query a riga singola che viene in mente è che quelle query possono essere in qualche modo più facili da esaminare. A parte questo, però, sono perplesso. Personalmente, preferisco le query più leggibili e suddivise.

I commenti multilinea sono buoni, quasi vitali quando si tratta di grandi volumi di SQL. E se il tuo linguaggio di programmazione ha citazioni ereditate, è ancora meglio (poiché molti editor possono evidenziare la sintassi SQL in esse).

Esempio:

$a = SQL<<<
    SELECT a, b, c, d
    FROM Foo f
    WHERE f.a = ?
SQL;

Quando si lavora con query di dozzine di righe (o centinaia) sia il rientro che lo spazio bianco rendono il testo praticabile.

Sembra che si tratti in particolare di definire una grande query all'interno di un linguaggio di programmazione di sorta, vedendoti mettere la query in una stringa letterale e concatenarla.

Se è un linguaggio compilato, non dovrebbe fare alcuna differenza: una delle prime ottimizzazioni che il compilatore farebbe è concatenare automaticamente i valori letterali delle stringhe, in modo da finire comunque con una stringa grande.

Per quanto riguarda la sintassi, dovresti effettivamente considerare di spostare la query al di fuori del tuo codice: memorizzala in un file di risorse .sql separato e fai leggere il file al tuo software. Utilizzare istruzioni preparate per le variabili, se non è una query creata in modo dinamico (ovvero clausole where ecc. Aggiunte in base a determinati parametri). Se viene creato in modo dinamico, è possibile aggiungere variabili di sostituzione personalizzate, inserendo parametri aggiuntivi dove e quando necessario.

Per quanto riguarda le 1600 colonne, raccomando seriamente di costruirne una vista, quindi invece di

SELECT column1, column2, .... column1600 from X where Y

otterresti

SELEZIONA * DA viewX DOVE y

Molto più conciso nel tuo codice.

Uso spesso il formato proposto da @glasnt per la risoluzione di una query complicata, tuttavia in genere sono presenti query su una sola riga.

Questo potrebbe non rispondere alla tua domanda, ma suggerirei caldamente di suddividere la query in query più piccole. Ovviamente questo dipende dalla query, ma più clausole e join si aggiungono alla query, meno il motore SQL è in grado di ottimizzare la query.

Il tuo fornitore di database dovrebbe avere strumenti come EXPLAIN di MySQL (o l'impostazione SHOWPLAN_ALL di MSSQL) che ti mostreranno cosa sta facendo il database dietro le quinte per ottimizzare la tua query, ogni volta che il database deve creare una tabella temporanea o qualcosa del genere, stai aggiungendo enormi ritardi quando si parla di più utenti simultanei.

Spostando quella che potrebbe sembrare una banale logica fuori dall'SQL e nel tuo codice, puoi fornire notevoli aumenti delle prestazioni: SQL è eccezionale nelle operazioni semplici.

L'ovvio vantaggio di ciò, poiché potrebbe riguardare te, è che le tue query sono molto meno complesse e facili da leggere - facili da gestire (non> 1600 colonne) e più veloci. Sicuramente una vittoria a tutto tondo.

Spero che sia di aiuto :)