memset để khởi tạo trong C++

Question

memset đôi khi được sử dụng để khởi tạo dữ liệu trong một hàm tạo như ví dụ dưới đây. Nó có hoạt động nói chung không? Đó có phải là một ý tưởng hay không?

class A { 
public: 
    A(); 
private: 
    int a; 
    float f; 
    char str[35]; 
    long *lp; 
}; 

A::A() 
{ 
    memset(this, 0, sizeof(*this)); 
} 

Answer 1

Không sử dụng memset. Đó là một sự tiếp quản từ C và sẽ không hoạt động trên những người không phải là POD. Cụ thể, sử dụng nó trên một lớp dẫn xuất có chứa bất kỳ hàm ảo nào - hoặc bất kỳ lớp nào có chứa một non-builtin - sẽ dẫn đến thảm họa.

C++ cung cấp một cú pháp cụ thể để khởi tạo:

class A { 
public: 
    A(); 
private: 
    int a; 
    float f; 
    char str[35]; 
    long *lp; 
}; 

A::A() 
    : a(0), f(0), str(), lp(NULL) 
{ 
} 

Thành thật mà nói, tôi không chắc chắn, nhưng memset cũng có thể là một ý tưởng tồi trên diện rộng các điểm kể từ khi định dạng của họ là không xác định.

Answer 2

Đó là một ý tưởng tồi tệ. Bạn chỉ đang tromping dữ liệu, không chú ý đến cách các đối tượng nên được khởi tạo. Nếu lớp của bạn là ảo, bạn có khả năng quét sạch con trỏ vtable.

memset hoạt động trên dữ liệu thô, nhưng C++ không phải là dữ liệu thô. C++ tạo trừu tượng, vì vậy nếu bạn muốn an toàn, bạn sử dụng những trừu tượng đó. Sử dụng danh sách khởi tạo để khởi tạo thành viên.

Bạn thể làm điều đó với các loại POD:

struct nothing_fancy_here 
{ 
    bool b; 
    int i; 
    void* p; 
}; 

nothing_fancy_here x; 
memset(&x, 0, sizeof(x)); 

Nhưng nếu bạn đang làm nó trên this, có nghĩa là bạn đang ở trong một nhà xây dựng người dùng định nghĩa và không còn đủ điều kiện như một loại POD . (Mặc dù nếu tất cả các thành viên của bạn là POD nó có thể làm việc, miễn là không chứa 0 như một giá trị bẫy. Tôi chắc chắn không chắc chắn nếu bất kỳ nguồn khác của hành vi không xác định đi vào chơi ở đây.)