Kiểm tra xem boolean có đúng không?

Question

bool foo = true; 

// Do this? 
if (foo) 
{ 
} 

// Or this? 
if (foo == true) 
{ 
} 

Tôi thích một trong số họ và đồng nghiệp của tôi. Kết quả là như nhau, nhưng (chính xác hơn) là gì?

Answer 1

Hầu như tất cả mọi người tôi đã nhìn thấy đưa ra ý kiến ​​thích

if (foo) 
{ 
} 

Thật vậy, tôi đã nhìn thấy nhiều người chỉ trích việc so sánh rõ ràng, và tôi thậm chí có thể đã làm như vậy bản thân mình trước khi bây giờ. Tôi muốn nói kiểu "ngắn" là thành ngữ.

EDIT:

Lưu ý rằng điều này không có nghĩa là dòng mã luôn không chính xác. Xem xét:

bool? maybeFoo = GetSomeNullableBooleanValue(); 
if (maybeFoo == true) 
{ 
    ... 
} 

Điều đó sẽ biên dịch, nhưng không có "== true" nó sẽ không, như không có chuyển đổi ngầm từ bool? để bool.

Answer 2

Tùy thuộc vào hoàn cảnh của bạn.

Tôi xin nói, nếu bool bạn có một cái tên tốt, sau đó:

if (control.IsEnabled) // Read "If control is enabled." 
{ 
} 

sẽ được ưa thích.

Nếu, tuy nhiên, biến có tên không rõ ràng, việc kiểm tra đối với true sẽ rất hữu ích trong việc hiểu logic.

if (first == true) // Read "If first is true." 
{ 
} 

Answer 3

Ví dụ đầu tiên gần như luôn luôn thắng trong cuốn sách của tôi:

if(foo) 
{ 
} 

Đó là ngắn hơn và ngắn gọn hơn. Tại sao thêm một kiểm tra thêm vào một cái gì đó khi nó hoàn toàn không cần thiết? Chỉ cần lãng phí các chu kỳ ...

Tôi đồng ý rằng đôi khi cú pháp chi tiết hơn làm cho mọi thứ dễ đọc hơn (điều cuối cùng quan trọng hơn miễn là hiệu suất có thể chấp nhận được) trong trường hợp các biến được đặt tên kém.

Answer 4

Cả hai đều chính xác.

Bạn có thể có một số tiêu chuẩn mã hóa trong công ty của mình - chỉ cần xem theo dõi thông qua. Nếu bạn không có - bạn nên :)

Answer 5

cá nhân tôi muốn

if(true == foo) 
{ 
} 

không có cơ hội cho ==/= nhập nhầm và tôi thấy nó biểu cảm hơn về loại foo của. Nhưng đó là một câu hỏi rất chủ quan.

Answer 6

Không phải là "chính xác hơn". Sở thích cá nhân của tôi là cho hình thức ngắn gọn hơn nhưng một trong hai là tốt. Đối với tôi, cuộc sống quá ngắn ngủi để có thể nghĩ đến việc tranh cãi về những thứ như thế này.

Answer 7

Nếu bạn đang đi để lựa chọn

if(foo == true) 

tại sao không đi tất cả các cách và làm

if(foo == true == true == true == true == true == true == true == true == true) 

Đó là điều tương tự.

Tôi không đồng ý rằng nếu tên được đặt tên rõ ràng (ví dụ: IsSomething) thì không sao để so sánh với đúng, nhưng nếu không bạn nên làm như vậy. Nếu nó trong một tuyên bố nếu rõ ràng nó có thể được so sánh với sự thật.

if(monday) 

là giống như mô tả như

if(monday == true) 

Tôi cũng thích cùng một tiêu chuẩn cho không:

if(!monday) 

như trái ngược với

if(monday == false)