[РЕШЕНО] Выход из приложения

[Urvin]

В конструкторе главной формы имею следующее:

Код

C++ (Qt)
MainWindow::MainWindow(QWidget *parent)
    : QMainWindow(parent), ui(new Ui::MainWindow)
{
    ui->setupUi(this);
 
    //...
    //...
 
    if (!openPort()) // собственная функция
    {
        QMessageBox::critical(this,"Port error.","Application will be terminated",QMessageBox::Ok);
 
        // Здесь должен быть выход из программы
 
    }
}

Никак не могу завершить программу после выскакивания сообщения об ошибке.. Команды close(), this->close(), qApp->quit()  не возымеют никакого влияния - все также появляется окно программы.

Где я неправ и как все же завершить работу программы?  

[BRE]

Никак не могу завершить программу после выскакивания сообщения об ошибке.. Команды close(), this->close(), qApp->quit()  не возымеют никакого влияния - все также появляется окно программы.

Где я неправ и как все же завершить работу программы? 

Когда отрабатывает конструктор MainWindow цикл обработки событий еще не запущен и окно еще не открыто. Поэтому ни close ни quit, эффекта не дают.

Может проще пытаться открывать порт в функции main и если открытие невозможно, то выходить из программы не создавая главного окна.

[Urvin]

блин.. косяк...
Может, в этом блоке выставлять некую переменную bool closeMePlease в true и закрывать откуда-то еще, но при старте программы?
Появится при этом главное окно, не появится - не важно

[BRE]

Может, в этом блоке выставлять некую переменную bool closeMePlease в true и закрывать откуда-то еще, но при старте программы?
Появится при этом главное окно, не появится - не важно

Да можно и так, еще можно исключение вызвать.
Только для чего лишние телодвижения? 

[Urvin]

сейчас голова работает откровенно плохо и что-то мне подсказывает, что перенести эту муть в main получится оооооой как не сразу =)

куды бы вставить проверку флага на закрытие?

[BRE]

сейчас голова работает откровенно плохо и что-то мне подсказывает, что перенести эту муть в main получится оооооой как не сразу =)

куды бы вставить проверку флага на закрытие?

В main, сразу за строкой создания MainWindow.

[Urvin]

То, что надо! Безмерное и безграничное спасибо!

[ритт]

что за сорт хоть?)

[Urvin]

Hefeweizen

[Alex Custov]

Код:

exit(0);

[sergeyvl12]

сейчас голова работает откровенно плохо и что-то мне подсказывает, что перенести эту муть в main получится оооооой как не сразу =)

куды бы вставить проверку флага на закрытие?

Флаг на закрытие? Некрасиво же! Исключение будет правильнее имхо.

[spectre71]

В конструкторе главной формы имею следующее:
...
...
Никак не могу завершить программу после выскакивания сообщения об ошибке.. Команды close(), this->close(), qApp->quit()  не возымеют никакого влияния - все также появляется окно программы.

Где я неправ и как все же завершить работу программы?  

exit(X), где:
int X != 0; поскольку 0 - нормальное завершение!
Соответственно, конкретное заначение X != 0 - тип ошибки в твоем приложении,  если лениво классифицировать ошибки делай exit(1)

[ритт]

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

из всех моих проектов на кутэ только в одном используется exit(X) - в дебаге при выбросе qCriticalType сообщения...

[sergeyvl12]

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

из всех моих проектов на кутэ только в одном используется exit(X) - в дебаге при выбросе qCriticalType сообщения...

Абсолютно согласен. Для лабораторки это приемлемо. Но для большого серьезного приложения это недопустимо. С флагом тоже некрасиво.

лучше всего так:

Код:

int main()
{

MyWindow window = NULL;
try // Initialization
{
....
    window = new MyWindow(...);
....

}
catch( const MyException& e )
{
    MessageBox(e.what());
    return 1;
}
catch(std::exception e)
{
    MessageBox(e.what());
    return 2;
}
catch(...)
{
    MessageBox("Unknown exception");
    return 3;
}

...
// Main loop
...

return 0;

}

Заодно в этом же месте перехватим и ошибки выделения памяти и длугие ошибки.

Теперь из конструктора нужно вызвать throw(MyException("text")); и все...

P.S. Вместо exit() в main() можно использовать return.

[BRE]

Абсолютно согласен. Для лабораторки это приемлемо. Но для большого серьезного приложения это недопустимо. С флагом тоже некрасиво.

А тебя только момент с флагом насторожил (для большого серьезного проекта)? 
А то что весь функционал работы с устройством находиться в классе MainWindow - это как?