Câu lệnh IF - ELSE ngắn gọn

Tôi đang cố gắng làm cho mã của mình dễ đọc hơn, vì vậy tôi quyết định sử dụng một số câu lệnh IF ngắn.

Đây là mã của tôi không hoạt động ("không phải là một câu lệnh"):

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

Có gì sai với điều này? Cần dấu ngoặc? Ở đâu?

"Biểu thức bậc ba" x ? y : zchỉ có thể được sử dụng để gán có điều kiện . Đó là, bạn có thể làm điều gì đó như:

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

bởi vì biểu thức bậc ba đang trả về một cái gì đó (loại Stringtrong ví dụ này).

Nó không thực sự được sử dụng như một đoạn ngắn, nội dòng if-else. Đặc biệt, bạn không thể sử dụng nó nếu các phần riêng lẻ không trả về giá trị hoặc trả về giá trị của các kiểu không tương thích. (Vì vậy, trong khi bạn có thể làm điều này nếu cả hai phương thức đều trả về cùng một giá trị, bạn không nên gọi nó chỉ cho các mục đích tác dụng phụ).

Vì vậy, cách thích hợp để làm điều này sẽ chỉ là với một khối if-else:

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

tất nhiên có thể được rút ngắn thành

jXPanel6.setVisible(jXPanel6.isVisible());

Đối với tôi, cả hai biểu thức sau đều dễ đọc hơn ở chỗ chúng truyền đạt rõ ràng hơn những gì bạn đang cố gắng làm. (Và nhân tiện, bạn đã hiểu sai điều kiện của mình chưa? Có vẻ như đây là một lựa chọn không sử dụng, thay vì bật tắt).

Đừng trộn lẫn số lượng ký tự thấp với khả năng đọc . Điểm mấu chốt là những gì dễ hiểu nhất; và việc lạm dụng nhẹ các đặc điểm ngôn ngữ là một cách rõ ràng để gây nhầm lẫn cho người đọc, hoặc ít nhất là khiến họ phải suy nghĩ kỹ.

jXPanel6.setVisible(jXPanel6.isVisible());

hoặc ở dạng của bạn:

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

Toán tử bậc ba chỉ có thể là bên phải của một phép gán chứ không phải là một câu lệnh của chính nó.

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

Như những người khác đã chỉ ra, một cái gì đó có dạng

x ? y : z

là một biểu thức, không phải là một câu lệnh (hoàn chỉnh). Đó là một giá trị cần được sử dụng ở một nơi nào đó - như ở bên phải của một phép gán hoặc một tham số cho một hàm, v.v.

Có lẽ bạn có thể xem tại đây: http://download.oracle.com/javase/tutorial/java/nutsandbolts/expressions.html

Tôi đến bữa tiệc hơi muộn nhưng dành cho những độc giả tương lai.

Từ những gì tôi có thể nói, bạn chỉ muốn chuyển đổi trạng thái hiển thị đúng không? Tại sao không chỉ sử dụng !toán tử?

jxPanel6.setVisible(!jxPanel6.isVisible);

Nó không phải là câu lệnh if nhưng tôi thích phương pháp này hơn cho mã liên quan đến ví dụ của bạn.

Bạn có thể làm điều đó đơn giản như sau, tôi đã làm điều đó trong các móc phản ứng:

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

nó bằng với hàm if dài này bên dưới:

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

Hy vọng nó sẽ giúp ^ _ ^