MySQLi Posted July 9, 2017 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
Viston Posted July 9, 2017 Posted July 9, 2017 I suggest using walkPath, since you have a static area. Take a loot at it. Other than that, gz
Viston Posted July 9, 2017 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
Rekt Posted July 9, 2017 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
Viston Posted July 9, 2017 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
Rekt Posted July 9, 2017 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
mercylad Posted July 9, 2017 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
Apaec Posted July 9, 2017 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
MySQLi Posted July 9, 2017 Author 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!
Prolax Posted July 12, 2017 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.