Tôi có nên kiểm tra tăng :: shared_ptr hoặc std :: shared_ptr trước khi tôi sử dụng không?

Question

void f(boost::shared_ptr<int> ptr) 
{ 
    if (ptr) // should we check? 
     // do something 
} 

void f2(int *p) 
{ 
    if (p) // good practice to check before using it 
     // do something 
} 

Câu hỏi: Chúng tôi có nên xác thực shared_ptr trước khi chúng tôi sử dụng không?

Answer 1

Có. shared_ptr chỉ cung cấp cho bạn lợi ích của nó khi biết khi nào phát hành bộ nhớ được cấp phát bằng cách đếm các tham chiếu đến nó. Bạn vẫn nên xác minh xem nó có hợp lệ không trước khi sử dụng nó, giả sử bạn không có cách nào khác để biết trước rằng nó chắc chắn sẽ không rỗng, nhưng đó là vấn đề của việc bạn đang sử dụng shared_ptr hay không.

Answer 2

No. Nếu nó nằm trong hợp đồng của hàm phải hợp lệ, thì cách nhanh nhất để thu hút sự chú ý đến thực tế là người gọi có lỗi là lỗi. Thất bại càng sớm càng tốt.

Answer 3

Phụ thuộc vào việc nó thực sự là một khả năng trong quá trình thực thi chương trình bình thường mà shared_ptr là không. Nếu không, hãy đẩy trách nhiệm như một điều kiện tiên quyết về người dùng của f và có thể assert(ptr);. Lưu ý rằng nó không chỉ áp dụng cho shared_ptr, nhưng bất kỳ con trỏ nào sử dụng thực sự. Mặc dù bạn có thể chỉ nên sử dụng tài liệu tham khảo ở đây.

Answer 4

Phụ thuộc vào kịch bản của bạn:

Nếu f() là một hàm public function/API mà bạn không kiểm soát được những gì ai đó có thể vượt qua bạn sau đó có bạn nên kiểm tra.

Nếu hàm là một hàm thành viên riêng tư hoặc được ghi lại là yêu cầu một con trỏ hợp lệ thì tôi sẽ sử dụng assert. Nó có một số lượng lớn không có phí trong bản phát hành bản phát hành và nhanh chóng hiển thị vấn đề trong quá trình gỡ lỗi.

void f(shared_ptr<T> ptr) 
{ 
    assert(ptr && "ptr is null!"); 

    ..... 
} 

Answer 5

Hãy nói: nếu void foo(shared_ptr<int> ptr) của bạn là một thư viện mục đích chung và bạn muốn thông báo cho người sử dụng trong thời gian chạy rằng giá trị nullptr không được hỗ trợ (ném một ngoại lệ) hoặc nó được xử lý bằng một codepath khác nhau có, nó là đúng để kiểm tra. Nếu bạn không cần lưu trữ shared_ptr ở đâu đó với số foo() và bạn chỉ cần đảm bảo chức năng sẽ không hỗ trợ con trỏ null, chỉ cần chuyển một tham chiếu, ví dụ void foo(int &integer). Đối với các trình biên dịch mà tôi biết, các tham chiếu là phần lớn các con trỏ không có giá trị null (tùy thuộc vào các lựa chọn tối ưu hóa).