Use error messages instead of NPE (#115)

* repo: Use error messages instead of NPE when package cannot be found in MiraiRepo * repo: use Local Variable Type Inference
iTXTech · Jun 2, 2022 · f716a26 · f716a26
1 parent c86d10b
commit f716a26
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/.gitignore b/.gitignore
@@ -4,3 +4,8 @@ build
 out
 
 /testing
+
+
+/gradle
+gradlew
+gradlew.bat
diff --git a/src/main/java/org/itxtech/mcl/module/builtin/Repo.java b/src/main/java/org/itxtech/mcl/module/builtin/Repo.java
@@ -77,8 +77,16 @@ public void cli() {
 
             if (loader.cli.hasOption("i")) {
                 var pkg = loader.cli.getOptionValue("i");
+
                 loader.logger.info("Fetching channel info for package \"" + pkg + "\"");
-                for (var chan : loader.repo.fetchPackage(pkg).channels.entrySet()) {
+                var fetchedPackageInfo = loader.repo.fetchPackage(pkg);
+                if (null == fetchedPackageInfo) {
+                    loader.logger.error("Package \"" + pkg + "\" is not found in MiraiRepo");
+                    loader.exit(1);
+                    return;
+                }
+
+                for (var chan : fetchedPackageInfo.channels.entrySet()) {
                     loader.logger.info("---------- Channel: " + chan.getKey() + " ----------");
                     loader.logger.info("Version: " + Utility.join(", ", chan.getValue()));
                     loader.logger.info("");