#c #qt #segmentation-fault #qt5
#c #qt #ошибка сегментации #qt5
Вопрос:
Мой проект иногда выходит из строя при закрытии, но ненадежно. При закрытии с помощью кнопки «Отмена» он выходит из строя гораздо реже, чем при закрытии с помощью клавиши escape, что еще больше меня смущает. У меня есть простой диалог поиска (небольшая часть более крупного проекта) в виде тестового примера.
Попытки отладки с помощью GDB приводят к тому, что он никогда не выходит из строя. Дамп ядра — это мусор (плохая память).
Ниже приведен исходный код. Я не смог точно определить, что на самом деле происходит не так; Я видел (с небольшими изменениями) double free, освобождение NULL, освобождение указателя, который никогда не существовал, и т.д., И я начинаю задаваться вопросом, не является ли это частично ошибкой Qt.
Заголовок:
#ifndef NIDE_FINDDIALOG_HPP
#define NIDE_FINDDIALOG_HPP
#include <QDialog>
#include <QString>
#include <QGridLayout>
#include <QCheckBox>
#include <QLineEdit>
#include <QPushButton>
#include <QWidget>
class FindDialog: public QDialog {
public:
bool regex, caseSensitive, inSelection, wholeWord, forward, wrap;
QString expr;
private:
struct {
QGridLayout *layout;
QCheckBox *regex, *caseSense, *select, *whole, *forward, *wrap;
QLineEdit *exprBox;
QPushButton *cancel, *find;
}ui;
void onCancel();
void onFind();
void onRegex();
void onCase();
void onSelect();
void onWhole();
void onForward();
void onWrap();
public:
FindDialog(QWidget *parent);
~FindDialog();
};
#endif/*NIDE_FINDDIALOG_HPP*/
Источник:
#include <NIDE/FindDialog.hpp>
FindDialog::FindDialog(QWidget *parent): QDialog(parent),
regex(false), caseSensitive(true), inSelection(false), wholeWord(true),
forward(true), wrap(true), expr() {
ui.layout = new QGridLayout(this);
ui.regex = new QCheckBox("Regex", this);
ui.caseSense = new QCheckBox("Match case", this);
ui.select = new QCheckBox("In Selection", this);
ui.whole = new QCheckBox("Whole Word", this);
ui.forward = new QCheckBox("Forward", this);
ui.wrap = new QCheckBox("Wrap around", this);
ui.exprBox = new QLineEdit(this);
ui.cancel = new QPushButton("Cancel", this);
ui.find = new QPushButton("Find", this);
ui.regex->setChecked(regex);
connect(ui.regex, amp;QCheckBox::stateChanged, this, amp;FindDialog::onRegex);
ui.caseSense->setChecked(caseSensitive);
connect(ui.caseSense, amp;QCheckBox::stateChanged, this, amp;FindDialog::onCase);
ui.select->setChecked(inSelection);
connect(ui.select, amp;QCheckBox::stateChanged, this, amp;FindDialog::onSelect);
ui.whole->setChecked(wholeWord);
connect(ui.whole, amp;QCheckBox::stateChanged, this, amp;FindDialog::onWhole);
ui.forward->setChecked(forward);
connect(ui.forward, amp;QCheckBox::stateChanged, this, amp;FindDialog::onForward);
ui.wrap->setChecked(wrap);
connect(ui.wrap, amp;QCheckBox::stateChanged, this, amp;FindDialog::onWrap);
ui.exprBox->setPlaceholderText(tr("Find expr..."));
connect(ui.exprBox, amp;QLineEdit::returnPressed, this, amp;FindDialog::onFind);
connect(ui.cancel, amp;QPushButton::clicked, this, amp;FindDialog::onCancel);
ui.find->setDefault(true);
connect(ui.find, amp;QPushButton::clicked, this, amp;FindDialog::onFind);
ui.layout->addWidget(ui.regex, 0, 0);
ui.layout->addWidget(ui.caseSense, 0, 1);
ui.layout->addWidget(ui.select, 0, 2);
ui.layout->addWidget(ui.whole, 1, 0);
ui.layout->addWidget(ui.forward, 1, 1);
ui.layout->addWidget(ui.wrap, 1, 2);
ui.layout->addWidget(ui.exprBox, 2, 0, 1, 3);
ui.layout->addWidget(ui.cancel, 3, 1);
ui.layout->addWidget(ui.find, 3, 2);
setLayout(ui.layout);
setWindowTitle(tr("Find"));
}
FindDialog::~FindDialog() {
delete ui.layout;
delete ui.regex;
delete ui.caseSense;
delete ui.select;
delete ui.whole;
delete ui.forward;
delete ui.wrap;
delete ui.exprBox;
delete ui.cancel;
delete ui.find;
}
void FindDialog::onCancel() {
done(QDialog::Rejected);
}
void FindDialog::onFind() {
expr = ui.exprBox->text();
done(QDialog::Accepted);
}
void FindDialog::onRegex() { regex = ui.regex->isChecked(); }
void FindDialog::onCase() { caseSensitive = ui.caseSense->isChecked(); }
void FindDialog::onSelect() { inSelection = ui.select->isChecked(); }
void FindDialog::onWhole() { wholeWord = ui.whole->isChecked(); }
void FindDialog::onForward() { forward = ui.forward->isChecked(); }
void FindDialog::onWrap() { wrap = ui.wrap->isChecked(); }
Главная:
#include <NIDE/FindDialog.hpp>
#include <QApplication>
int main(int argc, char **argv) {
QApplication *app = new QApplication(argc, argv);
FindDialog fd(NULL);
app->setApplicationName("NIDE");
fd.exec();
}
Комментарии:
1. попробуйте выполнить удаление деструктора в порядке, обратном порядку создания
2. В вашем dtor не должно быть никаких
delete
. Действительно, вы установили родителей для своего QObject (this
которые вы передали в каждом ctor), и они будут обрабатывать освобождение памяти своего дочернего объекта.3. Кроме того, в настоящее время нет необходимости в динамически выделяемом
QApplication
в вашем основном. Просто создайте статически выделенную переменную. И попробуйтеfd.exec()
также заменить наapp->exec(); fd.show();
в основном. Я не могу сказать, что это решит вашу проблему на 100%, но есть большие шансы, что это произойдет.4. Кроме того, обычный способ подключения сигнала / слота — это сделать
connect(amp;sender, SIGNAL(sender_signal(param_type)), amp;receiver, SLOT(receiver_slot(param_type)));
. Могу ли я предложить найти действительно хороший учебник по созданию приложений Qt, например, этот учебник о небольшом текстовом редакторе , который поможет вам понять основы Qt.5. @JBL Я добавил удаления только после того, как заметил, что он сбой. Изменение статического распределения QApplication и вызов fd.show(), а затем app.exec() полностью решили проблему (или заставили ее происходить намного реже; еще не видел, чтобы она зависала). Я специально избегаю традиционного Qt-сигнала / слота, поскольку я пытаюсь избежать использования Q_OBJECT Qt «языкового расширения».
Ответ №1:
Избегая использования Q_OBJECT
макроса, вы полагаетесь на детали реализации Qt. Подключение к методам, которые были объявлены в классе без, является неопределенным поведением Q_OBJECT
. Тот факт, что он работает в определенной версии Qt, является счастливым совпадением.
Увы, ваш код делает много ненужных вещей.
-
Нет необходимости хранить элементы пользовательского интерфейса или
QApplication
экземпляр в куче. -
Для всех методов нет причин
onXxxx
, поскольку вы не будете читать состояние диалогового окна во время его просмотраexec()
. Вы заботитесь о состоянии диалогового окна только после того, как оно было принято (или, возможно, отклонено). -
Вы можете использовать
QDialog::accept
иQDialog::reject
слоты. -
Чтобы диалог выглядел корректно на нескольких платформах, вы должны использовать a
QDialogButtonBox
вместо отдельных кнопок. -
Наконец, using
QDialog::exec()
повторно входит в цикл событий и заставляет вас думать, что ваш код синхронен, хотя на самом деле это не так. Это источник труднодоступных ошибок. Вы должны простоshow()
открыть диалоговое окно и отреагировать наaccepted
сигнал, который он выдает при нажатии кнопки поиска.
Приведенный ниже код пытается быть разумно правильным решением. Он работает как в Qt 4, так и в Qt 5.
// Interface
#include <QDialog>
#include <QGridLayout>
#include <QCheckBox>
#include <QLineEdit>
#include <QDialogButtonBox>
#if QT_VERSION<QT_VERSION_CHECK(5,0,0)
#define Q_DECL_OVERRIDE
#endif
class FindDialog: public QDialog {
struct Ui {
QGridLayout layout;
QCheckBox regex, caseSense, select, whole, forward, wrap;
QLineEdit exprBox;
QDialogButtonBox buttonBox;
Ui(QWidget * widget);
} m_ui;
void get();
public:
bool regex, caseSensitive, inSelection, wholeWord, forward, wrap;
QString expr;
FindDialog(QWidget *parent = 0);
~FindDialog();
void set();
void done(int r) Q_DECL_OVERRIDE;
};
// Implementation
FindDialog::Ui::Ui(QWidget * widget) :
layout(widget),
regex(tr("Regex")),
caseSense(tr("Match case")),
select(tr("In Selection")),
whole(tr("Whole Word")),
forward(tr("Forward")),
wrap(tr("Wrap around")),
buttonBox(QDialogButtonBox::Cancel)
{
layout.addWidget(amp;regex, 0, 0);
layout.addWidget(amp;caseSense, 0, 1);
layout.addWidget(amp;select, 0, 2);
layout.addWidget(amp;whole, 1, 0);
layout.addWidget(amp;forward, 1, 1);
layout.addWidget(amp;wrap, 1, 2);
layout.addWidget(amp;exprBox, 2, 0, 1, 3);
layout.addWidget(amp;buttonBox, 3, 0, 1, 3);
exprBox.setPlaceholderText(tr("Find expr..."));
buttonBox.addButton(tr("Find"), QDialogButtonBox::AcceptRole);
}
FindDialog::FindDialog(QWidget *parent): QDialog(parent), m_ui(this),
regex(false), caseSensitive(true), inSelection(false), wholeWord(true),
forward(true), wrap(true)
{
set();
connect(amp;m_ui.buttonBox, SIGNAL(rejected()), SLOT(reject()));
connect(amp;m_ui.buttonBox, SIGNAL(accepted()), SLOT(accept()));
setWindowTitle("Find");
}
FindDialog::~FindDialog() {}
void FindDialog::done(int result)
{
get();
QDialog::done(result);
}
void FindDialog::get()
{
regex = m_ui.regex.isChecked();
caseSensitive = m_ui.caseSense.isChecked();
inSelection = m_ui.select.isChecked();
wholeWord = m_ui.whole.isChecked();
forward = m_ui.forward.isChecked();
wrap = m_ui.wrap.isChecked();
expr = m_ui.exprBox.text();
}
void FindDialog::set()
{
m_ui.regex.setChecked(regex);
m_ui.caseSense.setChecked(caseSensitive);
m_ui.select.setChecked(inSelection);
m_ui.whole.setChecked(wholeWord);
m_ui.forward.setChecked(forward);
m_ui.wrap.setChecked(wrap);
}
// Main
#include <QApplication>
#include <QMessageBox>
int main(int argc, char **argv) {
QApplication app(argc, argv);
app.setApplicationName("NIDE");
FindDialog fd;
fd.show();
#if QT_VERSION>=QT_VERSION_CHECK(5,0,0)
// This can't be done in Qt4 without using moc
QObject::connect(amp;fd, amp;QDialog::accepted, [amp;fd]{
QMessageBox::information(NULL, "Find",
QString("The user wants to find "%1"").arg(fd.expr));
});
#endif
return app.exec();
}