关于类内部指针的问题

Question

考虑这个类：

class Packet {
public:
  Packet();
  ~Packet();
  void allocateBuffer(unsigned int size);
  void deallocateBuffer();
  char* getBuffer() const;
private:
  char* buffer;
};

使用以下方法：

Packet::Packet():
  buffer(NULL) {
  std::cout << "[DEBUG] Construct Packet class..." << std::endl;
}

void Packet::allocateBuffer(unsigned int size) {
  if (!(buffer)) {
    buffer = new char[size];
#ifdef DEBUG
    std::cout << "Allocate buffer memory..." << std::endl;
#endif
  }
}

void Packet::deallocateBuffer() {
  if (buffer) {
   delete[] buffer;
   buffer = NULL;
#ifdef DEBUG
    std::cout << "Deallocate buffer memory..." << std::endl;
#endif
  }
}

这里有一些问题：

1. 因为在 C 中指针等于 NULL 如果它们不指向任何内容，那么上面列出的实现是处理类内部指针的好方法吗？我要求这样做是因为如果 buffer 变量未在构造函数中初始化，则默认情况下似乎会为其分配一个奇怪的 \v 值。
2. 如果没有，您能提出更优雅的方法吗？

Answer 1

1. 因为在 C 中，如果指针不指向任何内容，它们就等于 NULL，那么上面列出的实现是处理类内部指针的好方法吗？我要求这样做是因为如果缓冲区变量未在构造函数中初始化，默认情况下似乎会为其分配一个奇怪的 \v 值。

您应该始终注意类构造函数中非静态成员变量的初始化，除非它们是非原始类型，并且它们自己负责它们的初始化。
否则访问它们会暴露未定义的行为。

您应该注意，非静态类成员变量没有默认初始化。

同样对于 c++，您应该使用 nullptr 而不是 NULL 来初始化原始指针。

1. 如果没有，您能提出更优雅的方法吗？

当然，只需使用std::vector<char> 成员变量，它会自动为您完成所有分配/释放内存管理。

要通过原始指针访问原始数据，请使用std::vector::data() 函数重载。

正如您在 cmets 中提到的，您必须处理 TCP/IP 低级 API¹，我建议您使用 std::vector<uint8_t> 进行缓冲，因为您无法确定只会传输有效的签名char 值。

---

¹⁾_{在处理通过网络发送的数据时，请注意机器字节序中立性，尤其是在发送和接收数据包大小时。 htonx() / ntohx() 函数族可以派上用场正确地做到这一点。}