cú pháp java tốt hơn là gì: if (isSomething() == false) {hoặc nếu {

Question

Nếu bạn nhìn vào

if (!x) { 
if (x == false) { 

Dường như x là tốt hơn, nhưng

(isSomething()!)!

if (!isSomething()) { 
if (isSomething() == false) { 

bạn có thể dễ dàng giám sát!

Việc cần làm? Có đề xuất nào không?

Answer 1

Tùy chọn thứ ba ẩn là đặt tên biến và phương thức của bạn đúng cách.

Thay vì

if (!isDisabled()) { 
    ... 
} 

sử dụng

if (isEnabled()) { 
    ... 
} 

hoặc nếu bạn muốn kiểm tra sự tiêu cực:

boolean disabled = !isEnabled(); 
if (disabled) { 
    ... 
} 

hoặc thêm cả hai phương pháp:

boolean isDisabled() { 
    return !isEnabled(); 
} 

Chỉnh sửa: Tôi đã tìm thấy câu hỏi này: Is it bad to explicitly compare against boolean constants e.g. if (b == false) in Java?

Answer 2

Tôi sẽ gắn với ký hiệu if (!isSomething()) {. Nếu bạn hoặc những người khác cảm thấy khó đọc, bạn luôn có thể thêm một khoảng trắng nhỏ xung quanh '!' để làm cho nó nổi bật:

if (! isSomething()) { hoặc if (!isSomething()) {

Bên cạnh đó, nhiều báo cáo có điều kiện có thể trở thành áp đảo với các ký hiệu sau

if (isSomething() == false && isSomethingElse() == false && ..),

trong khi thay thế của nó là ngắn và gọn gàng. Sau một thời gian nó trở nên tự nhiên để đọc '!' cùng với các câu lệnh là "not isSomething() và không phải là isSomethingElse()".

Answer 3

Tôi không nghĩ có bất kỳ đề xuất nào mà mọi người sẽ theo dõi.

Hãy làm theo cách của bạn, personnally, tôi sẽ chọn if (!isSomething()) phong cách :)

Đặc biệt kể từ khi tôi đã chọn if (!x) phong cách.

Answer 4

if (!isSomething()) { 

sẽ là tốt nhất theo ý kiến ​​của tôi. Bằng cách này, bạn đang giữ cho nhân vật đếm ngược, mã của bạn có thể đọc được và ! không kết nối ngay từ đầu, vì vậy chỉ bằng cách lướt qua mã, những người khác có thể thấy ý định của nó.