Чтение нарушения прав доступа при создании QDomElement во второй раз

Я написал класс для обработки XML-файла. Класс загружает файл при построении и сохраняет его при уничтожении. Пока класс активен, у меня есть несколько функций получения и установки для изменения значений. Одна из них — функция регистрации некоторых переименований файлов. При каждом вызове он будет создавать новый дочерний элемент для elem_renames.

void DataElementHandle::renamed(QString new_name, QString old_name)
{
    QDomElement elem_ren = xml_doc.createElement("renamed");
    QDomAttr att = xml_doc.createAttribute("time");
    att.setValue(QDateTime::currentDateTime().toString(Qt::ISODate));
    elem_ren.setAttributeNode(att);

    QDomText t = xml_doc.createTextNode(old_name + " -> " + new_name);
    elem_ren.appendChild(t);

    elem_renames.appendChild(elem_ren);
}

Проблема. Теперь я создаю класс DataElementHandle и вызываю функцию переименования для каждого изменения в файлах. Но каждый второй раз, когда я вызываю функцию, моя программа вылетает с этим сообщением об ошибке:

Возникло исключение: нарушение доступа для чтения. Ошибка выдается в первой строке функции.

Я не понимаю, почему это происходит. Я думаю, что не могу переопределить QDomElement, потому что все еще есть ссылка на элемент, созданный при первом вызове. Но как? Он должен исчезнуть в конце функции.

Я использую Qt 5.8 с Visual Studio 2015 и Visual Leak Detector.

Заголовочный файл:

// Version
const quint32 version = 1;

// Doc file path
QString file_path;
bool load_file_ok;

// Doc
QDomDocument xml_doc;
QDomElement root;

// First root elements
QDomElement elem_renames;

Если файл XML не существует, новый шаблон создается таким образом и будет сохранен в файле в деструкторе.

xml_doc = QDomDocument("data_xml");
root = xml_doc.createElement("root");
root.setAttribute("version", QString::number(version));
xml_doc.appendChild(root);

elem_renames= xml_doc.createElement("renames");

root.appendChild(elem_renames);

Редактировать 1: я настроил тестовый проект, и он работает нормально. Я должен изучить проблему более подробно.


c++ qt qdomdocument
person Darkproduct    schedule 20.11.2017    source источник

Ответы (1)


arrow_upward
2
arrow_downward

Решено! Я такой глупый. У меня была такая установка. И я объявил и инициализировал указатель на класс XML в цикле for. И поскольку я удаляю старый и создаю новый класс в операторе if ниже, я инициализировал указатель в операторе if в первый раз, но затем удалил его во второй раз.

#include <QCoreApplication>
#include <QDomDocument>
#include <QFile>
#include <QDateTime>
#include <QString>
#include <QDebug>

class TestDomClass
{
public:
    TestDomClass(QString file_path) :
        path(file_path)
    {
        xml_doc = QDomDocument("data_xml");
        root = xml_doc.createElement("root");
        root.setAttribute("version", QString::number(version));
        xml_doc.appendChild(root);

        elem_renames= xml_doc.createElement("renames");

        root.appendChild(elem_renames);
    }
    ~TestDomClass()
    {
        QFile file(path);
        if(!file.open(QIODevice::WriteOnly | QIODevice::Text))
        {
            // TODO
            qDebug() << "Failed to open file for writing!";
        }
        else
        {
            QTextStream stream(&file);
            stream << xml_doc.toString();
        }

        if(file.isOpen())
        {
            file.close();
        }
    }

    void renamed(QString new_name, QString old_name)
    {
        QDomElement elem_ren = xml_doc.createElement("renamed");
        QDomAttr att = xml_doc.createAttribute("time");
        att.setValue(QDateTime::currentDateTime().toString(Qt::ISODate));
        elem_ren.setAttributeNode(att);

        QDomText t = xml_doc.createTextNode(old_name + " -> " + new_name);
        elem_ren.appendChild(t);

        elem_renames.appendChild(elem_ren);
    }

private:
    QString path;

    // Version
    const quint32 version = 1;

    // Doc
    QDomDocument xml_doc;
    QDomElement root;

    // First root elements
    QDomElement elem_renames;
};

int main(int argc, char *argv[])
{
    QCoreApplication a(argc, argv);

    TestDomClass *t = Q_NULLPTR; // <-- This line was in the for loop

    for(int a = 0; a < 10; ++a)
    {
        delete t;
        t = new TestDomClass("test_" + QString::number(a) + ".xml");
        for(int i = 0; i < 10; ++i)
        {
            qDebug() << "Round: " << i;
            t->renamed(QString::number(10*a + i), QString::number(10*a + 10-i));
        }
    }

    delete t;

    return a.exec();
}
person Darkproduct    schedule 21.11.2017