help plz

[scarscaper4life]

hey this my first script and i was wondering what i am doing wrong because i set the monsters color and name and it doesn't click on the monster it just laggs here the script anyhelp will be very greatly apprechiated(soz can't spell that)



program monsterkiller;
{.include SRL/SRL.scar}
{.include SRL/SRL/Skill/Fighting.scar}


var
tol,monsterColor1,monsterColor2,monsterColor3:inte ger;
MonsterName :string;

procedure setup;
begin
MonsterColor1:=1188253;//monster color
MonsterColor2:=1188250;//monster color
MonsterColor3:=3033954;//monster color
tol:=10;//tolarence for finding monster
MonsterName:='Guard';
end;


procedure FindRandoms;
begin
FindNormalRandoms;
end;

procedure ownMonster;
begin
wait(2000)random(200)
KAttackMonster(monsterColor1, MonsterColor2, MonsterColor3, Tol, 'MonsterName');
end;





begin
setupsrl;
Findrandoms
repeat
ownMonster;

until false

end.

[Infantry001]

hmm...well you should fix inte ger to integer in the variables seciton. Aslo, you didnt put setup in your main loop. otherwixe, it should work. if it doenst, then... post agian i guess?

[WhiteShadow]

program monsterkiller;
{.include SRL/SRL.scar}
{.include SRL/SRL/Skill/Fighting.scar}


var
Tol, MonsterColor1, MonsterColor2, MonsterColor3 : Integer;//Integer;
MonsterName : String;

procedure Setup;
begin
MonsterColor1:=1188253;//monster color
MonsterColor2:=1188250;//monster color
MonsterColor3:=3033954;//monster color
tol:=10;//tolarence for finding monster
MonsterName:='Guard';
end;


procedure FindRandoms;
begin
FindNormalRandoms;
end;

procedure OwnMonster;
begin
wait(2000)random(200)
KAttackMonster(monsterColor1, MonsterColor2, MonsterColor3, Tol, MonsterName);//!!Not 'MonsterName' but MonsterName
end;


begin
setupsrl;
Findrandoms
repeat
ownMonster;
until false
end.

The main reason was because you put it to look for 'MonsterName', which means it tries to find that in the top left hand corner. It's Monstername. So it's just the string variable not the string. Btw, your doing good. :)

[scarscaper4life]

o i see thanks

[Jagex_Fagex]

also you don't need that randoms procedure until you are going to look for more random things at once.

but for now, instead of that proc, you cld just call FindNormalRandoms;

[WhiteShadow]

also you don't need that randoms procedure until you are going to look for more random things at once.

but for now, instead of that proc, you cld just call FindNormalRandoms;

Yea it's redundent, but some people just like doing that. They can't cope with names they don't like. :D