SRL Archive - Archive

Jack Shannon - SRL Member Application Form


ReadySteadyGo

2008-09-15 18:22:28

<!– BEGIN TEMPLATE: formanswers –>
Jack Shannon - SRL Member Application Form
Jack Shannon

What is the meaning to life?


Where you Referred to SRL? If yes, by whom?
no


Programming Knowledge
N/A

Other Communitys
N/A

How Long have You Played Runescape
On and off 3-4 years

Are you affilated in anyway with Jagex LTD?
No
—————————————————–
Please write down a paragraph about Yourself.

I SCAR scripted a few years ago but stopped when I quit Runescape. Started Runescape again a couple of months ago, but thought that SCAR had died out until about 3-4 weeks ago when I found SRL. Researched it a bit, read a few tutorials to remind myself but also to learn all the new methods of doing things.

The Script I am posting is a WIP, started it yesterday.

——————————————————
Please Post a Sample of your Work. Only SCAR Scripts Accepted

//Yew Cutter by Jack Shannon

program YewCutter;
{.Include SRL\SRL.scar}
{.Include SRL\SRL\Skill\WoodCutting.scar}

var
x, y: Integer;
Deposits: Integer;

{-------------------------------------------------------------------------------
Name: MyMouse
Description: Combines Mouse and FFlag
-------------------------------------------------------------------------------}

Procedure MyMouse(x, y: integer);
begin
Mouse(x, y, 3, 3, True);
Wait(800 + Random(500));
FFlag(5 + Random(5));
end;

{-------------------------------------------------------------------------------
Name: WalkToYews
Description: Walks to the Yew trees
-------------------------------------------------------------------------------}

procedure WalkToYews;
var
RoadColor, DTM1, DTM2: integer;
begin
MakeCompass('N');
SetRun(True);
RoadColor := FindRoadColor;
if RadialWalk(RoadColor, 225, 270, 66, -1, 0) then
begin
Wait(500);
FFlag(5 + Random(5));
end;

MyMouse(565, 88);
MyMouse(571, 91);

DTM1 := DTMFromString('78DA634C67626008644001FCBDA20CFF81342' +
'310FF0702C65CA09A185435D619AA0CFC503520C0580854E34340' +
'4D2D504D05AA9ADCEA09286A0020010C25');
DTM2 := DTMFromString('78DA634C63626008624001FCBDA20CFF81342' +
'310FF0702C608A01A575435D619AA0CFC503520C05803545389AA' +
'26B77A02AA9A66A09A6802E6F402D524E257030012960DE3');

if FindSymbol(x, y, 'Tree') then
begin
MyMouse(x, y);
end else
if DTMRotated(DTM1, x, y, MMX1, MMY1, MMX2, MMY2) then
begin
MyMouse(x, y);
FreeDTM(DTM1);
end else
if DTMRotated(DTM2, x, y, MMX1, MMY1, MMX2, MMY2) then
begin
MyMouse(x, y);
FreeDTM(DTM2);
end;
end;

{-------------------------------------------------------------------------------
Name: WalkToBank
Description: Walks to Catherby bank
-------------------------------------------------------------------------------}

procedure WalkToBank;
var
BankDTM: integer;
begin
MakeCompass('N');
BankDTM := DTMFromString('78DA639CCEC4C060C680028A0A3319FE03694' +
'620FE0F048C3D4035DAA86AFEFC6162E087AA0101C6094035EA04' +
'D4CC01AAD1C0AD0628CB000018F30F74');

MyMouse(690, 88);
MyMouse(687, 81);

RoadColor := FindRoadColor;
if RadialWalk(RoadColor, 50, 130, 66, -1, 0) then
begin
Wait(500);
FFlag(5 + Random(5));
end;
if DTMRotated(BankDTM, x, y, MMX1, MMY1, MMX2, MMY2) then
begin
Mouse(x, y, 3, 3, True);
Wait(800 + Random(500));
FFlag(5 + Random(5));
end;
FreeDTM(BankDTM);
end;

{-------------------------------------------------------------------------------
Name: FindYew
Description: Finds the nearest Yew tree
-------------------------------------------------------------------------------}

