Results 1 to 13 of 13

Thread: How to show a message to user when app crashes

  1. #1
    Join Date
    Jul 2019
    Posts
    32
    Thanks
    5
    Qt products
    Qt5
    Platforms
    Windows

    Default How to show a message to user when app crashes

    Hello! I want to show a QMessageBox when my app crashes, but I don't know how to handle the crash if it happens. I know that the best sollution of this is to make my code very well, avoiding the crash, but I want to show the message even if a crash happens. Thank you!

  2. #2
    Join Date
    Jan 2008
    Location
    Alameda, CA, USA
    Posts
    5,230
    Thanks
    302
    Thanked 864 Times in 851 Posts
    Qt products
    Qt5
    Platforms
    Windows

    Default Re: How to show a message to user when app crashes

    Hello! I want to show a QMessageBox when my app crashes, but I don't know how to handle the crash if it happens.
    The usual methods are:

    1. Use try/catch clauses around areas of the code that could throw an exception, making sure your catch() clauses include the right exceptions
    2. Add a signal handler. There is an example here.

    Generally a crash occurs due to a segmentation fault (SIGSEGV) because your code has made an invalid memory access. You usually cannot fix such a crash at runtime, because once memory is corrupted, there isn't much you can do except halt the program.

    Whether you will be able to display a QMessageBox or not depends on how broken your program is. If the code has corrupted some critical section needed by Qt's event loop or other functions, trying to display a message might just cause more faults.

    I know that the best sollution of this is to make my code very well, avoiding the crash
    Yep. Learn how to program defensively:

    1. Check all return values from functions that return some kind of status that indicates whether they succeeded or not
    2. Check any pointers for valid values (eg. non-null) before you use them, especially pointers returned from function calls. Don't assume that when a method returns a pointer you can just use it without checking.
    3. Check array indexes before using them to make sure you don't try to access memory that is out of range of your array
    4. Add assert() statements to your code to check that variables, pointers, etc. have valid values.
    5. Always, always make a debug version of your program and run it in the debugger. Learn to use the debugger output to backtrack where the error occurred and fix it.
    6. If you are writing code that others will use (i.e. a library), document the methods in your classes to help others avoid mistakes. In particular, if methods must be called in a certain order or variables initialized, make sure that is clear in your comments.
    <=== The Great Pumpkin says ===>
    Please use CODE tags when posting source code so it is more readable. Click "Go Advanced" and then the "#" icon to insert the tags. Paste your code between them.

  3. #3
    Join Date
    Jul 2019
    Posts
    32
    Thanks
    5
    Qt products
    Qt5
    Platforms
    Windows

    Default Re: How to show a message to user when app crashes

    Quote Originally Posted by d_stranz View Post
    The usual methods are:

    1. Use try/catch clauses around areas of the code that could throw an exception, making sure your catch() clauses include the right exceptions
    2. Add a signal handler. There is an example here.

    Generally a crash occurs due to a segmentation fault (SIGSEGV) because your code has made an invalid memory access. You usually cannot fix such a crash at runtime, because once memory is corrupted, there isn't much you can do except halt the program.

    Whether you will be able to display a QMessageBox or not depends on how broken your program is. If the code has corrupted some critical section needed by Qt's event loop or other functions, trying to display a message might just cause more faults.



    Yep. Learn how to program defensively:

    1. Check all return values from functions that return some kind of status that indicates whether they succeeded or not
    2. Check any pointers for valid values (eg. non-null) before you use them, especially pointers returned from function calls. Don't assume that when a method returns a pointer you can just use it without checking.
    3. Check array indexes before using them to make sure you don't try to access memory that is out of range of your array
    4. Add assert() statements to your code to check that variables, pointers, etc. have valid values.
    5. Always, always make a debug version of your program and run it in the debugger. Learn to use the debugger output to backtrack where the error occurred and fix it.
    6. If you are writing code that others will use (i.e. a library), document the methods in your classes to help others avoid mistakes. In particular, if methods must be called in a certain order or variables initialized, make sure that is clear in your comments.
    Thank you very much! I found out how to handle the crash, but I need some help with displaying the QMessageBox after app closes. How can I achieve that?
    My code:
    Qt Code:
    1. #include <QApplication>
    2. #include "mainwindow.h"
    3. #include <QMessageBox>
    4. #include <signal.h>
    5.  
    6. void signalHandler(int signum){
    7. signal(signum, SIG_DFL);
    8. qDebug() << "App crashed";
    9. QMessageBox::critical(nullptr, "App crashed", "Application has unexpectedly close! Try restarting the application.");
    10. }
    11.  
    12.  
    13. int main(int argl,char *argv[])
    14. {
    15. Q_INIT_RESOURCE(resources);
    16. signal(SIGSEGV, signalHandler);
    17. QApplication app(argl,argv);
    18. mainWindow *window = new mainWindow();
    19. window->setWindowTitle("Test");
    20. window->setFixedSize(700, 400);
    21. window->show();
    22.  
    23. return app.exec();
    24. }
    To copy to clipboard, switch view to plain text mode 

    In my code, the QMessageBox appears first, and then the app is closed. I want to close the mainWindow or making it dissapear and the displaying the QMessageBox.
    Last edited by INeedADollar; 10th September 2019 at 20:14.

  4. #4
    Join Date
    Jan 2008
    Location
    Alameda, CA, USA
    Posts
    5,230
    Thanks
    302
    Thanked 864 Times in 851 Posts
    Qt products
    Qt5
    Platforms
    Windows

    Default Re: How to show a message to user when app crashes

    You will probably need to do something like the following:

    In main(), call QApplication::setQuitOnLastWindowClosed() with the value false before you show the mainWindow. This is to prevent what happens in the signal handler (see below) from causing the app to quit too soon.

    In your signal handler, call QApplication::closeAllWindows() first, using the global qApp pointer.
    Next, display your QMessageBox.
    When the user closes the message box, call QApplication::quit() and then exit the handler.

    You might also need to add a call to QApplication::quit() after app.exec() in main, but probably not since main() exits at that point anyway.

    Another option would be to define your mainwindow pointer as a global variable in main.cpp so you can get to it from your signal handler. The all you have to do is call mainwindow->hide() from inside the handler before you display your message box. You should probably also call QApplication::quit() after the message box is closed so Qt can have a chance to clean up instead of just crashing out.
    <=== The Great Pumpkin says ===>
    Please use CODE tags when posting source code so it is more readable. Click "Go Advanced" and then the "#" icon to insert the tags. Paste your code between them.

  5. #5
    Join Date
    Jul 2019
    Posts
    32
    Thanks
    5
    Qt products
    Qt5
    Platforms
    Windows

    Default Re: How to show a message to user when app crashes

    Quote Originally Posted by d_stranz View Post
    You will probably need to do something like the following:

    In main(), call QApplication::setQuitOnLastWindowClosed() with the value false before you show the mainWindow. This is to prevent what happens in the signal handler (see below) from causing the app to quit too soon.

    In your signal handler, call QApplication::closeAllWindows() first, using the global qApp pointer.
    Next, display your QMessageBox.
    When the user closes the message box, call QApplication::quit() and then exit the handler.

    You might also need to add a call to QApplication::quit() after app.exec() in main, but probably not since main() exits at that point anyway.

    Another option would be to define your mainwindow pointer as a global variable in main.cpp so you can get to it from your signal handler. The all you have to do is call mainwindow->hide() from inside the handler before you display your message box. You should probably also call QApplication::quit() after the message box is closed so Qt can have a chance to clean up instead of just crashing out.
    Thank you very, very much! The second method worked very well. You are the best!

  6. #6
    Join Date
    Jan 2008
    Location
    Alameda, CA, USA
    Posts
    5,230
    Thanks
    302
    Thanked 864 Times in 851 Posts
    Qt products
    Qt5
    Platforms
    Windows

    Default Re: How to show a message to user when app crashes

    The second method worked very well. You are the best!
    It may not work so well if you have a modal dialog window open when a SIGSEGV happens - hiding the main window may not hide the dialog.

    You could test this by adding a QDialog-based class to your app and override some event (like showEvent()) for the class. In that event, call "raise( SIGSEGV )" to trigger a segmentation fault signal, which should go into your handler. If hiding the main window also hides the dialog, all is good.
    <=== The Great Pumpkin says ===>
    Please use CODE tags when posting source code so it is more readable. Click "Go Advanced" and then the "#" icon to insert the tags. Paste your code between them.

  7. #7
    Join Date
    Jul 2019
    Posts
    32
    Thanks
    5
    Qt products
    Qt5
    Platforms
    Windows

    Default Re: How to show a message to user when app crashes

    Quote Originally Posted by d_stranz View Post
    It may not work so well if you have a modal dialog window open when a SIGSEGV happens - hiding the main window may not hide the dialog.

    You could test this by adding a QDialog-based class to your app and override some event (like showEvent()) for the class. In that event, call "raise( SIGSEGV )" to trigger a segmentation fault signal, which should go into your handler. If hiding the main window also hides the dialog, all is good.
    I have a problem. I cannot do this because QDialog also has a function named raise() but without any arguments and I cannot call the function raise(SIGSEGV) from signal.h
    Do you know how can I call it without any error?

    P.S. the error is "too many arguments for function call". I think the compiler confuses the functions.

  8. #8
    Join Date
    Jan 2008
    Location
    Alameda, CA, USA
    Posts
    5,230
    Thanks
    302
    Thanked 864 Times in 851 Posts
    Qt products
    Qt5
    Platforms
    Windows

    Default Re: How to show a message to user when app crashes

    Do you know how can I call it without any error?
    You need to remember your C++ basics. To call a non-member function, eg. one in the global namespace, you use the global scope qualifier:

    Qt Code:
    1. ::raise( SIGSEGV );
    To copy to clipboard, switch view to plain text mode 
    <=== The Great Pumpkin says ===>
    Please use CODE tags when posting source code so it is more readable. Click "Go Advanced" and then the "#" icon to insert the tags. Paste your code between them.

  9. #9
    Join Date
    Jul 2019
    Posts
    32
    Thanks
    5
    Qt products
    Qt5
    Platforms
    Windows

    Default Re: How to show a message to user when app crashes

    Quote Originally Posted by d_stranz View Post
    You need to remember your C++ basics. To call a non-member function, eg. one in the global namespace, you use the global scope qualifier:

    Qt Code:
    1. ::raise( SIGSEGV );
    To copy to clipboard, switch view to plain text mode 
    Thank you very much! Not all windows closed when signal was sent, but I wrote this part inside signalHandler() function before showing the QMessageBox:
    Qt Code:
    1. const QWidgetList topLevelWidgets = QApplication::topLevelWidgets();
    2. for (QWidget *widget : topLevelWidgets) {
    3. widget->hide();
    4. }
    To copy to clipboard, switch view to plain text mode 

    Now it is working very well. Thank you again!

  10. #10
    Join Date
    Jan 2008
    Location
    Alameda, CA, USA
    Posts
    5,230
    Thanks
    302
    Thanked 864 Times in 851 Posts
    Qt products
    Qt5
    Platforms
    Windows

    Default Re: How to show a message to user when app crashes

    Great! Now start practicing defensive programming and debug your programs before you release them so you will never have a need for this kind of message.

    If you want to make this kind of signal handler really useful instead of just displaying a message like "Oops, something broke and there is nothing you can do to fix it", you would implement something in your QApplication class that could save any open files and basically try to do whatever is possible to clean up what the user was doing and leave it in a recoverable state.

    Just displaying a message and then crashing out isn't much different from simply crashing out as far as the user is concerned if their work is trashed and they lose it all.
    <=== The Great Pumpkin says ===>
    Please use CODE tags when posting source code so it is more readable. Click "Go Advanced" and then the "#" icon to insert the tags. Paste your code between them.

  11. #11
    Join Date
    Jul 2019
    Posts
    32
    Thanks
    5
    Qt products
    Qt5
    Platforms
    Windows

    Default Re: How to show a message to user when app crashes

    Quote Originally Posted by d_stranz View Post
    Great! Now start practicing defensive programming and debug your programs before you release them so you will never have a need for this kind of message.

    If you want to make this kind of signal handler really useful instead of just displaying a message like "Oops, something broke and there is nothing you can do to fix it", you would implement something in your QApplication class that could save any open files and basically try to do whatever is possible to clean up what the user was doing and leave it in a recoverable state.

    Just displaying a message and then crashing out isn't much different from simply crashing out as far as the user is concerned if their work is trashed and they lose it all.
    What a great idea! You are right, it is much better when the app saves your progress automatically If it crashes. Thank you for reminding me of this!

  12. #12
    Join Date
    Sep 2019
    Posts
    1
    Qt products
    Qt5
    Platforms
    Windows

    Default Re: How to show a message to user when app crashes

    Hello,

    I can't find correct solution still.

    Thanks & Regards

    Cyberops Infosec LLP

  13. #13
    Join Date
    Jan 2008
    Location
    Alameda, CA, USA
    Posts
    5,230
    Thanks
    302
    Thanked 864 Times in 851 Posts
    Qt products
    Qt5
    Platforms
    Windows

    Default Re: How to show a message to user when app crashes

    I can't find correct solution still.
    What is your question? If you are trying to implement something in the same way as INeedADollar is doing, then there is more than enough information in the code and advice that has been posted in response to his questions for you to do the same thing.
    <=== The Great Pumpkin says ===>
    Please use CODE tags when posting source code so it is more readable. Click "Go Advanced" and then the "#" icon to insert the tags. Paste your code between them.

Similar Threads

  1. Replies: 3
    Last Post: 16th May 2015, 11:08
  2. How to get user-defined message?
    By ponponfish in forum Qt Programming
    Replies: 3
    Last Post: 17th May 2011, 17:19
  3. Replies: 10
    Last Post: 5th August 2009, 12:53
  4. Replies: 4
    Last Post: 12th October 2008, 14:47
  5. StatusBar show message
    By Pang in forum Qt Programming
    Replies: 3
    Last Post: 23rd August 2007, 11:22

Tags for this Thread

Bookmarks

Posting Permissions

  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts
  •  
Digia, Qt and their respective logos are trademarks of Digia Plc in Finland and/or other countries worldwide.