Rays

Personally for me, the methods provided with the API already does a good job on this. It's more of a hassle when other people are reading your script though. Didn't notice your use of the sleep - my bad. Look good.

Basically, OSBot will spas out if you forget to check for null.

You should definitely look into Conditional Sleeps some more. Also this: private boolean atBank() { return Banks.FALADOR_EAST.contains(myPlayer()); } private boolean atCowField() { return cowField.contains(myPlayer()); } private boolean atGate() { return gateArea.contains(myPlayer()); } private void walkToBank() { getWalking().walk(Banks.FALADOR_EAST); } private void walkToCows() { getWalking().walk(cowField); } private void walkToGate() { getWalking().walk(gateArea); } private RS2Object fieldGate() { return getObjects().closest("Gate"); } private boolean gateClosed() { return fieldGate().hasAction("Open"); }  private boolean gateExists() { return fieldGate() != null; } These functions are essentially waste of space since all they do is return a value of another function.

I'd probably not bot it again if you care about the account.

Jokes on you guys; he's obviously trolling. Right?

Welcome to OSBot!