function FindYew(var x, y: Integer): Boolean;
var
Points: TPointArray;
YewBoxes: T2DPointArray;
BoxCenter: TPoint;
i: integer;
begin
Result := False;

if (not FindColorsTolerance(Points, 3175783, MSX1, MSY1, MSX2, MSY2, 15)) then
Exit;
Result := True;
SortTPAFrom(Points, Point(MSCX, MSCY));
YewBoxes := TPAtoATPAEx(Points, 30, 30);
for i := 0 to High(YewBoxes) do
begin
BoxCenter := MiddleTPA(YewBoxes[i]);
MMouse(BoxCenter.x, BoxCenter.y, 10, 10);
if IsUpText('ew') then
begin
Result := True;
GetMousePos(x, y);
Exit;
end;
end;
end;

{-------------------------------------------------------------------------------
Name: FollowYewPoint
Description: Outputs the new co-ords of Yew
-------------------------------------------------------------------------------}

function FollowYewPoint(var x, y: integer): Boolean;
var
TempP: Tpoint;
Points: TpointArray;
begin
Result:= False;
TempP.x := x;
TempP.y := y;
repeat
if FindColorsTolerance(Points, 3175783, TempP.x - 40, TempP.y - 40, TempP.x + 40, TempP.y + 40, 15) then
TempP := MiddleTPA(Points);
until (not FlagPresent);
Result := True;
end;

{-------------------------------------------------------------------------------
Name: ChopYews
Description: Chops down Yews until inventory is full
-------------------------------------------------------------------------------}

procedure ChopYews;
var
tm: integer;
begin
MarkTime(tm);
repeat
if (not FindYew(x, y)) then
begin
Exit;
WriteLn('Could not find yew')
end;
Mouse(x, y, 0, 0, True);
Wait(200 - random(100));
if (not FollowYewPoint(x, y)) then
begin
Exit;
WriteLn('Could not follow yew point')
end;
MMouse(x, y, 3, 3);
repeat
Wait(500 + random(100));
FindEnt(x, y, True);
until (not IsUpText('ew')) or (InvFull);
FindNormalRandoms;
until(InvFull) or (TimeFromMark(tm) > (15 *60000));
end;

{-------------------------------------------------------------------------------
Name: OpenBankFast_cb
Description: Taken from SRL, added the arguments: 'Catherby' , 'cb'
-------------------------------------------------------------------------------}

function CaOpenBankFast(Location: String): Boolean;
var
x, y, i, l, c, z: Integer;
Info: TIntegerArray;
Booths, t: TPointArray;
begin
Wait(400);
case LowerCase(Location) of
'akb', 'al kharid': Info := [MSCX, MSCY, 7969203, 25, 1];
'lb', 'lumbridge': Info := [MSCX, MSCY, 4807788, 10, 1];
'veb', 'varrock east': Info := [MSCX, MSCY, 2971236, 10, 1];
'vwb', 'varrock west': Info := [270, 270, 5689318, 25, 1];
'feb', 'falador east': Info := [MSCX, MSCY + 50, 4612728, 14, 1];
'fwb', 'falador west': Info := [MSCY, MSCY, 4612728, 14, 1];
'db', 'draynor': Info := [MSCX, MSCY, 605006, 4, 1];
'eb', 'edgeville bank': Info := [MSCX + 50, MSCY, 3230295, 9, 2];
'cb', 'catherby': Info := [MSCX, MSCY, 7763833, 10, 1];
else
begin
srl_Warn('OpenBankFast', 'Unknown bank: '+Location, warn_AllVersions);
Exit;
end;
end;

