First of all, not a bad script. Simple and decent for a first one. Here are some things to help :
1) Your walk to willows procedure looks for the bank symbol. You need to change it to tree.
Change :
procedure WalkToWillows;
begin
if not LoggedIn then Exit;
symbolAccuracy := 0.5;
if FindSymbol(x, y, 'bank') then
Mouse(x-33, y+33, 6, 6, True);
end;
To :
procedure WalkToWillows;
begin
if(not LoggedIn)then
Exit;
symbolAccuracy := 0.5;
if FindSymbol(x, y, 'tree') then
Mouse(x-33, y+33, 6, 6, True);
end;
2) Why would you log out if the player is logged in?
3) You kinda have a multiplayer going, but you only have one set in the Declare Players.
4) Work on standards.
Overall, not a bad script. You're getting it. Here is the changed version :)
program WillaKilla;
{.include SRL\SRL.scar}
{.include SRL\SRL\skill\WoodCutting.scar}
const
WillowColor = 330249;//Color of Willow; Change this if it won't work.
LoadsPerPlayer = 5;//Loads Per Player(Self-Explanatory) :)
var
x, y, CurrentLoads: Integer;
procedure DeclarePlayers;
begin
HowManyPlayers := 1;
NumberOfPlayers(HowManyPlayers);
CurrentPlayer := 0;
Players[0].Name := '';//Name of Player
Players[0].Pass := '';//Password of Player
Players[0].Nick := '';//3-4 letters from Player's name
Players[0].Active := True;
end;
procedure AntiBan;
begin
if(not LoggedIn)then
Exit;
case Random(36) of
0..5 : RandomRClick;
6..12 : HoverSkill('woodcutting', False);
13..17: BoredHuman;
18..24: PickUpMouse;
25..31: RandomMovement;
32..36: Wait(1750 + Random(500));
end;
end;
procedure ProgressReport;
begin
WriteLn('********************************');
WriteLn('');
WriteLn('Tree Chopper! :)');
WriteLn('');
WriteLn('Worked for ' + TimeRunning);
WriteLn('********************************');
end;
procedure KillaWilla;
begin
if(not LoggedIn)then
Exit;
repeat
if FindObj(x, y, 'illow', WillowColor, 10) then
begin
Mouse(x, y, 2, 3, True);
Wait(2000 + Random(800));
end;
until(InChat('full to'));
end;
procedure WalkToBank;
begin
if(not LoggedIn)then
Exit;
SetRun(True);
symbolAccuracy := 0.5;
if FindSymbol(x, y, 'bank') then
Mouse(x, y, 4, 4, True);
end;
procedure BankWillows;
begin
if(not LoggedIn)then
Exit;
OpenBankQuiet('db');
DepositAll;
CloseBank;
Inc(CurrentLoads);
end;
procedure WalkToWillows;
begin
if(not LoggedIn)then
Exit;
symbolAccuracy := 0.5;
if FindSymbol(x, y, 'tree') then
Mouse(x-33, y+33, 6, 6, True);
end;
begin
SetUpSRL;
DeclarePlayers;
LoginPlayer;
repeat
WalkToWillows;
KillaWilla;
WalkToBank;
BankWillows;
AntiBan;
Wait(3000);
ProgressReport;
if CurrentLoads = LoadsPerPlayer then
NextPlayer(true);
if(not LoggedIn)then
NextPlayer(false);
until IsFKeyDown(12);
end.
~Camo