I quickly analyzed your script a bit.
- You must use the PlayerForm or DeclarePlayers;
-
Code:
procedure progressreport;
Begin
Writeln('*>*>*>*>*>*>*>*>*>*>*>*>*>*>*>*>*>*>*>*>*>*>*>*>');
Writeln('Just A Simple Report');
Writeln('Number Of Monks Killed: '+ IntToStr(killed));
Writeln('Strenght Lvl when Started: '+ IntToStr(strengthlvl));
strngthlvl := GetSkillLevel('strength');
Writeln('Strenght Lvl NOW: '+ IntToStr(strngthlvl));
Writeln('Number Of Times Healed: '+ IntToStr(timesheald));
Writeln('*>*>*>*>*>*>*>*>*>*>*>*>*>*>*>*>*>*>*>*>*>*>*>*>');
end;
- Rawr, learn scripting standards. ;D
- Stay consistent, use your if then's. For example of why it's bad:
Code:
FindObjFast(x,y,'onk',monk2,tolerance);
MMouse(x,y,1,1);
What if it did not find that color? x, y would remain 0, therefore it would move to the top left corner.
- Use TimeMarkers
- OutFight is not a good choice. If has no anti randoms, mod it a bit and paste it in your script.
- You should make it logout or something when colors aren't found.
- !!Don't limit yourself to color, try bitmaps and etc.
- Multiple players.
Nice though, your a pretty great scripter. Good job! I hope you become even better.