Cách thích hợp nhất để gán và kiểm tra biến trong câu lệnh if là gì?

Question

Trong số những điều sau đây, đó là cách ưa thích để làm việc, và tại sao? Có bất kỳ tình huống cụ thể nào trong đó nó tạo ra bất kỳ sự khác biệt nào, giả sử rằng thanh hàm() không lấy giá trị bằng không tại bất kỳ thời điểm nào?

Trường hợp 1: Kiểm tra các giá trị chân lý của cả hai điều kiện

if ((foo = bar()) && foo < 0) 
    error(); 

Trường hợp 2: Kiểm tra chỉ biến

if ((foo = bar()) < 0) 
    error(); 

Answer 1

Cách ưa thích giao là để tách chúng:

foo = bar(); 
if (foo < 0) 
    error(); 

Chỉnh sửa: Đây là cách tốt hơn cho cả khả năng đọc và tránh các lỗi, chẳng hạn như trong trường hợp đầu tiên của bạn:

if (foo = bar() && foo < 0) 
    error(); 

Đó có lẽ nên được:

if ((foo = bar()) && foo < 0) 
    error(); 

Answer 2

Người đầu tiên là đồng bằng sai. Vì các quy tắc ưu tiên, bạn nhận được:

if (foo = (bar() && foo < 0)) 
    error(); 

thường không phải là những gì bạn mong đợi.

Answer 3

Ví dụ đầu tiên hoạt động, nhưng nếu ai đó cố gắng để thay đổi <-<= hoặc >=, nó sẽ nổ tung: cho bar trở về 0, nó sẽ kiểm tra phần đầu tiên của và tuyên bố và sẽ xem xét nó sai, hủy thứ hai kiểm tra, điều này đúng.

Ngoài ra, ví dụ đầu tiên của bạn không làm những gì nó phải làm: nhà điều hành ưu tiên của = là ít hơn &&, vì vậy bạn sẽ có được chuyển nhượng kết quả của & & -operation để foo

Vì vậy, tôi muốn các ví dụ thứ hai.

Answer 4

Trong hầu hết các trường hợp không quan trọng. Trường hợp đầu tiên của bạn có một số dự phòng vì vậy tôi muốn đề nghị thứ hai cho sự ngắn ngủi.

Có một số trường hợp mà cách tiếp cận này không đủ tốt, cụ thể trong môi trường nhiều luồng trong đó biến của bạn là tài sản được chia sẻ (đặt hoạt động kiểm tra là &). Trong trường hợp này, truy cập vào biến cần được bảo vệ đúng cách (tức là bằng cách khóa) để tránh các cuộc đua và tham nhũng dữ liệu.

Answer 5

Hầu hết các trình biên dịch nếu không phải tất cả sẽ đưa ra cảnh báo nếu bạn cố thực hiện nhiệm vụ trong dấu ngoặc đơn phụ có điều kiện. Tuy nhiên, trong hầu hết các trường hợp, tôi cho rằng các câu lệnh phức hợp ít có thể đọc được như vậy, trong khi Ruby có nghĩa là không phổ biến trong C. Nếu không có lý do sẽ không bao giờ vượt qua đánh giá mã IMO.