Jump to content

Dema

Members
  • Posts

    4
  • Joined

  • Last visited

  • Feedback

    0%

Posts posted by Dema

  1. The problemo is here:

    
        switch(getState()){
            case KILL:
            NPC skeleton = npcs.closest("Skeleton");
            if (skeleton.isVisible() && getTabs().getOpen().equals(magic)){
            getMagic().castSpellOnEntity(Spells.NormalSpells.CRUMBLE_UNDEAD, skeleton);
            } else sleep(random(100,250));
            getTabs().open(Tab.MAGIC);
            break;
    

    more importantly:

     if (skeleton.isVisible() && getTabs().getOpen().equals(magic)){
    

    Firstly, nullcheck for the skeleton. Secondly, it would have to be 

    getTabs().getOpen().equals(Tab.MAGIC);
    

    (not sure why you had just magic there, but i would expect it to give you an error for that). The reason it opens the magic tab then does nothing is because that if statement returns false every time as you just wrote magic in there. that means it always goes to the else part which says open the magic tab (which already checks if it is open or not before performing). So ye, replace all that with this:

            case KILL:
            NPC skeleton = npcs.closest("Skeleton");
            if (skeleton != null && skeleton.exists() && skeleton.isVisible() && getTabs().getOpen().equals(Tab.MAGIC)){
            getMagic().castSpellOnEntity(Spells.NormalSpells.CRUMBLE_UNDEAD, skeleton);
    sleep(3000L);
            } else {
    sleep(random(100,250));
            getTabs().open(Tab.MAGIC);
    }
            break;
    

    Let me know if anything is unclear

     

    Apaec

    Oh hot damn, that was a pretty stupid mistake. Thanks man, its working now.

    • Like 1
  2. Im new to scripting and i made my first script, the point is to make my character continuosly cast crumble undead from the spellbook (without a staff) on the nearest skeleton (i will add Ghosts later). The bot is opening the magic tab, then doing nothing. I have no idea what might cause this issue.

    
    
    import org.osbot.rs07.api.ui.Spells;
    import org.osbot.rs07.api.ui.Tab;
    import org.osbot.rs07.script.Script;
    import org.osbot.rs07.script.ScriptManifest;
     
     
     
    import java.awt.*;
     
    @ScriptManifest(author = "Dema", info = "Fast magic and hitpoints exp", name = "Crumble Undead", version = 0.13, logo = "")
    public class main extends Script {
     
        @Override
        public void onStart() {
            log("Get money, Fuck bitches");
        }
        
        private enum State {
    KILL, WAIT
    };
     
    private State getState() {
    NPC skeleton = npcs.closest("Skeleton");
    if (skeleton != null)
    return State.KILL;
    return State.WAIT;
     
    }
        @Override
        public int onLoop() throws InterruptedException {
        switch(getState()){
            case KILL:
            NPC skeleton = npcs.closest("Skeleton");
            if (skeleton.isVisible() && getTabs().getOpen().equals(magic)){
            getMagic().castSpellOnEntity(Spells.NormalSpells.CRUMBLE_UNDEAD, skeleton);
            } else sleep(random(100,250));
            getTabs().open(Tab.MAGIC);
            break;
        case WAIT:
        sleep(random(500,700));
        break;
        }
        return random(150, 350);
        onLoop();
        }
        
        @Override
        public void onExit() {
            log("Thanks for running my CRUMBLAH!");
        }
     
        @Override
        public void onPaint(Graphics2D g) {
     
        }
     
    }
×
×
  • Create New...