Ritorno dell'indice PHP vs JavaScript

Question

Mentre mi rendo conto che ogni lingua ha una propria convenzione per l'indentazione, non posso fare a meno di essere infastidita da qualcosa che ho scoperto di recente. Considera questo codice dal manuale PHP:

Si noti che ogni caso è rientrato dall'istruzione switch. Questo ha senso, poiché il codice è più facile da leggere e il corpo del blocco è contenuto un livello al suo interno.

Tuttavia, quando prova la dichiarazione equivalente JavaScript interruttore nella JSLint:

switch (i) { 
    case "apple": 
     alert("i is apple"); 
     break; 
    case "bar": 
     alert("i is bar"); 
     break; 
    case "cake": 
     alert("i is cake"); 
     break; 
} 

... visualizza un errore che mi diceva che dovrebbe apparire come questo, invece:

switch (i) { 
case "apple": 
    alert("i is apple"); 
    break; 
case "bar": 
    alert("i is bar"); 
    break; 
case "cake": 
    alert("i is cake"); 
    break; 
} 

Sembra controintuitivo, poiché ogni caso è ora in linea con il blocco dell'interruttore stesso. Non riesco a immaginare alcun motivo per cui ciò sarebbe considerato migliore, tanto meno che si innesca un errore.

JSLint è errato o segue solo la convenzione? Se quest'ultimo è vero, perché la convenzione non dovrebbe essere indentata per chiarezza?

Answer 1

È il tuo codice. Formatta come vuoi. Utilizza jsLint, ma se non sei d'accordo sul fatto che i suoi consigli migliorano il tuo codice, non implementarli. jsLint ferisce i tuoi sentimenti.

Answer 2

Fintanto che il tuo rientro può essere logicamente giustificato, devi indentare lo stile che preferisci. Se JSLint si lamenta davvero di ciò, è troppo pedante.

Answer 3

È solo convenzione per JS (come JSLint definisce convenzione). Personalmente, penso che PHP sarebbe nel torto qui (non so se il manuale segue qualsiasi tipo di convenzione, so che la libreria standard non lo fa). Preferisco lo stile JS perché può diventare davvero cattivo se ci si trova in alcuni blocchi annidati. Ad esempio (codice PHP):

class Foo{ 
     function Bar($arr) { 
      foreach($arr as $item) { 
       switch ($item) { 
        case "foo": 
         // Do something 
         break; 
         // You get the idea 

In definitiva, la vostra scelta. Non userei il manuale PHP come guida di stile; se non puoi usare stili diversi per lingue diverse, usa lo stile JS ben definito.

Answer 4

Mentre la formattazione è importante, alla fine è la tua formattazione. Fallo come ti piace Lo stile particolare che scegli è una scelta personale e molti stili possono essere "buoni". Tuttavia, qualsiasi stile di formattazione "buono" deve essere coerente: scegli le regole che ti piacciono e rispondi a loro.

È un po 'strano per me come si dibattono i dibattiti su cose come collocare {sulla stessa riga del codice precedente o no, o "stringere parentesi graffe" attorno a uno else. Penso che solo i comunisti pongono un {sulla stessa linea della loro dichiarazione if. :)

Answer 5

Gli spazi bianchi extra vengono analizzati da JavaScript e PHP quando vengono interpretati, quindi è possibile rendere il codice brutto come si desidera. Personalmente preferisco aggiungere parentesi graffe in più per le mie istruzioni switch in modo che non mi manca errori autunno-through:

switch ($someVar) 
{ 
    case 'value': 
    { 
    //your code here 
    } break; 
    case 'something': 
    { 
    //more code here 
    } 
    case 'something else': 
    { 
    //some more code here 
    } break; 
    default: 
    { 
    //default code here 
    } break; 
} 

scansione verso il basso il codice alla parentesi di fine mi permette di controllare se ho aggiunto le istruzioni break corrette . È possibile vedere che nel caso 'something' manca un'istruzione di interruzione (forse di proposito, forse un errore).

Alcuni non saranno d'accordo con questo formato, ma è proprio quello che sto ottenendo. Il formato non è poi così importante. I parser sono piuttosto indulgenti. Trova qualcosa che funzioni per te e atteniti ad esso.

Answer 6

Nel libro di Crockford, afferma che i blocchi non introducono un nuovo scopo. "JSLint si aspetta blocchi con funzioni, if, switch, while, for, do e try statements e da nessun'altra parte."

inoltre che

if (condition){ 
    statements; 
} 

è il metodo consigliato di fare blocchi; in quanto è "più resistente". Dubito fortemente che sia stato strutturato in questo modo per errore.

Answer 7

E 'il suo background Java che scorre attraverso. Il modo JSLint imita lo official Java conventions for switch case. Preferisco il tuo primo esempio, ma sto imparando a tollerare il modo JSLint.

Answer 8

Ho trovato anche questo molto confusa, e abbiamo trovato questo motivo in convenzioni Codice di Douglas Crockford per il JavaScript linguaggio di programmazione a http://javascript.crockford.com/code.html

clausole (case, catch, di default, altrimenti, finalmente) non sono dichiarazioni e quindi non dovrebbero essere dichiarazioni come indentate.