Good. You are starting to get the hang of things. And quick too. Not bad at all.
A couple of notes:
- Use
Mouse,
MMouse and
GetMousePos instead of
ClickMouse and
MoveMouseSmooth. The latter are quite detectable.
- Replace
IsUpText('Chop down') with
IsUpText('ow') or better
IsUpTextMulti('ho','op d','dow'). Makes it a lot quicker on the detection, experiment with those.
-
findcolorspiral(xs,ye,wood,1,1,512,336). We have constants for that: MSX1 (MainScreen X1), MSY1, MSX2, MSY2, so you can replace it with:
findcolorspiral(xs,ye,wood,MSX1, MSY1, MSX2, MSY2). Try using findcolorspiraltolerance, and try raising the tolerance by lil' steps.
-
if(invfull = true) then end; Doesnt do anything. You can leave this out.
- Procedure
dropal can be replaced by
DropAllLogs.
-
Check and
FindTree4 basically do the same thing, no? Replace them with one Function, that returns False if the tree is not found. Thus:
Check does:
- Function Check:Boolean (Returns true if found)
- Function Check(var xs, ye:integer):Boolean. Returns true if found and returns the found coords in xs and ye
- Try expanding your loop, by adding a lil' callibration to your script. Add a little walking or anything that releaves the stress from the tight loop.
- You are being quite creative and have put in a lot of effort into this. Good job. Keep up the good work, man. If you want I can releave you from the Clown biz.