Krótkie stwierdzenie IF

82

Staram się, aby mój kod był bardziej czytelny, więc zdecydowałem się użyć krótkich instrukcji IF.

Oto mój kod, który nie działa („nie jest to instrukcja”):

jXPanel6.isVisible() ? jXPanel6.setVisible(true) : jXPanel6.setVisible(false);

Co w tym złego? Potrzebuje nawiasów? Gdzie?

java if-statement

— monczek
źródło

2

Chociaż mautetto zapewnił lepszy sposób pisania tego, ta struktura jest zwykle nazywana operatorem trójskładnikowym. (Na wypadek, gdybyś się zastanawiał;))

— Reese Moore

To operator warunkowy . „Trójskładnikowy” oznacza po prostu, że ma 3 operandy.

Link do specyfikacji języka dla operatora warunkowego: docs.oracle.com/javase/specs/jls/se10/html/…

215

„Wyrażenie trójskładnikowe” x ? y : zmoże być używane tylko do przypisania warunkowego . Oznacza to, że możesz zrobić coś takiego:

String mood = inProfit() ? "happy" : "sad";

ponieważ wyrażenie trójargumentowe zwraca coś (typu Stringw tym przykładzie).

Tak naprawdę nie ma być używany jako krótki, liniowy if-else. W szczególności nie można go użyć, jeśli poszczególne części nie zwracają wartości lub zwracają wartości niezgodnych typów. (Więc chociaż możesz to zrobić, jeśli obie metody zwracają tę samą wartość, nie powinieneś wywoływać jej tylko w celu uzyskania efektu ubocznego).

Zatem właściwym sposobem na zrobienie tego byłoby użycie bloku if-else:

if (jXPanel6.isVisible()) {
    jXPanel6.setVisible(true);
}
else {
    jXPanel6.setVisible(false);
}

który oczywiście można skrócić do

jXPanel6.setVisible(jXPanel6.isVisible());

Oba te ostatnie wyrażenia są dla mnie bardziej czytelne, ponieważ wyraźniej wyrażają to, co próbujesz zrobić. (Nawiasem mówiąc, czy twoje warunki były niewłaściwe? Wygląda na to, że to nie jest operacja, a nie przełącznik).

Nie myl małej liczby znaków z czytelnością . Kluczową kwestią jest to, co jest najłatwiejsze do zrozumienia; a lekkie nadużywanie cech językowych jest zdecydowanym sposobem na zdezorientowanie czytelników lub przynajmniej zmusić ich do zastanowienia się.

— Andrzej Doyle
źródło

1

Po prostu nie jest prawdą, że do przypisania można używać tylko operatora warunkowego. Możesz go używać wszędzie tam, gdzie wymagane jest wyrażenie.

28

jXPanel6.setVisible(jXPanel6.isVisible());

lub w swoim formularzu:

jXPanel6.setVisible(jXPanel6.isVisible()?true:false);

— mauretto
źródło

14

FWIW, czuję, że nigdy nie ma potrzeby pisać, <bool condition> ? true : falseponieważ jest to dokładnie równoważne z just <bool condition>.

— Andrzej Doyle

2

Oczywiście! Ale monczek zapytał o instrukcję Krótkie Jeśli-Inaczej, więc zakodowałem ją tylko po to, by pokazać składnię.

— mauretto

4

Operator trójskładnikowy może być tylko prawą stroną przypisania, a nie własnym oświadczeniem.

http://www.devdaily.com/java/edu/pj/pj010018/

— perdian
źródło

Nie jest prawdą, że operator warunkowy może być używany tylko po prawej stronie przypisania. Można go używać wszędzie tam, gdzie można użyć wyrażenia.

3

Jak wskazywali inni, coś w formie

x ? y : z

jest wyrażeniem, a nie (kompletną) instrukcją. Jest to wartość r, która musi zostać gdzieś wykorzystana - na przykład po prawej stronie przypisania, parametr funkcji itp.

Być może możesz spojrzeć na to: http://download.oracle.com/javase/tutorial/java/nutsandbolts/expressions.html

— Anshuman Fotedar
źródło

2

Trochę spóźniłem się na imprezę, ale dla przyszłych czytelników.

Z tego, co mogę powiedzieć, chcesz tylko przełączyć stan widoczności, prawda? Dlaczego po prostu nie skorzystać z !operatora?

jxPanel6.setVisible(!jxPanel6.isVisible);

To nie jest instrukcja if, ale wolę tę metodę dla kodu związanego z twoim przykładem.

— Andy Body
źródło

Ponadto, jeśli próbujesz przełączyć stan, myślę, że instrukcja jest wsteczna. Czy to dlatego to nie zadziałało? Czy powinna to być jXPanel6.isVisible ()? jXPanel6.setVisible (false): jXPanel6.setVisible (true);

— Andy Body

Jeśli nie jest to instrukcja przełączania, to nie rozumiem celu uczynienia czegoś widocznym, co już jest ustawione w ten sposób. IsVisible pobiera stan, a setVisible ustawia stan, prawda? Jeśli instrukcja isVisible zwraca wartość true, po co ustawiać ją ponownie na true. Czy coś mi brakuje?

— Andy Body

1

Możesz to zrobić tak prosto, zrobiłem to w hookach:

 (myNumber == 12) ? "true" : "false"

była równa tej długiej funkcji if poniżej:

if (myNumber == 12) {
  "true"
} else {
  "false"
}

Mam nadzieję, że to pomoże ^ _ ^

— Steven S.
źródło

Krótkie stwierdzenie IF - ELSE