Глобальный обработчик исключений

[mejuhafr]

Есть ли аналог TApplication.OnException ?

[kambala]

заверни содержимое main() в try-catch

[mejuhafr]

Чукча не читатель проверятель, чукча писатель?

Код

C++ (Qt)
#include <QtGui/QApplication>
#include "mainwindow.h"
 
int main(int argc, char *argv[])
{
    try
    {
        QApplication a(argc, argv);
        MainWindow w;
        w.show();
        return a.exec();
    }
    catch ( ... )
    {
        qDebug( "Eception" );
    }
}

Код

C++ (Qt)
void MainWindow::on_b1_clicked()
{
    int i=0;
    setWindowTitle(QString::number(1/i));
}
 
void MainWindow::on_b2_clicked()
{
    throw new std::exception();
}

Не ловит ни то, ни то.

[kambala]

видимо надо было уточнить:

Код

C++ (Qt)
int main(int argc, char *argv[])
{
    QApplication a(argc, argv);
    try
    {
        MainWindow w;
        w.show();
    }
    catch ( ... )
    {
        qDebug( "Exception" );
    }
    return a.exec();
}

[mejuhafr]

Вообще не проверятель?

[Авварон]

и правильно не ловит

[brankovic]

и правильно не ловит

непонятно, можете пояснить? (первый пример c catch ( ... ))

mejuhafr офтоп, но всё же: надо делать throw std::exception (), без new

[kambala]

Вообще не проверятель?

см. скриншот

Код

C++ (Qt)
#include <QtGui/QApplication>
#include <QtGui/QWidget>
 
class Widget : public QWidget
{
    Q_OBJECT
 
public:
    Widget(QWidget *parent = 0) { throw std::exception(); }
};
 
int main(int argc, char *argv[])
{
    QApplication a(argc, argv);
 
    try
    {
        Widget w;
        w.show();
    }
    catch (...)
    {
        qDebug("exception");
    }
 
    return a.exec();
}

mejuhafr офтоп, но всё же: надо делать throw std::exception (), без new

из-за утечки памяти?

[Авварон]

То есть вы считаете правильным, что эксепшн, брошенный в слоте, должен приходить в место вызова сигнала и сворачивать там стек (и 99% там не будет кода, к-ый бы их отлавливал и корректно очищал ресурсы - например такого вида char * buffer = new char[1024])
Посему внутрях сигнал/слотов, в тч в цикле обработки событий (если мне не изменяет память) стоит try {} catch {}, к-ый ловит все эксепшны из слотов и "гасит" их; возможно что-то печатает в консоль/вызывает подменяемый хэндлер

[LisandreL]

kambala, если исключения не будет, то ваш код уничтожит окно до того, как мы попадём в обработчик событий, что несколько неправильно. Да и исключение не из конструктора/show/setVisible() он не поймает.

Посему внутрях сигнал/слотов, в тч в цикле обработки событий (если мне не изменяет память) стоит try {} catch {}, к-ый ловит все эксепшны из слотов и "гасит" их; возможно что-то печатает в консоль/вызывает подменяемый хэндлер

Попробовал, у меня exception'ы, порождаемые в слотах никем не гасятся. Программа просто умирает.

[brankovic]

надо делать throw std::exception (), без new

из-за утечки памяти?

И из-за утечки тоже, но в основном чтобы не писать лишнего, странного кода, который запутает читателя. В C++ можно и число (int) кинуть, но так никто не делает почему-то.

[Авварон]

Попробовал, у меня exception'ы, порождаемые в слотах никем не гасятся. Программа просто умирает.

А вы только директ соединение пробовали? Я процентов на 80 уверен, что в эвент лупе стоят QT_TRY

[kambala]

kambala, если исключения не будет, то ваш код уничтожит окно до того, как мы попадём в обработчик событий, что несколько неправильно. Да и исключение не из конструктора/show/setVisible() он не поймает.

да, точно, спасибо. чуть переделал:

Код

C++ (Qt)
#include <QtGui/QApplication>
#include <QtGui/QWidget>
#include <QtGui/QPushButton>
 
class Widget : public QWidget
{
    Q_OBJECT
 
public:
    Widget(QWidget *parent = 0)
    {
        QPushButton *b = new QPushButton("exception", this);
        connect(b, SIGNAL(clicked()), SLOT(throwException()));
    }
 
private slots:
    void throwException() { throw std::exception(); }
};
 
int main(int argc, char *argv[])
{
    QApplication a(argc, argv);
 
    Widget w;
    try
    {
        w.show();
    }
    catch (...)
    {
        qDebug("exception");
    }
 
    return a.exec();
}

и тут же получил "ответ" в консоли:

Qt has caught an exception thrown from an event handler. Throwing
exceptions from an event handler is not supported in Qt. You must
reimplement QApplication::notify() and catch all exceptions there.

Программа неожиданно завершилась.

[Akon]

Есть ли аналог TApplication.OnException ?

TApplication.OnException возникает при генерации исключения в цикле обработки сообщений. Заворачивание тела main() в try/catch к этому отношения не имеет.

Для перехвата исключений из цикла необходимо переопределить Q[Core]Application::notify().

Код:

class App : public QApplication
{
    Q_OBJECT
    typedef QApplication Base;

...

Q_SIGNALS:
    void onException(const std::exception& exception, bool& handled);

protected:
    virtual bool notify(QObject* receiver, QEvent* event)
    { 
        try { 
            return Base::notify(receiver, event);
        } 
        catch (const std::exception& exception) {
            bool handled = false;
            Q_EMIT onException(exception, handled);
            if (!handled)
                QMessageBox::critical(0, QApplication::instance()->applicationName(),
                    exception.what(), QMessageBox::Ok, QMessageBox::NoButton);
        } 
        catch (...) {
	    QMessageBox::critical(0, QApplication::instance()->applicationName(),
	        "Application general failure", QMessageBox::Ok, QMessageBox::NoButton);             
        } 

        return false; 
    }
};

Но все это не вяжется с дизайном Qt. Об ошибках здесь сообщается не через исключения.

Для того, чтобы ловить исключения из слотов необходимо патчить QMetaObject::activate(). В частности, будет возможен вызов второго и всех остальных приконнекченных к данному сигналу слотов, если первый слот выбросит исключение. Это согласуется с дизайном Qt, поскольку формально порядок вызова слотов не определен, а, следовательно, слоты несут независимый друг от друга код.

[LisandreL]

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

В последних версиях Qt эту неопределённость убрали и в справке порядок официально закрепили порядок (хотя, говорят, и раньше оно работало именно так):

If a signal is connected to several slots, the slots are activated in the same order as the order the connection was made, when the signal is emitted.

Хотя надо понимать, что в случае соединения через очередь «slot is activated» значит лишь постановку в очередь.