Thread: [PI] Woodcutting Help Again

Results 1 to 9 of 9
  1. #1 [PI] Woodcutting Help Again 
    Registered Member Ired's Avatar
    Join Date
    May 2012
    Posts
    160
    Thanks given
    13
    Thanks received
    10
    Rep Power
    12
    Alright, I figured out the Better Woodcutting tutorial and how I needed to add an interger for my boolean I had for new objects. But now I'm using a different tutorial and keep running into the same problem maybe someone can help me with. I will try and do the best I can on explaining it. I used this tutorial for it (Mopar)
    [PI] 90% - 100% woodcutting || Perfection (No link, because it's on Mopar).

    Problem: So I was trying to make my woodcutting better with stumps and what not, but ran into a problem during that way. I don't know what f'ed it up, but it's pretty f'ed up now. My player will woodcut the tree (At level 1 woodcutting on normal tree) for nearly 5 times, then just randomly stop and stand there(probably has to do with animation), but anyone got any help?



    --No Errors compiling--


    Spoiler for Woodcutting.java:
    package server.model.players.skills;

    import server.model.players.*;
    import server.Config;
    import server.util.Misc;
    import server.Server;
    import java.lang.Math;
    import server.model.players.Client;
    import server.model.players.PacketType;
    import server.model.objects.Object;
    import server.model.players.PlayerHandler;

    /**
    * @author Sanity
    * @note Heavily modified by Cres || rebellion
    */

    public class Woodcutting {

    Client c;

    private final int VALID_AXE[] = {1351,1349,1353,1361,1355,1357,1359,6739};
    private final int[] AXE_REQS = {1,1,6,6,21,31,41,61};
    private int logType;
    private int exp;
    private int levelReq;
    private int axeType;
    private int treeX;
    private int treeY;
    private int density;
    private int treeType;
    public int logsCutFromTree;
    public int tempTimer = 0;

    public static int TREE_TIMER = 2000;

    public Woodcutting(Client c) {
    this.c = c;
    }

    public void startWoodcutting(int logType, int levelReq, int exp, int treeX, int treeY, int density, int treeType) {
    if (goodAxe() > 0) {
    c.turnPlayerTo(treeX, treeY);
    if (c.playerLevel[c.playerWoodcutting] >= levelReq) {
    this.logType = logType;
    this.exp = exp;
    this.levelReq = levelReq;
    this.treeX = treeX;
    this.treeY = treeY;
    this.density = density;
    this.treeType = treeType;
    this.axeType = goodAxe();
    c.wcTimer = getWcTimer(density);
    c.startAnimation(getWcEmote());
    } else {
    c.getPA().resetVariables();
    c.startAnimation(65535);
    c.sendMessage("You need a woodcutting level of " + levelReq + " to cut this tree.");
    }
    } else {
    c.startAnimation(65535);
    c.sendMessage("You don't have an axe for which you have the level to use.");
    c.getPA().resetVariables();
    }
    }

    public void resetWoodcut() {
    this.logType = -1;
    this.exp = -1;
    this.levelReq = -1;
    this.axeType = -1;
    this.density = -1;
    this.treeX = -1;
    this.treeY = -1;
    this.treeType = -1;
    c.wcTimer = -1;
    logsCutFromTree = 0;
    }

    public int getWcEmote() {
    if (axeType == 1351) // bronze
    return 879;
    if (axeType == 1349) // iron
    return 877;
    if (axeType == 1353) // steel
    return 875;
    if (axeType == 1355) // mith
    return 871;
    if (axeType == 1357) // addy
    return 869;
    if (axeType == 1359) // rune
    return 867;
    if (axeType == 6739) // d axe
    return 2846;
    if (axeType == 1361) // black
    return 873;
    else
    return 0;
    }

    public void cutWood() {
    int maxLogsPerTree = cutForChop(treeType);
    if (logsCutFromTree < maxLogsPerTree) {
    if (c.getItems().addItem(logType,1)) {
    c.startAnimation(getWcEmote());
    logsCutFromTree += 1;
    c.sendMessage("You get some logs.");
    c.getPA().addSkillXP(exp * Config.WOODCUTTING_EXPERIENCE, c.playerWoodcutting);
    c.getPA().refreshSkill(c.playerWoodcutting);
    c.wcTimer = getWcTimer(density);
    } else {
    c.getPA().resetVariables();
    return;
    }
    } else {
    for (int j = 0; j < Server.playerHandler.players.length; j++) {
    if (Server.playerHandler.players[j] != null) {
    Client c2 = (Client)Server.playerHandler.players[j];
    new Object(1343, treeX, treeY, 0, 1, 10, treeType, getTicksForTree(), false);
    c.getPA().resetVariables();
    resetWoodcut();
    }
    }
    return;
    }
    }

    public int getTicksForTree() {
    return (2000-PlayerHandler.getPlayerCount())/100 + additionalTicksPerTree();
    }

    public int additionalTicksPerTree() {
    if (treeType == 1276) // reg
    return 1;
    if (treeType == 1278) // reg
    return 1;
    if (treeType == 1281) // oak
    return 2;
    if (treeType == 1308) // willow
    return 3;
    if (treeType == 1307) // maple
    return 4;
    if (treeType == 1309) // yew
    return 6;
    if (treeType == 1306) // magic
    return 7;
    else
    return 0;
    }

    public int cutForChop(int tree) {
    if (tree == 1276 || tree == 1278)
    return 3;
    if (tree == 1281)
    return 5;
    if (tree == 1308)
    return 6;
    if (tree == 1307)
    return 7;
    if (tree == 1309)
    return 8;
    if (tree == 1306)
    return 10;
    else
    return 0;
    }

    public int axePower(int axe) {
    if (axe == 1351) // bronze
    return 1;
    if (axe == 1349) // iron
    return 1;
    if (axe == 1353) // steel
    return 2;
    if (axe == 1361) // black
    return 2;
    if (axe == 1355) // mith
    return 3;
    if (axe == 1357) // addy
    return 4;
    if (axe == 1359) // rune
    return 5;
    if (axe == 6739) //daxe
    return 7;
    else
    return 0;
    }

    public int goodAxe() {
    for (int j = VALID_AXE.length - 1; j >= 0; j--) {
    if (c.playerEquipment[c.playerWeapon] == VALID_AXE[j]) {
    if (c.playerLevel[c.playerWoodcutting] >= AXE_REQS[j])
    return VALID_AXE[j];
    }
    }
    for (int i = 0; i < c.playerItems.length; i++) {
    for (int j = VALID_AXE.length - 1; j >= 0; j--) {
    if (c.playerItems[i] == VALID_AXE[j] + 1) {
    if (c.playerLevel[c.playerWoodcutting] >= AXE_REQS[j])
    return VALID_AXE[j];
    }
    }
    }
    return - 1;
    }

    public int getWcTimer(int density) {
    int time = Misc.random(2);
    return density + time - axePower(goodAxe());
    }

    }


    Spoiler for ActionHandler.java:
    package server.model.players;

    import server.Config;
    import server.Server;
    import server.event.CycleEvent;
    import server.event.CycleEventContainer;
    import server.event.CycleEventHandler;
    import server.model.objects.Object;
    import server.util.Misc;
    import server.util.ScriptManager;

    public class ActionHandler {

    private Client c;

    public ActionHandler(Client Client) {
    this.c = Client;
    }

    public void firstClickObject(int objectType, int obX, int obY) {
    c.clickObjectType = 0;
    if (c.agilityEmote)
    return;
    c.getAgil().appendObstacles(objectType);
    // c.sendMessage("Object type: " + objectType);
    switch (objectType) {
    case 2492:
    if (c.killCount >= 20) {
    c.getDH().sendOption4("Armadyl", "Bandos", "Saradomin",
    "Zamorak");
    c.dialogueAction = 20;
    } else {
    c.sendMessage("You need 20 kill count before teleporting to a boss chamber.");
    }
    break;

    case 190:
    c.getPA().movePlayer(2461, 3385, 0);
    break;
    case 12045:
    case 12047:
    c.getDH().sendDialogues(66, 0);
    break;

    case 2408:
    c.getDH().sendDialogues(58, 656);
    break;

    case 2406:
    if (c.playerEquipment[c.playerWeapon] == 772) {
    if (c.lostcity == 5) {
    c.getQuestHandler().LostCityComplete();
    c.getPA().movePlayer(2452, 4471, 0);
    } else if (c.lostcity >= 6) {
    c.getPA().movePlayer(2452, 4471, 0);
    }
    }
    break;

    case 1292:
    if (c.lostcity == 3) {
    Server.npcHandler.spawnNpc(c, 655, 2860, 9733 - 0, 0, 0, 120, 14, 101, 96, true, true);
    c.sendMessage("A tree spirit appears!");
    c.lostcity = 4;
    } else if(c.lostcity >=5) {
    if(c.getItems().playerHasItem(1351) || c.getItems().playerHasItem(1349) || c.getItems().playerHasItem(1353) || c.getItems().playerHasItem(1355) || c.getItems().playerHasItem(1357) || c.getItems().playerHasItem(1359) && c.playerLevel[8] >= 36) {
    c.getItems().addItem(771, 1);
    } else if(!c.getItems().playerHasItem(1351) || c.getItems().playerHasItem(1349) || c.getItems().playerHasItem(1353) || c.getItems().playerHasItem(1355) || c.getItems().playerHasItem(1357) || c.getItems().playerHasItem(1359)) {
    c.sendMessage("You need an axe.");
    }
    if (c.playerLevel[8] <= 35) {
    c.sendMessage("You need 36 woodcutting.");
    }
    }
    break;
    case 2213:
    case 14367:
    case 11758:
    case 3193:
    case 11402:
    case 27663:
    case 26972:
    case 16700:
    case 4483:
    c.getPA().openUpBank();
    break;
    case 9299:
    if (c.objectX == 3240 && c.objectY == 3190 && c.lumbyswamp == false) {
    c.getPA().movePlayer(3240, 3190, 0);
    c.sendMessage("You squeeze through the fence...");
    c.lumbyswamp = true;
    } else if (c.objectX == 3240 && c.objectY == 3190 && c.lumbyswamp == true) {
    c.getPA().movePlayer(3240, 3191, 0);
    c.sendMessage("You squeeze through the fence...");
    c.lumbyswamp = false;
    }
    break;


    case 1765:
    c.getPA().movePlayer(3067, 10256, 0);
    break;
    case 2882:
    case 2883:
    if (c.objectX == 3268) {
    if (c.absX < c.objectX) {
    c.getPA().walkTo(1, 0);
    } else {
    c.getPA().walkTo(-1, 0);
    }
    }
    break;
    case 272:
    c.getPA().movePlayer(c.absX, c.absY, 1);
    break;

    case 273:
    c.getPA().movePlayer(c.absX, c.absY, 0);
    break;
    case 245:
    c.getPA().movePlayer(c.absX, c.absY + 2, 2);
    break;
    case 246:
    c.getPA().movePlayer(c.absX, c.absY - 2, 1);
    break;
    case 1766:
    c.getPA().movePlayer(3016, 3849, 0);
    break;
    case 6552:
    if (c.playerMagicBook == 0) {
    c.playerMagicBook = 1;
    c.setSidebarInterface(6, 12855);
    c.sendMessage("An ancient wisdomin fills your mind.");
    c.getPA().resetAutocast();
    } else {
    c.setSidebarInterface(6, 1151); // modern
    c.playerMagicBook = 0;
    c.sendMessage("You feel a drain on your memory.");
    c.autocastId = -1;
    c.getPA().resetAutocast();
    }
    break;

    case 410:
    if (c.playerMagicBook == 0) {
    c.playerMagicBook = 2;
    c.setSidebarInterface(6, 29999);
    c.sendMessage("Lunar wisdom fills your mind.");
    c.getPA().resetAutocast();
    } else {
    c.setSidebarInterface(6, 1151); // modern
    c.playerMagicBook = 0;
    c.sendMessage("You feel a drain on your memory.");
    c.autocastId = -1;
    c.getPA().resetAutocast();
    }
    break;
    case 1816:
    c.getPA().startTeleport2(2271, 4680, 0);
    break;
    case 1817:
    c.getPA().startTeleport(3067, 10253, 0, "modern");
    break;
    case 1814:
    // ardy lever
    c.getPA().startTeleport(3153, 3923, 0, "modern");
    break;

    case 9356:
    // c.getPA().enterCaves();
    c.sendMessage("Temporarily removed due to bugs.");
    break;
    case 1733:
    c.getPA().movePlayer(c.absX, c.absY + 6393, 0);
    break;

    case 1734:
    c.getPA().movePlayer(c.absX, c.absY - 6396, 0);
    break;

    case 9357:
    c.getPA().resetTzhaar();
    break;

    case 10177:
    c.getPA().movePlayer(1890, 4407, 0);
    break;
    case 10230:
    c.getPA().movePlayer(2900, 4449, 0);
    break;
    case 10229:
    c.getPA().movePlayer(1912, 4367, 0);
    break;
    case 2623:
    if (c.absX >= c.objectX)
    c.getPA().walkTo(-1, 0);
    else
    c.getPA().walkTo(1, 0);
    break;
    // pc boat
    case 14315:
    c.getPA().movePlayer(2661, 2639, 0);
    break;
    case 14314:
    c.getPA().movePlayer(2657, 2639, 0);
    break;

    case 1596:
    case 1597:
    if (c.getY() >= c.objectY)
    c.getPA().walkTo(0, -1);
    else
    c.getPA().walkTo(0, 1);
    break;

    case 14235:
    case 14233:
    if (c.objectX == 2670)
    if (c.absX <= 2670)
    c.absX = 2671;
    else
    c.absX = 2670;
    if (c.objectX == 2643)
    if (c.absX >= 2643)
    c.absX = 2642;
    else
    c.absX = 2643;
    if (c.absX <= 2585)
    c.absY += 1;
    else
    c.absY -= 1;
    c.getPA().movePlayer(c.absX, c.absY, 0);
    break;

    case 14829:
    case 14830:
    case 14827:
    case 14828:
    case 14826:
    case 14831:
    // Server.objectHandler.startObelisk(objectType);
    Server.objectManager.startObelisk(objectType);
    break;
    case 4387:
    // Server.castleWars.joinWait(c,1);
    break;

    case 4388:
    // Server.castleWars.joinWait(c,2);
    break;

    case 4408:
    // Server.castleWars.joinWait(c,3);
    break;

    case 9369:
    if (c.getY() > 5175)
    c.getPA().movePlayer(2399, 5175, 0);
    else
    c.getPA().movePlayer(2399, 5177, 0);
    break;

    case 9368:
    if (c.getY() < 5169) {
    Server.fightPits.removePlayerFromPits(c.playerId);
    c.getPA().movePlayer(2399, 5169, 0);
    }
    break;
    case 4411:
    case 4415:
    case 4417:
    case 4418:
    case 4419:
    case 4420:
    case 4469:
    case 4470:
    case 4911:
    case 4912:
    case 1747:
    case 1757:
    // Server.castleWars.handleObjects(c, objectType, obX, obY);
    break;

    case 2286:
    case 154:
    case 4058:
    case 2295:
    case 2285:
    case 2313:
    case 2312:
    case 2314:
    c.getAgility().handleGnomeCourse(objectType, obX, obY);
    break;

    /*
    * Barrows Chest
    */
    case 10284:
    if (c.barrowsKillCount < 5) {
    c.sendMessage("You haven't killed all the brothers");
    }
    if (c.barrowsKillCount == 5
    && c.barrowsNpcs[c.randomCoffin][1] == 1) {
    c.sendMessage("I have already summoned this npc.");
    }
    if (c.barrowsNpcs[c.randomCoffin][1] == 0
    && c.barrowsKillCount >= 5) {
    Server.npcHandler.spawnNpc(c, c.barrowsNpcs[c.randomCoffin][0],
    3551, 9694 - 1, 0, 0, 120, 30, 200, 200, true, true);
    c.barrowsNpcs[c.randomCoffin][1] = 1;
    }
    if ((c.barrowsKillCount > 5 || c.barrowsNpcs[c.randomCoffin][1] == 2)
    && c.getItems().freeSlots() >= 2) {
    c.getPA().resetBarrows();
    c.getItems().addItem(c.getPA().randomRunes(),
    Misc.random(150) + 100);
    if (Misc.random(2) == 1)
    c.getItems().addItem(c.getPA().randomBarrows(), 1);
    c.getPA().startTeleport(3564, 3288, 0, "modern");
    } else if (c.barrowsKillCount > 5 && c.getItems().freeSlots() <= 1) {
    c.sendMessage("You need at least 2 inventory slot opened.");
    }
    break;
    /*
    * Doors
    */
    case 6749:
    if (obX == 3562 && obY == 9678) {
    c.getPA().object(3562, 9678, 6749, -3, 0);
    c.getPA().object(3562, 9677, 6730, -1, 0);
    } else if (obX == 3558 && obY == 9677) {
    c.getPA().object(3558, 9677, 6749, -1, 0);
    c.getPA().object(3558, 9678, 6730, -3, 0);
    }
    break;
    case 6730:
    if (obX == 3558 && obY == 9677) {
    c.getPA().object(3562, 9678, 6749, -3, 0);
    c.getPA().object(3562, 9677, 6730, -1, 0);
    } else if (obX == 3558 && obY == 9678) {
    c.getPA().object(3558, 9677, 6749, -1, 0);
    c.getPA().object(3558, 9678, 6730, -3, 0);
    }
    break;
    case 6727:
    if (obX == 3551 && obY == 9684) {
    c.sendMessage("You cant open this door..");
    }
    break;
    case 6746:
    if (obX == 3552 && obY == 9684) {
    c.sendMessage("You cant open this door..");
    }
    break;
    case 6748:
    if (obX == 3545 && obY == 9678) {
    c.getPA().object(3545, 9678, 6748, -3, 0);
    c.getPA().object(3545, 9677, 6729, -1, 0);
    } else if (obX == 3541 && obY == 9677) {
    c.getPA().object(3541, 9677, 6748, -1, 0);
    c.getPA().object(3541, 9678, 6729, -3, 0);
    }
    break;
    case 6729:
    if (obX == 3545 && obY == 9677) {
    c.getPA().object(3545, 9678, 6748, -3, 0);
    c.getPA().object(3545, 9677, 6729, -1, 0);
    } else if (obX == 3541 && obY == 9678) {
    c.getPA().object(3541, 9677, 6748, -1, 0);
    c.getPA().object(3541, 9678, 6729, -3, 0);
    }
    break;
    case 6726:
    if (obX == 3534 && obY == 9684) {
    c.getPA().object(3534, 9684, 6726, -4, 0);
    c.getPA().object(3535, 9684, 6745, -2, 0);
    } else if (obX == 3535 && obY == 9688) {
    c.getPA().object(3535, 9688, 6726, -2, 0);
    c.getPA().object(3534, 9688, 6745, -4, 0);
    }
    break;
    case 6745:
    if (obX == 3535 && obY == 9684) {
    c.getPA().object(3534, 9684, 6726, -4, 0);
    c.getPA().object(3535, 9684, 6745, -2, 0);
    } else if (obX == 3534 && obY == 9688) {
    c.getPA().object(3535, 9688, 6726, -2, 0);
    c.getPA().object(3534, 9688, 6745, -4, 0);
    }
    break;
    case 6743:
    if (obX == 3545 && obY == 9695) {
    c.getPA().object(3545, 9694, 6724, -1, 0);
    c.getPA().object(3545, 9695, 6743, -3, 0);
    } else if (obX == 3541 && obY == 9694) {
    c.getPA().object(3541, 9694, 6724, -1, 0);
    c.getPA().object(3541, 9695, 6743, -3, 0);
    }
    break;
    case 6724:
    if (obX == 3545 && obY == 9694) {
    c.getPA().object(3545, 9694, 6724, -1, 0);
    c.getPA().object(3545, 9695, 6743, -3, 0);
    } else if (obX == 3541 && obY == 9695) {
    c.getPA().object(3541, 9694, 6724, -1, 0);
    c.getPA().object(3541, 9695, 6743, -3, 0);
    }
    break;
    /*
    * Cofins
    */
    case 6707: // verac
    c.getPA().movePlayer(3556, 3298, 0);
    break;

    case 6823:
    if (server.model.minigames.Barrows.selectCoffin(c, objectType)) {
    return;
    }
    if (c.barrowsNpcs[0][1] == 0) {
    Server.npcHandler.spawnNpc(c, 2030, c.getX(), c.getY() - 1, -1,
    0, 120, 25, 200, 200, true, true);
    c.barrowsNpcs[0][1] = 1;
    } else {
    c.sendMessage("You have already searched in this sarcophagus.");
    }
    break;

    case 6706: // torag
    c.getPA().movePlayer(3553, 3283, 0);
    break;

    case 6772:
    if (server.model.minigames.Barrows.selectCoffin(c, objectType)) {
    return;
    }
    if (c.barrowsNpcs[1][1] == 0) {
    Server.npcHandler.spawnNpc(c, 2029, c.getX() + 1, c.getY(), -1,
    0, 120, 20, 200, 200, true, true);
    c.barrowsNpcs[1][1] = 1;
    } else {
    c.sendMessage("You have already searched in this sarcophagus.");
    }
    break;

    case 6705: // karil stairs
    c.getPA().movePlayer(3565, 3276, 0);
    break;
    case 6822:
    if (server.model.minigames.Barrows.selectCoffin(c, objectType)) {
    return;
    }
    if (c.barrowsNpcs[2][1] == 0) {
    Server.npcHandler.spawnNpc(c, 2028, c.getX(), c.getY() - 1, -1,
    0, 90, 17, 200, 200, true, true);
    c.barrowsNpcs[2][1] = 1;
    } else {
    c.sendMessage("You have already searched in this sarcophagus.");
    }
    break;

    case 6704: // guthan stairs
    c.getPA().movePlayer(3578, 3284, 0);
    break;
    case 6773:
    if (server.model.minigames.Barrows.selectCoffin(c, objectType)) {
    return;
    }
    if (c.barrowsNpcs[3][1] == 0) {
    Server.npcHandler.spawnNpc(c, 2027, c.getX(), c.getY() - 1, -1,
    0, 120, 23, 200, 200, true, true);
    c.barrowsNpcs[3][1] = 1;
    } else {
    c.sendMessage("You have already searched in this sarcophagus.");
    }
    break;

    case 6703: // dharok stairs
    c.getPA().movePlayer(3574, 3298, 0);
    break;
    case 6771:
    if (server.model.minigames.Barrows.selectCoffin(c, objectType)) {
    return;
    }
    if (c.barrowsNpcs[4][1] == 0) {
    Server.npcHandler.spawnNpc(c, 2026, c.getX(), c.getY() - 1, -1,
    0, 120, 45, 250, 250, true, true);
    c.barrowsNpcs[4][1] = 1;
    } else {
    c.sendMessage("You have already searched in this sarcophagus.");
    }
    break;

    case 6702: // ahrim stairs
    c.getPA().movePlayer(3565, 3290, 0);
    break;
    case 6821:
    if (server.model.minigames.Barrows.selectCoffin(c, objectType)) {
    return;
    }
    if (c.barrowsNpcs[5][1] == 0) {
    Server.npcHandler.spawnNpc(c, 2025, c.getX(), c.getY() - 1, -1,
    0, 90, 19, 200, 200, true, true);
    c.barrowsNpcs[5][1] = 1;
    } else {
    c.sendMessage("You have already searched in this sarcophagus.");
    }
    break;

    /**
    * @param 0 , the log you get for cutting the tree.
    * @param 1 , the level required to cut the tree.
    * @param 2 , the experience gained for cutting the tree.
    * @param 3 , the X coordinate of the tree.
    * @param 4 , the Y coordinate of the tree.
    * @param 5 , the density of the tree.
    * @param 6 , the id of the tree.
    */

    case 1276:
    case 1278://trees
    c.woodcut[0] = 1511;
    c.woodcut[1] = 1;
    c.woodcut[2] = 25;
    c.woodcut[3] = obX;
    c.woodcut[4] = obY;
    c.woodcut[5] = 8;
    c.woodcut[6] = objectType;
    c.getWoodcutting().startWoodcutting(c.woodcut[0], c.woodcut[1], c.woodcut[2], c.woodcut[3], c.woodcut[4], c.woodcut[5], c.woodcut[6]);
    break;
    case 1281: //oak
    c.woodcut[0] = 1521;
    c.woodcut[1] = 15;
    c.woodcut[2] = 37;
    c.woodcut[3] = obX;
    c.woodcut[4] = obY;
    c.woodcut[5] = 9;
    c.woodcut[6] = objectType;
    c.getWoodcutting().startWoodcutting(c.woodcut[0], c.woodcut[1], c.woodcut[2], c.woodcut[3], c.woodcut[4], c.woodcut[5], c.woodcut[6]);
    break;

    case 1308: //willow
    c.woodcut[0] = 1519;
    c.woodcut[1] = 30;
    c.woodcut[2] = 68;
    c.woodcut[3] = obX;
    c.woodcut[4] = obY;
    c.woodcut[5] = 10;
    c.woodcut[6] = objectType;
    c.getWoodcutting().startWoodcutting(c.woodcut[0], c.woodcut[1], c.woodcut[2], c.woodcut[3], c.woodcut[4], c.woodcut[5], c.woodcut[6]);
    break;

    case 1307: //maple
    c.woodcut[0] = 1517;
    c.woodcut[1] = 45;
    c.woodcut[2] = 100;
    c.woodcut[3] = obX;
    c.woodcut[4] = obY;
    c.woodcut[5] = 11;
    c.woodcut[6] = objectType;
    c.getWoodcutting().startWoodcutting(c.woodcut[0], c.woodcut[1], c.woodcut[2], c.woodcut[3], c.woodcut[4], c.woodcut[5], c.woodcut[6]);
    break;

    case 1309: //yew
    c.woodcut[0] = 1515;
    c.woodcut[1] = 60;
    c.woodcut[2] = 175;
    c.woodcut[3] = obX;
    c.woodcut[4] = obY;
    c.woodcut[5] = 12;
    c.woodcut[6] = objectType;
    c.getWoodcutting().startWoodcutting(c.woodcut[0], c.woodcut[1], c.woodcut[2], c.woodcut[3], c.woodcut[4], c.woodcut[5], c.woodcut[6]);
    break;

    case 1306: // magic
    c.woodcut[0] = 1513;
    c.woodcut[1] = 75;
    c.woodcut[2] = 250;
    c.woodcut[3] = obX;
    c.woodcut[4] = obY;
    c.woodcut[5] = 13;
    c.woodcut[6] = objectType;
    c.getWoodcutting().startWoodcutting(c.woodcut[0], c.woodcut[1], c.woodcut[2], c.woodcut[3], c.woodcut[4], c.woodcut[5], c.woodcut[6]);
    break;

    case 2090:// copper
    case 2091:
    c.mining[0] = 436;
    c.mining[1] = 1;
    c.mining[2] = 18;
    c.getMining().startMining(c.mining[0], c.mining[1], c.mining[2]);
    break;

    case 2094:// tin
    c.mining[0] = 438;
    c.mining[1] = 1;
    c.mining[2] = 18;
    c.getMining().startMining(c.mining[0], c.mining[1], c.mining[2]);
    break;

    case 145856:
    case 2092:
    case 2093: // iron
    c.mining[0] = 440;
    c.mining[1] = 15;
    c.mining[2] = 35;
    c.getMining().startMining(c.mining[0], c.mining[1], c.mining[2]);
    break;

    case 14850:
    case 14851:
    case 14852:
    case 2096:
    case 2097: // coal
    c.mining[0] = 453;
    c.mining[1] = 30;
    c.mining[2] = 50;
    c.getMining().startMining(c.mining[0], c.mining[1], c.mining[2]);
    break;

    case 2098:
    case 2099:
    c.mining[0] = 444;
    c.mining[1] = 40;
    c.mining[2] = 65;
    c.getMining().startMining(c.mining[0], c.mining[1], c.mining[2]);
    break;

    case 2102:
    case 2103:
    case 14853:
    case 14854:
    case 14855: // mith ore
    c.mining[0] = 447;
    c.mining[1] = 55;
    c.mining[2] = 80;
    c.getMining().startMining(c.mining[0], c.mining[1], c.mining[2]);
    break;

    case 2105:
    case 14862: // addy ore
    c.mining[0] = 449;
    c.mining[1] = 70;
    c.mining[2] = 95;
    c.getMining().startMining(c.mining[0], c.mining[1], c.mining[2]);
    break;

    case 14859:
    case 14860: // rune ore
    c.mining[0] = 451;
    c.mining[1] = 85;
    c.mining[2] = 125;
    c.getMining().startMining(c.mining[0], c.mining[1], c.mining[2]);
    break;

    case 8143:
    if (c.farm[0] > 0 && c.farm[1] > 0) {
    c.getFarming().pickHerb();
    }
    break;

    // DOORS
    case 1516:
    case 1519:
    if (c.objectY == 9698) {
    if (c.absY >= c.objectY)
    c.getPA().walkTo(0, -1);
    else
    c.getPA().walkTo(0, 1);
    break;
    }
    case 1530:
    case 1531:
    case 1533:
    case 1534:
    case 11712:
    case 11711:
    case 11707:
    case 11708:
    case 6725:
    case 3198:
    case 3197:
    Server.objectHandler.doorHandling(objectType, c.objectX, c.objectY,
    0);
    break;

    case 9319:
    if (c.heightLevel == 0)
    c.getPA().movePlayer(c.absX, c.absY, 1);
    else if (c.heightLevel == 1)
    c.getPA().movePlayer(c.absX, c.absY, 2);
    break;

    case 9320:
    if (c.heightLevel == 1)
    c.getPA().movePlayer(c.absX, c.absY, 0);
    else if (c.heightLevel == 2)
    c.getPA().movePlayer(c.absX, c.absY, 1);
    break;

    case 4496:
    case 4494:
    if (c.heightLevel == 2) {
    c.getPA().movePlayer(c.absX - 5, c.absY, 1);
    } else if (c.heightLevel == 1) {
    c.getPA().movePlayer(c.absX + 5, c.absY, 0);
    }
    break;

    case 4493:
    if (c.heightLevel == 0) {
    c.getPA().movePlayer(c.absX - 5, c.absY, 1);
    } else if (c.heightLevel == 1) {
    c.getPA().movePlayer(c.absX + 5, c.absY, 2);
    }
    break;

    case 4495:
    if (c.heightLevel == 1) {
    c.getPA().movePlayer(c.absX + 5, c.absY, 2);
    }
    break;

    case 5126:
    if (c.absY == 3554)
    c.getPA().walkTo(0, 1);
    else
    c.getPA().walkTo(0, -1);
    break;

    case 1755:
    if (c.objectX == 2884 && c.objectY == 9797)
    c.getPA().movePlayer(c.absX, c.absY - 6400, 0);
    break;
    case 1759:
    if (c.objectX == 2884 && c.objectY == 3397)
    c.getPA().movePlayer(c.absX, c.absY + 6400, 0);
    break;
    case 24343:
    case 409:
    if (c.playerLevel[5] < c.getPA().getLevelForXP(c.playerXP[5])) {
    c.startAnimation(645);
    c.playerLevel[5] = c.getPA().getLevelForXP(c.playerXP[5]);
    c.sendMessage("You recharge your prayer points.");
    c.getPA().refreshSkill(5);
    } else {
    c.sendMessage("You already have full prayer points.");
    }
    break;
    case 2873:
    if (!c.getItems().ownsCape()) {
    c.startAnimation(645);
    c.sendMessage("Saradomin blesses you with a cape.");
    c.getItems().addItem(2412, 1);
    }
    break;
    case 2875:
    if (!c.getItems().ownsCape()) {
    c.startAnimation(645);
    c.sendMessage("Guthix blesses you with a cape.");
    c.getItems().addItem(2413, 1);
    }
    break;
    case 2874:
    if (!c.getItems().ownsCape()) {
    c.startAnimation(645);
    c.sendMessage("Zamorak blesses you with a cape.");
    c.getItems().addItem(2414, 1);
    }
    break;
    case 2879:
    c.getPA().movePlayer(2538, 4716, 0);
    break;
    case 2878:
    c.getPA().movePlayer(2509, 4689, 0);
    break;
    case 5960:
    c.getPA().startTeleport2(3090, 3956, 0);
    break;

    case 1815:
    c.getPA().startTeleport2(Config.EDGEVILLE_X, Config.EDGEVILLE_Y, 0);
    break;

    case 9706:
    c.getPA().startTeleport2(3105, 3951, 0);
    break;
    case 9707:
    c.getPA().startTeleport2(3105, 3956, 0);
    break;

    case 5959:
    c.getPA().startTeleport2(2539, 4712, 0);
    break;

    case 2558:
    c.sendMessage("This door is locked.");
    break;

    case 9294:
    if (c.absX < c.objectX) {
    c.getPA().movePlayer(c.objectX + 1, c.absY, 0);
    } else if (c.absX > c.objectX) {
    c.getPA().movePlayer(c.objectX - 1, c.absY, 0);
    }
    break;

    case 9293:
    if (c.absX < c.objectX) {
    c.getPA().movePlayer(2892, 9799, 0);
    } else {
    c.getPA().movePlayer(2886, 9799, 0);
    }
    break;
    case 10529:
    case 10527:
    if (c.absY <= c.objectY)
    c.getPA().walkTo(0, 1);
    else
    c.getPA().walkTo(0, -1);
    break;
    case 3044:
    c.getSmithing().sendSmelting();
    break;
    case 2409:
    if (c.lostcity == 1) {
    c.getDH().sendDialogues(41, 654);
    }
    break;

    default:
    ScriptManager.callFunc("objectClick1_" + objectType, c, objectType,
    obX, obY);
    break;

    }
    }

    public void secondClickObject(int objectType, int obX, int obY) {
    c.clickObjectType = 0;
    // c.sendMessage("Object type: " + objectType);
    switch (objectType) {
    case 11666:
    case 3044:
    case 2781:
    c.getSmithing().sendSmelting();
    break;
    case 2213:
    case 14367:
    case 11758:
    case 3193:
    case 11402:
    case 27663:
    case 26972:
    case 16700:
    case 4483:
    c.getPA().openUpBank();
    break;

    case 4874:
    c.getThieving().stealFromStall(1625, 10, 1); // uncut opal
    break;
    case 4875:
    c.getThieving().stealFromStall(371, 30, 45); // raw swordie fish
    break;
    case 4876:
    c.getThieving().stealFromStall(1700, 60, 60); // diamond amulet
    break;
    case 4877:
    c.getThieving().stealFromStall(2529, 100, 80); // orb of light
    break;
    case 4878:
    c.getThieving().stealFromStall(1613, 170, 95); // red topaz
    break;
    case 6163:
    c.getThieving().stealFromStall(1897, 10, 1);
    break;
    case 6165:
    c.getThieving().stealFromStall(950, 30, 25);
    break;
    case 6166:
    c.getThieving().stealFromStall(1635, 60, 50);
    break;
    case 6164:
    c.getThieving().stealFromStall(7650, 100, 75);
    break;
    case 6162:
    c.getThieving().stealFromStall(1613, 170, 90);
    break;
    case 2558:
    if (System.currentTimeMillis() - c.lastLockPick < 3000
    || c.freezeTimer > 0)
    break;
    if (c.getItems().playerHasItem(1523, 1)) {
    c.lastLockPick = System.currentTimeMillis();
    if (Misc.random(10) <= 3) {
    c.sendMessage("You fail to pick the lock.");
    break;
    }
    if (c.objectX == 3044 && c.objectY == 3956) {
    if (c.absX == 3045) {
    c.getPA().walkTo2(-1, 0);
    } else if (c.absX == 3044) {
    c.getPA().walkTo2(1, 0);
    }

    } else if (c.objectX == 3038 && c.objectY == 3956) {
    if (c.absX == 3037) {
    c.getPA().walkTo2(1, 0);
    } else if (c.absX == 3038) {
    c.getPA().walkTo2(-1, 0);
    }
    } else if (c.objectX == 3041 && c.objectY == 3959) {
    if (c.absY == 3960) {
    c.getPA().walkTo2(0, -1);
    } else if (c.absY == 3959) {
    c.getPA().walkTo2(0, 1);
    }
    }
    } else {
    c.sendMessage("I need a lockpick to pick this lock.");
    }
    break;
    default:
    ScriptManager.callFunc("objectClick2_" + objectType, c, objectType,
    obX, obY);
    break;
    }
    }

    public void thirdClickObject(int objectType, int obX, int obY) {
    c.clickObjectType = 0;
    c.sendMessage("Object type: " + objectType);
    switch (objectType) {
    default:
    ScriptManager.callFunc("objectClick3_" + objectType, c, objectType,
    obX, obY);
    break;
    }
    }

    public void firstClickNpc(int i) {
    c.clickNpcType = 0;
    c.npcClickIndex = 0;
    switch (i) {
    case 706:
    c.getDH().sendDialogues(9, i);
    break;

    case 2538:
    c.getShops().openShop(9);
    break;

    case 2258:
    c.getDH().sendDialogues(17, i);
    break;

    case 747:
    if (c.dragonslayer == 0) {
    c.getDH().sendDialogues(68, i);
    }
    break;

    case 650:
    if (c.lostcity == 0) {
    c.getDH().sendDialogues(20, 650);
    } else {
    c.getDH().sendDialogues(40, 650);
    }
    break;

    case 564:
    c.getShops().openShop(19);
    break;

    case 654:
    if (c.lostcity == 2) {
    c.getDH().sendDialogues(43, i);
    }
    break;

    case 1599:
    if (c.slayerTask <= 0) {
    c.getDH().sendDialogues(11, i);
    } else {
    c.getDH().sendDialogues(13, i);
    }
    break;

    case 1304:
    c.getDH().sendOption5("Home", "Edgeville", "Island",
    "Dagannoth Kings", "Next Page");
    c.teleAction = 1;
    break;

    case 528:
    c.getShops().openShop(1);
    break;

    case 461:
    c.getShops().openShop(2);
    break;

    case 683:
    c.getShops().openShop(3);
    break;

    case 586:
    c.getShops().openShop(4);
    break;

    case 555:
    c.getShops().openShop(6);
    break;

    case 519:
    c.getShops().openShop(8);
    break;

    case 1700:
    c.getShops().openShop(19);
    break;

    case 251:
    c.getShops().openShop(60);
    break;

    case 1282:
    c.getShops().openShop(7);
    break;

    case 1152:
    c.getDH().sendDialogues(16, i);
    break;

    case 494:
    c.getPA().openUpBank();
    break;

    case 2566:
    c.getShops().openSkillCape();
    break;

    case 3789:
    c.sendMessage((new StringBuilder()).append("You currently have ")
    .append(c.pcPoints).append(" points.")
    .toString());
    break;

    case 3788:
    c.getShops().openVoid();
    break;

    case 905:
    c.getDH().sendDialogues(5, i);
    break;

    case 460:
    c.getDH().sendDialogues(3, i);
    break;

    case 462:
    c.getDH().sendDialogues(7, i);
    break;

    case 316:
    c.getFishing().setupFishing(317);
    break;

    case 334:
    c.getFishing().setupFishing(389);
    break;

    case 324:
    c.getFishing().setupFishing(359);
    break;

    case 314:
    c.getFishing().setupFishing(335);
    break;

    case 326:
    c.getFishing().setupFishing(7944);
    break;

    case 522:
    case 523:
    c.getShops().openShop(1);
    break;

    case 549:
    c.getShops().openShop(4);
    break;

    case 599:
    c.getPA().showInterface(3559);
    c.canChangeAppearance = true;
    break;

    case 656:
    c.getDH().sendDialogues(58, i);
    break;


    case 904:
    c.sendMessage((new StringBuilder()).append("You have ")
    .append(c.magePoints).append(" points.").toString());
    break;
    default:
    ScriptManager.callFunc("npcClick1_"+i, c, i);
    if(c.playerRights == 3)
    Misc.println("First Click Npc : "+i);
    break;
    }
    }

    public void secondClickNpc(int i) {
    c.clickNpcType = 0;
    c.npcClickIndex = 0;
    switch (i) {
    case 1282:
    c.getShops().openShop(7);
    break;

    case 334:
    c.getFishing().setupFishing(383);
    break;

    case 3788:
    c.getShops().openVoid();
    break;

    case 564:
    c.getShops().openShop(19);
    break;

    case 494:
    c.getPA().openUpBank();
    break;

    case 324:
    c.getFishing().setupFishing(359);
    break;

    case 904:
    c.getShops().openShop(17);
    break;

    case 522:
    case 523:
    c.getShops().openShop(1);
    break;

    case 541:
    c.getShops().openShop(5);
    break;

    case 461:
    c.getShops().openShop(2);
    break;

    case 683:
    c.getShops().openShop(3);
    break;

    case 519:
    c.getShops().openShop(8);
    break;

    case 3789:
    c.getShops().openShop(21);
    break;

    case 1: // '\001'
    case 9: // '\t'
    case 18: // '\022'
    case 20: // '\024'
    case 21: // '\025'
    case 26: // '\032'
    c.getThieving().stealFromNPC(i);
    break;
    case 657:
    c.sendMessage("After a quick search, the monk smiles at you and allows you to board.");
    c.getPA().movePlayer(2834, 3335, 0);
    break;
    }
    }

    public void thirdClickNpc(int npcType) {
    c.clickNpcType = 0;
    c.npcClickIndex = 0;
    switch (npcType) {
    case 1599:
    c.getShops().openShop(20);
    c.sendMessage((new StringBuilder()).append("You currently have ")
    .append(c.slayerPoints).append(" Slayer Points.")
    .toString());
    break;
    default:
    ScriptManager.callFunc("npcClick3_" + npcType, c, npcType);
    if (c.playerRights == 3)
    Misc.println("Third Click NPC : " + npcType);
    break;

    }
    }

    }



    Media:



    Those have been the modified files, player.java just needed an integer added to it, so I doubt it has to do with that...
    Reply With Quote  
     

  2. #2  
    Donator
    Darkness's Avatar
    Join Date
    Nov 2012
    Age
    26
    Posts
    876
    Thanks given
    121
    Thanks received
    75
    Rep Power
    47
    Code:
    public int getWcTimer(int density) {
    int time = Misc.random(2);
    return density + time - axePower(goodAxe());
    }
    
    }
    At the far end, i just thought it semt pretty f'ed up, try just once to change misc random to 1000 and cut a few trees, cause yeah it's said somewhere that ; c.wcTimer = getWcTimer(density);
    And wcTimer is Density + time - axepower but idfk whats density, time is the Misc.random( set to 1 or two seconds randomly) minus the axe power which is defined here
    Code:
    public int axePower(int axe) {
    if (axe == 1351) // bronze
    return 1;
    if (axe == 1349) // iron
    return 1;
    if (axe == 1353) // steel
    return 2;
    if (axe == 1361) // black
    return 2;
    if (axe == 1355) // mith
    return 3;
    if (axe == 1357) // addy
    return 4;
    if (axe == 1359) // rune
    return 5;
    if (axe == 6739) //daxe
    return 7;
    else
    return 0;
    }
    Tbh im not sure of anything as this is a really strange woodcutting but i'd try that
    Attached image
    Reply With Quote  
     

  3. #3  
    Registered Member Ired's Avatar
    Join Date
    May 2012
    Posts
    160
    Thanks given
    13
    Thanks received
    10
    Rep Power
    12
    Hmm, I updated the topic with media of what happens. I think i might of deleted a loop that was defined somewhere because I couldn't get the "[PI]Better Woodcutting" working. But when I try woodcutting on a backup client it works fine. Here's what it looks like in-game:

    Reply With Quote  
     

  4. #4  
    Registered Member Ired's Avatar
    Join Date
    May 2012
    Posts
    160
    Thanks given
    13
    Thanks received
    10
    Rep Power
    12
    Anyone got any help for me? Still don't know what's wrong..
    Reply With Quote  
     

  5. #5  
    Registered Member Aintaro.'s Avatar
    Join Date
    Sep 2010
    Posts
    998
    Thanks given
    188
    Thanks received
    213
    Rep Power
    37
    where is your code to loop the process of your woodcutting?
    Reply With Quote  
     

  6. #6  
    Registered Member Ired's Avatar
    Join Date
    May 2012
    Posts
    160
    Thanks given
    13
    Thanks received
    10
    Rep Power
    12
    Quote Originally Posted by Aintaro. View Post
    where is your code to loop the process of your woodcutting?
    That's what I think is the problem, where would that be located? I might of deleted it :\ Or what would the code "look" like?
    Reply With Quote  
     

  7. #7  
    Registered Member Aintaro.'s Avatar
    Join Date
    Sep 2010
    Posts
    998
    Thanks given
    188
    Thanks received
    213
    Rep Power
    37
    well from what I can ana-lyse your cutWood() method should be in a process
    Reply With Quote  
     

  8. Thankful user:


  9. #8  
    Registered Member Ired's Avatar
    Join Date
    May 2012
    Posts
    160
    Thanks given
    13
    Thanks received
    10
    Rep Power
    12
    Quote Originally Posted by Aintaro. View Post
    well from what I can ana-lyse your cutWood() method should be in a process
    LIFE SAVER Reped and Thanked: I was missing this in my Client.java

    Code:
    		if (wcTimer > 0 && woodcut[0] > 0) {
    			wcTimer--;
    		} else if (wcTimer == 0 && woodcut[0] > 0) {
    			getWoodcutting().cutWood();
    Reply With Quote  
     

  10. #9  
    Registered Member Aintaro.'s Avatar
    Join Date
    Sep 2010
    Posts
    998
    Thanks given
    188
    Thanks received
    213
    Rep Power
    37
    Quote Originally Posted by Ired View Post
    LIFE SAVER Reped and Thanked: I was missing this in my Client.java

    Code:
    		if (wcTimer > 0 && woodcut[0] > 0) {
    			wcTimer--;
    		} else if (wcTimer == 0 && woodcut[0] > 0) {
    			getWoodcutting().cutWood();
    Alright, have fun with it
    Reply With Quote  
     


Thread Information
Users Browsing this Thread

There are currently 1 users browsing this thread. (0 members and 1 guests)


User Tag List

Similar Threads

  1. [PI] Woodcutting Help
    By Ired in forum Help
    Replies: 7
    Last Post: 06-06-2013, 08:27 PM
  2. [PI] Woodcutting help
    By mc wobbla in forum Help
    Replies: 2
    Last Post: 12-06-2012, 02:47 AM
  3. [PI] Woodcutting help
    By mc wobbla in forum Help
    Replies: 0
    Last Post: 12-06-2012, 01:34 AM
  4. PI woodcutting help
    By corndogs in forum Help
    Replies: 1
    Last Post: 07-01-2012, 01:24 AM
  5. PI client help again please.. :$
    By andriy81 in forum Help
    Replies: 1
    Last Post: 01-22-2012, 11:47 PM
Posting Permissions
  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts
  •