Add missing 'enabled' state to various AbstractDebuggers + only send debug info if when enabled

src/main/java/org/rev317/min/debug/DBank.java

@@ -1,6 +1,7 @@
 package org.rev317.min.debug;
 
 import org.parabot.core.paint.AbstractDebugger;
+import org.parabot.core.ui.Logger;
 import org.rev317.min.api.methods.Bank;
 import org.rev317.min.api.wrappers.Item;
 
@@ -8,28 +9,38 @@ import java.awt.*;
 
 public class DBank extends AbstractDebugger {
 
+    private boolean enabled;
+
     @Override
     public void paint(Graphics g) {
 
+        if (!Bank.isOpen()) {
+            return;
+        }
     }
 
     @Override
     public boolean isEnabled() {
-        return false;
+        return enabled;
     }
 
     @Override
     public void toggle() {
-        if (!Bank.isOpen()) {
+        enabled = !enabled;
-            return;
-        }
 
-        Item[] items;
+        if (enabled) {
-        if ((items = Bank.getBankItems()) != null) {
+            if (!Bank.isOpen()) {
-            for (int i = items.length - 1; i >= 0; i--) {
+                Logger.addMessage("The bank is not open, cannot debug.");
-                System.out.println("ID: " + items[i].getId()
+                return;
-                        + " Stack: " + items[i].getStackSize()
+            }
-                        + " Slot: " + items[i].getSlot());
+
+            Item[] items;
+            if ((items = Bank.getBankItems()) != null) {
+                for (int i = items.length - 1; i >= 0; i--) {
+                    System.out.println("ID: " + items[i].getId()
+                            + " Stack: " + items[i].getStackSize()
+                            + " Slot: " + items[i].getSlot());
+                }
             }
         }
     }

src/main/java/org/rev317/min/debug/DGroundItems.java

@@ -1,31 +1,49 @@
 package org.rev317.min.debug;
 
+import org.parabot.core.Context;
 import org.parabot.core.paint.AbstractDebugger;
+import org.parabot.core.paint.PaintDebugger;
+import org.parabot.core.ui.Logger;
 import org.rev317.min.api.methods.GroundItems;
+import org.rev317.min.api.methods.Interfaces;
 import org.rev317.min.api.wrappers.GroundItem;
 
 import java.awt.*;
 
 public class DGroundItems extends AbstractDebugger {
 
+    private boolean enabled;
+    private long lastCheck = System.currentTimeMillis();
+    private int cached;
+
     @Override
     public void paint(Graphics g) {
-
+        if (System.currentTimeMillis() - lastCheck > 1000L) {
+            lastCheck = System.currentTimeMillis();
+            cached = GroundItems.getGroundItems().length;
+        }
+        PaintDebugger p = Context.getInstance().getPaintDebugger();
+        p.addLine("Close Ground Items: " + cached);
     }
 
     @Override
     public boolean isEnabled() {
-        return false;
+        return enabled;
     }
 
     @Override
     public void toggle() {
-        GroundItem[] items = GroundItems.getNearest();
+        enabled = !enabled;
-        if (items == null || items.length == 0) {
+
-            return;
+        if (enabled) {
-        }
+            GroundItem[] items = GroundItems.getNearest();
-        for (GroundItem item : items) {
+            if (items == null || items.length == 0) {
-            System.out.println("ID: " + item.getId() + " Location: " + item.getLocation());
+                Logger.addMessage("There are no Ground Items close to you.");
+                return;
+            }
+            for (GroundItem item : items) {
+                System.out.println("ID: " + item.getId() + " Location: " + item.getLocation());
+            }
         }
     }
 }

src/main/java/org/rev317/min/debug/DNpcs.java

@@ -1,27 +1,47 @@
 package org.rev317.min.debug;
 
+import org.parabot.core.Context;
 import org.parabot.core.paint.AbstractDebugger;
+import org.parabot.core.paint.PaintDebugger;
+import org.parabot.core.ui.Logger;
 import org.rev317.min.api.methods.Npcs;
+import org.rev317.min.api.methods.Players;
 import org.rev317.min.api.wrappers.Npc;
 
 import java.awt.*;
 
 public class DNpcs extends AbstractDebugger {
 
+    private boolean enabled;
+    private long lastCheck = System.currentTimeMillis();
+    private int cached;
+
     @Override
     public void paint(Graphics g) {
-
+        if (System.currentTimeMillis() - lastCheck > 1000L) {
+            lastCheck = System.currentTimeMillis();
+            cached = Npcs.getNearest().length;
+        }
+        PaintDebugger p = Context.getInstance().getPaintDebugger();
+        p.addLine("Close NPCs: " + cached);
     }
 
     @Override
     public boolean isEnabled() {
-        return false;
+        return enabled;
     }
 
     @Override
     public void toggle() {
-        for (Npc n : Npcs.getNearest()) {
+        enabled = !enabled;
-            System.out.println("ID: " + n.getDef().getId() + " Distance: " + n.distanceTo() + " Location: " + n.getLocation().toString());
+        if (enabled) {
+            if (Npcs.getNearest().length == 0) {
+                Logger.addMessage("There are no NPCs close to you.");
+                return;
+            }
+            for (Npc n : Npcs.getNearest()) {
+                System.out.println("ID: " + n.getDef().getId() + " Distance: " + n.distanceTo() + " Location: " + n.getLocation().toString());
+            }
         }
     }
 }

src/main/java/org/rev317/min/debug/DPlayers.java

@@ -1,6 +1,10 @@
 package org.rev317.min.debug;
 
+import org.parabot.core.Context;
 import org.parabot.core.paint.AbstractDebugger;
+import org.parabot.core.paint.PaintDebugger;
+import org.parabot.core.ui.Logger;
+import org.rev317.min.api.methods.GroundItems;
 import org.rev317.min.api.methods.Players;
 import org.rev317.min.api.wrappers.Player;
 
@@ -10,19 +14,37 @@ import java.awt.*;
  * @author JKetelaar
  */
 public class DPlayers extends AbstractDebugger {
+
+    private boolean enabled = false;
+    private long lastCheck = System.currentTimeMillis();
+    private int cachedClosePlayerCount;
+
     @Override
     public void toggle() {
-        for (Player player : Players.getNearest()) {
+        enabled = !enabled;
-            System.out.println("Username: " + player.getName() + " Index: " + player.getIndex() + " Distance: " + player.distanceTo() + " Location: " + player.getLocation().toString());
+        if (enabled) {
+            if (Players.getNearest().length == 0) {
+                Logger.addMessage("There are no Players close to you.");
+                return;
+            }
+            for (Player player : Players.getNearest()) {
+                System.out.println("Username: " + player.getName() + " Index: " + player.getIndex() + " Distance: " + player.distanceTo() + " Location: " + player.getLocation().toString());
+            }
         }
     }
 
     @Override
     public boolean isEnabled() {
-        return false;
+        return enabled;
     }
 
     @Override
     public void paint(Graphics graphics) {
+        if (System.currentTimeMillis() - lastCheck > 1000L) {
+            lastCheck = System.currentTimeMillis();
+            cachedClosePlayerCount = Players.getNearest().length;
+        }
+        PaintDebugger p = Context.getInstance().getPaintDebugger();
+        p.addLine("Close Players: " + cachedClosePlayerCount);
     }
 }

src/main/java/org/rev317/min/debug/DSceneObjects.java

@@ -1,6 +1,10 @@
 package org.rev317.min.debug;
 
+import org.parabot.core.Context;
 import org.parabot.core.paint.AbstractDebugger;
+import org.parabot.core.paint.PaintDebugger;
+import org.parabot.core.ui.Logger;
+import org.rev317.min.api.methods.GroundItems;
 import org.rev317.min.api.methods.SceneObjects;
 import org.rev317.min.api.wrappers.SceneObject;
 
@@ -8,28 +12,43 @@ import java.awt.*;
 
 public class DSceneObjects extends AbstractDebugger {
 
+    private boolean enabled;
+    private long lastCheck = System.currentTimeMillis();
+    private int cached;
+
     @Override
     public void paint(Graphics g) {
+        if (System.currentTimeMillis() - lastCheck > 1000L) {
+            lastCheck = System.currentTimeMillis();
+            cached = SceneObjects.getNearest().length;
+        }
+        PaintDebugger p = Context.getInstance().getPaintDebugger();
+        p.addLine("Close SceneObjects: " + cached);
     }
 
     @Override
     public boolean isEnabled() {
-        return false;
+        return enabled;
     }
 
     @Override
     public void toggle() {
-        SceneObject[] objects = SceneObjects.getNearest();
+        enabled = !enabled;
-        if (objects == null || objects.length == 0) {
-            return;
-        }
 
-        for (int i = objects.length - 1; i >= 0; i--) {
+        if (enabled) {
-            System.out.println(
+            SceneObject[] objects = SceneObjects.getNearest();
-                    " ID: " + objects[i].getId() +
+            if (objects == null || objects.length == 0) {
-                            " UID: " + objects[i].getHash() +
+                Logger.addMessage("There are no GameObjects around you.");
-                            " Location: " + objects[i].getLocation() +
+                return;
-                            " Distance: " + objects[i].distanceTo());
+            }
+
+            for (int i = objects.length - 1; i >= 0; i--) {
+                System.out.println(
+                        " ID: " + objects[i].getId() +
+                                " UID: " + objects[i].getHash() +
+                                " Location: " + objects[i].getLocation() +
+                                " Distance: " + objects[i].distanceTo());
+            }
         }
     }
 }

src/main/java/org/rev317/min/debug/DSkills.java

@@ -7,16 +7,21 @@ import java.awt.*;
 
 public class DSkills extends AbstractDebugger {
 
+    private boolean enabled;
+
     @Override
     public void toggle() {
-        for (Skill skill : Skill.values()){
+        enabled = !enabled;
-            System.out.println(String.format("%s level: %d / %d", skill.getName(), skill.getLevel(), skill.getRealLevel()));
+        if (enabled) {
+            for (Skill skill : Skill.values()) {
+                System.out.println(String.format("%s level: %d / %d (exp: %s)", skill.getName(), skill.getLevel(), skill.getRealLevel(), skill.getExperience()));
+            }
         }
     }
 
     @Override
     public boolean isEnabled() {
-        return false;
+        return enabled;
     }
 
     @Override