July 29, 20178 yr Author 7 minutes ago, Eliot said: Looks good, pretty similar to progressiveWC. You have a lot of nested if statements that could be shortened (e.g. if (tree != null && tree.interact("Chop down")). Tree names and areas should be final constants declared once instead of returning a new area/string each time. Logic of the script has some quirks such as not switching to the next tree when the current tree is chopped down (by someone else) and waiting until the animation ends. I'll leave that for your one. But yes, that would be added normally
August 2, 20178 yr if (Banks.VARROCK_WEST.contains(myPlayer())) { getWalking().webWalk(Banks.VARROCK_WEST); Can somebody explain this to me? Trying to learn the code From how I read it, if The Bank contains player, walk to the bank? or am i reading this wrong?
August 2, 20178 yr 3 hours ago, kingbutton said: if (Banks.VARROCK_WEST.contains(myPlayer())) { getWalking().webWalk(Banks.VARROCK_WEST); Can somebody explain this to me? Trying to learn the code From how I read it, if The Bank contains player, walk to the bank? or am i reading this wrong? yea, think it's missing an ! at the if ;o
August 2, 20178 yr 4 hours ago, kingbutton said: if (Banks.VARROCK_WEST.contains(myPlayer())) { getWalking().webWalk(Banks.VARROCK_WEST); Can somebody explain this to me? Trying to learn the code From how I read it, if The Bank contains player, walk to the bank? or am i reading this wrong? if (!Banks.VARROCK_WEST.contains(myPlayer())) { getWalking().webWalk(Banks.VARROCK_WEST);
August 3, 20178 yr Easier way to get tree area; Area yewArea = new Area(3203, 3506, 3225, 3497); Area oakArea = new Area(3171, 3425, 3159, 3399); Area areaToChop = getSkills().getDynamic(Skills.WOODCUTTING) >= 60 ? yewArea : oakArea;
August 3, 20178 yr Author On 8/2/2017 at 3:37 AM, kingbutton said: if (Banks.VARROCK_WEST.contains(myPlayer())) { getWalking().webWalk(Banks.VARROCK_WEST); Can somebody explain this to me? Trying to learn the code From how I read it, if The Bank contains player, walk to the bank? or am i reading this wrong? ! was left out, fixed
Create an account or sign in to comment