Mysteryy

What you posted wont even paint anything. It looks like you had the right idea, but the syntax wasnt there.

Ill hint what you need to do, you will have to put in the effort of coding it. -When the mouse is clicked, check if the mouse is in the paint area. -If the mouse is in the paint area, update the x, y values of the paint to the mouse x, y coordinates (in the onPaint method, use variables for this). This will move the paint around to the mouse position while the mouse is being held down. -when the mouse is released, stop moving the paint. You should think of a logical way to do it before just trying to code it.

Rs. Hue hue

http://imgur.com/FnynuaL.png

Rekt

You can add a check. There is a method to get your current minimap destination. For example, only click the next tile of you are < 4 tiles from the current destination. I made my own walker and used something like this in it.

Seems legit

What am I looking at.

Of course the first one spam clicks. You are telling your script to walk to x no matter what. If you added a check in there saying to only walk to x if your not already at x, then it would work. As for the second one, I'm guessing it's something with the code in the statement. Since you never post your code we can't help. ^_^

case SPIN: if (!player.isAnimating()){ if (SPIN_AREA.contains(player) && (inventory.contains("Flax"))){ objects.closest("Spinning wheel").interact("Spin"); sleep(random(200,600)); Inter.interactWithChild(459, 91, "Make X"); sleep(random(1000, 1500)); RS2Interface Inter = interfaces.get(118); if (Inter.getMessage(118) != null){ keyboard.typeString("99"); sleep(random(2000,4000)); }else{ sleep(random(60000, 62000)); } } } break; Jesus i cant read OP Holy crap your sleeping for a minute l0l

Not working how? Not working is pretty general and not helpful.

You need to add conditionals to your code. You cant just add two lines; walkToBank bank You need to add in the proper logic to make it know when to walk to the bank and when to bank.

dang. ur turn

ull never guess this one

Jar Jar? hue

If you are wanting to add it to the SDN look here: http://osbot.org/forum/forum/181-sdn-upload-requests/ You will need to set up a git account and then upload your source there. If you want to share it locally, you can build the jar and share a download link to the jar, or share the source code directly.

He means why are you doing this: npc.getName().contains("Cow") && !npc.getName().contains("Dairy") && npc != null null checking npc after you call npc.getName is pointless. If there is an NPE you will get it on npc.getName, and thus null checking after you call a method from that object is pointless. The NPE will not be avoided at all.

Make a filter for it. A filter will basically only accept npcs that meet a specific criteria, i.e. the cow is not under attack. There might be something in the api for this already, but this is a way you can do it without an api method.

Try to be more specific? Maybe post your code? Im guessing you didn't add a manifest or something of that nature.

Oh yea, dropAll is a slow action. There isnt a way to fix it that I know of other than writing your own method.

If im understanding you, you are asking if its necessary to have: if(inventory.isFull()) { return State.DROP; } and then again have this: case DROP: if(inventory.isFull()) { inventory.dropAllExcept("Small fishing net"); } break; ? If that is your question, then its not really necessary. What you are doing could be called "Defensive programming". Basically its checking the crap out of everything to make sure that its what you want, which prevents any unwanted results or nulls etc. I am not a fan of state based scripting, I always use task based. Case gets too messy imo, but thats just me. ^_^

Yay i like patches

Either way. If you make that enum one time, you can just call it from the enum each time you need that area. I keep all of my scripts in the same project, that way I can reuse every bit of my custom API code, web walker, etc. and I never have to rewrite code for any script twice if I have already written it. Then when I compile, I just compile the one script class that I want to submit. ^_^

I think you have a misunderstanding of the concept of reuse. Enums are made exactly for things such as this.