MySQLi Posted July 9, 2017 Share Posted July 9, 2017 Hey guys, today I'm releasing my first very basic script. With the release of this I'm also working on an AIO Woodcutter, I know there's a million but I'm just doing this to learn! Varrock east wood cutter Provides banking Inventory or equipped axe support Not much to the script, just my first basic contribution, as I am learning to do more, so if for nothing, give me suggestions. Pastebin Source : https://pastebin.com/nkrYCZqT Quote Link to comment Share on other sites More sharing options...
Viston Posted July 9, 2017 Share Posted July 9, 2017 I suggest using walkPath, since you have a static area. Take a loot at it. Other than that, gz Quote Link to comment Share on other sites More sharing options...
Rekt Posted July 9, 2017 Share Posted July 9, 2017 I dont see how this supports axe in inv/equipments tho Quote Link to comment Share on other sites More sharing options...
Viston Posted July 9, 2017 Share Posted July 9, 2017 (edited) 4 minutes ago, Get Rekt said: I dont see how this supports axe in inv/equipments tho It does, he is specifically banking Yew Logs, so the axes won't be banked. Edited July 9, 2017 by Visty Quote Link to comment Share on other sites More sharing options...
Rekt Posted July 9, 2017 Share Posted July 9, 2017 (edited) 4 minutes ago, Visty said: It does, he is specifically banking Yew Logs, so the axes won't be banked. No i meant what if inventory doesnt have an axe Edited July 9, 2017 by Get Rekt Quote Link to comment Share on other sites More sharing options...
Viston Posted July 9, 2017 Share Posted July 9, 2017 1 minute ago, Get Rekt said: No i meant what if inventory doesnt have an axe Then the script will fuck up I guess. But shouldn't be a worry, since you'll start the script with an axe anyways Quote Link to comment Share on other sites More sharing options...
Rekt Posted July 9, 2017 Share Posted July 9, 2017 3 minutes ago, Visty said: Then the script will fuck up I guess. But shouldn't be a worry, since you'll start the script with an axe anyways Well the one i made won't fuck up Quote Link to comment Share on other sites More sharing options...
Eliot Posted July 9, 2017 Share Posted July 9, 2017 Nice job! I love seeing new people learning to script. Quote Link to comment Share on other sites More sharing options...
mercylad Posted July 9, 2017 Share Posted July 9, 2017 Good stuff mate keep it up! 3 minutes ago, Get Rekt said: Well the one i made won't fuck up OT: That's why you're madtier scripter over 9001 Quote Link to comment Share on other sites More sharing options...
A one Posted July 9, 2017 Share Posted July 9, 2017 gz on script Quote Link to comment Share on other sites More sharing options...
Apaec Posted July 9, 2017 Share Posted July 9, 2017 (edited) Looks good! There are a few areas where you're running unnecessary checks e.g in your animation check in the CUT state, but other than that it looks good. As was mentioned above, it's probably best to use a pre-recorded path, but webwalking should be pretty reliable! Good luck! Edited July 9, 2017 by Apaec Quote Link to comment Share on other sites More sharing options...
MySQLi Posted July 9, 2017 Author Share Posted July 9, 2017 As far as the web walking goes, I see what you guys mean having a predefined path since it is such a close walk, although I did run the script for 6 hours without any errors! Quote Link to comment Share on other sites More sharing options...
Prolax Posted July 12, 2017 Share Posted July 12, 2017 case CUT: if (myPlayer().isAnimating()){ }else{ if(!myPlayer().isAnimating()){ } RS2Object yew = getObjects().closest(WC, "Yew"); if (yew != null){ log("Chopping yew"); yew.interact("Chop down"); } } break; The first 2 if functions are doing nothing. You can remove them. Quote Link to comment Share on other sites More sharing options...
