Jump to content

veedsFisher


Recommended Posts

This is just a simple fisher with basic GUI.

Script download - At bottom of post.

Current Features

  • Fish - Shrimp, Anchovies, Lobster, Trout, Salmon, Swordfish
  • Banking
  • Automatically gathers required items ( if there in the bank )
  • Profiles

Planned Updates

  • New fish - Karambwanji, Karambwan, Tuna, Anglerfish, Bass, Cod, Dark crab, Herring, Monkfish
  • Different drop patterns
  • Restocking
  • New Gui

 

Quote

v1.3

veedsFisher_V1.3.zip

-Minor bug fixes

-Improved anti-ban

 

Quote

v1.2

veedsFisher_V1.2.zip

 

-Reworked movement and interaction with fishing spots

-Minor improvements and bug fixes

 

Quote

v1.1

veedsFisher_V1.1.zip

- Added Swordfish

- Minor bug fixes

 

 

Edited by veedubman87
  • Thanks 1
Link to comment
Share on other sites

Noice! :D 

 

One thing that comes up from the top of my head, is the way you pass the CTX into the functions and also the class

 

Try doing what's below:

    private final Data data;
    private final Bank bank;
    private final Fish fish;
    private final Walk walk;
    private final Inventory inventory;

    @Override
    public boolean onStart(String... strings) {
      	data = new Data(getAPIContext());
        bank = new Bank(getAPIContext());
      	fish = new Fish(getAPIContext());
      	walk = new Walk(getAPIContext());
      	inventory = new Inventory(getAPIContext());
      
        new Gui(getAPIContext());
        System.out.println("Staring veedsFisher");
        return true;
    }

 

That way, you don't need to pass it for each function since you already declared it as ctx in your sub-class :)

Link to comment
Share on other sites

1 hour ago, Proto said:

Noice! 😄

 

One thing that comes up from the top of my head, is the way you pass the CTX into the functions and also the class

 

Try doing what's below:

    private final Data data;
    private final Bank bank;
    private final Fish fish;
    private final Walk walk;
    private final Inventory inventory;

    @Override
    public boolean onStart(String... strings) {
      	data = new Data(getAPIContext());
        bank = new Bank(getAPIContext());
      	fish = new Fish(getAPIContext());
      	walk = new Walk(getAPIContext());
      	inventory = new Inventory(getAPIContext());
      
        new Gui(getAPIContext());
        System.out.println("Staring veedsFisher");
        return true;
    }

 

That way, you don't need to pass it for each function since you already declared it as ctx in your sub-class 🙂

Thx I was wondering why that wasn’t working I’ll fix it when I get home.

Link to comment
Share on other sites

  • 4 weeks later...
  • 4 weeks later...
  • 4 weeks later...

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

 Share

×
×
  • Create New...