Casi switch Java: con o senza parentesi?

Question

Considerate i seguenti due frammenti, con bretelle:

switch (var) { 
    case FOO: { 
    x = x + 1; 
    break; 
    } 

    case BAR: { 
    y = y + 1; 
    break; 
    } 
} 

Senza bretelle:

switch (var) { 
    case FOO: 
    x = x + 1; 
    break; 

    case BAR: 
    y = y + 1; 
    break; 
} 

So che, nel frammento con bretelle, un nuovo ambito viene creato racchiudendo ogni caso in parentesi. Tuttavia, se per ogni caso non è necessario il nuovo ambito (ovvero non viene riutilizzato alcun nome di variabile), esiste un qualche tipo di penalità di prestazioni per l'utilizzo delle parentesi graffe con un caso?

Answer 1

C'è qualche tipo di penalità di prestazioni per l'utilizzo delle parentesi graffe con un caso?

Nessuno.

Le parentesi graffe servono per aiutare il compilatore a capire l'ambito di una variabile, condizione, dichiarazione di funzione, ecc. Non influisce sulle prestazioni di runtime una volta che il codice è stato compilato in un eseguibile.

Answer 2

Questa domanda probabilmente verrà chiusa come "argomentativa" (BRACE WAR!) Ma che diamine. In realtà mi piacciono le parentesi graffe dopo i casi. A mio avviso, la sintassi del brutto sintomo assomiglia più al resto dei costrutti linguistici. (Non vi è alcuna penalità per l'uso di parentesi graffe in questo "caso")

Answer 3

Si dice che le parentesi possono essere omesse perché nessun nome di variabili viene riutilizzato. Riutilizzando i nomi delle variabili, suppongo che intendi dichiarare una nuova variabile dello stesso tipo.

Le parentesi graffe sono in realtà più utili per garantire che non si rientri per errore riutilizzando la stessa variabile in diversi case s. Oggi non dichiarano alcuna variabile, ma qualcuno le aggiungerà domani e senza le parentesi il codice è soggetto a errori.

Answer 4

Questo è un estremo esempio di ottimizzazione prematura. Sarebbe una buona idea preoccuparsi di quale modo è più facile leggere e fare il debug.

Answer 5

Nessuna penale prestazioni dal punto di vista dell'esecuzione.

penalizzazione delle prestazioni Leggera da un punto di vista compilazione in quanto v'è di più per analizzare, ma se qualcuno fosse in realtà che preoccupa è che avrebbe dovuto scrivere il loro codice di tutto su una riga :-)

E ora la parte parere del nostro post ... ho sempre messo in {} e perché non v'è una sanzione per manutenibilità nel senso che si dovrà probabilmente per metterli in in un secondo momento, e può essere un dolore metterli in un secondo momento. .. ma questa è una preferenza personale del 103%.

Answer 6

Con bretelle.

Ci sono così tante cose che possono andare male con istruzioni switch cerco di evitarli dove posso, cioè

1. Forgetting pause e che hanno quindi caso di caduta-through
2. dimenticare un caso di default e, quindi, non prendere un non-soddisfatti condizione
3. accidentalmente riutilizzo variabili tra istruzioni case, o peggio ancora, che colpisce una variabile che è la condivisione di una variabile tra le istruzioni case.

Utilizzando le parentesi graffe è un modo per evitare che sia la condivisione intenzionale e accidentale di variabili tra casi di dichiarazioni

Answer 7

ho mai pensato prima. Non ho mai avuto bisogno delle parentesi graffe in una clausola, quindi non vedo davvero perché ne avresti bisogno. Personalmente non scelgo l'idea "Sarà più facile da mantenere", è solo spazzatura, sarà più facile mantenere se il codice ha senso ed è documentato.

No bretelle ... meno sintassi è più

Answer 8

Non vorrei usare le parentesi per i casi di commutazione.

• L'istruzione switch sembra già abbastanza barocca senza parentesi graffe.

• I casi di interruttori devono essere molto brevi. Quando hai bisogno di dichiarare variabili è un segno che stai sbagliando.

ora fuori per mantenere un po 'di codice C legacy che sport passano casi di oltre 500 linee ...

Answer 9

Come sappiamo parentesi graffe per i casi di commutazione non sono necessari. L'utilizzo di casi di bretelle può causare confusione sull'ambito di un caso.

Una parentesi aperta è generalmente associata a qualcosa di significativo come un inizio di una funzione o avviare un ciclo o dall'inizio dichiarazione di classe o dall'inizio matrice inizializzazione ecc ... Sappiamo che, un caso scoppia di blocco interruttore quando incontra una dichiarazione di rottura. Quindi l'uso di parentesi graffe sembra implicare l'idea di un diverso campo di applicazione per un lettore ignorante. Quindi, è meglio evitare l'uso di parentesi graffe per una migliore leggibilità della programmazione.

vale a dire quando ho qualcosa di simile,

switch(i) 
{ 
    case 1 : 
    { 
    //do something 
    } 
    System.out.println("Hello from 1"); 

    case 2: 
    .... 
} 

"Ciao da 1" viene stampato. Ma l'uso di parentesi graffa può suggerire a un lettore ignorante che il caso termina con "}", già sapendo quali parentesi graffe generalmente significano in caso di loop, metodi, ecc.

Come se avessimo istruzioni "jump-to-label" in "C" 'il controllo passa al caso e continua la sua esecuzione. Quindi, con questa comprensione, è solo una pratica CATTIVA utilizzare le parentesi graffe durante la scrittura di casi per switch.

Tecnicamente parlando è possibile circondare qualsiasi blocco del codice con una coppia aggiuntiva di parentesi graffe quando viene utilizzato con una sintassi valida. Usare le parentesi graffe in switch sembra così male almeno per me in quanto sembra dare una sensazione diversa come ho detto sopra.

Il mio suggerimento: Basta evitare di utilizzare le parentesi graffe circostanti per i casi di interruttore.