MakeCompass('n');
z := GetColorToleranceSpeed;
ColorToleranceSpeed(Info[4]);
FindColorsSpiralTolerance(Info[0], Info[1], Booths, Info[2], MSX1, MSY1, MSX2, MSY2, Info[3]);
ColorToleranceSpeed(z);
SetArrayLength(t, 0);
t := Booths;
SetArrayLength(Booths, 0);
Booths := ReArrangeAndShortenArray(t, 14);
l := GetArrayLength(Booths) -1;
for i := 0 to l do
begin
if (not LoggedIn) or BankScreen or PinScreen then
Break;
if (LowerCase(Location) = 'vwb') or (LowerCase(Location) = 'varrock west') then
begin
Booths[i].X := Booths[i].X + 5;
Booths[i].Y := Booths[i].Y - 20;
end;
MMouse(Booths[i].X, Booths[i].Y, 3, 3);
Wait(75 + Random(75));
if IsUpText('ooth') Then
begin
GetMousePos(x, y);
Mouse(x, y, 0, 0, False);
Wait(70 + Random(75));
if ChooseOption('uickly') then
begin
FFLag(0);
MarkTime(c);
repeat
Wait(100);
until (BankScreen) or (PinScreen) or (TimeFromMark(c) > 3000);
if (Length(Players) > 0) then
if (Players[CurrentPlayer].Pin <> '') then
InPin(Players[CurrentPlayer].Pin);
end;
end;
end;
Result := (BankScreen) or (PinScreen);
end;

{-------------------------------------------------------------------------------
Name: Bank
Description: Deposits all in Catherby Bank
-------------------------------------------------------------------------------}

procedure Bank;
begin
CaOpenBankFast('cb');
InPin('1604');
DepositAll;
CloseBank;
end;

{-------------------------------------------------------------------------------
Name: ProgressReport
Description: Displays a rundown of how the script is doing
-------------------------------------------------------------------------------}

procedure ProgressReport;
begin
ClearReport;
WriteLn('----- Yew Cutter')
WriteLn('----- Duration:' + TimeRunning)
WriteLn('----- Trips:' + IntToStr(Deposits))
WriteLn('----- Yews:' + IntToStr(Deposits * 27))
end;

{-------------------------------------------------------------------------------
Name: SetupScript
Description: Sets up the Main Loop
-------------------------------------------------------------------------------}

Procedure SetupScript;
begin
SetupSRL;
ClearDeBug;
ActivateClient;
wait(2000);
end;

{-------------------------------------------------------------------------------
Main Loop
-------------------------------------------------------------------------------}

begin
SetupScript;
repeat
WalkToYews;
ChopYews;
WalkToBank;
Bank;
until(false);
end.

——————————————————
<!– END TEMPLATE: formanswers –>

Runescapian321

2008-09-15 18:29:26

Anti-Randoms? Multi-player? Your other stuff looks OK (TPA, walking, etc), but you're lacking some basic stuff. All you need to do is put a little bit more work into that script, and then you might have a good chance. I'm leaning towards a no, but I'll see what other people say.

EDIT: And being more active around the forums wouldn't hurt either.

ReadySteadyGo

2008-09-16 15:26:33

Hi I have updated my script with your suggestions, how can I change it?

Thanks

sirlaughsalot

2008-09-16 15:40:26

From: Jack Shannon
Hi I have updated my script with your suggestions, how can I change it?

Thanks


Have the original on top, then post the edited one on the bottom, :D i havent looked over it yet, will edit in a min…

Edit: I tend to agree with Runescapian… I am going to vote no, because the single biggest function, isnt really yours (at least it doesnt look like it), and instead of having those DTMs use DDTMs not that hard and easier to update and such… so

*More failsafes
*MultiPlayer
*Make your own OpenBank function
*Make a DDTM instead of DTM's for mapwalking

it looks really good but the failsafes is the biggest issue to me, i think that is when people really shine and think outside the box… if this gets denied, i want to see what your next app will be like, you seem to have potential, keep it up :D

ReadySteadyGo

2008-09-16 17:04:37

Thanks Sirlaughsalot for the critique, yeah I agree with what you've said.

How come I should make my own open bank if SRL has one that works for me? Not trying to be difficult because I will if you think I should, I'd just like the resoning behind it.

My walking has just completely screwed up for some reason, it was working earlier, so I'll make some DDTM's. I have searched through all the DDTM tutorials but can't find one which shows you how to walk through grassy areas rather than just roads and paths? Is it possible to make ones for grassy areas and if so how? Like should i be using the trees, the water, the grassy area, which land marks would I use.

