i strongly suggest you remove this from the script:
{///////////Contact/////////////}
{ To contact me by Email: }
{ [email]Wrightri601@hotmail.com[/email] }
{ }
{ In Game: }
{ ******** }
{///////////////////////////////}
at least the ingame name. It's common knowledge and sense that jagex monitors this website, and they might ban or flag your accounts if they find your ingame username.
ok, so i'm gonna post my comments on the whole script, do notice i'm not even close to being one of the best scripters around, so this advice won't be something like extremely useful, only my two cents :p
pretty amazing for a first script btw ;)
procedure load+setvariables toguether
less procedures makes the script faster.
on procedure Login fighter you might want to make a couple changes,
begin
loginplayer;
to
begin
if (not loggedin) then loginplayer;
it's just a precaution in this case to make sure there isn't a player already logged in.
writeln(players[currentplayer].Nick+' as loged in');
has log
ged in. just a spelling mistake, not really a coding thing :p
sendkeys('...');
i suggest you use TypeSend('…') instead, TypeSend is less detectable, but don't forget to include srl.scar if you do so!
begin
case random(70) of
1: RandomRClick;
2:HoverSkill('Strenght',false);
3:PickUpMouse;
23:SayCurrentLevels('strength');
5:SayCurrentLevels('attack');
15:SayCurrentLevels('defence');
7:AlmostLogOut;
45:RandomMovement;
9:BoredHuman;
end
end;
well this is totally up to you, but i usually make a smaller random and make it do something no matter what number i get, even if it's just a wait to make sure it does some antiban activity
also you missed a ";" after the first end which is not necessary, but it's standarts.
~ RMagician