(Java, Bukkit) Использование методов расширенного класса не работает (Noob oop)

У меня есть один класс с именем 'Регионы', который создает регион и проверяет, есть ли в нем координаты. Тогда у меня есть другой класс 'ArrowTower', который расширяет 'Region'. Класс ArrowTower создает башни. Весь день я экспериментировал безрезультатно. Я хочу назначить первое и второе местоположение классу "Регионы", затем создать регион и проверить, принадлежат ли ему координаты. У меня также есть события - класс 'BlockTouch', который создает объект 'ArrowTower'.. Когда я пытаюсь tower.insideRegion(e.getClickedBlock()); он дает мне мое местоположение и затем нули, потому что он не установил значения в public void buildArrowTower, но значения здесь. Так что я просто не понимаю, почему это не работает:/

Класс Мои События:

import eu.anavicius.TomTom1997.TomTowerDefence.Towers.ArrowTower;

public class BlockTouch implements Listener {   

    @EventHandler
    public void onPlayerInteract(PlayerInteractEvent e) {
        ArrowTower tower = new ArrowTower();
        if (e.getAction() == Action.RIGHT_CLICK_BLOCK ) {
            if (e.getItem().getType() == Material.ARROW) {
                tower.buildArrowTower(e.getClickedBlock(),e.getPlayer());
            } else if (e.getItem().getType() == Material.BONE) {
                Bukkit.broadcastMessage("Bone");
                tower.insideRegion(e.getClickedBlock());
            }
        }
    }
}

Мой Region.class:

public class Regions {
    private int xl,yl,zl,xh,yh,zh;

    public void setLRegion (int x, int y, int z) {
        xl = x;
        yl = y;
        zl = z;
        //Bukkit.broadcastMessage("SetLMethod" + " \t " +    "|" + xl + "|"+ xh + "||" + "|" + yl + "|" + yh + "||"  + "|" +zl + "|" + zh + "||");
    }

    public void setHRegion (int x, int y, int z) {
        xh = x;
        yh = y;
        zh = z;
        //Bukkit.broadcastMessage("SetHMethod" + " \t " +    "|" + xl + "|"+ xh + "||" + "|" + yl + "|" + yh + "||"  + "|" +zl + "|" + zh + "||");
    }


    public void insideRegion (Block l) {
        int x,y,z;
        x = l.getX();
        y = l.getY();
        z =l.getZ();
        Bukkit.broadcastMessage("InsideMethod" + " \\t " + x  + "|" + xl + "|"+ xh + "||" +y + "|" + yl + "|" + yh + "||" + z + "|" +zl + "|" + zh + "||");

        if (x >= xl && x <= xh ) {
            Bukkit.broadcastMessage("Region check 1");
            if (z >= zl && z <= zh) {
                Bukkit.broadcastMessage("Region check 2");
                if (y >= yl && y >= yh) {
                    Bukkit.broadcastMessage("Regione");
                }
            }
        }

    }
}

Мой ArrowTower.class:

public class ArrowTower extends Regions {

    public ArrowTower () {

    }

    public void buildArrowTower (Block b,Player p) {
        if (b.getType().equals(Material.EMERALD_BLOCK)) {
            Location loc = b.getLocation();
            for (int y = 0; y < 4;y++) {
                int layloc = 0;
                int by = loc.getBlock().getY()+1 + y;
                for (int x = 0;x <3;x++) {
                    int bx = loc.getBlock().getX()-1 + x;
                    for (int z = 0;z < 3;z++) { // Pagalvot dar del delay, nes su kintamaisiais pirma blogai buvau
                        int bz = loc.getBlock().getZ()-1 + z; // sugalvojas, atsispausdina tuscios vietos
                        Location block = new Location(b.getWorld(),bx,by,bz); // pass loop values to method
                        if (y == 0 && layloc == 0) {
                            Bukkit.broadcastMessage("SetR L");
                            setLRegion(bx,by,bz);
                        } else if (y == 3 && layloc == 8) {
                            Bukkit.broadcastMessage("SetR H");
                            setHRegion(bx,by,bz);
                        }
                        block.getBlock().setType(Material.matchMaterial(towerl1(y,layloc)));
                        layloc++;
                    }
                }
            }
        }
    }

    public String towerl1(int h, int layloc) {
        String[] layer1 = { // - l
                "LOG","AIR","LOG",
                "AIR","AIR","AIR",
                "LOG","AIR","LOG"}; 
        String[] layer2 = { // - i
                "COBBLE_WALL","COBBLESTONE","COBBLE_WALL",
                "COBBLESTONE","AIR","COBBLESTONE",
                "COBBLE_WALL","COBBLESTONE","COBBLE_WALL"};
        String[] layer3 = { // - t
                "COBBLE_WALL","AIR","COBBLE_WALL",
                "COBBLE_WALL","MOSSY_COBBLESTONE","COBBLE_WALL",
                "COBBLE_WALL","AIR","COBBLE_WALL"};
        String[] layer4 = {
                "AIR","AIR","AIR",
                "AIR","JACK_O_LANTERN","AIR",
                "AIR","AIR","AIR"};
        if (h == 0) {
            return layer1[layloc];
        } else if (h == 1) {
            return layer2[layloc];
        } else if (h == 2) {
            return layer3[layloc];
        } else if (h == 3) {
            return layer4[layloc];
        } else {
            return null;
        }
    }
}

1 ответ

Решение

Проблема в том, что вы создаете новый ArrowTower объект на каждом взаимодействующем событии. Я предполагаю, что вы первый клик правой кнопкой мыши на блоке, держа стрелку в руке - вот когда tower.buildArrowTower(e.getClickedBlock(),e.getPlayer()); называется.

Затем вы берете кость в руку и нажимаете снова - но на этот раз ваш обработчик событий создаст совершенно новый ArrowTower с его первой строкой (на самом деле у вас даже нет ссылки на первую, поскольку она была объявлена ​​только в области действия функции):

ArrowTower tower = new ArrowTower();

Тогда вы звоните:

tower.insideRegion(e.getClickedBlock());

но это tower был только что создан - на самом деле это xh, yhи т. д. значения не были инициализированы.

Другие вопросы по тегам