Qt: Вылетает программа при закрытии главного окна

[FluffyMan2000]

Доброго времени суток.

Делаю лабораторную работу на тему "многодокументный интерфейс". писать за меня не прошу. просто подскажите в чем ошибка.

Имеется QMainWindow с QMdiArea в качестве centralWodget. В QMdiArea создаются окна (объекты, наследники QTextEdit со своими слотами "сохранить", "сохранить как"). Реализовал вызов диалогового окна при закрытии документа и при закрытии главного окна что-то вроде "документ не сохранен. сохранить?".

Проблема в следующем:
1)если создаю несколько окон, и закрываю один за одним, то вылетает программа при закрытии последнего дочернего окна. То есть если создам одно окно и его закрою - вылетает с ошибкой в консоль "программа неожиданно завершилась"
2) если закрываю главное окно и в нем имеются дочерние - тоже вылетает с ошибкой той же самой.

Проект во вложении. Кому не лень, посмотрите пожалуйста. у меня уже мозг взрывается.

[V1KT0P]

Как минимум у тебя в функции:

Код

C++ (Qt)
void MainWindow::checkForActionsAvailable() 

нету проверки:

Код

C++ (Qt)
ui->mdiArea->isActiveWindow()

Также при каждом касте qobject_cast надо проверять не возвращает ли он ноль.
Как по мне так у тебя проблемы с архитектурой из-за плохого понимания Qt.

[FluffyMan2000]

Также при каждом касте qobject_cast надо проверять не возвращает ли он ноль.
Как по мне так у тебя проблемы с архитектурой из-за плохого понимания Qt.

в чем непонимание? это (использование кастов) вообще в примерах Макса Шлее написано.

[V1KT0P]

в чем непонимание? это (использование кастов) вообще в примерах Макса Шлее написано.

qobject_cast это тебе не static_cast и может спокойно вернуть ноль и вот тут начнется веселье. Особенно если программа большая, а ошибка достаточно редкая.

[FluffyMan2000]

понял.

устроил проверку на ноль при сохранении в слотах delegateForSave и delegateForSaveAs - возвращает не ноль. но ошибка та же.

еще заметил, что если создать два документа и нажать в тулбаре сохранить, то диалог сохранения вылезет два раза.

[Mikhail]

Ты используешь QMdiArea и не прочитав описания класса подсовываешь ему не наследников QMdiSubWindow, а какую то "дрянь" наследников QTextEdit.

Из документации:
Subwindows in QMdiArea are instances of QMdiSubWindow. They are added to an MDI area with addSubWindow(). It is common to pass a QWidget, which is set as the internal widget, to this function, but it is also possible to pass a QMdiSubWindow directly.The class inherits QWidget, and you can use the same API as with a normal top-level window when programming. QMdiSubWindow also has behavior that is specific to MDI windows. See the QMdiSubWindow class description for more details.

[V1KT0P]

понял.
устроил проверку на ноль при сохранении в слотах delegateForSave и delegateForSaveAs - возвращает не ноль. но ошибка та же.
еще заметил, что если создать два документа и нажать в тулбаре сохранить, то диалог сохранения вылезет два раза.

Я же тебе уже указал на эту ошибку. Вот исправь для начала функцию "void MainWindow::checkForActionsAvailable()" на вот это, а затем уже выясняй что ты делаешь не так:

Код

C++ (Qt)
void MainWindow::checkForActionsAvailable()
{
    if( !ui->mdiArea->isActiveWindow() )
    {
        qDebug() << "Error: isActiveWindow() is false";
        return;
    }
    DocWindow *widget = qobject_cast<DocWindow*>( ui->mdiArea->activeSubWindow()->widget() );
    if( !widget )
    {
        qDebug() << "Error: activeSubWindow() is not DocWindow";
        return;
    }
    ui->actionUndo->setEnabled( widget->document()->isUndoAvailable() );
    ui->actionRedo->setEnabled( widget->document()->isRedoAvailable() );
    ui->actionPaste->setEnabled( widget->canPaste() );
}

[FluffyMan2000]

понял.
устроил проверку на ноль при сохранении в слотах delegateForSave и delegateForSaveAs - возвращает не ноль. но ошибка та же.
еще заметил, что если создать два документа и нажать в тулбаре сохранить, то диалог сохранения вылезет два раза.

Я же тебе уже указал на эту ошибку. Вот исправь для начала функцию "void MainWindow::checkForActionsAvailable()" на вот это, а затем уже выясняй что ты делаешь не так:

Код

