Refactored entry point to make it easier to read

sohstationviewer/main.py

@@ -27,21 +27,17 @@ if os_name == 'macOS':
     os.environ['QT_MAC_WANTS_LAYER'] = '1'
 
 
-def main():
-    # Change the working directory so that relative paths work correctly.
+def fix_relative_paths() -> None:
+    """
+    Change the working director so that the relative paths in the code work
+    correctly.
+    """
     current_file_path = os.path.abspath(__file__)
     current_file_dir = Path(current_file_path).parent
     os.chdir(current_file_dir)
 
-    app = QtWidgets.QApplication(sys.argv)
-    wnd = MainWindow()
-    config = ConfigProcessor()
-    config.load_config()
-    need_reset = False
-    try:
-        config.validate_config()
-        config.apply_config(wnd)
-    except (BadConfigError, ValueError) as e:
+
+def check_if_user_want_to_reset_config():
         bad_config_dialog = QMessageBox()
         bad_config_dialog.setText('Something went wrong when reading the '
                                   'config.')
@@ -53,20 +49,35 @@ def main():
         bad_config_dialog.setDefaultButton(QMessageBox.Ok)
         bad_config_dialog.setIcon(QMessageBox.Critical)
         reset_choice = bad_config_dialog.exec_()
-        if reset_choice == QMessageBox.Ok:
-            need_reset = True
-        else:
-            sys.exit(1)
-    if need_reset:
+        return reset_choice == QMessageBox.Ok
+
+
+def main():
+    # Change the working directory so that relative paths work correctly.
+    fix_relative_paths()
+
+    app = QtWidgets.QApplication(sys.argv)
+    wnd = MainWindow()
+
+    config = ConfigProcessor()
+    config.load_config()
+    do_reset = False
+    try:
+        config.validate_config()
+        config.apply_config(wnd)
+    except (BadConfigError, ValueError) as e:
+        do_reset = check_if_user_want_to_reset_config()
+    if do_reset:
         try:
             config.reset()
         except OSError:
             QMessageBox.critical(None, 'Cannot reset config',
-                                 'Config file cannot be reset.  Please '
-                                 'ensure that it is not opened in another '
-                                 'program.',
+                                 'Config file cannot be reset.  Please ensure '
+                                 'that it is not opened in another program.',
                                  QMessageBox.Close)
             sys.exit(1)
+    else:
+        sys.exit(1)
     config.apply_config(wnd)
 
     screen_width, screen_height = app.primaryScreen().size().toTuple()