Jump to content

Fishing Script Help


Popymon24

Recommended Posts



import org.osbot.rs07.api.model.NPC;
import org.osbot.rs07.api.model.RS2Object;
import org.osbot.rs07.api.ui.RS2Widget;
import org.osbot.rs07.script.Script;
import org.osbot.rs07.script.ScriptManifest;

import java.awt.*;
 
@ScriptManifest(author = "You", info = "My first script", name = "Tea thiever", version = 0, logo = "")
public class main extends Script {
 
    @Override
    public void onStart() {
        log("Let's get started!");
    }
    
    private enum State {
        FISH, COOK, DROP
    }
    
    private State getState() {
        if(inventory.isEmptyExcept(310, 17794, 309, 17795, 123, 17796) && !myPlayer().isAnimating())
            return State.FISH;
        if(inventory.isFull())
            return State.COOK;
        return State.DROP;
    }
 
    @Override
    public int onLoop() throws InterruptedException {
        switch(getState()) {
            case FISH:
                //Entity fishingSpot = objects.closest("Fishing Spot"); Wrong. Fishing spots are tied to NPCs

                //You meant to do:
                NPC fishingSpot = getNpcs().closest("Fishing Spot");
                if (fishingSpot != null) { //ALWAYS NULL CHECK BEFORE INTERACTING..Keep this in mind.   logic: if it exists -> interact
                    fishingSpot.interact("Lure");
                    //sleep
                }
                break;
            case COOK:
                //Entity fire = objects.closest("fire"); This is okay to do since Object,NPC,etc extend Entity
                //but should do it like this.
                RS2Object fire = getObjects().closest("Fire");
                RS2Widget cookMenu = widgets.get(307, 4);

                //I am going to throw some code in here for you. It will be your job to find it in the API docs and understand the methods I am implementing.
                if (cookMenu != null && cookMenu.isVisible()) {
                    cookMenu.interact("Cook All");
                    //sleep
                } else {
                    if (getInventory().isItemSelected()) {
                        if (fire != null) {
                            fire.interact("Use");
                            //sleep...look into ConditionalSleep
                        }
                    } else {
                        //item is not selected..select something
                        inventory.interact("Use", "Trout");
                        //sleep
                    }
                }
                break;
            case DROP:
                inventory.dropAll(331, 332, 333, 334, 25976);
                break;
        }
        return 600; //just return a server tick 600ms
    }
 
    @Override
    public void onExit() {
        log("Thanks for running my first script!");
    }
 
    @Override
    public void onPaint(Graphics2D g) {
 
    }
 
}

 

I wanted to make a script for OSBot, and decided that I wanted to make a script that fished at barbarian village and also cook the fish before dropping them. When I got to making the cooking part, I got stuck because I couldn't use interact on the fire like I could the fishing spots. Any help or touchups of what I have done are gladly welcomed!

 

 

 

Try this

Edited by Christopher
  • Like 1
Link to comment
Share on other sites



import org.osbot.rs07.api.model.NPC;
import org.osbot.rs07.api.model.RS2Object;
import org.osbot.rs07.api.ui.RS2Widget;
import org.osbot.rs07.script.Script;
import org.osbot.rs07.script.ScriptManifest;

import java.awt.*;
 
@ScriptManifest(author = "You", info = "My first script", name = "Tea thiever", version = 0, logo = "")
public class main extends Script {
 
    @Override
    public void onStart() {
        log("Let's get started!");
    }
    
    private enum State {
        FISH, COOK, DROP
    }
    
    private State getState() {
        if(inventory.isEmptyExcept(310, 17794, 309, 17795, 123, 17796) && !myPlayer().isAnimating())
            return State.FISH;
        if(inventory.isFull())
            return State.COOK;
        return State.DROP;
    }
 
    @Override
    public int onLoop() throws InterruptedException {
        switch(getState()) {
            case FISH:
                //Entity fishingSpot = objects.closest("Fishing Spot"); Wrong. Fishing spots are tied to NPCs

                //You meant to do:
                NPC fishingSpot = getNpcs().closest("Fishing Spot");
                if (fishingSpot != null) { //ALWAYS NULL CHECK BEFORE INTERACTING..Keep this in mind.   logic: if it exists -> interact
                    fishingSpot.interact("Lure");
                    //sleep
                }
                break;
            case COOK:
                //Entity fire = objects.closest("fire"); This is okay to do since Object,NPC,etc extend Entity
                //but should do it like this.
                RS2Object fire = getObjects().closest("Fire");
                RS2Widget cookMenu = widgets.get(307, 4);

                //I am going to throw some code in here for you. It will be your job to find it in the API docs and understand the methods I am implementing.
                if (cookMenu != null && cookMenu.isVisible()) {
                    cookMenu.interact("Cook All");
                    //sleep
                } else {
                    if (getInventory().isItemSelected()) {
                        if (fire != null) {
                            fire.interact("Use");
                            //sleep...look into ConditionalSleep
                        }
                    } else {
                        //item is not selected..select something
                        inventory.interact("Use", "Trout");
                        //sleep
                    }
                }
                break;
            case DROP:
                inventory.dropAll(331, 332, 333, 334, 25976);
                break;
        }
        return 600; //just return a server tick 600ms
    }
 
    @Override
    public void onExit() {
        log("Thanks for running my first script!");
    }
 
    @Override
    public void onPaint(Graphics2D g) {
 
    }
 
}

Try this

 

Holy crap. Thank you so much. I kinda rushed into scripting, and this helps a lot. Thanks for all the comments you left in it, I'll have a much better grasp next time I decide to write- and some more Java under my belt.

 

EDIT:

I went through and checked the ID's, and they were incorrect, which was why the script wasn't working. I had the wrong ID for the fishing rod, so it wasn't starting the FISH state.

Edited by Popymon24
Link to comment
Share on other sites

Holy crap. Thank you so much. I kinda rushed into scripting, and this helps a lot. Thanks for all the comments you left in it, I'll have a much better grasp next time I decide to write- and some more Java under my belt.

 

EDIT:

I went through and checked the ID's, and they were incorrect, which was why the script wasn't working. I had the wrong ID for the fishing rod, so it wasn't starting the FISH state.

Use names , no need to use IDs.

  • Like 1
Link to comment
Share on other sites

1) Few problems in your script, it will keep clicking the fishing spot bc you aren't checking if you are fishing. Check if your player is animating or not for this.

2) YOu don't check if you've done the previous steps, you will just keep looping non stop. 

3) You're missing brackets on your widgets

4) Why are you dropping 4 items?

5) Your script literally is just linear right now. Look into conditional sleeps and using if/else statements to check if you're doing an action.

6) Do the first few modules Code Academy to learn the basics of Java first. It will help you in the long wrong.

7) Use pastebin, code is ugly on OsBot and not formatted 

8) Shouldn't "fire" be capitalized? 

 

Brackets are good form for if/elses but they're not necessary if theres only like 1 line in code.

 

if (a == b) {

     ~~~~~;

}

 

is effectively the same as

 

if(a ==b)

  ~~~~~~~;

Edited by Imateamcape
Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

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.

  • Recently Browsing   0 members

    • No registered users viewing this page.
×
×
  • Create New...