Question

我创建了一个自定义QLineEdit小部件来处理文件的拖放。这一部分的一切都很好用，但是只要我添加一个Class变量，应用程序就会在类的构造或破坏时崩溃：

当我的变量是QString时，它在构造时崩溃
当我的变量是QString *时，它在销毁

我尝试在析构函数中删除变量（QString *），同样的问题......

有什么想法吗？

部首：

#ifndef DROPLINEEDIT_H
#define DROPLINEEDIT_H

#include <QLineEdit>
#include <QDragEnterEvent>
#include <QMimeData>
#include <QFileInfo>
#include <QString>
#include <QDebug>


class DropLineEdit : public QLineEdit
{
    Q_OBJECT
public:
    explicit DropLineEdit(QWidget *parent = 0);
    ~DropLineEdit();

protected:
    virtual void dragEnterEvent(QDragEnterEvent *event);
    virtual void dropEvent(QDropEvent *event);
    virtual void dragLeaveEvent(QDragLeaveEvent *event);

signals:

public slots:

private:

    QString * mFileName;

};

#endif // DROPLINEEDIT_H

来源：

#include "droplineedit.h"

DropLineEdit::DropLineEdit(QWidget *parent) :
    QLineEdit(parent)
{
    setAcceptDrops(true);
    this->setReadOnly(true);
    this->setStyleSheet("QLineEdit { border: 2px solid gray ; border-radius: 8px ; padding: 0 6px }");
}

DropLineEdit::~DropLineEdit()
{
    if(!mFileName){
        delete mFileName;
    }
}

// **************************************** PROTECTED METHODS **************************************** //

void DropLineEdit::dragEnterEvent(QDragEnterEvent *event){
    this->setStyleSheet("QLineEdit { border: 3px solid black ; border-radius: 8px ; padding: 0 6px }");
    event->accept();
}
void DropLineEdit::dragLeaveEvent(QDragLeaveEvent *event){
    this->setStyleSheet("QLineEdit { border: 2px solid gray ; border-radius: 8px ; padding: 0 6px }");
    event->accept();
}

void DropLineEdit::dropEvent(QDropEvent *event){
    // Get the data. If multiple files are dropped, take only the first one and fetch save its info
    QList<QUrl> list = event->mimeData()->urls();
    QFileInfo * fileInfo = new QFileInfo(list.at(0).toLocalFile());

    qDebug() << fileInfo->absoluteFilePath();
    mFileName = new QString(fileInfo->absoluteFilePath());

    this->setText(fileInfo->fileName());
    this->setStyleSheet("QLineEdit { border: 2px solid gray ; border-radius: 8px ; padding: 0 6px }");
    event->accept();
}

Answer 1

将其添加到构造函数：

mFileName = 0;

您需要初始化指针。否则它将具有随机值，您将无法检查是否已创建对象。

修改析构函数：

delete mFileName;

如果对象不是NULL（您的版本相反），则需要删除它。 delete将在内部执行检查。

请注意，如果多次执行mFileName = new QString...，则会创建多个对象。如果需要避免内存泄漏，则需要在创建新对象之前删除先前的对象。

但是，所有上述信息都是为普通教育提供的。你不应该在这里使用QString*。非指针QString类成员将更加正确。在这种情况下，您不需要使用new或delete，您根本不需要关心指针和内存。

Answer 2

我在调试中将此视为错误：“指定给rtlfreeheap的无效地址”

我对此做了一些研究，发现它可能与Qt的内部问题（调用对象构造函数的方式）有关。我在我的主应用程序中回过头来看看我是如何调用我的DropLineEdit对象的，并注意到我在创建DropLineEdit Classe之后忘记删除setAcceptDrops(true)（在我创建DropLineEdit之前，该对象以前是正常的QLineEdit）。

这是函数调用：

DropLineEdit* Instrument::createDropLineEdit(){
    DropLineEdit * lineEdit = new DropLineEdit();
    lineEdit->setAcceptDrops(true);
    return lineEdit;
}

更改为：

DropLineEdit* Instrument::createDropLineEdit(){
    DropLineEdit * lineEdit = new DropLineEdit();
    return lineEdit;
}

删除冗余调用（冗余，因为在构造函数中设置了acceptdrops）后，我没有使用QString或QString *作为DropLineEdit类的成员变量导致任何应用程序崩溃。我使用了一个QString（而不是指针），我能够让我的Class做到应该做的事。

感谢您的回复。非常感谢。

Qt5自定义LineEdit小部件（QLineEdit子类）私有变量崩溃应用程序

2 个答案: