From e00f1d8562dac24eaf9effbaa78425d89b533d0b Mon Sep 17 00:00:00 2001 From: Emma Stone Date: Wed, 1 Feb 2017 20:35:49 +0000 Subject: [PATCH 1/2] [CLEANUP] Code cleanup --- src/main/java/org/rev317/min/Loader.java | 4 ++- .../java/org/rev317/min/accessors/Client.java | 1 - .../rev317/min/accessors/CollisionMap.java | 1 - .../java/org/rev317/min/accessors/Deque.java | 1 - .../java/org/rev317/min/accessors/Ground.java | 1 - .../org/rev317/min/accessors/Interface.java | 6 ---- .../java/org/rev317/min/accessors/Item.java | 1 - .../java/org/rev317/min/accessors/Node.java | 1 - .../java/org/rev317/min/accessors/Npc.java | 1 - .../java/org/rev317/min/accessors/NpcDef.java | 1 + .../java/org/rev317/min/accessors/Player.java | 1 + .../java/org/rev317/min/accessors/Scene.java | 1 - .../rev317/min/accessors/SceneObjectTile.java | 1 - .../min/api/events/GameActionEvent.java | 1 - .../rev317/min/api/events/MessageEvent.java | 3 +- .../rev317/min/api/interfaces/Locatable.java | 1 - .../rev317/min/api/interfaces/TileFlags.java | 1 - .../java/org/rev317/min/api/methods/Bank.java | 14 ++++++--- .../rev317/min/api/methods/Calculations.java | 7 +++-- .../org/rev317/min/api/methods/Equipment.java | 1 + .../java/org/rev317/min/api/methods/Game.java | 3 +- .../rev317/min/api/methods/GroundItems.java | 15 ++++++--- .../org/rev317/min/api/methods/Items.java | 21 +++++++++---- .../java/org/rev317/min/api/methods/Menu.java | 14 +++------ .../java/org/rev317/min/api/methods/Npcs.java | 8 ++++- .../org/rev317/min/api/methods/Players.java | 5 +-- .../org/rev317/min/api/methods/Prayer.java | 9 +++--- .../rev317/min/api/methods/SceneObjects.java | 31 +++++++++++-------- .../org/rev317/min/api/methods/Skill.java | 22 +++++++------ .../org/rev317/min/api/methods/Walking.java | 4 +-- .../org/rev317/min/api/wrappers/Area.java | 7 +++-- .../rev317/min/api/wrappers/Character.java | 23 +++++++++----- .../rev317/min/api/wrappers/GroundItem.java | 1 - .../java/org/rev317/min/api/wrappers/Npc.java | 1 - .../org/rev317/min/api/wrappers/NpcDef.java | 4 +-- .../rev317/min/api/wrappers/SceneObject.java | 1 - .../org/rev317/min/api/wrappers/Tile.java | 2 +- .../org/rev317/min/api/wrappers/TilePath.java | 14 +++++---- .../org/rev317/min/callback/MenuAction.java | 4 +-- .../rev317/min/callback/MessageCallback.java | 1 - .../java/org/rev317/min/debug/DActions.java | 1 - .../java/org/rev317/min/debug/DAnimation.java | 1 - src/main/java/org/rev317/min/debug/DBank.java | 2 +- .../org/rev317/min/debug/DCollisionFlags.java | 1 - .../org/rev317/min/debug/DGroundItems.java | 1 - .../org/rev317/min/debug/DInterfaces.java | 1 - .../java/org/rev317/min/debug/DInventory.java | 1 + src/main/java/org/rev317/min/debug/DMap.java | 1 - .../java/org/rev317/min/debug/DMouse.java | 1 - src/main/java/org/rev317/min/debug/DNpcs.java | 2 -- .../java/org/rev317/min/debug/DPlayers.java | 1 - .../org/rev317/min/debug/DSceneObjects.java | 1 - .../org/rev317/min/script/ScriptEngine.java | 14 ++++----- src/main/java/org/rev317/min/ui/BotMenu.java | 2 +- 54 files changed, 143 insertions(+), 126 deletions(-) diff --git a/src/main/java/org/rev317/min/Loader.java b/src/main/java/org/rev317/min/Loader.java index cbc5802..7aaeca6 100644 --- a/src/main/java/org/rev317/min/Loader.java +++ b/src/main/java/org/rev317/min/Loader.java @@ -40,9 +40,11 @@ public class Loader extends ServerProvider { final ASMClassLoader classLoader = context.getASMClassLoader(); final Class clientClass = classLoader.loadClass(Context.getInstance().getServerProviderInfo().getClientClass()); Object instance = clientClass.newInstance(); + return (Applet) instance; } catch (Exception e) { e.printStackTrace(); + return null; } } @@ -100,4 +102,4 @@ public class Loader extends ServerProvider { @Override public void init() { } -} \ No newline at end of file +} diff --git a/src/main/java/org/rev317/min/accessors/Client.java b/src/main/java/org/rev317/min/accessors/Client.java index ecffb6d..2d61f4c 100644 --- a/src/main/java/org/rev317/min/accessors/Client.java +++ b/src/main/java/org/rev317/min/accessors/Client.java @@ -44,7 +44,6 @@ public interface Client { CollisionMap[] getCollisionMap(); - // args switched boolean walkTo(int clickType, int sizeX, int sizeY, int startX, int startY, int destX, int destY, int type, int face, boolean arbitrary, int rotation); boolean walkToPKH(boolean flag1, boolean flag2, int clickType, int sizeX, int sizeY, int startX, int startY, int destX, int destY, int type, int face, boolean arbitrary, int rotation); diff --git a/src/main/java/org/rev317/min/accessors/CollisionMap.java b/src/main/java/org/rev317/min/accessors/CollisionMap.java index 768dec6..af58c53 100644 --- a/src/main/java/org/rev317/min/accessors/CollisionMap.java +++ b/src/main/java/org/rev317/min/accessors/CollisionMap.java @@ -4,6 +4,5 @@ package org.rev317.min.accessors; public interface CollisionMap { int[][] getFlags(); - } diff --git a/src/main/java/org/rev317/min/accessors/Deque.java b/src/main/java/org/rev317/min/accessors/Deque.java index 6634828..5b1a39f 100644 --- a/src/main/java/org/rev317/min/accessors/Deque.java +++ b/src/main/java/org/rev317/min/accessors/Deque.java @@ -5,5 +5,4 @@ public interface Deque { Node getHead(); Node getCurrent(); - } diff --git a/src/main/java/org/rev317/min/accessors/Ground.java b/src/main/java/org/rev317/min/accessors/Ground.java index 9bcd69d..616467d 100644 --- a/src/main/java/org/rev317/min/accessors/Ground.java +++ b/src/main/java/org/rev317/min/accessors/Ground.java @@ -11,5 +11,4 @@ public interface Ground { SceneObjectTile getGroundItem(); SceneObjectTile[] getInteractiveObjects(); - } diff --git a/src/main/java/org/rev317/min/accessors/Interface.java b/src/main/java/org/rev317/min/accessors/Interface.java index f4ea49d..f1969ad 100644 --- a/src/main/java/org/rev317/min/accessors/Interface.java +++ b/src/main/java/org/rev317/min/accessors/Interface.java @@ -6,13 +6,7 @@ public interface Interface { int[] getStackSizes(); - /** - * This is meant for the clients that have a long value as their stacksizes, simply because they have no logic... - * - * @return long version of #getStackSizes - */ long[] getLongStackSizes(); String getMessage(); - } diff --git a/src/main/java/org/rev317/min/accessors/Item.java b/src/main/java/org/rev317/min/accessors/Item.java index 113cf71..8257423 100644 --- a/src/main/java/org/rev317/min/accessors/Item.java +++ b/src/main/java/org/rev317/min/accessors/Item.java @@ -3,6 +3,5 @@ package org.rev317.min.accessors; public interface Item { int getId(); - } diff --git a/src/main/java/org/rev317/min/accessors/Node.java b/src/main/java/org/rev317/min/accessors/Node.java index 32c5aae..0705085 100644 --- a/src/main/java/org/rev317/min/accessors/Node.java +++ b/src/main/java/org/rev317/min/accessors/Node.java @@ -3,5 +3,4 @@ package org.rev317.min.accessors; public interface Node { Node getNext(); - } diff --git a/src/main/java/org/rev317/min/accessors/Npc.java b/src/main/java/org/rev317/min/accessors/Npc.java index 22830f8..042d3ab 100644 --- a/src/main/java/org/rev317/min/accessors/Npc.java +++ b/src/main/java/org/rev317/min/accessors/Npc.java @@ -3,5 +3,4 @@ package org.rev317.min.accessors; public interface Npc extends Character { NpcDef getDef(); - } \ No newline at end of file diff --git a/src/main/java/org/rev317/min/accessors/NpcDef.java b/src/main/java/org/rev317/min/accessors/NpcDef.java index 3e71fba..5022faf 100644 --- a/src/main/java/org/rev317/min/accessors/NpcDef.java +++ b/src/main/java/org/rev317/min/accessors/NpcDef.java @@ -4,4 +4,5 @@ public interface NpcDef { int getId(); + long getLongId(); } diff --git a/src/main/java/org/rev317/min/accessors/Player.java b/src/main/java/org/rev317/min/accessors/Player.java index 10b4d6a..c924f41 100644 --- a/src/main/java/org/rev317/min/accessors/Player.java +++ b/src/main/java/org/rev317/min/accessors/Player.java @@ -1,5 +1,6 @@ package org.rev317.min.accessors; public interface Player extends Character { + String getName(); } diff --git a/src/main/java/org/rev317/min/accessors/Scene.java b/src/main/java/org/rev317/min/accessors/Scene.java index d6bc2dd..69952f4 100644 --- a/src/main/java/org/rev317/min/accessors/Scene.java +++ b/src/main/java/org/rev317/min/accessors/Scene.java @@ -3,5 +3,4 @@ package org.rev317.min.accessors; public interface Scene { Ground[][][] getGroundArray(); - } diff --git a/src/main/java/org/rev317/min/accessors/SceneObjectTile.java b/src/main/java/org/rev317/min/accessors/SceneObjectTile.java index 51875e5..ad9e48d 100644 --- a/src/main/java/org/rev317/min/accessors/SceneObjectTile.java +++ b/src/main/java/org/rev317/min/accessors/SceneObjectTile.java @@ -3,5 +3,4 @@ package org.rev317.min.accessors; public interface SceneObjectTile { int getHash(); - } \ No newline at end of file diff --git a/src/main/java/org/rev317/min/api/events/GameActionEvent.java b/src/main/java/org/rev317/min/api/events/GameActionEvent.java index c636730..07cc4b3 100644 --- a/src/main/java/org/rev317/min/api/events/GameActionEvent.java +++ b/src/main/java/org/rev317/min/api/events/GameActionEvent.java @@ -38,5 +38,4 @@ public final class GameActionEvent { public int getIndex() { return index; } - } diff --git a/src/main/java/org/rev317/min/api/events/MessageEvent.java b/src/main/java/org/rev317/min/api/events/MessageEvent.java index ce1dfdf..2a6e917 100644 --- a/src/main/java/org/rev317/min/api/events/MessageEvent.java +++ b/src/main/java/org/rev317/min/api/events/MessageEvent.java @@ -29,5 +29,4 @@ public final class MessageEvent { public final String getSender() { return name; } - -} \ No newline at end of file +} diff --git a/src/main/java/org/rev317/min/api/interfaces/Locatable.java b/src/main/java/org/rev317/min/api/interfaces/Locatable.java index 6424c91..d3bfcab 100644 --- a/src/main/java/org/rev317/min/api/interfaces/Locatable.java +++ b/src/main/java/org/rev317/min/api/interfaces/Locatable.java @@ -20,5 +20,4 @@ public interface Locatable { * @return distance from player to locatable */ int distanceTo(); - } diff --git a/src/main/java/org/rev317/min/api/interfaces/TileFlags.java b/src/main/java/org/rev317/min/api/interfaces/TileFlags.java index 4ff0911..4c592b0 100644 --- a/src/main/java/org/rev317/min/api/interfaces/TileFlags.java +++ b/src/main/java/org/rev317/min/api/interfaces/TileFlags.java @@ -14,5 +14,4 @@ public interface TileFlags { int UNKNOWN = 0x80000; int BLOCKED_TILE = 0x200000; int UNLOADED_TILE = 0x1000000; - } diff --git a/src/main/java/org/rev317/min/api/methods/Bank.java b/src/main/java/org/rev317/min/api/methods/Bank.java index 1c91a0c..39f6798 100644 --- a/src/main/java/org/rev317/min/api/methods/Bank.java +++ b/src/main/java/org/rev317/min/api/methods/Bank.java @@ -63,10 +63,10 @@ public class Bank { * @return true if successfully interacted */ public static boolean open() { - if (isOpen()) { return false; } + SceneObject bank = getBank(); Npc banker = getBanker(); @@ -100,7 +100,6 @@ public class Bank { } Item b = getItem(id); - if (b == null) { return; } @@ -139,6 +138,7 @@ public class Bank { } } } + return null; } @@ -153,6 +153,7 @@ public class Bank { return 0; } Item item; + return ((item = getItem(id)) != null ? item.getStackSize() : 0); } @@ -165,7 +166,6 @@ public class Bank { if (isOpen()) { return; } - if (bank.getLocation().distanceTo() > 8) { bank.getLocation().walkTo(); Time.sleep(new SleepCondition() { @@ -176,6 +176,7 @@ public class Bank { }, 5000); return; } + bank.interact(SceneObjects.Option.USE); } @@ -187,6 +188,7 @@ public class Bank { if (!isOpen()) { return; } + Menu.sendAction(200, -1, -1, settings.get("button_close_bank")); } @@ -229,6 +231,7 @@ public class Bank { if (!isOpen()) { return null; } + return Loader.getClient().getInterfaceCache()[settings.get("item_interface_id")].getItems(); } @@ -241,6 +244,7 @@ public class Bank { if (!isOpen()) { return null; } + return Loader.getClient().getInterfaceCache()[settings.get("item_interface_id")].getStackSizes(); } @@ -253,6 +257,7 @@ public class Bank { if (!isOpen()) { return null; } + ArrayList items = new ArrayList(); int[] ids = getBankItemIDs(); int[] stacks = getBankStacks(); @@ -263,6 +268,7 @@ public class Bank { } } } + return items.toArray(new Item[items.size()]); } @@ -275,6 +281,7 @@ public class Bank { if (!isOpen()) { return 0; } + int[] items; return ((items = getBankItemIDs()) != null ? items.length : 0); } @@ -287,5 +294,4 @@ public class Bank { public static boolean isOpen() { return Loader.getClient().getOpenInterfaceId() == settings.get("bank_interface_id"); } - } diff --git a/src/main/java/org/rev317/min/api/methods/Calculations.java b/src/main/java/org/rev317/min/api/methods/Calculations.java index 9f5730c..0f55bbc 100644 --- a/src/main/java/org/rev317/min/api/methods/Calculations.java +++ b/src/main/java/org/rev317/min/api/methods/Calculations.java @@ -28,6 +28,7 @@ public class Calculations { public static final double distanceBetween(Tile a, Tile b) { int x = b.getX() - a.getX(); int y = b.getY() - a.getY(); + return Math.sqrt((x * x) + (y * y)); } @@ -145,6 +146,7 @@ public class Calculations { dist[curr_x + 1][curr_y + 1] = cost; } } + return foundPath ? dist[curr_x][curr_y] : -1; } catch (Exception e) { return -1; @@ -185,6 +187,7 @@ public class Calculations { if (foundPath(from, to)) { return dijkstraDist(from.getX(), from.getY(), to.getX(), to.getY(), false); } + return -1; } @@ -200,6 +203,7 @@ public class Calculations { if (foundPath(from, to)) { return dijkstraDist(from.getX(), from.getY(), to.getX(), to.getY(), isObject); } + return -1; } @@ -267,5 +271,4 @@ public class Calculations { public static int gainedPerHour(Timer runtime, int start, int current) { return runtime.getPerHour(gained(start, current)); } - -} \ No newline at end of file +} diff --git a/src/main/java/org/rev317/min/api/methods/Equipment.java b/src/main/java/org/rev317/min/api/methods/Equipment.java index 4447e86..601446e 100644 --- a/src/main/java/org/rev317/min/api/methods/Equipment.java +++ b/src/main/java/org/rev317/min/api/methods/Equipment.java @@ -69,6 +69,7 @@ public class Equipment { return true; } } + return false; } diff --git a/src/main/java/org/rev317/min/api/methods/Game.java b/src/main/java/org/rev317/min/api/methods/Game.java index 392cc29..a04800f 100644 --- a/src/main/java/org/rev317/min/api/methods/Game.java +++ b/src/main/java/org/rev317/min/api/methods/Game.java @@ -146,9 +146,10 @@ public class Game { Time.sleep(new SleepCondition() { @Override public boolean isValid() { - return isLoggedIn(); + return !isLoggedIn(); } }, 2500); + return !isLoggedIn(); } diff --git a/src/main/java/org/rev317/min/api/methods/GroundItems.java b/src/main/java/org/rev317/min/api/methods/GroundItems.java index acd4790..8c79bfe 100644 --- a/src/main/java/org/rev317/min/api/methods/GroundItems.java +++ b/src/main/java/org/rev317/min/api/methods/GroundItems.java @@ -19,12 +19,10 @@ import java.util.HashMap; */ public class GroundItems { private static final Comparator NEAREST_SORTER = new Comparator() { - @Override public int compare(GroundItem n1, GroundItem n2) { return n1.distanceTo() - n2.distanceTo(); } - }; private static final Filter ALL_FILTER = new Filter() { @@ -60,6 +58,7 @@ public class GroundItems { } } } + return items.toArray(new GroundItem[items.size()]); } @@ -75,10 +74,12 @@ public class GroundItems { if (client == null) { client = Loader.getClient(); } + final Deque deque = client.getGroundItems()[Game.getPlane()][x][y]; if (deque == null) { return null; } + ArrayList list = new ArrayList<>(); final Node holder = deque.getHead(); Node curNode = holder.getNext(); @@ -88,10 +89,12 @@ public class GroundItems { list.add(new GroundItem(groundItem, x, y)); curNode = curNode.getNext(); } + return list.toArray(new GroundItem[list.size()]); } catch (Exception e) { e.printStackTrace(); } + return null; } @@ -125,6 +128,7 @@ public class GroundItems { if (objects == null || objects.length == 0) { return null; } + return objects[0]; } @@ -139,6 +143,7 @@ public class GroundItems { if (objects == null || objects.length == 0) { return null; } + return objects[0]; } @@ -151,6 +156,7 @@ public class GroundItems { public static final GroundItem[] getNearest(Filter filter) { final GroundItem[] objects = getGroundItems(filter); Arrays.sort(objects, NEAREST_SORTER); + return objects; } @@ -171,7 +177,6 @@ public class GroundItems { */ public static final GroundItem[] getNearest(final int... ids) { return getNearest(new Filter() { - @Override public boolean accept(GroundItem object) { for (final int id : ids) { @@ -179,9 +184,9 @@ public class GroundItems { return true; } } + return false; } - }); } @@ -199,7 +204,7 @@ public class GroundItems { EXAMINE(settings.get("menu_ground_item_examine_interaction")); - int actionId; + private int actionId; Option(int actionId) { this.actionId = actionId; diff --git a/src/main/java/org/rev317/min/api/methods/Items.java b/src/main/java/org/rev317/min/api/methods/Items.java index ff54b96..d1ba388 100644 --- a/src/main/java/org/rev317/min/api/methods/Items.java +++ b/src/main/java/org/rev317/min/api/methods/Items.java @@ -40,9 +40,11 @@ public class Items { return name; } } - } catch (MalformedURLException | ParseException ignored) { + } catch (MalformedURLException | ParseException e) { + e.printStackTrace(); } } + return null; } @@ -60,8 +62,10 @@ public class Items { return ids; } } - } catch (MalformedURLException | ParseException ignored) { + } catch (MalformedURLException | ParseException e) { + e.printStackTrace(); } + return new int[0]; } @@ -85,8 +89,10 @@ public class Items { return ids; } } - } catch (MalformedURLException | ParseException ignored) { + } catch (MalformedURLException | ParseException e) { + e.printStackTrace(); } + return new int[0]; } @@ -110,8 +116,10 @@ public class Items { return ids; } } - } catch (MalformedURLException | ParseException ignored) { + } catch (MalformedURLException | ParseException e) { + e.printStackTrace(); } + return new int[0]; } @@ -130,7 +138,8 @@ public class Items { return price; } } - } catch (MalformedURLException | ParseException ignored) { + } catch (MalformedURLException | ParseException e) { + e.printStackTrace(); } } return 0; @@ -181,7 +190,7 @@ public class Items { SEVENTH(settings.get("menu_item_seventh_interaction")), USE_WITH(settings.get("menu_item_seventh_interaction")); - int actionId; + private int actionId; Option(int actionId) { this.actionId = actionId; diff --git a/src/main/java/org/rev317/min/api/methods/Menu.java b/src/main/java/org/rev317/min/api/methods/Menu.java index 703a228..fd27426 100644 --- a/src/main/java/org/rev317/min/api/methods/Menu.java +++ b/src/main/java/org/rev317/min/api/methods/Menu.java @@ -108,6 +108,7 @@ public class Menu { actionId = 478; break; } + sendAction(actionId, character.getIndex(), 0, 0); } @@ -118,10 +119,8 @@ public class Menu { * @param action * @param interfaceParentId */ - public static void transformItem(Item item, Items.Option action, - int interfaceParentId) { - sendAction(action.getActionId(), item.getId() - 1, item.getSlot(), - interfaceParentId); + public static void transformItem(Item item, Items.Option action, int interfaceParentId) { + sendAction(action.getActionId(), item.getId() - 1, item.getSlot(), interfaceParentId); } /** @@ -132,8 +131,7 @@ public class Menu { * @param interfaceParentId * @deprecated */ - public static void transformItem(Item item, int actionIndex, - int interfaceParentId) { + public static void transformItem(Item item, int actionIndex, int interfaceParentId) { int actionId = Items.Option.TRANSFORM_FIRST.getActionId(); switch (actionIndex) { case 0: @@ -264,8 +262,7 @@ public class Menu { * @param item */ public static void drop(Item item) { - sendAction(settings.get("button_drop_item"), item.getId() - 1, item.getSlot(), - settings.get("inventory_index")); + sendAction(settings.get("button_drop_item"), item.getId() - 1, item.getSlot(), settings.get("inventory_index")); } /** @@ -323,7 +320,6 @@ public class Menu { } client.getMenuActionId()[index] = action; - client.doAction(index); } } diff --git a/src/main/java/org/rev317/min/api/methods/Npcs.java b/src/main/java/org/rev317/min/api/methods/Npcs.java index cd81490..e3efd79 100644 --- a/src/main/java/org/rev317/min/api/methods/Npcs.java +++ b/src/main/java/org/rev317/min/api/methods/Npcs.java @@ -54,6 +54,7 @@ public class Npcs { npcList.add(npc); } } + return npcList.toArray(new Npc[npcList.size()]); } @@ -77,6 +78,7 @@ public class Npcs { if (npcs == null || npcs.length == 0) { return null; } + return npcs[0]; } @@ -91,6 +93,7 @@ public class Npcs { if (npcs == null || npcs.length == 0) { return null; } + return npcs[0]; } @@ -103,6 +106,7 @@ public class Npcs { public static final Npc[] getNearest(final Filter filter) { final Npc[] npcs = getNpcs(filter); Arrays.sort(npcs, NEAREST_SORTER); + return npcs; } @@ -124,11 +128,13 @@ public class Npcs { } } } + return false; } }); Arrays.sort(npcs, NEAREST_SORTER); + return npcs; } @@ -168,7 +174,7 @@ public class Npcs { EXAMINE(settings.get("menu_character_examine")); - int actionId; + private int actionId; Option(int actionId) { this.actionId = actionId; diff --git a/src/main/java/org/rev317/min/api/methods/Players.java b/src/main/java/org/rev317/min/api/methods/Players.java index fb8a638..7dd6c66 100644 --- a/src/main/java/org/rev317/min/api/methods/Players.java +++ b/src/main/java/org/rev317/min/api/methods/Players.java @@ -43,7 +43,7 @@ public class Players { */ public static final Player[] getPlayers(final Filter filter) { final Client client = Loader.getClient(); - ArrayList playerList = new ArrayList(); + ArrayList playerList = new ArrayList<>(); final org.rev317.min.accessors.Player[] accPlayers = client.getPlayers(); for (int i = 0; i < accPlayers.length; i++) { if (accPlayers[i] == null) { @@ -54,6 +54,7 @@ public class Players { playerList.add(player); } } + return playerList.toArray(new Player[playerList.size()]); } @@ -116,7 +117,7 @@ public class Players { EXAMINE(settings.get("menu_character_examine")); - int actionId; + private int actionId; Option(int actionId) { this.actionId = actionId; diff --git a/src/main/java/org/rev317/min/api/methods/Prayer.java b/src/main/java/org/rev317/min/api/methods/Prayer.java index 182b339..9596d3c 100644 --- a/src/main/java/org/rev317/min/api/methods/Prayer.java +++ b/src/main/java/org/rev317/min/api/methods/Prayer.java @@ -52,6 +52,7 @@ public class Prayer { prayers.add(curse); } } + return prayers.toArray(new Book[prayers.size()]); } @@ -225,12 +226,12 @@ public class Prayer { } public interface Book { - public int getSetting(); + int getSetting(); - public int getAction(); + int getAction(); - public int getLevel(); + int getLevel(); - public String getName(); + String getName(); } } diff --git a/src/main/java/org/rev317/min/api/methods/SceneObjects.java b/src/main/java/org/rev317/min/api/methods/SceneObjects.java index 86bbc59..d760638 100644 --- a/src/main/java/org/rev317/min/api/methods/SceneObjects.java +++ b/src/main/java/org/rev317/min/api/methods/SceneObjects.java @@ -40,16 +40,17 @@ public class SceneObjects { * @return scene objects */ public static final SceneObject[] getSceneObjects(Filter filter) { - ArrayList sceneObjects = new ArrayList(); + ArrayList sceneObjects = new ArrayList<>(); for (int x = 0; x < 104; x++) { for (int y = 0; y < 104; y++) { - final SceneObject sceneObjectAtTile = getSceneObjectAtTile(x, y, true); + final SceneObject sceneObjectAtTile = getSceneObjectAtTile(x, y); if (sceneObjectAtTile != null && filter.accept(sceneObjectAtTile)) { sceneObjects.add(sceneObjectAtTile); } } } + return sceneObjects.toArray(new SceneObject[sceneObjects.size()]); } @@ -71,6 +72,7 @@ public class SceneObjects { public static final SceneObject[] getNearest(Filter filter) { final SceneObject[] objects = getSceneObjects(filter); Arrays.sort(objects, NEAREST_SORTER); + return objects; } @@ -99,6 +101,7 @@ public class SceneObjects { return true; } } + return false; } @@ -107,7 +110,6 @@ public class SceneObjects { public static final SceneObject getClosest(final int... ids) { SceneObject[] nearestObjects = getNearest(new Filter() { - @Override public boolean accept(SceneObject object) { for (final int id : ids) { @@ -115,34 +117,37 @@ public class SceneObjects { return true; } } + return false; } - }); if (nearestObjects == null || nearestObjects.length == 0) { return null; } + return nearestObjects[0]; } - private static SceneObject getSceneObjectAtTile(int x, int y, boolean useCached) { + private static SceneObject getSceneObjectAtTile(int x, int y) { Ground sceneTile = Loader.getClient().getScene().getGroundArray()[Game.getPlane()][x][y]; if (sceneTile == null) { return null; } + final SceneObjectTile[] interactiveObjects = sceneTile.getInteractiveObjects(); if (interactiveObjects != null) { for (final SceneObjectTile interactiveObject : interactiveObjects) { - // get top if (interactiveObject != null) { return new SceneObject(interactiveObject, SceneObject.TYPE_INTERACTIVE); } } } + SceneObjectTile sceneObjectTile = sceneTile.getWallObject(); if (sceneObjectTile != null) { return new SceneObject(sceneObjectTile, SceneObject.TYPE_WALL); } + return null; } @@ -152,15 +157,16 @@ public class SceneObjects { * @return every loaded scene object in game */ public static final SceneObject[] getAllSceneObjects() { - ArrayList sceneObjects = new ArrayList(); + ArrayList sceneObjects = new ArrayList<>(); for (int x = 0; x < 104; x++) { for (int y = 0; y < 104; y++) { - final Collection sceneObjectsAtTile = getSceneObjectsAtTile(x, y, true); + final Collection sceneObjectsAtTile = getSceneObjectsAtTile(x, y); if (sceneObjectsAtTile != null && !sceneObjectsAtTile.isEmpty()) { sceneObjects.addAll(sceneObjectsAtTile); } } } + return sceneObjects.toArray(new SceneObject[sceneObjects.size()]); } @@ -169,10 +175,9 @@ public class SceneObjects { * * @param x * @param y - * @param useCached * @return array of sceneobjects, or null if there aren't any */ - public static final Collection getSceneObjectsAtTile(int x, int y, boolean useCached) { + public static final Collection getSceneObjectsAtTile(int x, int y) { Ground sceneTile = Loader.getClient().getScene().getGroundArray()[Game.getPlane()][x][y]; ArrayList sceneObjects = null; if (sceneTile != null) { @@ -219,9 +224,11 @@ public class SceneObjects { sceneObjects.add(new SceneObject(sceneObjectTile, SceneObject.TYPE_GROUNDITEM)); } } + return sceneObjects; } + public enum Option { FIRST(settings.get("menu_scene_object_first_interaction")), TALK_TO(settings.get("menu_scene_object_first_interaction")), @@ -253,7 +260,7 @@ public class SceneObjects { EXAMINE(settings.get("menu_scene_object_examine")); - int actionId; + private int actionId; Option(int actionId) { this.actionId = actionId; @@ -263,6 +270,4 @@ public class SceneObjects { return actionId; } } - } - diff --git a/src/main/java/org/rev317/min/api/methods/Skill.java b/src/main/java/org/rev317/min/api/methods/Skill.java index 8bc08d1..8b1eca7 100644 --- a/src/main/java/org/rev317/min/api/methods/Skill.java +++ b/src/main/java/org/rev317/min/api/methods/Skill.java @@ -66,6 +66,7 @@ public enum Skill { if (level > 99 || level < 1) { return 0; } + return EXPERIENCE[level]; } @@ -81,6 +82,7 @@ public enum Skill { return i; } } + return 1; } @@ -95,6 +97,7 @@ public enum Skill { if (level >= 99 || level < 1) { return 0; } + return EXPERIENCE[(level + 1)] - getCurrentExperience(index); } @@ -111,11 +114,12 @@ public enum Skill { || nextLevel < 1) { return 0; } + return (int) (100f * ((float) getCurrentExperience(index) / (float) EXPERIENCE[nextLevel])); } /** - * @Deprecated use Skill.ordinal() instead Returns the skill's index. + * @deprecated use Skill.ordinal() instead Returns the skill's index. */ public int getIndex() { return ordinal(); @@ -125,22 +129,21 @@ public enum Skill { * Returns the name of the skill. */ public final String getName() { - return Character.toUpperCase(name().charAt(0)) - + name().toLowerCase().substring(1); + return Character.toUpperCase(name().charAt(0)) + name().toLowerCase().substring(1); } /** * Returns the current experience. */ public final int getExperience() { - return Skill.getCurrentExperience(this.getIndex()); + return Skill.getCurrentExperience(this.ordinal()); } /** * Returns the real level. */ public final int getRealLevel() { - return Skill.getRealLevel(this.getIndex()); + return Skill.getRealLevel(this.ordinal()); } /** @@ -148,21 +151,21 @@ public enum Skill { * stat; it will return the buffed level.) Done by Bears */ public final int getLevel() { - return Skill.getCurrentLevel(this.getIndex()); + return Skill.getCurrentLevel(this.ordinal()); } /** * Returns the remaining experience until the next level. */ public final int getRemaining() { - return Skill.getRemainingExperience(this.getIndex()); + return Skill.getRemainingExperience(this.ordinal()); } /** * Returns the percentage until the next level. */ public final int getPercentage() { - return Skill.getPercentToNextLevel(this.getIndex()); + return Skill.getPercentToNextLevel(this.ordinal()); } /** @@ -170,7 +173,6 @@ public enum Skill { */ @Override public final String toString() { - return "Skill: [" + this.getName() + ": " + this.getLevel() + " / " - + this.getRealLevel() + "]"; + return "Skill: [" + this.getName() + ": " + this.getLevel() + " / " + this.getRealLevel() + "]"; } } diff --git a/src/main/java/org/rev317/min/api/methods/Walking.java b/src/main/java/org/rev317/min/api/methods/Walking.java index 4cc026a..2ae0906 100644 --- a/src/main/java/org/rev317/min/api/methods/Walking.java +++ b/src/main/java/org/rev317/min/api/methods/Walking.java @@ -42,6 +42,7 @@ public class Walking { if (tilePath.hasReached()) { return true; } + tilePath.traverse(); return false; } @@ -61,8 +62,7 @@ public class Walking { tile = new Tile((loc.getX() + tile.getX()) / 2, (loc.getY() + tile.getY()) / 2); } + return null; } - } - diff --git a/src/main/java/org/rev317/min/api/wrappers/Area.java b/src/main/java/org/rev317/min/api/wrappers/Area.java index 4a23fb8..d0c0972 100644 --- a/src/main/java/org/rev317/min/api/wrappers/Area.java +++ b/src/main/java/org/rev317/min/api/wrappers/Area.java @@ -37,8 +37,10 @@ public class Area { */ public Tile[] getPoints() { Tile[] tiles = new Tile[p.npoints]; - for (int i = 0; i < tiles.length; i++) + for (int i = 0; i < tiles.length; i++) { tiles[i] = new Tile(p.xpoints[i], p.ypoints[i]); + } + return tiles; } @@ -76,6 +78,7 @@ public class Area { } } } + return t.toArray(new Tile[t.size()]); } @@ -107,7 +110,7 @@ public class Area { result = !result; } } + return result; } - } diff --git a/src/main/java/org/rev317/min/api/wrappers/Character.java b/src/main/java/org/rev317/min/api/wrappers/Character.java index d918f29..e94f345 100644 --- a/src/main/java/org/rev317/min/api/wrappers/Character.java +++ b/src/main/java/org/rev317/min/api/wrappers/Character.java @@ -142,6 +142,7 @@ public class Character implements Locatable { return Players.getMyPlayer(); } } + return null; } @@ -161,26 +162,32 @@ public class Character implements Locatable { result = prime * result + ((accessor == null) ? 0 : accessor.hashCode()); result = prime * result + index; + return result; } @Override public boolean equals(Object obj) { - if (this == obj) + if (this == obj) { return true; - if (obj == null) + } + if (obj == null) { return false; - if (getClass() != obj.getClass()) + } + if (getClass() != obj.getClass()) { return false; + } + Character other = (Character) obj; if (accessor == null) { - if (other.accessor != null) + if (other.accessor != null) { return false; - } else if (!accessor.equals(other.accessor)) + } + } else if (!accessor.equals(other.accessor)) { return false; - if (index != other.index) - return false; - return true; + } + + return index == other.index; } public org.rev317.min.accessors.Character getAccessor() { diff --git a/src/main/java/org/rev317/min/api/wrappers/GroundItem.java b/src/main/java/org/rev317/min/api/wrappers/GroundItem.java index ca8e7c6..f3fe9e8 100644 --- a/src/main/java/org/rev317/min/api/wrappers/GroundItem.java +++ b/src/main/java/org/rev317/min/api/wrappers/GroundItem.java @@ -100,5 +100,4 @@ public class GroundItem implements Locatable { public RefClass getRefClass() { return new RefClass(this.accessor); } - } diff --git a/src/main/java/org/rev317/min/api/wrappers/Npc.java b/src/main/java/org/rev317/min/api/wrappers/Npc.java index f267692..f4db678 100644 --- a/src/main/java/org/rev317/min/api/wrappers/Npc.java +++ b/src/main/java/org/rev317/min/api/wrappers/Npc.java @@ -42,5 +42,4 @@ public final class Npc extends Character { public void interact(Npcs.Option option) { Menu.interact(this, option); } - } diff --git a/src/main/java/org/rev317/min/api/wrappers/NpcDef.java b/src/main/java/org/rev317/min/api/wrappers/NpcDef.java index 4c7e9f3..caacffd 100644 --- a/src/main/java/org/rev317/min/api/wrappers/NpcDef.java +++ b/src/main/java/org/rev317/min/api/wrappers/NpcDef.java @@ -23,12 +23,13 @@ public class NpcDef { } catch (Exception e) { e.printStackTrace(); } + return -1; } private long getLongId() { if (accessor != null) { - if (accessor.getId() < Integer.MAX_VALUE) { + if (accessor.getLongId() < Integer.MAX_VALUE) { return (long) accessor.getId(); } } @@ -45,5 +46,4 @@ public class NpcDef { public RefClass getRefClass() { return new RefClass(this.accessor); } - } diff --git a/src/main/java/org/rev317/min/api/wrappers/SceneObject.java b/src/main/java/org/rev317/min/api/wrappers/SceneObject.java index 5060778..9a63c58 100644 --- a/src/main/java/org/rev317/min/api/wrappers/SceneObject.java +++ b/src/main/java/org/rev317/min/api/wrappers/SceneObject.java @@ -121,5 +121,4 @@ public class SceneObject implements Locatable { public String toString() { return String.format("[ID: %d, X: %d, Y: %d]", getId(), getLocalRegionX(), getLocalRegionY()); } - } diff --git a/src/main/java/org/rev317/min/api/wrappers/Tile.java b/src/main/java/org/rev317/min/api/wrappers/Tile.java index 0ca9382..561cf75 100644 --- a/src/main/java/org/rev317/min/api/wrappers/Tile.java +++ b/src/main/java/org/rev317/min/api/wrappers/Tile.java @@ -116,6 +116,7 @@ public final class Tile implements TileFlags, Locatable { hash = 31 * hash + this.x; hash = 31 * hash + this.y; hash = 31 * hash + this.z; + return hash; } @@ -171,5 +172,4 @@ public final class Tile implements TileFlags, Locatable { public Tile getLocation() { return this; } - } diff --git a/src/main/java/org/rev317/min/api/wrappers/TilePath.java b/src/main/java/org/rev317/min/api/wrappers/TilePath.java index 52e5cce..5c587b3 100644 --- a/src/main/java/org/rev317/min/api/wrappers/TilePath.java +++ b/src/main/java/org/rev317/min/api/wrappers/TilePath.java @@ -27,11 +27,12 @@ public class TilePath { */ public final Tile getNextTile() { Tile next = null; - for (int x = 0; x < tiles.length; x++) { - if (tiles[x].isOnMinimap()) { - next = tiles[x]; + for (Tile tile : tiles) { + if (tile.isOnMinimap()) { + next = tile; } } + return next; } @@ -61,6 +62,7 @@ public class TilePath { if (next == null) { return; } + next.walkTo(); } @@ -71,10 +73,10 @@ public class TilePath { */ public final TilePath reverse() { Tile[] newTiles = new Tile[tiles.length]; - for (int i = 0; i < tiles.length; i++) + for (int i = 0; i < tiles.length; i++) { newTiles[i] = tiles[tiles.length - i - 1]; + } + return new TilePath(newTiles); } - } - diff --git a/src/main/java/org/rev317/min/callback/MenuAction.java b/src/main/java/org/rev317/min/callback/MenuAction.java index fdf0947..c174f98 100644 --- a/src/main/java/org/rev317/min/callback/MenuAction.java +++ b/src/main/java/org/rev317/min/callback/MenuAction.java @@ -8,9 +8,7 @@ import org.rev317.min.debug.DActions; import org.rev317.min.script.ScriptEngine; /** - * @author Everel - * @author JKetelaar - * @author Matt123337 + * @author Everel, JKetelaar, Matt123337 */ public class MenuAction { diff --git a/src/main/java/org/rev317/min/callback/MessageCallback.java b/src/main/java/org/rev317/min/callback/MessageCallback.java index 946c7b4..00b2b56 100644 --- a/src/main/java/org/rev317/min/callback/MessageCallback.java +++ b/src/main/java/org/rev317/min/callback/MessageCallback.java @@ -10,5 +10,4 @@ public class MessageCallback { final MessageEvent messageEvent = new MessageEvent(type, name, message); ScriptEngine.getInstance().dispatch(messageEvent); } - } diff --git a/src/main/java/org/rev317/min/debug/DActions.java b/src/main/java/org/rev317/min/debug/DActions.java index acd7794..a36ff63 100644 --- a/src/main/java/org/rev317/min/debug/DActions.java +++ b/src/main/java/org/rev317/min/debug/DActions.java @@ -24,5 +24,4 @@ public class DActions extends AbstractDebugger { public void toggle() { enabled = !enabled; } - } diff --git a/src/main/java/org/rev317/min/debug/DAnimation.java b/src/main/java/org/rev317/min/debug/DAnimation.java index 555b261..2e57168 100644 --- a/src/main/java/org/rev317/min/debug/DAnimation.java +++ b/src/main/java/org/rev317/min/debug/DAnimation.java @@ -23,5 +23,4 @@ public class DAnimation extends AbstractDebugger { public void toggle() { enabled = !enabled; } - } diff --git a/src/main/java/org/rev317/min/debug/DBank.java b/src/main/java/org/rev317/min/debug/DBank.java index c08991f..234c17c 100644 --- a/src/main/java/org/rev317/min/debug/DBank.java +++ b/src/main/java/org/rev317/min/debug/DBank.java @@ -23,6 +23,7 @@ public class DBank extends AbstractDebugger { if (!Bank.isOpen()) { return; } + Item[] items; if ((items = Bank.getBankItems()) != null) { for (int i = items.length - 1; i >= 0; i--) { @@ -32,5 +33,4 @@ public class DBank extends AbstractDebugger { } } } - } diff --git a/src/main/java/org/rev317/min/debug/DCollisionFlags.java b/src/main/java/org/rev317/min/debug/DCollisionFlags.java index 48db0fe..9a79012 100644 --- a/src/main/java/org/rev317/min/debug/DCollisionFlags.java +++ b/src/main/java/org/rev317/min/debug/DCollisionFlags.java @@ -35,5 +35,4 @@ public class DCollisionFlags extends AbstractDebugger { public void toggle() { enabled = !enabled; } - } diff --git a/src/main/java/org/rev317/min/debug/DGroundItems.java b/src/main/java/org/rev317/min/debug/DGroundItems.java index 3095bf9..aad7ee2 100644 --- a/src/main/java/org/rev317/min/debug/DGroundItems.java +++ b/src/main/java/org/rev317/min/debug/DGroundItems.java @@ -28,5 +28,4 @@ public class DGroundItems extends AbstractDebugger { System.out.println("ID: " + item.getId() + " Location: " + item.getLocation()); } } - } diff --git a/src/main/java/org/rev317/min/debug/DInterfaces.java b/src/main/java/org/rev317/min/debug/DInterfaces.java index a4cd85d..2645d5a 100644 --- a/src/main/java/org/rev317/min/debug/DInterfaces.java +++ b/src/main/java/org/rev317/min/debug/DInterfaces.java @@ -26,5 +26,4 @@ public class DInterfaces extends AbstractDebugger { public void toggle() { enabled = !enabled; } - } diff --git a/src/main/java/org/rev317/min/debug/DInventory.java b/src/main/java/org/rev317/min/debug/DInventory.java index 5ec332d..2a4a4bf 100644 --- a/src/main/java/org/rev317/min/debug/DInventory.java +++ b/src/main/java/org/rev317/min/debug/DInventory.java @@ -47,6 +47,7 @@ public class DInventory extends AbstractDebugger { return item.getId(); } } + return 0; } } diff --git a/src/main/java/org/rev317/min/debug/DMap.java b/src/main/java/org/rev317/min/debug/DMap.java index bd51d3b..a7d9cbe 100644 --- a/src/main/java/org/rev317/min/debug/DMap.java +++ b/src/main/java/org/rev317/min/debug/DMap.java @@ -27,5 +27,4 @@ public class DMap extends AbstractDebugger { public void toggle() { enabled = !enabled; } - } diff --git a/src/main/java/org/rev317/min/debug/DMouse.java b/src/main/java/org/rev317/min/debug/DMouse.java index 39459e6..730ddc7 100644 --- a/src/main/java/org/rev317/min/debug/DMouse.java +++ b/src/main/java/org/rev317/min/debug/DMouse.java @@ -24,5 +24,4 @@ public class DMouse extends AbstractDebugger { public void toggle() { enabled = !enabled; } - } diff --git a/src/main/java/org/rev317/min/debug/DNpcs.java b/src/main/java/org/rev317/min/debug/DNpcs.java index 802d979..cd6a6ea 100644 --- a/src/main/java/org/rev317/min/debug/DNpcs.java +++ b/src/main/java/org/rev317/min/debug/DNpcs.java @@ -23,7 +23,5 @@ public class DNpcs extends AbstractDebugger { for (Npc n : Npcs.getNearest()) { System.out.println("ID: " + n.getDef().getId() + " Distance: " + n.distanceTo() + " Location: " + n.getLocation().toString()); } - } - } diff --git a/src/main/java/org/rev317/min/debug/DPlayers.java b/src/main/java/org/rev317/min/debug/DPlayers.java index ed7f25d..859416a 100644 --- a/src/main/java/org/rev317/min/debug/DPlayers.java +++ b/src/main/java/org/rev317/min/debug/DPlayers.java @@ -24,6 +24,5 @@ public class DPlayers extends AbstractDebugger { @Override public void paint(Graphics graphics) { - } } diff --git a/src/main/java/org/rev317/min/debug/DSceneObjects.java b/src/main/java/org/rev317/min/debug/DSceneObjects.java index d56163e..cba2291 100644 --- a/src/main/java/org/rev317/min/debug/DSceneObjects.java +++ b/src/main/java/org/rev317/min/debug/DSceneObjects.java @@ -31,5 +31,4 @@ public class DSceneObjects extends AbstractDebugger { " Distance: " + objects[i].distanceTo()); } } - } diff --git a/src/main/java/org/rev317/min/script/ScriptEngine.java b/src/main/java/org/rev317/min/script/ScriptEngine.java index 5090cb7..21776e1 100644 --- a/src/main/java/org/rev317/min/script/ScriptEngine.java +++ b/src/main/java/org/rev317/min/script/ScriptEngine.java @@ -20,7 +20,7 @@ import java.util.HashMap; * @author matt123337 */ public class ScriptEngine { - private static HashMap instances = new HashMap(); + private static HashMap instances = new HashMap<>(); private ArrayList mouseListeners; private ArrayList mouseMotionListeners; private ArrayList messageListeners; @@ -29,10 +29,10 @@ public class ScriptEngine { private Script script = null; private ScriptEngine() { - this.mouseListeners = new ArrayList(); - this.mouseMotionListeners = new ArrayList(); - this.messageListeners = new ArrayList(); - this.actionListeners = new ArrayList(); + this.mouseListeners = new ArrayList<>(); + this.mouseMotionListeners = new ArrayList<>(); + this.messageListeners = new ArrayList<>(); + this.actionListeners = new ArrayList<>(); instances.put(Context.getInstance(), this); } @@ -103,6 +103,7 @@ public class ScriptEngine { if (script instanceof Paintable) { Context.getInstance().removePaintable((Paintable) script); } + this.script = null; } @@ -134,6 +135,7 @@ public class ScriptEngine { if (!(event instanceof MouseEvent)) { return; } + final MouseEvent e = (MouseEvent) event; for (final MouseListener m : mouseListeners) { switch (e.getID()) { @@ -176,6 +178,4 @@ public class ScriptEngine { a.onGameAction(event); } } - } - diff --git a/src/main/java/org/rev317/min/ui/BotMenu.java b/src/main/java/org/rev317/min/ui/BotMenu.java index bec560f..a8ff048 100644 --- a/src/main/java/org/rev317/min/ui/BotMenu.java +++ b/src/main/java/org/rev317/min/ui/BotMenu.java @@ -87,6 +87,7 @@ public class BotMenu implements ActionListener { private JMenuItem newItem(String name) { JMenuItem item = new JCheckBoxMenuItem(name); item.addActionListener(this); + return item; } @@ -94,5 +95,4 @@ public class BotMenu implements ActionListener { public void actionPerformed(ActionEvent e) { Context.getInstance().getPaintDebugger().toggle(e.getActionCommand()); } - } From 7767ae1b859dd6aef4884cac274116d1acf6ce4e Mon Sep 17 00:00:00 2001 From: Emma Stone Date: Fri, 3 Feb 2017 16:10:47 +0000 Subject: [PATCH 2/2] [CLEANUP] Readded the return --- src/main/java/org/rev317/min/accessors/Interface.java | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/main/java/org/rev317/min/accessors/Interface.java b/src/main/java/org/rev317/min/accessors/Interface.java index f1969ad..bc19c43 100644 --- a/src/main/java/org/rev317/min/accessors/Interface.java +++ b/src/main/java/org/rev317/min/accessors/Interface.java @@ -6,6 +6,9 @@ public interface Interface { int[] getStackSizes(); + /** + * @return long version of #getStackSizes + */ long[] getLongStackSizes(); String getMessage();