Problem z pętlą white - wykonywanie się tylko raz

[Zientez]

Witajcie,

Dlaczego petla WHILE sie tylko raz wykonuje przeciez warunek jest? W czym blad?

    @EventHandler
    public void onBlockPlace(BlockPlaceEvent e) {
        int i = -1;
        Location loc = e.getBlockPlaced().getLocation().add(0, i, 0);
        if(e.getPlayer().getItemInHand().hasItemMeta()) {
            if(e.getPlayer().getItemInHand().isSimilar(trololololalalal())) {
                if(wg.canBuild(e.getPlayer(), e.getBlock())) {
                    while(loc.getBlock().getType().equals(Material.AIR)) {
                        loc.getBlock().setType(Material.OBSIDIAN);
                        i--;
                        e.getPlayer().sendMessage("Petla... ["+i+"]");
                    }
                    e.getPlayer().sendMessage("Koniec Petli...");
                }
            }
        }
    }

[Wampir7858]

Może warunek jest spełniony tylko raz dlatego nigdy więcej sie nie wykona??

 

Ps: Zły dział

[Veasst]

Twoja pętla wykonuje się tylko wtedy gdy blok jest powietrzem. Ty w prętli zamieniasz sprawdzany blok na obsydian.

[Zientez]

2UP twoja odpowiedz nic nie wniosla do tematu kolego... A czy zly dzial czy dobry nie ty decydujesz wiec shout up...

Warunek sie tylko raz wykonuje to akurat wiem tylko nie wiem jak to naprawic...

@Veasst wiesz jak to naprawic...?

[Veasst]

Nie wiem co chcesz dokładnie zrobić. Zamieniasz blok na obsydian jeśli ten jest powietrzem. Nie widzę potrzeby używania pętli tutaj.

[Zientez]

@Veasst to ma dzialac jak boy farmer...

Dobra temat do zamkniecia zsumuje to a pozniej roznice walne

[rafszy122]

 while(loc.getBlock().getType().equals(Material.AIR))

zauwaz ze w tej petli bierzesz pod uwage caly czas jeden i ten sam blok 

[AdamGrzegorz]

Skoro boyfarmer, zrób to trochę inaczej:

­

@EventHandler
    public void onBlockPlace(BlockPlaceEvent e) {
        int i = -1;
        Location loc = e.getBlockPlaced().getLocation().add(0, i, 0);
        if(e.getPlayer().getItemInHand().hasItemMeta()) {
            if(e.getPlayer().getItemInHand().isSimilar(trololololalalal())) {
                if(wg.canBuild(e.getPlayer(), e.getBlock())) {
                    while(loc.getBlock().getType().equals(Material.BEDROCK)) {
                        loc.getBlock().setType(Material.OBSIDIAN);
                        i--;
                        loc = e.getBlockPlaced().getLocation().add(0, i, 0);
                        e.getPlayer().sendMessage("Petla... ["+i+"]");
                    }
                    e.getPlayer().sendMessage("Koniec Petli...");
                }
            }
        }
    }

Zrobiłbym to na twoim miejscu jeszcze inaczej, ale jestem leniem i nie chce mi się pisać dla ciebie gotowego kodu. Powinno działac, c*****o ale powinno.

[Zientez]

Ja to zrobilem juz inaczej lepszy efekt daje nie to co te same boiyfarmery z neta, ale dzieki za chec pomocy