LazoYoung / Naru-ACARS

Flight tracking solution
0 stars 1 forks source link

Error messages #13

Open cristian9751 opened 6 days ago

cristian9751 commented 6 days ago

You should implement another layer to catch the error and present a messsage not the plain exception

LazoYoung commented 6 days ago

You should implement another layer to catch the error and present a messsage not the plain exception

Thanks for the comment but would you mention the line of code that you concern please. @cristian9751

LazoYoung commented 6 days ago

https://github.com/LazoYoung/Naru-ACARS/blob/120188537ec3e35b8e41c86201f046c7be8b55ba/src/main/java/com/flylazo/naru_acars/gui/panel/ACARS_Form.java#L135 If this part is your concern, presenting the error by using logger.log() was intentional as the logger is hooked by a LogHandler to present every warnings or stacktraces on the console page of this application. Users can check the console to review the error if desired. Also, alert window pops up automatically when a SEVERE log is printed to the console. You can check the code here. Do you think I need a different approach to deal with errors?