如何返回 QList<specificObject*> 指针并且没有内存泄漏？

Question

我的数据库中有一个问题和答案表。这两者之间的关系是一个问题可以有多个答案。在我的 C++ 代码中，我有一个 Question 普通对象，它存储问题文本和答案的 QList。 Answer 普通对象只存储答案文本。

我想阅读与这些问题相关的所有问题和所有答案，并将它们存储在我的问题对象中。这就是我所做的。

QList<Question*> *ExamPersistance::readQuestions(int examId)
{
    QList<Question*> *questions;
    QSqlQuery *query = new QSqlQuery(connection->getDb());
    query->prepare("select q.id, q.text from question q where id_exam =:examId");
    query->bindValue(":examId",examId);
    query->exec();
    while(query->next())
    {
        questions->append(new Question(
                             query->value(1).toString(),
                                       this->readAnswers(query->value(0).toInt())));
    }
    return questions;
}

QList<Answer*> *ExamPersistance::readAnswers(int questionId)
{
    QList<Answer*> *answers;
    QSqlQuery *query = new QSqlQuery(connection->getDb());
    query->prepare("select a.text from answer a where id_question = :questionId");
    query->bindValue(":questionId",questionId);
    query->exec();
    while(query->next())
    {
        answers->append(new Answer(query->value(0).toString()));
    }
    return answers;
}

我遇到分段错误，我不知道为什么。

这些是我的头文件中的构造函数

 Answer(QString text);
 Question(QString name, QList<Answer*> *answers);

也许我的指针做错了。我不得不承认我对这些不是很熟悉，因为我来自 Java。

编辑：

我还有一个类“考试”，其中存储了某个考试的问题，但我认为这并不重要。只是你知道这个“int ExamId”代表什么

Answer 1

我怎样才能返回一个 QList 指针并且没有内存 泄漏？

最简单也可能是最佳的方法是存储值而不是指针：

QList<Question> listOfQuestions;
QList<Answer> listOfAnswers;

另外，我们可以创建共享指针的集合。但这不是 QSharedPointer 上所读内容的预期用途。

QList<QSharedPointer<Question>> listOfQuestions;
QList<QSharedPointer<Answer>> listOfAnswers;

否则，您需要弄清楚是哪个对象负责释放作为指针传递给列表的对象（非常流行的 Qt 技术，通过提供“父”指针）。

实践中的价值观列表：

QList<Question> ExamPersistance::readQuestions(int examId)
{
    QList<Question> questions;
    QSqlQuery query(connection->getDb());
    query.prepare("select q.id, q.text from question q where id_exam =:examId");
    query.bindValue(":examId",examId);
    query.exec();
    while(query.next())
    {
        questions.append(Question(
                             query.value(1).toString(),
                                       this->readAnswers(query.value(0).toInt())));
    }
    return questions;

}

QList<Answer> ExamPersistance::readAnswers(int questionId)
{
    QList<Answer> answers;
    QSqlQuery query(connection->getDb());
    query.prepare("select a.text from answer a where id_question = :questionId");
    query.bindValue(":questionId",questionId);
    query.exec();
    while(query.next())
    {
        answers.append(Answer(query.value(0).toString()));
    }
    return answers;
}

当然要注意从您的问题中看不到的实际构造函数参数。使用 C++ 11 和“移动”构造函数应该非常有效。

Answer 2

你忘记初始化一些指针。

修复了崩溃和内存泄漏的版本：

QList<std::unique_ptr<Question>> ExamPersistance::readQuestions(int examId)
{
    QList<std::unique_ptr<Question>> questions;
    QSqlQuery query(connection->getDb());
    query.prepare("select q.id, q.text from question q where id_exam =:examId");
    query.bindValue(":examId",examId);
    query.exec();
    while(query.next())
    {
        questions.append(std::make_unique<Question>(
                             query.value(1).toString(),
                                       this->readAnswers(query.value(0).toInt())));
    }
    return questions;
}

Answer 3

您正在取消引用 QList 指针，但您从未初始化它们（即它们指向垃圾）。这段代码应该可以解决这个问题。

QList<Question*> *ExamPersistance::readQuestions(int examId)
{
    QList<Question*> *questions = new QList<Question*>();
    QSqlQuery *query = new QSqlQuery(connection->getDb());
    query->prepare("select q.id, q.text from question q where id_exam =:examId");
    query->bindValue(":examId",examId);
    query->exec();
    while(query->next())
    {
        questions->append(new Question(
                             query->value(1).toString(),
                                       this->readAnswers(query->value(0).toInt())));
        }
        return questions;

    }

QList<Answer*> *ExamPersistance::readAnswers(int questionId)
{
    QList<Answer*> *answers = new QList<Answer*>();
    QSqlQuery *query = new QSqlQuery(connection->getDb());
    query->prepare("select a.text from answer a where id_question = :questionId");
    query->bindValue(":questionId",questionId);
    query->exec();
    while(query->next())
    {
        answers->append(new Answer(query->value(0).toString()));
    }
   return answers;

}

请注意，这只是我发现的问题。即使此代码运行，它也存在其他问题。您肯定会泄漏与变量查询关联的内存，并且可能会将内存泄漏到所有问题和答案对象（以及我们使用它时的 QLists），除非您稍后在某个时间点删除它们。在 C++ 中，每次调用 new 时都需要调用 delete，否则会泄漏内存。除非您确实需要，否则在 C++ 中通常不赞成使用指针。这与必须使用 new 来创建对象的 Java 大不相同。这个answer 对 C++ 和 Java 中 new 运算符的区别进行了很好的讨论，是一个很好的起点。