SRL Archive - First Scripts

Arrow Killer

blazingrobin

2012-04-21 03:03:21

Arrow Killer
By: Blazingrobin

Here is my script let me know how things work and if you find any bugs.

What it does
It fletches shafts from logs or fletches headless arrows from the shafts
Also fletches Reg Long (u), Oak long (u), Willow long (u), Maple long (u), Yew long (u), and Magic long (u)
Also works at any bank with a normal banker (aka bankers like the ones at varrock or yanille)

Setup
1)Start Character at Soul Wars near banking chest
2)Fill out form (read instructions carefully)
3)if headless then have both in inventory
4)Make sure "withdraw-x" is set to 28

Updates
V2-Added Proggies and Maple Long(u) support, Fixed Banking problem
v3-Fixed Progress Report adding error
v3.5-Compressed parts of the script (credit to NKN for the suggestion)
v3.75-Compressed even more making banking all one function, also added small fail safes for now until I can get more in
v4- Contains "walk to bank" Fail Safe
v4.1-edited Headless Arrow Making procedure, Added fail safe to all crafting/banking procedures, freed dtms added
v4.15-Fixed error with script not finding feathers at all, fixed DTM error
v4.2- No longer need to click past knife script does it automatically
v4.75- Added support for Reg Long(u), Oak Long(u), Willow long (u), Yew long (u), and Magic long(u)
v5- Added support for all banks with normal npc



Coming Soon
-Possibly other fletching support
-Compress script more
-Add other bank support
-Add Anti-Randoms for areas other than SW
-Add support for Maple Long (u)
-Add Progress Reports

Known Bugs
-Progress Report count is off fixing in next update

Progress Reports

Shaft Progress Reports
***********Arrow Killer**********
*         ~By Blazingrobin~               
*Shafts Made: 9660 (19266 P/H)
*XP earned: 3220 (6422 P/H)
*GP Made: 115920 (231197 P/H)
*Total Time: 30 Minutes and 5 Seconds
********************************
-
-
-
Headless Progress Reports
***********Arrow Killer**********
*        ~By Blazingrobin~               
*Headless Made: 14550 (28811 P/H)
*XP earned: 14550 (28811 P/H)
*GP Made: 654750 (1065933 P/H)
*Total Time: 30 Minutes and 19 Seconds
********************************
-
-
Reg Long (u) Progress Reports
-
-
-
Oak Long (u) Progress Reports
-
-
-
Willow Long (u) Progress Reports
-
-
-
Maple Long (u) Progress Reports
***********Arrow Killer**********
*         ~By Blazingrobin~               
*Maples Made: 6496 (1422 P/H)
*XP earned: 383264 (83906 P/H)
*GP Made: 519680 (113770 P/H)
*Total Time: 4 Hours, 34 Minutes and 4 Seconds
********************************

Yew Long (u) Progress Reports
-
-
-
Magic Long(u) Progress Reports
-
-
-

-

begginer

2012-04-21 05:30:24

Well done on your first script.

alexes

2012-04-21 05:54:28

Well done. It will be good for me to start lvling up my fletch :D

Thanks ^^

NKN

2012-04-21 13:37:02

Not bad.

Have you considered combining banking methods?

You could also make a Mainloop procedure, and have all the procs in that, then just put MainLoop in the part that runs the script.

blazingrobin

2012-04-21 14:22:41

From: NKN
Not bad.

Have you considered combining banking methods?

You could also make a Mainloop procedure, and have all the procs in that, then just put MainLoop in the part that runs the script.


ive done a lot of changes in the new script and i might have a look and see how well that would work (with the combined banking) because before ive had a lot of issues with banking. i'll probably be uploading the script within the hour with a lot of updates

blazingrobin

2012-04-21 14:48:50

updated script. let me know what you guys think and thank you for the support

blazingrobin

2012-04-21 16:09:52

Originally Posted by NKN
Not bad.

Have you considered combining banking methods?

You could also make a Mainloop procedure, and have all the procs in that, then just put MainLoop in the part that runs the script.

oh and thank you for posting the script on making progress reports. it helped a lot and i gave credit in the script

NKN

2012-04-21 20:55:03

You could combine progress reports, using an If statement. So, if doing this, show this part, if doing this, show this other part.

blazingrobin

2012-04-21 22:30:01

ty updated. Oh and the reason I have two seperate withdraw statements is because i couldn't get it stable at the time when withdrawing. i might take a crack at it tomorrow or later today if i get a chance.

blazingrobin

2012-04-23 15:08:02

Updated quite a bit. send in any proggies you guys get it really helps me out to see how the script is doing

NKN

2012-04-23 17:43:00

Good luck! If you need anything, PM me. -nod-

blazingrobin

2012-04-24 08:40:35

UPDATE: fully update script with a lot more fletching support read above for details

Mr Stoj

2012-04-24 16:22:12

just looked at this, and its a very good script impressive but a misleading title

NKN

2012-04-24 16:23:57

