Tại sao nó không phải là tốt để sử dụng để kiểm tra nếu đối số được truyền

Question

Từ cuốn sách duy trì hoạt Javascript nó đề cập:

// Bad: Testing to see if an argument was passed 
function doSomething(arg1, arg2, arg3, arg4){ 
if (arg4 != null){ 
    doSomethingElse(); 
    } 
} 

nhưng tôi cảm thấy sử dụng != null ở đây là khá hiệu quả, nó được lọc cả trường hợp đối số không được thông qua hoặc nó được thông qua là null

bất kỳ lý do gì khiến tác giả đề xuất xấu?

Answer 1

!= null là sai cho undefined, về cơ bản có nghĩa là x !== null && x !== undefined. != loại cưỡng chế, và một trong những tác động của nó là nó xử lý null và undefined tương đương (null == undefined là đúng).

Vì vậy, trừ khi bạn muốn điều đó, một số người khuyên bạn không nên sử dụng != với null (hoặc undefined).

Và bạn hoàn toàn đúng rằng mã bảo vệ chống lại hai (cũng thực sự ba) khả năng:

1. Không có gì đã được thông qua cho các đối số ở tất cả.

2. null đã được chuyển.

3. undefined đã được chuyển.

Nhưng những nhận xét nói đó là một cách xấu để kiểm tra xem một cuộc tranh cãi đã được thông qua, và nó là: Bởi vì nó không bắt đi null hoặc undefined. (Đi qua số null và không chuyển qua số mọi thứ là các trường hợp khác nhau.) Điều đó không nhất thiết có nghĩa là đó là cách xấu để kiểm tra đối số cho một hàm cụ thể.

Answer 2

Vì bạn có thể chuyển bất kỳ thứ gì vào hàm, cũng null. Nếu bạn muốn những điều sau đây:

var obj = new MyObject(); 
// ... 
obj = null; 
// ... 
doSomething(arg1, arg2, arg3, obj); 

Bạn sẽ trôi qua một cuộc tranh cãi nhưng việc kiểm tra của bạn sẽ thất bại vì các tài liệu tham khảo thông qua là rỗng. Luôn sử dụng các mục sau:

if(typeof arg4 != 'undefined') 
    // ... 

Answer 3

Chắc chắn. Trong javascript, bạn có thể có giá trị null và giá trị undefined. Và chúng không giống nhau. Một cách tiếp cận nhanh chóng và dễ dàng để kiểm tra xem arg là hợp lệ có lẽ sẽ là:

if (arg4)

Và bằng cách này bạn kiểm tra arg4 không phải là null, không xác định, và không sai. Vui lòng cẩn thận với điều kiện cuối cùng này, bạn có thể thông báo giá trị sai có ý thức ở đây :-)

CHỈNH SỬA: T.J. Crowder là ngay tại đây. Rất dễ sử dụng và sẽ hoạt động trong hầu hết các trường hợp, nhưng hãy cẩn thận.

Answer 4

Tôi rất muốn giới thiệu xem http://www.infoq.com/presentations/Null-References-The-Billion-Dollar-Mistake-Tony-Hoare - một bài thuyết trình tuyệt vời về vấn đề này từ người tự tạo ra nó.