Kiểm tra nếu biến null trước khi gán cho null?

Question

Biến nhượng có đắt hơn so với kiểm tra không? Ví dụ, có đáng để kiểm tra rằng foo không phải là null trước khi gán nó không?

if (foo != null) { 
    foo = null; 
} 

Hoặc điều này đáng lo ngại về không có gì?

Answer 1

Đây là một vi tối ưu hóa (và có thể là một cái gì đó được xử lý bởi trình biên dịch anyways). Đừng lo lắng về điều đó. Bạn sẽ nhận được một sự trở lại lớn hơn nhiều bằng cách tập trung vào chương trình thuật toán thực tế của bạn.

Chúng ta nên quên hiệu quả nhỏ, nói khoảng 97% thời gian: tối ưu hóa sớm là gốc rễ của mọi điều ác. - Donald Knuth

Answer 2

Đây thực sự là (rất, rất nhẹ) ít hiệu quả hơn. Các phép gán biến tương đương với kiểm tra null, cộng thêm một nhánh phụ có thể. Không phải là nó tạo ra nhiều khác biệt.

Hoặc điều này đáng lo ngại về không có gì?

Bạn đã hiểu.

Answer 3

Tôi sẽ không lo lắng về điều đó - đó chỉ là các dòng mã bổ sung để duy trì. Đây là loại tối ưu hóa vi mô mà bạn không bao giờ nên làm trừ khi bạn có lô bằng chứng được chứng minh rằng đó là nút cổ chai của bạn.

Answer 4

foo = null; 

if (foo != null) 
    foo = null; 

Nếu tôi nhìn vào mã khối thứ hai tôi sẽ nghĩ rằng bạn chỉ muốn đặt biến foo null nếu nó không phải là null trước khi, và nếu tôi nhìn vào các mã đầu tiên tôi sẽ nghĩ bạn muốn đặt biến foo thành null.

Tôi biết điều này là do ví dụ bạn đã viết, nhưng cuối cùng loại tối ưu hóa vi mô này chỉ thêm nhầm lẫn (không đáng giá).

Answer 5

Điều này sẽ làm cho mã của bạn khó đọc hơn rất nhiều ngay cả khi đó là một tối ưu hóa, nó sẽ không đáng để gây rắc rối.

Và nó không phải là tối ưu hóa. Trên hầu hết các cpu hiện đại nếu báo cáo là khá tốn kém.

Answer 6

Điều này sẽ có ít hoặc không có hiệu lực. Tôi không nghĩ rằng bạn thậm chí có thể tạo ra một điểm chuẩn để chứng minh sự khác biệt.

Trong thực tế, một số người lập luận rằng gán null ở tất cả là một mùi code (xem PMD detector for NullAssignment):

Gán một "null" cho một biến (bên ngoài kê khai của nó) là thường hình thức xấu. Một số lần, nhiệm vụ là dấu hiệu cho thấy lập trình viên không hoàn toàn hiểu điều gì đang xảy ra trong mã . LƯU Ý: Loại nhiệm vụ này có thể trong các trường hợp hiếm hoi có ích cho việc khuyến khích thu gom rác thải.Nếu đó là những gì bạn đang sử dụng nó cho, bởi mọi cách, bỏ qua quy tắc này :-)

Nói chung, tôi thận trọng cá nhân của bất cứ điều gì mà cố gắng để khuyến khích thu gom rác thải (bạn hầu như luôn luôn có được hiệu ứng mà bạn không mong đợi).

Answer 7

Trước hết, đó là tối ưu hóa vi mô. Vì vậy, không cần phải lo lắng nhiều về nó.

Nhưng để trả lời câu hỏi của bạn, bạn cần phải giảm nó xuống chỉ một dòng. (như tất cả các mã của bạn hiện là để thiết lập nó để NULL).

foo = NULL; 

Lý do được rằng,

So sánh là một hoạt động nhiều tốn kém hơn nhiều so với chuyển nhượng. (Như so sánh ăn nhiều hướng dẫn lắp ráp tương đối. Nói chung một phép trừ và so sánh với số không hoặc một XOR và so sánh với số không). Nhiệm vụ chỉ dẫn ít hơn.

Answer 8

Nếu bạn có trình biên dịch tốt, chúng sẽ tạo mã giống hệt nhau. Nếu bạn có trình biên dịch crappy thì trình biên dịch có số if sẽ tồi tệ hơn. Về phân bổ phần cứng năm 2009 cho các biến rất rẻ, và các nhánh có điều kiện đôi khi có thể tốn kém.