Blazin. You considered removing old download links?
Getting kind of large!

blazingrobin

2012-04-24 20:41:14

From: Mr Stoj
just looked at this, and its a very good script impressive but a misleading title


Yea lol probably will end up when I move this to the fetching thread area I'll end up renaming it

blazingrobin

2012-04-24 20:42:37

From: NKN
Blazin. You considered removing old download links?
Getting kind of large!


Yea was half asleep when I finished doing all of this. I'll delete all but the newest and the new update I will be releasing

NKN

2012-04-24 21:45:24

You should wait between releases. Debug it, make it look nice, ect.

I don't know if it's buggy or not, just pointing that out for future reference. I made the mistake up uploading with a bug, had to re-upload every 10 minutes.


Edit: Ogod! The LARGE amount of If statements, use a case!

Bad: 

Procedure MasterPlan;
begin
 If Players[0].Integers[0] = 1 then
         begin
           WithdrawItems;
           Wait(1500)
           CraftItems;
           Progress;
          end;
 If Players[0].Integers[0] = 2 then
        begin
         HeadlessMaker;
         Progress;
        end;



 If Players[0].Integers[0] = 3 then
         begin
           WithdrawItems;
           Wait(1500)
           RegMaker;
           Progress;
          end;
If Players[0].Integers[0] = 4 then
         begin
           WithdrawItems;
           Wait(1500)
           OakMaker;
           Progress;
          end;
     If Players[0].Integers[0] = 5 then
         begin
           WithdrawItems;
           Wait(1500)
           WillowMaker;
           Progress;
          end;


 If Players[0].Integers[0] = 6 then
         begin
           WithdrawItems;
           Wait(1500)
           MapleMaker;
           Progress;
          end;



If Players[0].Integers[0] = 7 then
         begin
           WithdrawItems;
           Wait(1500)
           YewMaker;
           Progress;
          end;

If Players[0].Integers[0] = 8 then
         begin
           WithdrawItems;
           Wait(1500)
           MagicMaker;
           Progress;
          end;
end;
Good(Tabbing is off, I did this in reply.):
Procedure MasterPlan;
begin
 case  Integers[0] of
     1:
        begin
           WithdrawItems;
           Wait(1500)
           CraftItems;
           Progress;
          end;
    2:
        begin
         HeadlessMaker;
         Progress;
        end;



  3:
         begin
           WithdrawItems;
           Wait(1500)
           RegMaker;
           Progress;
          end;
  4:
         begin
           WithdrawItems;
           Wait(1500)
           OakMaker;
           Progress;
          end;
  5:
         begin
           WithdrawItems;
           Wait(1500)
           WillowMaker;
           Progress;
          end;


 6;
         begin
           WithdrawItems;
           Wait(1500)
           MapleMaker;
           Progress;
          end;



   7:
         begin
           WithdrawItems;
           Wait(1500)
           YewMaker;
           Progress;
          end;

   8:
         begin
           WithdrawItems;
           Wait(1500)
           MagicMaker;
           Progress;
          end;
  end;
end;


I've also said this to you a few times!
Bad:

repeat
   SetAngle(SRL_ANGLE_HIGH);
   MasterPlan;
  until false;
Good: 

repeat
   SetAngle(SRL_ANGLE_HIGH);
   MasterPlan;
  until AllPlayersInactive;

This one is really important!

I might fix up your script later, I don't know.

blazingrobin

2012-04-24 22:16:36

I'll check into the script more man don't worry about it. And before I ever release an update I make sure it fixed the errors it had or has no errors.

blazingrobin

2012-04-27 23:41:33

updated to version 5

blazingrobin

2012-05-05 13:40:29

anyone have any proggies so i can know how its working?

Evil_Joe

2012-05-06 00:49:29

hey for me it seem to not find the banking chest it stuck on saying finding npc bank and the cursor just seem to hover over the chest



Edit: It dont seem to work in SW but when i do in a normal bank like edgeville it works fine look into that
ps: thnx for this script! i have over 150k steel arrow head and got tire of making arrow shaft =D

blazingrobin

2012-05-08 18:05:38

From: Evil_Joe
hey for me it seem to not find the banking chest it stuck on saying finding npc bank and the cursor just seem to hover over the chest



Edit: It dont seem to work in SW but when i do in a normal bank like edgeville it works fine look into that
ps: thnx for this script! i have over 150k steel arrow head and got tire of making arrow shaft =D


did you set the setting correct in the form? if not ill have to finish updating it when i get a chance.

Joepetey

2012-05-08 18:29:06

Hope you fix your progress report, will use after the fix

josephh

2012-05-08 21:11:17

going to get 55 fletching on an account by making shafts, just started it and its been running for 10 minutes fine, looking good.
Will check back and tell you how it goes in a while, thanks for the script.

EDIT: am having issues with "out of logs"
attempts to get it to check twice before logging out are failing and i find the script stopping after 3-10 invs.
Help would be appreciated