Explv

Looks like a fantastic investment, buy buy buy buy buy I expect a price of $1 by the end of the day buy buy invest invest buy buy forget bitcoin, safecoin will rule the world haha invest invest

Can you confirm the output of the log if you run from the command line. Are you using Java 8 or Java 9? Are you using a proxy or anything like that?

Wrong section bud. This is for help with writing scripts.

Your entire existence makes me cringe

Not against a fsm, just against ur trash implementation of one. Reeeeeeeeeee

Yes, you are correct my friend. But when you're using the janky ass "Task pattern" with dumb shit like WalkingToTheBankOnTheOtherSideOfVarrockHaHaTask.java like a lot of people do, then you're just shooting yourself in the foot.

@liverare ur doing the lord's work, preach the gospel, preachhhhhh

Pseudo code? Not sure you know what pseudo code is fren and also the "Task pattern" is trash

Your FindCrab task will always execute unless isFighting or isUnderAttack is true. In this case neither are true so it will continue to try and walk. Web walking will not walk to the exact position you specify, it will stop walking a few tiles away. To walk to a closer tile / an exact tile you will need to use a custom WalkingEvent and set the minDistanceThreshold. The minDistanceThreshold specifies how close the player must be from the destination position before the WalkingEvent finishes. You will also need to account for when mr crab does not wake up from his peaceful sleep. If the crab doesn't attack your player, then you will continue to execute the FindCrab task and try to walk.

Can do this for $350

Taking a look at these issues now. Smithing one has been fixed and will be available when the SDN is next updated.

I think it's hardcoded to require at least 120 coins in the inventory. If you have more than that and it still doesn't work, or if you think the behaviour should be different, I can take a look into the issue.

I have just pushed a fix for this, it will be available when the SDN is next updated. I'm not sure what you mean here. The script will only keep the best axe the player can use in the inventory / equipment, when the player levels up and can use a better axe, it will go to the bank to retrieve it, depositing the previous lower level axe.

"cause im 99% sure the logic in my script isn't causing this" I wouldn't be so sure about that... Post your code, otherwise we cannot help.

Ok well apparently I *still* have to do this for you... If you look at the docs for Timer: https://docs.oracle.com/javase/8/docs/api/java/util/Timer.html You will see there is no constructor that takes a number as the parameter. There is an empty constructor though. So just change new Timer(0) to new Timer() ..............

All of the logic in this code is wrong. If there is a single fire that is inside FireArea and not in line1 then it will walk to a random position.. what? Assuming this is for firemaking, you shouldn't be using a for loop here at all. You should be checking if any of the positions in line1 has a fire on top of it. Either way this design won't work well for firemaking. ?

Yes you need: import java.util.Timer; Wherever you are instantiating the Timer, you're doing it wrong. Check the Java docs for the Timer class. And follow some basic Java tutorials. Also use a proper Java IDE, it will highlight where and what your errors are in your code.

Thank you

Issue solved. OP should have been using getWidgets().getWidgetContainingText not getWidgets().containingText And also the message "You must ask the foreman's permisson before using the blast furnace" should contain a line break tag "<br>" This line break tag is only visible if you look at the message in the widget debugger. "You must ask the foreman's permisson before using the blast<br>furnace"

It ignores widgets with root id 162 I believe. The root id of this widget is supposedly 229, so that shouldn't be an issue.

getWidgets().containingText returns a List<RS2Widget> You are checking that the result of getWidgets().containingText() is not null, however that will always be true, because containingText will never return a null List. What you should be using instead is getWidgets().getWidgetContainingText() which returns an RS2Widget.

Yes, isAttackable makes use of getHealthPercent and isUnderAttack, both of which I believe make use of the health bar. If the health bar is not visible, isUnderAttack will return false.

Dear god that script makes me throw up in my mouth every time I see it. Wouldn't really call him a genius. Sure, he made a popular bug-free script, but he could certainly use a lesson or two on code design.

Yeah that sounds like you will just be using an existing game engine rather than creating your own. If you provide us with more information on the syllabus we would be able to tell you more precisely what to expect. Have fun taking the course anyways, even though it wasn't your first choice i'm sure it'll be useful and interesting in some regard.