C++: Tôi có nên khởi tạo thành viên con trỏ được chỉ định trong phần thân hàm tạo thành NULL không?

Question

Giả sử tôi có:

// MyClass.h 
class MyClass 
{ 
    public: 
    MyClass(); 

    private: 
    Something *something_; 
} 

// MyClass.cpp 
MyClass::MyClass() 
{ 
    something_ = new Something(); 
} 

Tôi có nên khởi something_ để NULL (hoặc 0) trong các nhà xây dựng danh sách khởi tạo của các nhà xây dựng MyClass? Hay điều đó là không cần thiết bởi vì tôi đang gán cho nó trong phần thân của nhà xây dựng? Thực hành được khuyến nghị là gì?

Answer 1

Nói chung, bạn chỉ gán nó một lần, trong danh sách khởi tạo hoặc cơ thể, trừ khi khởi tạo cơ thể có thể hoặc không thể xảy ra, hoặc có mã điều kiện tiên quyết:

MyClass::MyClass() 
{ 
    //this code must happen first 
    // _now_ max is known 
    something_ = new Something(max); 
} 

MyClass::MyClass() 
{ 
    if (max) 
     something_ = new Something(max); 
    else 
     something_ = NULL; 
} 

MyClass::MyClass() 
    : something_(new Something()) 
//as pointed out in the comments, use smart pointers 
{ 
} 

Answer 2

Nói chung - không. Nhưng nếu một nơi nào đó trong con trỏ constructor của bạn được sử dụng trước khi nó được khởi tạo thì bạn sẽ nhận được hành vi không xác định. Tuy nhiên, vấn đề lớn nhất mà bạn có là điều này - nếu ngoại lệ được ném vào hàm tạo, hàm hủy của nó không được gọi. Vì vậy, hãy tưởng tượng bạn có hai con trỏ đến các đối tượng và phân bổ đối tượng đầu tiên thành công trong khi phân bổ thứ hai không thành công, trong trường hợp đó bạn kết thúc với một rò rỉ tài nguyên. Điều này có thể được giải quyết bằng cách sử dụng con trỏ "thông minh". Nhưng trong trường hợp đó, chúng sẽ được khởi tạo trong danh sách khởi tạo, và nếu bạn không muốn gán giá trị cho chúng hai lần thì bạn nên cấp phát bộ nhớ trong đó hơn là trong phần thân của hàm tạo.

Answer 3

Không cần thiết, đặc biệt là nếu bạn ngay lập tức khởi tạo trong các nhà xây dựng cơ thể, tuy nhiên nếu khởi tạo không phải là quá rõ ràng thì tại sao không NULL nó để tránh tình cờ truy cập vào biến uninitialized.

• tối thiểu trên không hiệu quả
• tiết kiệm rất nhiều thời gian gỡ lỗi/xử lý sự cố với săn lỗi điên