Có an toàn để xóa một con trỏ NULL?

Có an toàn để xóa một con trỏ NULL?

Và nó có phải là một phong cách mã hóa tốt?

deletethực hiện kiểm tra bằng mọi cách, vì vậy kiểm tra nó về phía bạn thêm chi phí và trông xấu hơn. Một thực hành rất tốt là đặt con trỏ thành NULL sau delete(giúp tránh xóa hai lần và các vấn đề hỏng bộ nhớ tương tự khác).

Tôi cũng thích nếu deletetheo mặc định là đặt tham số thành NULL như trong

#define my_delete(x) {delete x; x = NULL;}

(Tôi biết về các giá trị R và L, nhưng nó sẽ không tốt sao?)

Từ tiêu chuẩn dự thảo C ++ 0x.

$ 5,3,5 / 2 - "[...] Trong cả hai cách khác, giá trị của toán hạng xóa có thể là giá trị con trỏ null. [... '"

Tất nhiên, không ai sẽ thực hiện 'xóa' một con trỏ có giá trị NULL, nhưng nó an toàn để làm. Lý tưởng nhất là không nên có mã xóa con trỏ NULL. Nhưng đôi khi nó rất hữu ích khi xóa các con trỏ (ví dụ trong một thùng chứa) xảy ra trong một vòng lặp. Vì việc xóa giá trị con trỏ NULL là an toàn, người ta thực sự có thể viết logic xóa mà không cần kiểm tra rõ ràng cho toán hạng NULL để xóa.

Bên cạnh đó, Tiêu chuẩn C $ 7,20.3.2 cũng nói rằng 'miễn phí' trên con trỏ NULL không có hành động.

Hàm miễn phí làm cho không gian được trỏ bởi ptr bị phân bổ, nghĩa là, được tạo sẵn để phân bổ thêm. Nếu ptr là một con trỏ null, không có hành động xảy ra.

Có nó là an toàn.

Không có hại trong việc xóa một con trỏ null; nó thường làm giảm số lượng kiểm tra ở phần đuôi của hàm nếu các con trỏ chưa được phân bổ được khởi tạo về 0 và sau đó chỉ cần xóa.

Vì câu trước đã gây nhầm lẫn, một ví dụ - không phải là ngoại lệ an toàn - về những gì đang được mô tả:

void somefunc(void)
{
    SomeType *pst = 0;
    AnotherType *pat = 0;

    …
    pst = new SomeType;
    …
    if (…)
    {
        pat = new AnotherType[10];
        …
    }
    if (…)
    {
        …code using pat sometimes…
    }

    delete[] pat;
    delete pst;
}

Có tất cả các loại nits có thể được chọn với mã mẫu, nhưng khái niệm này (tôi hy vọng) rõ ràng. Các biến con trỏ được khởi tạo về 0 để các deletehoạt động ở cuối hàm không cần kiểm tra xem chúng có phải là null trong mã nguồn hay không; mã thư viện thực hiện kiểm tra nào.

Xóa một con trỏ null không có hiệu lực. Đó không phải là phong cách mã hóa tốt bởi vì nó không cần thiết, nhưng nó cũng không tệ.

Nếu bạn đang tìm kiếm các thực tiễn mã hóa tốt, hãy cân nhắc sử dụng các con trỏ thông minh thay vào đó, vậy thì bạn không cần phải làm gì deletecả.

Để bổ sung cho câu trả lời của ruslik , trong C ++ 14, bạn có thể sử dụng cấu trúc này:

delete std::exchange(heapObject, nullptr);

Nó là an toàn trừ khi bạn quá tải toán tử xóa. nếu bạn quá tải toán tử xóa và không xử lý điều kiện null thì nó không an toàn chút nào.

Tôi đã trải nghiệm rằng không an toàn (VS2010) để xóa [] NULL (tức là cú pháp mảng). Tôi không chắc liệu đây có phải là theo tiêu chuẩn C ++ hay không.

Nó là an toàn để xóa NULL (cú pháp vô hướng).