Output error messages when using UI

ChachyDev · Jun 4, 2021 · c8aac3c · c8aac3c
1 parent 2964c1b
commit c8aac3c
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 9 deletions.
diff --git a/src/main/java/club/chachy/multimc4forge/installer/Installers.java b/src/main/java/club/chachy/multimc4forge/installer/Installers.java
@@ -26,7 +26,7 @@ public static Installer getInstaller(File forgeJar) {
         }
 
         if (installer == null) {
-            throw new IllegalStateException("There are no installers currently available for the provided jar...");
+            throw new IllegalStateException("There are no installers currently available for the provided jar, is it a Forge jar...?");
         }
 
         return installer;

diff --git a/user-interface/src/main/java/club/chachy/multimc4forge/ui/UI.java b/user-interface/src/main/java/club/chachy/multimc4forge/ui/UI.java
@@ -10,18 +10,28 @@
 import java.io.File;
 
 public class UI {
-    public static void main(String[] args) throws Throwable {
-        UIManager.setLookAndFeel(UIManager.getSystemLookAndFeelClassName());
+    public static void main(String[] args) {
+        try {
+            UIManager.setLookAndFeel(UIManager.getSystemLookAndFeelClassName());
 
-        File forgeJar = getFile(new JFileChooser(), "Please select a Forge INSTALLER jar", "You MUST provide a Forge Installer jar...", new JarFilter(), new ZipFilter());
+            File forgeJar = getFile(new JFileChooser(), "Please select a Forge INSTALLER jar", "You MUST provide a Forge Installer jar...", new JarFilter(), new ZipFilter());
 
-        if (forgeJar != null) {
-            File multiMcDirectory = getFile(new JFileChooser(), "Please provide your Multi", "You MUST provide a MultiMC install directory", new DirectoryFilter());
+            if (forgeJar != null) {
+                File multiMcDirectory = getFile(new JFileChooser(), "Please provide your Multi", "You MUST provide a MultiMC install directory", new DirectoryFilter());
 
-            if (multiMcDirectory != null) {
-                File location = Installers.getInstaller(forgeJar).install(multiMcDirectory, forgeJar);
-                JOptionPane.showMessageDialog(null, "Successfully installed Forge to " + location.getAbsolutePath() + "! Make sure to restart MultiMC if it was already opened", "Success! :)", JOptionPane.PLAIN_MESSAGE);
+                if (multiMcDirectory != null) {
+                    File location = Installers.getInstaller(forgeJar).install(multiMcDirectory, forgeJar);
+                    JOptionPane.showMessageDialog(null, "Successfully installed Forge to " + location.getAbsolutePath() + "! Make sure to restart MultiMC if it was already opened", "Success! :)", JOptionPane.PLAIN_MESSAGE);
+                }
             }
+        } catch (Throwable t) {
+            String message = t.getMessage();
+
+            if (message != null) {
+                JOptionPane.showMessageDialog(null, message, "An error occurred :(", JOptionPane.ERROR_MESSAGE);
+            }
+
+            t.printStackTrace();
         }
     }