Add some switch defaults to make Codacy happy

AddstarMC · Mar 3, 2024 · cf22602 · cf22602
1 parent 023e749
commit cf22602
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/Minigames/src/main/java/au/com/mineauz/minigames/Events.java b/Minigames/src/main/java/au/com/mineauz/minigames/Events.java
@@ -664,8 +664,6 @@ private void clickMenu(InventoryClickEvent event) {
             } else if (!mgPlayer.getMenu().getAllowModify()) {
                 Inventory topInv = event.getView().getTopInventory();
                 switch (event.getAction()) {
-                    case NOTHING, DROP_ALL_CURSOR, DROP_ONE_CURSOR, UNKNOWN -> {
-                    } // nothing
                     case PICKUP_ALL, PICKUP_SOME, PICKUP_HALF, PICKUP_ONE, DROP_ALL_SLOT, DROP_ONE_SLOT, HOTBAR_MOVE_AND_READD, // may take
                             PLACE_ALL, PLACE_SOME, PLACE_ONE, /*may place*/
                             SWAP_WITH_CURSOR, HOTBAR_SWAP /*may give and take*/ -> {
@@ -681,6 +679,8 @@ private void clickMenu(InventoryClickEvent event) {
                     case MOVE_TO_OTHER_INVENTORY -> {
                         event.setCancelled(true);
                     } // definitely one or the other
+                    default -> {
+                    } // do nothing for any other action
                 }
             }
 

diff --git a/Minigames/src/main/java/au/com/mineauz/minigames/commands/ScoreCommand.java b/Minigames/src/main/java/au/com/mineauz/minigames/commands/ScoreCommand.java
@@ -230,6 +230,9 @@ public boolean onCommand(@NotNull CommandSender sender, @Nullable Minigame minig
 
                     return true;
                 } // end case
+                default -> {
+                    sender.sendMessage(ChatColor.RED + "Invalid argument!");
+                }
             } // end switch
         } else {
             sender.sendMessage(ChatColor.RED + "Not enough arguments!");
@@ -276,6 +279,8 @@ public List<String> onTabComplete(@NotNull CommandSender sender, @Nullable Minig
                         } // not a number
                     } // not add / set
                 } // more than 4 arguments
+                default -> {
+                } // do nothing
             } // end switch
         } // args == null