memelord123 Posted December 11, 2018 Share Posted December 11, 2018 (edited) Fly fishes trout and salmon at Barbarian Village then cooks the raw fish on the permanent fire. Option to drop cooked fish or to bank them. Current Version 1.1 Download: http://www.mediafire.com/file/35ftk4721uz144f/BarbVilleFisher%26Cooker.zip/file Progress Reports: Spoiler Change Log: Spoiler 1.0 - Initial release 1.1 - Fixed the bot occasionally not picking up the fire on default zoom, optimised script Edited September 28, 2023 by memelord123 1 Quote Link to comment Share on other sites More sharing options...
jca Posted December 11, 2018 Share Posted December 11, 2018 (edited) Good job on the release... I would try and refactor your code a little to be more maintainable. Rather than have a giant onLoop() you can break it down to functions and boolean checks. if ( ! myPlayer().isAnimating() ){ if ( !getInventory().isFull() ){ fish(); } else { if ( shouldBank() ){ bank(); } else { cook(); } } } else { // Wait as we are fishing or cooking } Also wrap your interactions in conditionals if ( fishingSpot.interact("Lure") ){ status = "Fishing"; //handle post interaction logic } This ensures you can check for interaction success / failure. Look into walking#walkPath as you shouldn't need to use webWalk for this. Edited December 11, 2018 by jca 1 Quote Link to comment Share on other sites More sharing options...
Juggles Posted December 11, 2018 Share Posted December 11, 2018 Start the bracket { on the line above please. Easier to read xD Quote Link to comment Share on other sites More sharing options...
memelord123 Posted December 11, 2018 Author Share Posted December 11, 2018 34 minutes ago, jca said: Good job on the release... I would try and refactor your code a little to be more maintainable. Rather than have a giant onLoop() you can break it down to functions and boolean checks. if ( ! myPlayer().isAnimating() ){ if ( !getInventory().isFull() ){ fish(); } else { if ( shouldBank() ){ bank(); } else { cook(); } } } else { // Wait as we are fishing or cooking } Also wrap your interactions in conditionals if ( fishingSpot.interact("Lure") ){ status = "Fishing"; //handle post interaction logic } This ensures you can check for interaction success / failure. Look into walking#walkPath as you shouldn't need to use webWalk for this. Thanks for the advice. I plan on breaking the onLoop down to a few methods but as I initially wrote it like this I kept it like this in the thread so others can easily read it without scrolling to each method. Second point makes total sense and I should be putting booleans like that in conditional statements but as I viewed it as "just an action" I didn't really think to write it the way you describe. Finally, I just used webWalk because I knew about it so I didn't look into any other methods of walking but I realised while using webWalk that it's not ideal, so I'll definitely look into walkPath. 20 minutes ago, Juggles said: Start the bracket { on the line above please. Easier to read xD Imo it looks prettier and easier to follow the way I do it and doesn't break the code conventions. But I realise most people write like the way you described and to them it probably seems like a lot of brackets flying around 1 Quote Link to comment Share on other sites More sharing options...
Kushitious Posted October 13, 2019 Share Posted October 13, 2019 [INFO][Bot #1][10/13 04:53:22 PM]: Terminating script BarbVilleFisher&Cooker... [WARN][Bot #1][10/13 04:53:27 PM]: Event executor is taking too long to suspend; terminating now... [INFO][Bot #1][10/13 04:53:27 PM]: Time ran: 15:28:54 [INFO][Bot #1][10/13 04:53:27 PM]: Fishing Xp gained: 365,250 [INFO][Bot #1][10/13 04:53:27 PM]: Cooking Xp gained: 471,640 [INFO][Bot #1][10/13 04:53:27 PM]: Script BarbVilleFisher&Cooker has exited! Ran for nearly 15.5 hours good script 1 Quote Link to comment Share on other sites More sharing options...
z10n Posted November 11, 2019 Share Posted November 11, 2019 (edited) Works great, ty! Does it have any reaction time variation? Edited November 11, 2019 by z10n 1 Quote Link to comment Share on other sites More sharing options...
memelord123 Posted November 11, 2019 Author Share Posted November 11, 2019 6 hours ago, z10n said: Works great, ty! Does it have any reaction time variation? Yes it's built in. 1 Quote Link to comment Share on other sites More sharing options...
z10n Posted November 11, 2019 Share Posted November 11, 2019 Ran from 20-58, no ban yet. =) Thanks, really nice script. 1 Quote Link to comment Share on other sites More sharing options...
Trum Trum Posted July 31, 2020 Share Posted July 31, 2020 Anyone used this lately, looking for a good f2p fishing script Quote Link to comment Share on other sites More sharing options...
stoodge Posted December 17, 2020 Share Posted December 17, 2020 On 8/1/2020 at 12:44 AM, Trum Trum said: Anyone used this lately, looking for a good f2p fishing script Did u end up trying this? Seeing if it’s still viable Quote Link to comment Share on other sites More sharing options...
SlurepieeG Posted January 23, 2022 Share Posted January 23, 2022 Looks interesting Quote Link to comment Share on other sites More sharing options...
boerlo Posted July 30, 2022 Share Posted July 30, 2022 Had it running for a few hours today, if i get banned i'll update. But still viable atm tbh. Not the best script but does what it says Quote Link to comment Share on other sites More sharing options...
CIeetZ Posted August 1, 2022 Share Posted August 1, 2022 can it tick fish with kebbit clawws? Quote Link to comment Share on other sites More sharing options...
skang Posted November 17, 2022 Share Posted November 17, 2022 Bot works well, really fills the niche in free scripts. Quote Link to comment Share on other sites More sharing options...