Khaleesi
Script Officer
-
Joined
-
Last visited
Everything posted by Khaleesi
-
Ok so for this function: @Override public boolean canProcess() throws InterruptedException { return script.getGroundItems().closest(GroundID) != null && script.getGroundItems().closest(GroundID).isOnScreen(); //script.getGroundItems().closest(GroundID).isVisible() ; } You are loading a new item 2 times, if you would be walking it could be loading different items or even return Null on the isOnScreen method. + You should check in here if your inventory is full or not. A better way is to load it once and check on it. is this @Override public boolean canProcess() throws InterruptedException { if(script.getInvenotry().isFull){ return false; } grountItem item = script.getGroundItems().closest(GroundID); return item != null && item.isOnScreen(); } __________________________________________________________________________________________________________________________________________________________________ For the next Part: @Override public void process() throws InterruptedException { script.currentState = Status.LOOTING; GroundItem loot = script.getGroundItems().closest(GroundID); //if (GroundID != null && !script.getInventory().isFull()) if (!script.inventory.isFull()){ loot.interact("Take"); sleep(random(999,1777)); } } You are not Null checking if there is loot on the ground. (Commented out) Also move the inventory check to canProcess, else it might get stuck in trying to loot but your inventory is full so it will never loot. @Override public void process() throws InterruptedException { script.currentState = Status.LOOTING; GroundItem loot = script.getGroundItems().closest(GroundID); if (loot != null){ loot.interact("Take"); sleep(random(999,1777)); } } Try to consider using Conditional sleeps instead of a random sleep. This might click the loot multiple times for no reason. You could do this instead: @Override public void process() throws InterruptedException { script.currentState = Status.LOOTING; GroundItem loot = script.getGroundItems().closest(GroundID); if (loot != null){ if(loot.interact("Take")){ int prevAmount = script.getInventory().getAmount(loot.getName()); new ConditionalSleep(random(5000, 7500)) { @Override public boolean condition() throws InterruptedException { return script.getInventory().getAmount(loot.getName()) > prevAmount; } }.sleep(); } } } I Hope this helped you ^^ -
-
Mirror does exactly the same as the default client
-
Not giving trials on this script, sorry.
-
Ya zoom out almost to the max Just so it clearly sees all objects ^^ Sure! Enjoy
-
-
Awesome! sorry for teh delay Enjoy! I'm not using imgur Can you empty them once by hand before starting the script? Enjoy!
-
That's the plan indeed Enjoy!
-
Enjoy!
-
Not giving trials on this
-
-
This script is the most flawless script ever made, over 5000+ accs with 99 firemaking ^^ Just Zoom out and everything is fixed, That's all you gotta do (Or read the FAQ next time :D)
-
Could even use the bank right above the new dungeon as option, I guess it won't matter that much ^^ Will take a look at bothi options asap
-
-
Sure! Enjoy your trial
-
-
Enjoy! Ya mirror mode is worth it, it helps a lot to avoid banns
-
I will take a look at it!
-
-
Welcome! Goodluck with your plans! If you have any questions, feel free to pm me on discord ^^
-
-
Let me know how it works for you ^^
-
-
-
Tried restarting the whole client?