Also, should I still add multi player even if i don't use it or have two members so can't test it?

Sorry if my questions are a pain.

I will add more failsafes tommorow.

Thanks again.

oh, almost forgot, here is the update with randoms and loads of new things…

{ _ __ _____ _ _ _    _____ _____ _____ _____ _____ _____
| | | __| | | | | | | |_ _|_ _| __| __ |
|_ _| __| | | | | --| | | | | | | | __| -|
|_| |_____|_____| |_____|_____| |_| |_| |_____|__|__|
Catherby

//////////////////////////////////////////////////
// //
// Title: Catherby Yew Cutter [SMART version] //
// Version: 1.0 //
// Author: Jack Shannon //
// Description: Cuts Yew trees in Catherby and //
// banks them //
// //
// Instructions: //
// - Start in Catherby Bank //
// - Wield axe //
// - Run Script //
// //
/////////////////////////////////////////////////}


program YewCutter;
{.Include SRL\SRL\Misc\SMART.scar}
{.Include SRL\SRL.scar}
{.Include SRL\SRL\Misc\Users.scar}
{.Include SRL\SRL\Skill\WoodCutting.scar}

var
x, y: Integer;
Deposits: Integer;

{-------------------------------------------------------------------------------
Name: MyMouse
Description: Combines Mouse and FFlag
-------------------------------------------------------------------------------}

Procedure MyMouse(x, y: integer);
begin
Mouse(x, y, 3, 3, True);
Wait(800 + Random(500));
FFlag(5 + Random(5));
end;

{-------------------------------------------------------------------------------
Name: WalkToYews
Description: Walks to the Yew trees
-------------------------------------------------------------------------------}

procedure WalkToYews;
var
RoadColor, DTM1, DTM2: integer;
begin
MakeCompass('N');
SetRun(True);
RoadColor := FindRoadColor;
if RadialWalk(RoadColor, 225, 270, 66, -1, 0) then
begin
Wait(500);
FFlag(5 + Random(5));
end;

MyMouse(565, 88);
MyMouse(571, 91);

DTM1 := DTMFromString('78DA634C67626008644001FCBDA20CFF81342' +
'310FF0702C65CA09A185435D619AA0CFC503520C0580854E34340' +
'4D2D504D05AA9ADCEA09286A0020010C25');
DTM2 := DTMFromString('78DA634C63626008624001FCBDA20CFF81342' +
'310FF0702C608A01A575435D619AA0CFC503520C05803545389AA' +
'26B77A02AA9A66A09A6802E6F402D524E257030012960DE3');

if FindSymbol(x, y, 'Tree') then
begin
MyMouse(x, y);
FTWait(2);
FindNormalRandoms;
end else
if DTMRotated(DTM1, x, y, MMX1, MMY1, MMX2, MMY2) then
begin
MyMouse(x, y);
FTWait(2);
FindNormalRandoms;
FreeDTM(DTM1);
end else
if DTMRotated(DTM2, x, y, MMX1, MMY1, MMX2, MMY2) then
begin
MyMouse(x, y);
FTWait(2);
FindNormalRandoms;
FreeDTM(DTM2);
end else
begin
WriteLn('Could not find Yew Symbol and backup DTM''s failed');
end;
end;

{-------------------------------------------------------------------------------
Name: WalkToBank
Description: Walks to Catherby bank
-------------------------------------------------------------------------------}

procedure WalkToBank;
var
BankDTM: integer;
begin
if(not(LoggedIn))then Exit;
MakeCompass('N');
BankDTM := DTMFromString('78DA639CCEC4C060C680028A0A3319FE03694' +
'620FE0F048C3D4035DAA86AFEFC6162E087AA0101C6094035EA04' +
'D4CC01AAD1C0AD0628CB000018F30F74');

MyMouse(690, 88);
FTWait(2);
FindNormalRandoms;
MyMouse(687, 81);
FTWait(2);
FindNormalRandoms;

RoadColor := FindRoadColor;
if RadialWalk(RoadColor, 50, 130, 66, -1, 0) then
begin
FTWait(2);
FindNormalRandoms;
Wait(500);
FFlag(5 + Random(5));
end;
if DTMRotated(BankDTM, x, y, MMX1, MMY1, MMX2, MMY2) then
begin
Mouse(x, y, 3, 3, True);
FTWait(2);
FindNormalRandoms;
Wait(800 + Random(500));
FFlag(5 + Random(5));
end;
FreeDTM(BankDTM);
end;

{-------------------------------------------------------------------------------
Name: FindYew
Description: Finds the nearest Yew tree
-------------------------------------------------------------------------------}

function FindYew(var x, y: Integer): Boolean;
var
Points: TPointArray;
YewBoxes: T2DPointArray;
BoxCenter: TPoint;
i: integer;
begin
Result := False;

if (not FindColorsTolerance(Points, 3175783, MSX1, MSY1, MSX2, MSY2, 15)) then
Exit;
Result := True;
SortTPAFrom(Points, Point(MSCX, MSCY));
YewBoxes := TPAtoATPAEx(Points, 30, 30);
for i := 0 to High(YewBoxes) do
begin
BoxCenter := MiddleTPA(YewBoxes[i]);
MMouse(BoxCenter.x, BoxCenter.y, 10, 10);
if IsUpText('ew') then
begin
Result := True;
GetMousePos(x, y);
Exit;
end;
end;
end;

{-------------------------------------------------------------------------------
Name: FollowYewPoint
Description: Outputs the new co-ords of Yew
-------------------------------------------------------------------------------}

function FollowYewPoint(var x, y: integer): Boolean;
var
TempP: Tpoint;
Points: TpointArray;
begin
Result:= False;
TempP.x := x;
TempP.y := y;
repeat
if FindColorsTolerance(Points, 3175783, TempP.x - 40, TempP.y - 40, TempP.x + 40, TempP.y + 40, 15) then
TempP := MiddleTPA(Points);
Wait(1);
until (not FlagPresent);
Result := True;
end;

{-------------------------------------------------------------------------------
Name: ChopYews
Description: Chops down Yews until inventory is full
-------------------------------------------------------------------------------}

procedure ChopYews;
var
tm, tm2: integer;
begin
if(not(LoggedIn))then Exit;
MarkTime(tm);
repeat
if (not FindYew(x, y)) then
begin
Exit;
WriteLn('Could not find yew')
end;
Mouse(x, y, 0, 0, True);
FTWait(2);
FindNormalRandoms;
FindEnt(x, y, True);
Wait(200 - random(100));
if (not FollowYewPoint(x, y)) then Continue;
MMouse(x, y, 3, 3);
FTWait(2);
FindNormalRandoms;
FindEnt(x, y, True);
MarkTime(tm2);
repeat
Wait(500 + random(100));
FindNormalRandoms;
FindBirdsNest;
FindEnt(x, y, True);
if FindFight then RunAway('N', True, 1, 2000);
if (TimeFromMark(tm2) > (4000 + Random(5000))) then
begin
if (Random(2) = 0) then
Mouse(x, y, 0, 0, True) else
Mouse(x, y, 5, 5, True);
FTWait(2);
FindNormalRandoms;
FindBirdsNest;
FindEnt(x, y, True);
MarkTime(tm2);
end;
until (not IsUpText('ew')) or (InvFull) or (not LoggedIn);
FTWait(2);
FindNormalRandoms;
FindEnt(x, y, True);
until(InvFull) or (TimeFromMark(tm) > (20 *60000)) or (not LoggedIn);
end;

{-------------------------------------------------------------------------------
Name: OpenBankFast_cb
Description: Taken from SRL, added the arguments: 'Catherby' , 'cb'
-------------------------------------------------------------------------------}

function CaOpenBankFast(Location: String): Boolean;
var
x, y, i, l, c, z: Integer;
Info: TIntegerArray;
Booths, t: TPointArray;
begin
Wait(400);
case LowerCase(Location) of
'akb', 'al kharid': Info := [MSCX, MSCY, 7969203, 25, 1];
'lb', 'lumbridge': Info := [MSCX, MSCY, 4807788, 10, 1];
'veb', 'varrock east': Info := [MSCX, MSCY, 2971236, 10, 1];
'vwb', 'varrock west': Info := [270, 270, 5689318, 25, 1];
'feb', 'falador east': Info := [MSCX, MSCY + 50, 4612728, 14, 1];
'fwb', 'falador west': Info := [MSCY, MSCY, 4612728, 14, 1];
'db', 'draynor': Info := [MSCX, MSCY, 605006, 4, 1];
'eb', 'edgeville bank': Info := [MSCX + 50, MSCY, 3230295, 9, 2];
'cb', 'catherby': Info := [MSCX, MSCY, 7763833, 10, 1];
else
begin
srl_Warn('OpenBankFast', 'Unknown bank: '+Location, warn_AllVersions);
Exit;
end;
end;

MakeCompass('n');
z := GetColorToleranceSpeed;
ColorToleranceSpeed(Info[4]);
FindColorsSpiralTolerance(Info[0], Info[1], Booths, Info[2], MSX1, MSY1, MSX2, MSY2, Info[3]);
ColorToleranceSpeed(z);
SetArrayLength(t, 0);
t := Booths;
SetArrayLength(Booths, 0);
Booths := ReArrangeAndShortenArray(t, 14);
l := GetArrayLength(Booths) -1;
for i := 0 to l do
begin
if (not LoggedIn) or BankScreen or PinScreen then
Break;
if (LowerCase(Location) = 'vwb') or (LowerCase(Location) = 'varrock west') then
begin
Booths[i].X := Booths[i].X + 5;
Booths[i].Y := Booths[i].Y - 20;
end;
MMouse(Booths[i].X, Booths[i].Y, 3, 3);
Wait(75 + Random(75));
if IsUpText('ooth') Then
begin
GetMousePos(x, y);
Mouse(x, y, 0, 0, False);
Wait(70 + Random(75));
if ChooseOption('uickly') then
begin
FFLag(0);
MarkTime(c);
repeat
Wait(100);
until (BankScreen) or (PinScreen) or (TimeFromMark(c) > 3000);
if (Length(Players) > 0) then
if (Players[CurrentPlayer].Pin <> '') then
InPin(Players[CurrentPlayer].Pin);
end;
end;
end;
Result := (BankScreen) or (PinScreen);
end;

{-------------------------------------------------------------------------------
Name: Bank
Description: Deposits all in Catherby Bank
-------------------------------------------------------------------------------}

procedure Bank;
begin
if(not(LoggedIn))then Exit;
CaOpenBankFast('cb');
InPin('1604');
DepositAll;
CloseBank;
Deposits := Deposits + 1;
end;

{-------------------------------------------------------------------------------
Name: ProgressReport
Description: Displays a rundown of how the script is doing
-------------------------------------------------------------------------------}

procedure ProgressReport;
begin
ClearReport;
WriteLn('----- Yew Cutter')
WriteLn('----- Duration:' + TimeRunning)
WriteLn('----- Trips:' + IntToStr(Deposits))
WriteLn('----- Yews:' + IntToStr(Deposits * 27))
SRLRandomsReport;
SendSRLReport;
end;

{-------------------------------------------------------------------------------
Name: SetupScript
Description: Sets up the Main Loop
-------------------------------------------------------------------------------}

Procedure SetupScript;
begin
SRLPlayerForm(True, [], [], [], []);
SMARTSetUp('world39', True, True, False);
SetTargetDC(SMARTGetDC);
SetupSRL;
ClearDeBug;
LoginPlayer;
SetAngle(True);
wait(200 + Random(100));
end;

{-------------------------------------------------------------------------------
Main Loop
-------------------------------------------------------------------------------}

begin
SetupScript;
repeat
WalkToYews;
ChopYews;
WalkToBank;
Bank;
ProgressReport;
until(not LoggedIn);
end.

sirlaughsalot

2008-09-16 17:21:25

From: Jack Shannon
Thanks Sirlaughsalot for the critique, yeah I agree with what you've said.

How come I should make my own open bank if SRL has one that works for me? Not trying to be difficult because I will if you think I should, I'd just like the resoning behind it.

My walking has just completely screwed up for some reason, it was working earlier, so I'll make some DDTM's. I have searched through all the DDTM tutorials but can't find one which shows you how to walk through grassy areas rather than just roads and paths? Is it possible to make ones for grassy areas and if so how? Like should i be using the trees, the water, the grassy area, which land marks would I use.

Also, should I still add multi player even if i don't use it or have two members so can't test it?

Sorry if my questions are a pain.

I will add more failsafes tommorow.

Thanks again.

oh, almost forgot, here is the update with randoms and loads of new things…


@ The DDTM question, you need to find a way to make it individual… prolly use like the road, the water and make the main point the grassy area,
@ the multiplayer, its not hard to add multiplayer, you really dont need to test it, read a multiplayer tut and you should be fine…
@ The Banking question, it just will make your app look better, you dont have to but i think it would make it 100% sure…

ReadySteadyGo

2008-09-16 17:23:38

From: sirlaughsalot
@ The DDTM question, you need to find a way to make it individual… prolly use like the road, the water and make the main point the grassy area,
@ the multiplayer, its not hard to add multiplayer, you really dont need to test it, read a multiplayer tut and you should be fine…
@ The Banking question, it just will make your app look better, you dont have to but i think it would make it 100% sure…


I'll do it then, thanks for the help.

Gu3st...

2008-09-17 15:28:30

I see you added the SRLPlayerForm. This is a notice to SRL Members to not vote no because you cant see and DeclarePlayers procedure.

I hold my vote.
You know the basics: TPA, Walking, Banking etc. :)

