Czy należy unikać wielu instrukcji if, takich jak "if (warunek) if (warunek) ..." w Javie?

Question

Mój IDE (IntelliJ IDEA) mówi mi, że mam możliwość usunięcia szelki na ten 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) { 
    ... 
} 

Stać:

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) { 
    ... 
} 

mogę zrozumieć, jak to ma sens, i kusi, by zgubić wcięcie, ale moim zmartwieniem jest to, że czytelność może ucierpieć. Ten ostatni wygląda czysto, ale czy zaoszczędzona przestrzeń jest warta potencjalnego zamieszania?

Zakładam, że różnica w wydajności między nimi jest banalna, jeśli w ogóle.

I jako pytanie uzupełniające: czy istnieje ograniczenie liczby "jeśli (stan) można zmieścić w pojedynczej linii, a raczej w jakim punkcie staje się zbyt wiele?

Answer 1

Głosuję za tym, jak już masz.

ja nawet nie korzystać z tego:

if(foo) 
    return bar; 

Lubię to zamiast:

if(foo){ 
    return bar; 
} 

"programy muszą być pisane dla ludzi do czytania, a jedynie incydentalnie dla maszyny do wykonywania"

Answer 2

Wolałbym pierwszy. Myślę, że bit z wieloma ifs w jednym wierszu jest nieczytelny.

Przepraszam, ale zamierzam głosować, aby zamknąć. To będzie debata bez odpowiedzi.

Answer 3

Zawsze używaj szelek. Pewnego dnia będziesz chciał umieścić drugie stwierdzenie w swoim bloku, jeśli lub w innym miejscu, a potem żałujesz, że nie miałeś. Czy jednak naprawdę sprawdzisz kontrole instanceof? Czy możesz przerobić program, aby zamiast tego zmienić go w zachowanie polimorficzne?

Answer 4

Zaleca się, aby zawsze używać nawiasów, ale tam jest sytuacja, kiedy nie ma sensu ciała szelki i to nie lepiej je za pomocą przyczyna jest bardziej czytelny

if(condition){ 

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

} 

jeśli zawsze używać szelek to będzie jak ten. Prawdopodobnie w którymś miejscu popełniam błąd.

if(condition){ 

} else{ 

     if (condition) { 
      ... 
     }else { 

      if (condition3){ 

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

więc myślę, że za pomocą zawsze zależy od czytelności, jak wyżej wspomniane programy muszą być pisane dla ludzi do czytania i tylko incydentalnie dla maszyny do wykonania.