Switch to Early-Return

Author: Test-Account666

src/main/java/me/entity303/serversystem/utils/interceptors/invsee/InvseeSetItemInterceptor.java

@@ -66,56 +66,56 @@ public void intercept(@This Object obj, @AllArguments Object[] allArguments, @Mo
             }
         }
 
-        if (allArguments[0] instanceof Integer) {
-            var index = (int) allArguments[0];
-
-            if (index > 45 - 5)
-                if (allArguments[1] != null) {
-
-                    if (this._setCountMethod == null)
-                        try {
-                            this._setCountMethod = allArguments[1].getClass().getDeclaredMethod("setCount", int.class);
-                        } catch (NoSuchMethodException ignored) {
-                        }
+        if (!(allArguments[0] instanceof Integer))
+            return;
 
+        var index = (int) allArguments[0];
 
+        if (index > 45 - 5)
+            if (allArguments[1] != null) {
+                if (this._setCountMethod == null)
                     try {
-                        var itemStack = (ItemStack) this._asCraftMirrorMethod.invoke(null, allArguments[1]);
-                        if (!itemStack.getType().name().contains("AIR"))
-                            this._victim.getWorld()
-                                        .dropItem(this._victim.getEyeLocation().add(0, -0.33, 0), itemStack)
-                                        .setVelocity(new Vector(0.0, 0.0, 0.0).add(this._victim.getLocation().getDirection().multiply(0.35)));
-                    } catch (IllegalAccessException | InvocationTargetException exception) {
-                        exception.printStackTrace();
-                        return;
+                        this._setCountMethod = allArguments[1].getClass().getDeclaredMethod("setCount", int.class);
+                    } catch (NoSuchMethodException ignored) {
                     }
 
-                    var timer = new Timer();
-                    timer.schedule(new MyTimerTask(allArguments), 1000L / 20);
+
+                try {
+                    var itemStack = (ItemStack) this._asCraftMirrorMethod.invoke(null, allArguments[1]);
+                    if (!itemStack.getType().name().contains("AIR"))
+                        this._victim.getWorld()
+                                    .dropItem(this._victim.getEyeLocation().add(0, -0.33, 0), itemStack)
+                                    .setVelocity(new Vector(0.0, 0.0, 0.0).add(this._victim.getLocation().getDirection().multiply(0.35)));
+                } catch (IllegalAccessException | InvocationTargetException exception) {
+                    exception.printStackTrace();
                     return;
                 }
 
-            if (this._setItemMethod == null)
-                this._setItemMethod = Arrays.stream(this._masterInventoryNms.getClass().getDeclaredMethods())
-                                            .filter(method1 -> method1.getName().equalsIgnoreCase("setItem"))
-                                            .findFirst()
-                                            .orElse(null);
-
-            if (this._setItemMethod == null)
-                for (var declaredMethod : this._masterInventoryNms.getClass().getDeclaredMethods())
-                    if (declaredMethod.getReturnType().getName().equalsIgnoreCase(void.class.getName()))
-                        if (declaredMethod.getParameters().length == 2)
-                            if (declaredMethod.getParameters()[0].getType().getName().equalsIgnoreCase(int.class.getName()))
-                                if (declaredMethod.getParameters()[1].getType().getName().toLowerCase(Locale.ROOT).contains("itemstack")) {
-                                    this._setItemMethod = declaredMethod;
-                                    break;
-                                }
-
-            try {
-                this._setItemMethod.invoke(this._masterInventoryNms, allArguments[0], allArguments[1]);
-            } catch (IllegalAccessException | InvocationTargetException exception) {
-                exception.printStackTrace();
+                var timer = new Timer();
+                timer.schedule(new MyTimerTask(allArguments), 1000L / 20);
+                return;
             }
+
+        if (this._setItemMethod == null)
+            this._setItemMethod = Arrays.stream(this._masterInventoryNms.getClass().getDeclaredMethods())
+                                        .filter(method1 -> method1.getName().equalsIgnoreCase("setItem"))
+                                        .findFirst()
+                                        .orElse(null);
+
+        if (this._setItemMethod == null)
+            for (var declaredMethod : this._masterInventoryNms.getClass().getDeclaredMethods())
+                if (declaredMethod.getReturnType().getName().equalsIgnoreCase(void.class.getName()))
+                    if (declaredMethod.getParameters().length == 2)
+                        if (declaredMethod.getParameters()[0].getType().getName().equalsIgnoreCase(int.class.getName()))
+                            if (declaredMethod.getParameters()[1].getType().getName().toLowerCase(Locale.ROOT).contains("itemstack")) {
+                                this._setItemMethod = declaredMethod;
+                                break;
+                            }
+
+        try {
+            this._setItemMethod.invoke(this._masterInventoryNms, allArguments[0], allArguments[1]);
+        } catch (IllegalAccessException | InvocationTargetException exception) {
+            exception.printStackTrace();
         }
     }
 
@@ -130,7 +130,7 @@ public MyTimerTask(Object... allArguments) {
         public void run() {
             if (InvseeSetItemInterceptor.this._setCountMethod != null)
                 try {
-                    InvseeSetItemInterceptor.this._setCountMethod.invoke(_allArguments[1], 0);
+                    InvseeSetItemInterceptor.this._setCountMethod.invoke(this._allArguments[1], 0);
                 } catch (IllegalAccessException | InvocationTargetException exception) {
                     if (exception instanceof InvocationTargetException) {
                         if (!(exception.getCause() instanceof AssertionError))
@@ -140,7 +140,7 @@ public void run() {
                 }
 
             try {
-                var itemStack = (ItemStack) InvseeSetItemInterceptor.this._asCraftMirrorMethod.invoke(null, _allArguments[1]);
+                var itemStack = (ItemStack) InvseeSetItemInterceptor.this._asCraftMirrorMethod.invoke(null, this._allArguments[1]);
                 itemStack.setAmount(0);
             } catch (IllegalAccessException | InvocationTargetException exception) {
                 exception.printStackTrace();