But I noticed this in ChopYews:
    if (not FindYew(x, y)) then
begin
Exit;
WriteLn('Could not find yew')
end;
It will exit the procedure BEFORE writing 'Could not find yew'. ;)

ReadySteadyGo

2008-09-17 17:34:49

From: Zytex!
I see you added the SRLPlayerForm. This is a notice to SRL Members to not vote no because you cant see and DeclarePlayers procedure.

I hold my vote.
You know the basics: TPA, Walking, Banking etc. :)

But I noticed this in ChopYews:
    if (not FindYew(x, y)) then
begin
Exit;
WriteLn('Could not find yew')
end;
It will exit the procedure BEFORE writing 'Could not find yew'. ;)


Thanks Zytex :)
Sugar! I'll change that. Well spotted.

Raskolnikov

2008-09-17 17:57:46

Hey,

You have potential but:

begin
SetupScript;
repeat
WalkToYews;
ChopYews;
WalkToBank;
Bank;
until(false);
end.

Should have a failsafe such as:

if not LoggedIn then Break;

Many functions should have a:

if not LoggedIn then Exit;

too. Sorry I lean to no, if you don't show that you know some little itty bitty things that are hugely important. You need a little more anti-randoms and failsafes make a huge difference.

You seem to have standards down except…

if RadialWalk(RoadColor, 50, 130, 66, -1, 0) then
begin
Wait(500);
FFlag(5 + Random(5));
end;

should be

if RadialWalk(RoadColor, 50, 130, 66, -1, 0) then
begin
Wait(500);
FFlag(5 + Random(5));
end;

Otherwise it is pretty good, but it lacks some things that are important.

Cut em2 it

EvilChicken!

2008-09-17 18:05:20

i liek u


yas


Nice script, very nice script!
I don't see why people vote no?

WT-Fakawi needs to vote on more apps, then people actually would realize what great people they're voting 'no' to ..

Raskolnikov

2008-09-18 00:07:20

Well I changed my mind:

I hadn't seen the updated version (my fault) and that looks 10 times better with a failsafe in the mainloop

Yes for me

Cut em2 it

Nava2

2008-09-18 16:21:15

Withheld. Its good, but not amazing.

You call FindNormalRandoms almost too much… :D

nielsie95

2008-09-20 11:20:13

It seems like you're not getting in this time, sorry.
You may try again in 2 weeks.
Closed.