Помогите Как сделать правильное условие?

Exception_Prototype · 18 сен 2016

Есть метод.
По ниму мы видим, если умирает игрок, то мы берём киллера, проверяем живой ли он и если да, то выдаём ему вещи. Проблема в том, что если игрок падает и умирает, возникает исключение, как это можно обойти? Какую проверку добавить?

Java:

@EventHandler
    public void EntityDeathEvent(EntityDeathEvent event) {
        if (event.getEntityType() == PLAYER) {
            Entity entity = event.getEntity();
            Player victim = (Player) entity;

            if (!event.getEntity().getKiller().getPlayer().isDead()) {
               //выдаю какие то вещи.
            }

            getServer().broadcastMessage(ChatColor.RED + event.getEntity().getKiller().getDisplayName() + ChatColor.WHITE + " убил " + ChatColor.GREEN + victim.getDisplayName());

        }
    }

Mr_RoboMan · 18 сен 2016

Exception_Prototype сказал(а):
Java:

getKiller().getPlayer()

Вырежи себе этот купон, используй где-нибуть, потом прочти ошибку (ту самую которая NullPointerException (Ага, я Ванга 100%)) и подумай сам как ее исправить.
И кстати подсказка. Это ошибка будет кидаться тебе не только когда "игрок падает и умирает", а вообще ВСЕГДА, когда убийца не игрок.

fall.us · 18 сен 2016

Java:

if(event.getEntity().getLastDamageCause().getCause() == DamageCause.FALL)
    return;

Mr_RoboMan · 18 сен 2016

fall.us сказал(а):
Java:

if(event.getEntity().getLastDamageCause().getCause() == DamageCause.FALL) return;

От падения это поможет, а от сгорания в лаве? Утопления в воде? Застревания в блоках?

Exception_Prototype · 18 сен 2016

Mr_RoboMan сказал(а):
Вырежи себе этот купон, используй где-нибуть, потом прочти ошибку (ту самую которая NullPointerException (Ага, я Ванга 100%)) и подумай сам как ее исправить.
И кстати подсказка. Это ошибка будет кидаться тебе не только когда "игрок падает и умирает", а вообще ВСЕГДА, когда убийца не игрок.

Ошибка у меня только одна. Когда игрок умирает не от игрока.
В остальных случаях жертва и киллер выводится корректно без исключений.
Ошибка возникает здесь.

Java:

if (!event.getEntity().getKiller().isDead())

Понятное дело, что у игрока, который упал и разбился нет киллера. Но повторяю вопрос первого поста, как тогда обходить это?

slavik123123123 · 18 сен 2016

Exception_Prototype сказал(а):
event.getEntity().getKiller()

instanceof LivingEntity || != null

Exception_Prototype · 18 сен 2016

slavik123123123 сказал(а):
instanceof LivingEntity || != null

Спасибо. Вот работает.

Java:

@EventHandler
    public void EntityDeathEvent(EntityDeathEvent event) {
        if (event.getEntityType() == PLAYER) {
            Entity entity = event.getEntity();
            Player victim = (Player) entity;

            if (event.getEntity().getKiller() instanceof LivingEntity || event.getEntity().getKiller() != null) {
                getServer().broadcastMessage(ChatColor.RED + event.getEntity().getKiller().getDisplayName() + ChatColor.WHITE + " убил " + ChatColor.GREEN + victim.getDisplayName());
                event.getEntity().getKiller().getInventory().addItem(new ItemStack(Material.EMERALD, 1));
                event.getEntity().getKiller().getInventory().addItem(new ItemStack(Material.RAW_BEEF, 1));
                event.getEntity().getKiller().getInventory().addItem(new ItemStack(Material.BONE, 1));

            } else {
                getServer().broadcastMessage(victim.getDisplayName() + " умер.");
            }
        }
    }

Reality_SC · 18 сен 2016

ПОЧЕМУ Вы так любите получать лист из всей глубоко вложенной структуры сразу?!?

Java:

