c++ - 类声明中的默认内存分配是否被认为是一种好的做法？

Question

我有一个类，其成员分配在堆上

// client.h

#include <QString>
#include <QTcpSocket>
#include <QTextEdit>
#include <QLineEdit>
#include <QWidget>

class Client : public QWidget
{
    Q_OBJECT

public:
    Client(const QString &computerName, int portNumber,
           QWidget *parent = nullptr);

    ~Client() = default;

// ...

private:
    QTcpSocket *_tcpSocket = new QTcpSocket(this);
    QTextEdit *_textInfo = new QTextEdit;
    QLineEdit *_textInput = new QLineEdit;
    quint16 _nextBlockSize = 0;
};

// client.cpp

Client::Client(const QString &computerName, int portNumber, QWidget *parent)
    : QWidget(parent)
{
    // ...
}

我从未见过以这种方式完成默认初始化 - 无论是在 ctor 中完成还是在其初始化部分的最佳情况下完成(例如 _tcpSocket(new QTcpSocket(this)))。我可以想到多种反对它的理由:

• 因为我们使用 new 调用我们分配的对象的构造函数，所以我们现在不能使用前向声明
• 在类定义中看到这些分配可能会建议用户该类有一个非平凡的 dtor，并且必须释放对象，而实际上它们没有，因为 Qt 将自行管理它们
• 在 _tcpSocket = new QTcpSocket(this) 中看到 this 对我来说很奇怪

那么在堆上默认分配对象的最佳方式是什么？

Answer 1

分配内存的默认成员初始化器并不是一个坏习惯。

<子>但是，不在析构函数中删除此类动态分配是一种不好的做法。首先拥有裸指针是一种不好的做法。此外，不必要的动态分配是一种不好的做法。 但这显然是 Qt 设计的问题。

we can't use forward declaration now

当然可以。但我认为你的意思是提前声明是不够的。那是个很好的观点。如果您需要避免定义这些成员类型，那么在这种情况下默认成员初始化器不是一个选项。