C++ (Qt)
void MainWindow::checkForActionsAvailable()
{
    if( !ui->mdiArea->isActiveWindow() )
    {
        qDebug() << "Error: isActiveWindow() is false";
        return;
    }
    DocWindow *widget = qobject_cast<DocWindow*>( ui->mdiArea->activeSubWindow()->widget() );
    if( !widget )
    {
        qDebug() << "Error: activeSubWindow() is not DocWindow";
        return;
    }
    ui->actionUndo->setEnabled( widget->document()->isUndoAvailable() );
    ui->actionRedo->setEnabled( widget->document()->isRedoAvailable() );
    ui->actionPaste->setEnabled( widget->canPaste() );
}

Объясните, зачем проверять mdiArea на isActiveWindow()? и еще, программа крашится и без вызова MainWindow::checkForActionsAvailable(). так что скорее всего не в ней дело.

[V1KT0P]

Объясните, зачем проверять mdiArea на isActiveWindow()? и еще, программа крашится и без вызова MainWindow::checkForActionsAvailable(). так что скорее всего не в ней дело.

По крайней мере у меня под Kubuntu в процессе работы твоей программы ui->mdiArea->activeSubWindow() возвращает ноль. И вот из-за этого программа падает.

[FluffyMan2000]

Ты используешь QMdiArea и не прочитав описания класса подсовываешь ему не наследников QMdiSubWindow, а какую то "дрянь" наследников QTextEdit.

Из документации:
Subwindows in QMdiArea are instances of QMdiSubWindow. They are added to an MDI area with addSubWindow(). It is common to pass a QWidget, which is set as the internal widget, to this function, but it is also possible to pass a QMdiSubWindow directly.The class inherits QWidget, and you can use the same API as with a normal top-level window when programming. QMdiSubWindow also has behavior that is specific to MDI windows. See the QMdiSubWindow class description for more details.

в примерах от Макса Шлее именно так и делается

[FluffyMan2000]

Объясните, зачем проверять mdiArea на isActiveWindow()? и еще, программа крашится и без вызова MainWindow::checkForActionsAvailable(). так что скорее всего не в ней дело.

По крайней мере у меня под Kubuntu в процессе работы твоей программы ui->mdiArea->activeSubWindow() возвращает ноль. И вот из-за этого программа падает.

а с твоим кодом слота MainWindow::checkForActionsAvailable() не крашится при закрытии программы? и еще нюанс, у тебя Qt установлена еще нокиевская или диджитевская? потому что у меня уже был казус - программа собранная на нокиевской кьют некоректно работала на диждитевской кьют

[V1KT0P]

а с твоим кодом слота MainWindow::checkForActionsAvailable() не крашится при закрытии программы? и еще нюанс, у тебя Qt установлена еще нокиевская или диджитевская? потому что у меня уже был казус - программа собранная на нокиевской кьют некоректно работала на диждитевской кьют

Да, не крашится. Сборка у меня из репозитория убунты 4.8.3.
Запусти под отладчиком и посмотри бэктрейс при падении, что может быть проще?

[FluffyMan2000]

а с твоим кодом слота MainWindow::checkForActionsAvailable() не крашится при закрытии программы? и еще нюанс, у тебя Qt установлена еще нокиевская или диджитевская? потому что у меня уже был казус - программа собранная на нокиевской кьют некоректно работала на диждитевской кьют

Да, не крашится. Сборка у меня из репозитория убунты 4.8.3.
Запусти под отладчиком и посмотри бэктрейс при падении, что может быть проще?

давайте по-порядку: ты содаешь два документа, жмешь крестик в главном окне, вылазит дилаог. жмешь да. вылазят два диалога. жмешь дискард и у тебя программа не крашится? у меня крашится.

Qt 4.7.4 (32-х битная)

[FluffyMan2000]

при запуске в режиме отладке в консоль вывалилось "Error: isActiveWindow() is false" и крашится все равно. спасибо за направление. буду дальше думать

[FluffyMan2000]

Решил таким образом:

Код:

void MainWindow::checkForActionsAvailable()
{
    if (ui->mdiArea->subWindowList().count() != 0)
    {
        DocWindow *pDoc = qobject_cast<DocWindow*>(ui->mdiArea->activeSubWindow()->widget());

        if (pDoc != 0)
        {
            ui->actionUndo->setEnabled(pDoc->document()->isUndoAvailable());
            ui->actionRedo->setEnabled(pDoc->document()->isRedoAvailable());
            ui->actionPaste->setEnabled(pDoc->canPaste());
        }
    }
}

проблема была в том, что я приводил к DocWindow несуществующий объект, так как он закрывался почему-то (до сих пор не понял почему он уничтожался раньше времени)