Помогите Помогите разобраться в коде:)

Всем привет!
Есть обработчик события:

Code:

    @EventHandler
    public void onUse(EntityDamageByEntityEvent event)
    {
        if (event.getEntityType() == EntityType.PLAYER)
        {
            Player p = (Player) event.getEntity();
            if(p.isBlocking())
            {
                ItemStack i = p.getItemInHand();
                if(i.getType() == Material.WOOD_SWORD || i.getType() == Material.DIAMOND_SWORD || i.getType() == Material.STONE_SWORD || i.getType() == Material.IRON_SWORD)
                {                                              
                    List<String> str = new ArrayList<String>();
                    str.add("Shield"); 
                    event.setDamage(0.0);  //это срабатывает                   
                    if (i.getItemMeta().hasLore() && i.getItemMeta().getLore().equals(str))
                    {
                        event.setDamage(0.0); // а вот это почему-то не срабатывает
                        getLogger().info("Shield"); // и это срабатывает                                                                                       
                        i.setDurability((short) (i.getDurability() + 5));                      
                        p.setItemInHand(i);                      
                        if ((int) p.getItemInHand().getDurability() <= (int) 0)
                        {
                            i.setType(Material.AIR);                      
                        }
                        p.setItemInHand(i);          
                    }  
                }
                                              
            }
        }
    }

Проблема в том, что if как бы true и часть if'a выполняется а часть нет.

Заранее благодарю.

P.S. Коменты в коде

 

Потому что event.setDamage(0.0) - говнокод, а event.setCancelled(true) - хороший код.
Выглядят по разному, а работают одинаково.

 

Я согласен что они выполняют одинаковую функцию, но есть одно НО:
Первый "говнокод"(как он написал) работает, а он спрашивает про вторую.

 

но по-моему это (даже хоть и чуть-чуть) сильнее нагружает. Все равно, если не работает, стоит попробовать setCancelled

 

А можно тут задать вопрос, чтобы не создавать лишние темы. Как "произвести" команду от лица игрока?

 

player.performCommand(),( п с смотри документацию)

 

Оу щит, забыл совсем. Спасибо.

 

Шта? Зачем event.setDamage(0.0) дважды выполнять?!

 

На сколько я знаю, в первом случае отправляется пакет с отображением нанесения урона, а во втором -- нет. Так же могут быть плагины которые ignoreCancelled = true и в них event.setCancelled(false), а в первом случае урон однозначно не пройдёт. Так же возможно сработает и отбрасывание/накладывание эффектов при уроне.[DOUBLEPOST=1391181127,1391180916][/DOUBLEPOST]

По поводу ошибки не скажу, но настораживает это место:
i.setDurability((short) (i.getDurability() + 5));
if ((int) p.getItemInHand().getDurability() <= (int) 0)
p.getItemInHand().getDurability() == 0 если вещь имеет полную прочность, если Вы хотите уничтожить "разбитую" вещь, то в условии нужно использовать p.getItemInHand().getDurability()>p.getItemInHand().getType().getMaxDurability()

 

Спасибо[DOUBLEPOST=1391185300][/DOUBLEPOST]

Первый(перед ифом) я не выполняю, просто написал, и говорю что первый срабатывает, а второй, который в ифе нет, хотя сообщение из ифа выводится.[DOUBLEPOST=1391192582][/DOUBLEPOST]Сделал так, но event.setDamage(0.0) все равно не работает хотя все остальное в операторных скобках работает

Code:

@EventHandler
    public void onUse(EntityDamageByEntityEvent event)
    {
        if (event.getEntityType() == EntityType.PLAYER)
        {
            Player p = (Player) event.getEntity();
            if(p.isBlocking())
            {
                List<String> str = new ArrayList<String>();
                str.add("Shield");
                ItemStack i = p.getItemInHand();
                if((i.getType() == Material.WOOD_SWORD || i.getType() == Material.DIAMOND_SWORD || i.getType() == Material.STONE_SWORD || i.getType() == Material.IRON_SWORD) && (i.getItemMeta().hasLore() && i.getItemMeta().getLore().equals(str)))
                {      
                    event.setDamage(0.0);                                                                                                 
                    i.setDurability((short) (i.getDurability() + 5));                                                                  
                    if (i.getDurability() > i.getType().getMaxDurability())
                    {
                        i.setType(Material.AIR);                      
                    }
                    p.setItemInHand(i);              
                }
                                              
            }
        }
    }

 

На каком ядре тестируешь?

 

Попробуй сделать
event.setCancelled(true);
player.damage(1);
вместо event.setDamage(0.0);
P.S. Хотя это может быть неправильно.

 

А зачем наносить 1 урон?

 

event.setCancelled(true) тоже не работает.[DOUBLEPOST=1391244669,1391244639][/DOUBLEPOST]

bukkit 1.7.2 R3.0

 

Мб Bukkit 1.7.2 R0.3 ?

 

Мне интересно: где вообще было взято такое ядро? Может это 1.7.2 R0.3? Попробуй сменить ядро на 1.6 и проверить на нём.

 

да 1.7.2 R0.3, опечаточка вышла, попробую на 1.6[DOUBLEPOST=1391267953,1391248038][/DOUBLEPOST]На 1.6 работает, это типа баг что ли?[DOUBLEPOST=1391269629][/DOUBLEPOST]Если сделать так, т.е итем в руке не менять, то работает:

Code:

    @EventHandler
    public void onUse(EntityDamageByEntityEvent event)
    {
        if (event.getEntityType() == EntityType.PLAYER)
        {
            Player p = (Player) event.getEntity();
            if(p.isBlocking())
            {
                ItemStack i = p.getItemInHand();
                if(i.getType() == Material.WOOD_SWORD || i.getType() == Material.DIAMOND_SWORD || i.getType() == Material.STONE_SWORD || i.getType() == Material.IRON_SWORD)
                {                                             
                    List<String> str = new ArrayList<String>();
                    str.add("Shield");                                   
                    if (i.getItemMeta().hasLore() && i.getItemMeta().getLore().equals(str))
                    {
                        event.setDamage(0);  //это срабатывает
                        getLogger().info("Shield"); // и это срабатывает                                                                                     
                        //i.setDurability((short) (i.getDurability() + 5));                               
                    }                                                           
                }                                         
            }
        }
      
    }

, но как это обойти, помогите пожалуйста.

 

Это назвается 1.7 билды ведра ещё не стабильны. У меня есть билд в котором при создании новых чанков(не начальная генерация мира) сервер падает.

 

Ну может как-то все таки обойти это можно?

 

Ждите нового билда.