DeEvade Posted September 16, 2021 Posted September 16, 2021 I want my script to sleep after hoppings worlds, I'm having problems when my onLoop tries to define objects from the game, before it is properly loaded. Which is causing some issues. It works if i add a static sleep, but something like a conditional sleep would make it a lot more reliable and faster. I tried using getGameState(), but it's always returning LOGGED_IN. Is there something that I've missed? @Override public int onLoop() throws InterruptedException { String state = getClient().getGameState().toString(); if(state != "LOADING" && state != "HOPPING") { npc = getNpcs().closest(npcName); if(!getStore().isOpen()) { if(hasItems()) { if(area(npcName).contains(myPlayer())) { if(npc != null) { if(npc.exists()) { npc.interact("Trade"); new Sleep(() -> getStore().isOpen(), 2000).sleep(); buyOrSell(); } } else { log("Npc is null"); } } else { walkTo(npcName); } } else { getItems(); } } else { buyOrSell(); } } else { log("State: " + state); } return 10; }
Hawk Posted September 16, 2021 Posted September 16, 2021 The proper way to compare strings is with .equals() if(!"LOADING".equals(state) && !"HOPPING".equals(state)) { You could also just compare directly to the enum Client.GameState state = getClient().getGameState(); if(!Client.GameState.LOADING.equals(state) && !Client.GameState.HOPPING.equals(state)) { There is a conditional sleep class: https://osbot.org/api/org/osbot/rs07/utility/ConditionalSleep2.html
Malcolm Posted September 17, 2021 Posted September 17, 2021 You may be able to get away with an easier check. Client#isLoggedIn will probably work for this