@EventHandler
    public void EntityDeathEvent(EntityDeathEvent event) {
        if (event.getEntityType() == PLAYER) {
            Entity entity = event.getEntity();
            Player victim = (Player) entity;
            Entity killer = victim.getKiller();

            if (killer != null && killer instanceof LivingEntity) {
                   getServer().broadcastMessage(victim.getDisplayName() + " умер.");
            } else {
                getServer().broadcastMessage(ChatColor.RED
                    + killer.getDisplayName()
                    + ChatColor.WHITE + " убил "
                    + ChatColor.GREEN + victim.getDisplayName());
                Inventory inv = killer.getInventory();
                inv.addItem(new ItemStack(Material.EMERALD, 1));
                inv.addItem(new ItemStack(Material.RAW_BEEF, 1));
                inv.addItem(new ItemStack(Material.BONE, 1));
            }
        }
    }

Ведь это быстрее работает, понятнее и красивее.

Exception_Prototype · 23 сен 2016

Reality_SC сказал(а):

ПОЧЕМУ Вы так любите получать лист из всей глубоко вложенной структуры сразу?!?
Посмотреть вложение 43065

Java:

@EventHandler
    public void EntityDeathEvent(EntityDeathEvent event) {
        if (event.getEntityType() == PLAYER) {
            Entity entity = event.getEntity();
            Player victim = (Player) entity;
            Entity killer = victim.getKiller();

            if (killer != null && killer instanceof LivingEntity) {
                   getServer().broadcastMessage(victim.getDisplayName() + " умер.");
            } else {
                getServer().broadcastMessage(ChatColor.RED
                    + killer.getDisplayName()
                    + ChatColor.WHITE + " убил "
                    + ChatColor.GREEN + victim.getDisplayName());
                Inventory inv = killer.getInventory();
                inv.addItem(new ItemStack(Material.EMERALD, 1));
                inv.addItem(new ItemStack(Material.RAW_BEEF, 1));
                inv.addItem(new ItemStack(Material.BONE, 1));
            }
        }
    }

Ведь это быстрее работает, понятнее и красивее.

Спасибо за помощь, дома буду гляну внимательнее.

Reality_SC сказал(а):

ПОЧЕМУ Вы так любите получать лист из всей глубоко вложенной структуры сразу?!?
Посмотреть вложение 43065

Java:

@EventHandler
    public void EntityDeathEvent(EntityDeathEvent event) {
        if (event.getEntityType() == PLAYER) {
            Entity entity = event.getEntity();
            Player victim = (Player) entity;
            Entity killer = victim.getKiller();

            if (killer != null && killer instanceof LivingEntity) {
                   getServer().broadcastMessage(victim.getDisplayName() + " умер.");
            } else {
                getServer().broadcastMessage(ChatColor.RED
                    + killer.getDisplayName()
                    + ChatColor.WHITE + " убил "
                    + ChatColor.GREEN + victim.getDisplayName());
                Inventory inv = killer.getInventory();
                inv.addItem(new ItemStack(Material.EMERALD, 1));
                inv.addItem(new ItemStack(Material.RAW_BEEF, 1));
                inv.addItem(new ItemStack(Material.BONE, 1));
            }
        }
    }

Ведь это быстрее работает, понятнее и красивее.

Смотрю и не могу понять, вы ошиблись с if else ?
Ведь если киллер жив, то мы ему даём вещи, а если его и в помине нет в живых, а тем более вообще нет (самойубийство), то мы просто выведем сообщение о том, что игрок умер.

Reality_SC · 23 сен 2016

Ну поменяй их местами, да.
Я вроде их твоего кода переделывал, но почему-то инвертировал, да.
Ввело в заблуждение твоё instanceof LivingEntity и потом || != null.

Exception_Prototype · 23 сен 2016

Reality_SC сказал(а):
Ну поменяй их местами, да.
Я вроде их твоего кода переделывал, но почему-то инвертировал, да.
Ввело в заблуждение твоё instanceof LivingEntity и потом || != null.

Хорошо, спасибо)

Reality_SC сказал(а):
Ну поменяй их местами, да.
Я вроде их твоего кода переделывал, но почему-то инвертировал, да.
Ввело в заблуждение твоё instanceof LivingEntity и потом || != null.

Ругается http://joxi.ru/12Mj9XRI4ekbLA

slavik123123123 · 23 сен 2016

скорее всего нужно поменять broadcast и код в esle метсами, и закостовать киллера в ливинг энтити

Exception_Prototype · 24 сен 2016

