È necessario evitare più istruzioni if ​​come "if (condition) if (condition) ..." in Java?

Question

mio IDE (IntelliJ IDEA) mi sta dicendo che ho la possibilità di rimuovere le parentesi graffe su questo if:

if (objectIsOfTypeFoo) { 
    if (objectOfTypeFooIsShared) { 
     // do something with Object of type Foo knowing that it's shared... 
    } else { 
     // do something with Object of type Foo knowing that it's not shared... 
    } 
} else if (objectIsOfTypeBar) { 
    ... 
} 

Per diventare:

if (objectIsOfTypeFoo) if (objectOfTypeFooIsShared) { 
    // do something with Object of type Foo knowing that it's shared... 
} else { 
    // do something with Object of type Foo knowing that it's not shared... 
} else if (objectIsOfTypeBar) { 
    ... 
} 

Capisco come questo ha un senso, e è allettante perdere il trattino, ma la mia preoccupazione è che la leggibilità possa risentirne. Quest'ultimo aspetto è più pulito, ma lo spazio risparmiato vale la potenziale confusione?

Suppongo che la differenza di prestazioni tra i due sia banale, se non del tutto.

E come domanda successiva: Esiste un limite al numero di "se" (condizione) che è possibile inserire in una singola riga, o piuttosto a che punto è diventato troppo?

Answer 1

Sto votando per il modo in cui hai già.

Non so nemmeno usare questo:

if(foo) 
    return bar; 

Mi piace questo posto:

if(foo){ 
    return bar; 
} 

"I programmi devono essere scritte per le persone a leggere, e solo incidentalmente per le macchine da eseguire"

Answer 2

Preferirei il primo. Penso che il bit con più if su una singola riga sia illeggibile.

Siamo spiacenti, ma ho intenzione di votare per chiudere. Sarà un dibattito senza risposta.

Answer 3

Utilizzare sempre bretelle. Un giorno, vorrai inserire una seconda affermazione nel tuo blocco if o else, e poi vorresti averlo. Tuttavia, stai davvero facendo i controlli instanceof? Puoi rielaborare il tuo programma per trasformarli invece in un comportamento di oggetti polimorfici?

Answer 4

Si consiglia di utilizzare sempre le parentesi graffe, ma c'è una situazione in cui nessuna parentesi uso del corpo ed è meglio non utilizzarli causa è più leggibile

if(condition){ 

} else if (condition) { 
    ... 
}else if (condition3){ 

} 

se si usa sempre le parentesi sarebbe come questo. Probabilmente sbaglio in qualche posto.

if(condition){ 

} else{ 

     if (condition) { 
      ... 
     }else { 

      if (condition3){ 

      }//end if 
     }//end else 
}//end else 

Quindi penso che l'uso sempre dipende di leggibilità, come sopra detto i programmi devono essere scritte per le persone a leggere e solo incidentalmente per le macchine da eseguire.