Cách đúng để cấp phát bộ nhớ trong hàm dựng C++ là gì?

Question

Cách nào là đúng cách để cấp phát bộ nhớ qua new trong hàm dựng C++. Cách thứ nhất trong danh sách đối số:

class Boda { 
    int *memory; 
    public: 
     Boda(int length) : memory(new int [length]) {} 
     ~Boda() { delete [] memory; } 
}; 

hoặc trong cơ thể của constructor:

class Boda { 
    int *memory; 
    public: 
     Boda(int length) { 
      memory = new int [length]; 
     } 
     ~Boda() { delete [] memory; } 
}; 

Cảm ơn, Boda Cydo.

Answer 1

Tôi nghĩ cách đơn giản nhất để thực hiện việc này là sử dụng boost scoped array và để mã thư viện được kiểm tra tốt của người khác xử lý tất cả cho bạn.

Vì vậy:

class Boda { 
    boost::scoped_array<int> memory; 
    public: 
     Boda(int length) : memory(new int [length]) {} 
     ~Boda() {} 
}; 

Hơn nữa, scoped mảng không thể sao chép - vì vậy bạn tránh được những bản sao constructor vấn đề deallocation khó chịu đề cập trong câu trả lời khác.

Answer 2

Nếu bạn muốn bắt lỗi phân bổ bộ nhớ (mà bạn có thể nên) thì bạn sẽ phải thực hiện cuộc gọi đến mới trong phần thân của hàm tạo.

Answer 3

Tôi có thể nói cả hai đều tương đương với hiệu quả mà chúng tạo ra và cả hai đều là "đúng cách". Tôi thích danh sách khởi tạo nhưng tôi sẽ đi với biến thể thứ hai chỉ để có thể kiểm tra đối số độ dài không hợp lệ trước khi cố gắng cấp phát bộ nhớ.

Answer 4

Sự cố chung chung hơn. Xem C++ FAQ Lite: [10.6] Should my constructors use "initialization lists" or "assignment"?

Answer 5

Bạn nên sử dụng các lớp quản lý tài nguyên sẽ xử lý các lớp đó cho bạn. Khác, bạn gặp phải một số vấn đề nghiêm trọng với sự an toàn ngoại lệ, ngoài việc không cần sao chép logic hiện có và bảo trì các toán tử gán/sao chép.

Answer 6

memory biến thành viên là một con trỏ, nếu bạn phân bổ nó trong danh sách khởi và nó không thành công, lớp học của bạn không được khởi tạo và bạn không cần phải giải phóng nó sau này (nhờ mẫu thiết kế RAII được sử dụng bởi C++ để khởi tạo lớp). Nếu bạn cấp phát bộ nhớ của nó bên trong phần thân của hàm tạo, hành vi tương tự sẽ xảy ra.

Nhưng nếu bạn muốn xử lý điều gì đó, hãy phân bổ bộ nhớ của nó trong phần thân của hàm tạo. Kiểm tra một cái gì đó hoặc thử/bắt nó hoặc in một số thông điệp hữu ích, nhưng ít nhất, bạn phải ném một ngoại lệ, bởi vì khởi tạo lớp của bạn bị hỏng.

Tôi nghĩ rằng phân bổ memory trong phần thân của hàm tạo dễ đọc hơn so với phương thức khác.