Qual è il modo migliore per indentare i casi in uno switch?

Question

Mentre stavo scrivendo un altro interruttore in Eclipse, ancora una volta sono imbattuto in un piuttosto strano (per me, almeno) rientranza di default, che viene applicato alle dichiarazioni 'switch':

switch (i) { 
case 1: 
    ... 
case n: 
    ... 
} 

tendo a preferire un'altra way:

switch (i) { 
    case 1: 
     ... 
    case n: 
     ... 
} 

Quale modo è più leggibile e piacevole per voi? Non sono ancora al cento per cento determinato per quello che è meglio per me, quindi mi piacerebbe attenermi a ciò che è meglio per le altre persone che leggono il mio codice.

BTW, sei libero di chiudere questa domanda se questo è troppo soggettivo. :)

Answer 1

io uso secondo modo:

switch (i) { 
    case 1: 
     ... 
    case n: 
     ... 
} 

Answer 2

io preferisco il secondo modo pure. Ma è più importante rimanere consistente all'interno di una particolare applicazione e/o all'interno di una squadra particolare piuttosto che rientrare in un modo o nell'altro.

Answer 3

tendo a rientrare tutti i corpi di struttura di controllo una scheda singola (4space) in questo modo:

switch (i) 
{ 
    case 1: 
     ... 
    case n: 
     ... 
} 

ritengo l'interruttore sia la struttura di controllo esterno e la parte direttive caso del corpo (anche se fanno parte della struttura di controllo).

vorrei poi ulteriormente scheda trattino ogni caso in questo modo:

switch (i) 
{ 
    case 1: 
     do_something(); 
    case n: 
     do_something_else(); 
} 

Trovo che questo sia il formato più leggibile per il caso interruttore costrutto.

Come jkohlhepp ha menzionato la conformità alle convenzioni di stile del codice del progetto su cui si sta lavorando è la cosa più importante, se si sta lavorando su un progetto che non ne ha, vale la pena di svilupparne alcuni.

Answer 4

Il primo metodo ha senso logico (per me), tuttavia preferisco anche il secondo metodo. Penso che molti di noi siano condizionati ad aspettarsi che il codice all'interno delle parentesi graffe sia rientrato.

Answer 5

Secondo lo "official" Java Code Conventions, è la prima variante (nessun rientro aggiuntivo per caso).

Answer 6

Il primo è il rientro del caso interruttore standard.

switch (i) { 

case 1: 
    .... // Here you can write more code in a row than using the second way 
    break; 
case n: 
    .... 
    break; 
default: 
    .... 
    break; 
} 

Avviso la nuova linea inserito tra interruttore (i) e il caso 1:

Answer 7

Se si preferisce il secondo caso, allora per coerenza si dovrebbe anche rientrare la parte else di un if dichiarazione:

if(test) 
    do_something(); 
    else 
     do_something_else(); 

La maggior parte delle persone non lo fa e la convenzione è di mantenere i rami sullo stesso livello dell'istruzione, quindi Java preferisce il primo come test dell'istruzione del caso e le diramazioni del codice sono coerenti nel livello di un if/then/altro costrutto t.

ho flip-flop tra loro pure, ma si è conclusa definitivamente preferendo la prima:

switch (i) { 
case 1: 
    ... 
case n: 
    ... 
}