Меня смущала вот эта строчка:

Java:

Entitykiller = victim.getKiller();

Поменял на это:

Java:

Player killer = victim.getKiller();

Ошибки пропали. Я правильно сделал?

Dereku · 24 сен 2016

Exception_Prototype сказал(а):
Ошибки пропали. Я правильно сделал?

Неа.
И прекращай скриншотить код - используй pastebin. Из-за вредности даже не буду писать код, который лишён некоторых недостатков.

Exception_Prototype · 24 сен 2016

Dereku сказал(а):
Неа.
И прекращай скриншотить код - используй pastebin. Из-за вредности даже не буду писать код, который лишён некоторых недостатков.

Как вы заметили Выше, вот так: http://pastebin.com/JSqY7GTj пользователь slavik123123123, любезно помог довести до ума поднятый вопрос в теме, но, при попытки применить его в действии, IDE ругается. Я попытался исправить это, но согласно Вашим словам, я сделал это не правильно.

Dereku · 24 сен 2016

Java:

@EventHandler
public void EntityDeathEvent(EntityDeathEvent event) {
    if (event.getEntityType() == PLAYER) {
        Entity entity = event.getEntity();
        Player victim = (Player) entity;
        Entity killer = victim.getKiller();

        if (killer != null && killer instanceof Player) {
            getServer().broadcastMessage(ChatColor.RED
                + ((Player)killer).getDisplayName()
                + ChatColor.WHITE + " убил "
                + ChatColor.GREEN + victim.getDisplayName());
            Inventory inv = ((Player)killer).getInventory();
            inv.addItem(new ItemStack(Material.EMERALD, 1));
            inv.addItem(new ItemStack(Material.RAW_BEEF, 1));
            inv.addItem(new ItemStack(Material.BONE, 1));
        } else {
               getServer().broadcastMessage(victim.getDisplayName() + " умер.");
        }
    }
}

Решение конечно не "ах", но работает.

_MediaNik_ · 24 сен 2016

Dereku сказал(а):

Java:

@EventHandler
public void EntityDeathEvent(EntityDeathEvent event) {
    if (event.getEntityType() == PLAYER) {
        Entity entity = event.getEntity();
        Player victim = (Player) entity;
        Entity killer = victim.getKiller();

        if (killer != null && killer instanceof Player) {
            getServer().broadcastMessage(ChatColor.RED
                + ((Player)killer).getDisplayName()
                + ChatColor.WHITE + " убил "
                + ChatColor.GREEN + victim.getDisplayName());
            Inventory inv = ((Player)killer).getInventory();
            inv.addItem(new ItemStack(Material.EMERALD, 1));
            inv.addItem(new ItemStack(Material.RAW_BEEF, 1));
            inv.addItem(new ItemStack(Material.BONE, 1));
        } else {
               getServer().broadcastMessage(victim.getDisplayName() + " умер.");
        }
    }
}

Не сработает. Для игрока есть PlayerDeathEvent. EntityDeathEvent не включает в себя игрока

Reality_SC · 24 сен 2016

Я бы написал внутри if-а: Player killerPlayer = (Player)killer;
и потом уже юзал эту переменную.

slavik123123123 · 24 сен 2016

_MediaNik_ сказал(а):
Не сработает. Для игрока есть PlayerDeathEvent. EntityDeathEvent не включает в себя игрока

как бы PlayerDeathEvent наследует EntityDeathEvent
public class PlayerDeathEvent extends EntityDeathEvent

Именно поэтому в PlayerDeathEvent не getPlayer(), а getEntity()

_MediaNik_ · 24 сен 2016

slavik123123123 сказал(а):
как бы PlayerDeathEvent наследует EntityDeathEvent
public class PlayerDeathEvent extends EntityDeathEvent

Именно поэтому в PlayerDeathEvent не getPlayer(), а getEntity()

Ну все же EntityDeathEvent не ловит игрока, для чего то же создали PlayerDeathEvent.

Помогите Как сделать правильное условие?

Активный участник

Старожил

Активный участник

Старожил

Активный участник

Старожил

Активный участник

Старожил

Активный участник

Старожил

Активный участник

Старожил

Активный участник

Старожил

Активный участник

Старожил

Активный участник

Старожил

Старожил

Активный участник