Jump to content
Search In
  • More options...
Find results that contain...
Find results in...

shadster4321

Registered
  • Content Count

    50
  • Joined

  • Last visited

  • Feedback

    N/A

Community Reputation

3 Neutral

About shadster4321

  • Rank
    Botter

Recent Profile Visitors

945 profile views
  1. This kind of thing usually happens if you use a data centre proxy to do tutorial island.
  2. The 'account disabled' screen changed very recently. That would be why. Use colour detection to fix it.
  3. Already checked that. Unfortunately on that page nothing is shown under "Launching examples:", it's just a blank space. The page shows how to add things like proxy and script name, but doesn't show what the actual string of text you're supposed to enter looks like.
  4. Are people still successfully running mass f2p farms?
  5. Wow I've spent way too much time manually walking to places to get the coordinates
  6. Wow. For an afk task where you're constantly moving your mouse to another window that must be a pretty blatant give away that you're a bot
  7. I'm impressed by the number of features this script has. With the automatic axe upgrading, how will the account gather the funds to buy the axes on the exchange? I'm also interested in the mathematical formula you use for determining which account to accept trades from. Mind sharing some details about it?
  8. shadster4321

    Delay Bans

    Nice, is that in f2p or p2p?
  9. Damn, that code is really nice. It fixes the 'arrow code' problem that I had, making it much easier to understand. I doubt I would have thought of this myself. Are there any ways I can improve my ability to write my code in a more logical way like yours?
  10. I considered that but I was worried it would break the principle of a function only doing one thing because the function would then create both oak and wooden chairs
  11. I have two different methods: public static void constructCrudeWoodenChairs() { boolean isAtHouse = isAtHouse(); if (isAtHouse && isInBuildingMode()) { if (Inventory.getCount("Plank") >= 2) { if (!isChairConstructed()) { if (!isBuildChairInterfaceOpen()) { openBuildChairInterface(); } else { clickBuildCrudeWoodenChair(); } } else { removeChair(); } } else { goToCastleWars(); } } else if (isAtHouse && !isInBuildingMode()) { enterBuildingMode(); } else if (isAtCastle()) { bankForChairs("Plank"); } else { goToHouse(); } } and public static void constructOakChairs() { boolean isAtHouse = isAtHouse(); if (isAtHouse && isInBuildingMode()) { if (Inventory.getCount("Oak plank") >= 2) { if (!isChairConstructed()) { if (!isBuildChairInterfaceOpen()) { openBuildChairInterface(); } else { clickBuildOakChair(); } } else { removeChair(); } } else { goToCastleWars(); } } else if (isAtHouse && !isInBuildingMode()) { enterBuildingMode(); } else if (isAtCastle()) { bankForChairs("Oak plank"); } else { goToHouse(); } } These methods are identical except for two things and I am trying to keep to the principle of avoiding repeated code. Is there a way for me to not use the same code twice here?
  12. Is it better to write a method like this: public static void constructCrudeWoodenChairs() { if (!isBuildChairInterfaceOpen()) { openBuildChairInterface(); } else { clickBuildCrudeWoodenChair(); } } or like this: public static void constructCrudeWoodenChairs() { if (isBuildChairInterfaceOpen()) { clickBuildCrudeWoodenChair(); } else { openBuildChairInterface(); } }
×
×
  • Create New...