Có bao nhiêu kiểm tra đối với null là thích hợp?

Question

Tôi gặp vấn đề này khi biến bị thiếu trường và người dùng nhận thấy cảnh báo rằng biến này hoặc biến đó không có thuộc tính này hoặc thuộc tính đó. Trong trường hợp đơn giản, nó rất thẳng về phía trước.

if(field) 
    doSomething(field.subField); 

Tuy nhiên, trong các trường hợp thực nghiệm, tôi thấy mình bị kiểm tra quá mức vô lý.

if(!data 
    || !data.records 
    || !data.records[0] 
    || !data.records[0].field 
    || !data.records[0].field.id) 
    return null; 
doSomething(data); 

Ý tôi là, điều này có vẻ như tôi là thợ sửa ống nước chứ không phải nhà phát triển. Vì vậy, tôi có một cảm giác rất mạnh mẽ rằng kiểm tra của tôi, trong khi đủ, có thể là một chút quá-quá-overkill. Có một quy ước nào trong JS về thời điểm thực hiện kiểm tra không?

Answer 1

Tôi sẽ chỉ đưa ra một ý kiến ​​gây tranh cãi.

Trong JavaScript, đừng bận tâm kiểm tra các giá trị null ở những nơi điều này thực tế không nên xảy ra. Nói cách khác, ý tưởng của bạn về việc kiểm tra từng thuộc tính lồng nhau cho null là một chút quá mức cần thiết, và chỉ phục vụ làm phức tạp kịch bản của bạn.

Theo kinh nghiệm của tôi, tôi đã học được cách chỉ để cho lỗi tập lệnh xảy ra. Điều này có phần phản trực giác đối với người viết mã C hoặc mã cơ sở dữ liệu, trong đó null không được giải quyết có thể làm hỏng máy chủ hoặc dữ liệu bị hỏng, tuy nhiên trong thế giới tập lệnh, tốt hơn để tìm hiểu về lỗi của bạn sớm hơn sau này. Nếu trang của bạn tiếp tục tải mà không có dấu hiệu cho thấy có điều gì đó bất ngờ xảy ra, nó sẽ tự biểu hiện dưới dạng các lỗi lạ sau này khi người dùng nhấp vào nút hoặc gửi biểu mẫu.

Lời khuyên của tôi:

Kiểm tra cho nullchỉ nếu bạn sẵn sàng để làm điều gì đó về nó. Nếu bạn có dịch vụ web có thể trả lại số null nếu có sự cố, hãy kiểm tra và hiển thị thông báo lỗi. Nếu bạn lấy lại một giá trị không null, giả sử nó là một giá trị hợp lệ và tiếp tục. Không có lý do để xả rác toàn bộ kịch bản của bạn với kiểm tra null mà sẽ không thực sự mang lại bất kỳ lợi ích thực sự cho chương trình của bạn.

Answer 2

Thông thường bạn sẽ đảm bảo rằng nếu một đối tượng tồn tại, nó luôn có một tập hợp các thuộc tính cơ bản mà làm cho nó có thể sử dụng được.

Ví dụ: nếu biến số data có giá trị, nó sẽ là đối tượng có thuộc tính luôn là một mảng, ngay cả khi nó trống. Nếu mảng chứa bất kỳ thứ gì, nó phải luôn là đối tượng có thuộc tính field, là đối tượng luôn có số lượng id. Điều đó sẽ cắt giảm chi phiếu thành:

if (!data || data.records.length == 0) { 
    return null; 
}