Помогите EntityDamageByEntityEvent проверка на Не игрока

AguRChek_TV · 15 сен 2020

пробовал сделать так но не работает

Код:

                if (!(e.getDamager() instanceof Player)) {
                    for(Player ps: Bukkit.getOnlinePlayers()) {
                        ps.sendMessage(ChatColor.GRAY + "" + ChatColor.BOLD + "player " + ChatColor.WHITE + "" + ChatColor.BOLD + p.getName() + ChatColor.GRAY + "" + ChatColor.BOLD + " killed by " + ChatColor.WHITE + "" + ChatColor.BOLD + e.getDamager().getType());
                        Location  loc = p.getLocation();
                        p.getWorld().strikeLightningEffect(loc);
                        p.getWorld().playSound(loc,Sound.ENTITY_LIGHTNING_BOLT_IMPACT, 1, 0);
                    }
                }

вот полный код

Код:

        @EventHandler
        public void dmg(EntityDamageByEntityEvent e) {
            Player p = (Player) e.getEntity();
            Player k = (Player) e.getDamager();
            if ((((Damageable) e.getEntity()).getHealth() - e.getFinalDamage() <= 0)) {
                p.setGameMode(GameMode.SPECTATOR);
                p.sendMessage(ChatColor.GOLD + "" + ChatColor.BOLD + "you died and moved to observer mode");
                e.setCancelled(true);
                if (e.getDamager() instanceof Player) {
                    for(Player ps: Bukkit.getOnlinePlayers()) {
                        ps.sendMessage(ChatColor.GRAY + "" + ChatColor.BOLD + "player " + ChatColor.WHITE + "" + ChatColor.BOLD + p.getName() + ChatColor.GRAY + "" + ChatColor.BOLD + " killed by " + ChatColor.WHITE + "" + ChatColor.BOLD + k.getName());
                        Location  loc = p.getLocation();
                        p.getWorld().strikeLightningEffect(loc);
                        p.getWorld().playSound(loc,Sound.ENTITY_LIGHTNING_BOLT_IMPACT, 1, 0);
                    }
                }
                if (!(e.getDamager() instanceof Player)) {
                    for(Player ps: Bukkit.getOnlinePlayers()) {
                        ps.sendMessage(ChatColor.GRAY + "" + ChatColor.BOLD + "player " + ChatColor.WHITE + "" + ChatColor.BOLD + p.getName() + ChatColor.GRAY + "" + ChatColor.BOLD + " killed by " + ChatColor.WHITE + "" + ChatColor.BOLD + e.getDamager().getType());
                        Location  loc = p.getLocation();
                        p.getWorld().strikeLightningEffect(loc);
                        p.getWorld().playSound(loc,Sound.ENTITY_LIGHTNING_BOLT_IMPACT, 1, 0);
                    }
                }
                int size = 0;
                ArrayList<Player> PE = new ArrayList<Player>();
                ArrayList<Player> PL = new ArrayList<Player>();
                PL.addAll(Bukkit.getOnlinePlayers());
                for(Player pe : PL) {
                    if(pe.getGameMode().equals(GameMode.SURVIVAL)) {
                        PE.add(pe);
                        size = PE.size();
                    }
                }
                if(size < 2) {
                    if(status == 1) {
                        for(Player ps: Bukkit.getOnlinePlayers()) {
                            // top1
                        }
                    }
                }
            }
        }

imDaniX · 15 сен 2020

_MRDen3000YT_ · 15 сен 2020

Так а что нужно то тебе?

AguRChek_TV сказал(а):
ChatColor.WHITE + "" +

AguRChek_TV · 15 сен 2020

_MRDen3000YT_ сказал(а):
Так а что нужно то тебе?

по преколу

alexandrage · 15 сен 2020

Мой мозг отключился как на код глянул

AguRChek_TV · 15 сен 2020

alexandrage сказал(а):
Мой мозг отключился как на код глянул

прости мы фортнайтеры не особо умные к сожалению

imDaniX · 15 сен 2020

AguRChek_TV сказал(а):

пробовал сделать так но не работает

Код:

                if (!(e.getDamager() instanceof Player)) {
                    for(Player ps: Bukkit.getOnlinePlayers()) {
                        ps.sendMessage(ChatColor.GRAY + "" + ChatColor.BOLD + "player " + ChatColor.WHITE + "" + ChatColor.BOLD + p.getName() + ChatColor.GRAY + "" + ChatColor.BOLD + " killed by " + ChatColor.WHITE + "" + ChatColor.BOLD + e.getDamager().getType());
                        Location  loc = p.getLocation();
                        p.getWorld().strikeLightningEffect(loc);
                        p.getWorld().playSound(loc,Sound.ENTITY_LIGHTNING_BOLT_IMPACT, 1, 0);
                    }
                }

вот полный код

Код:

        @EventHandler
        public void dmg(EntityDamageByEntityEvent e) {
            Player p = (Player) e.getEntity();
            Player k = (Player) e.getDamager();
            if ((((Damageable) e.getEntity()).getHealth() - e.getFinalDamage() <= 0)) {
                p.setGameMode(GameMode.SPECTATOR);
                p.sendMessage(ChatColor.GOLD + "" + ChatColor.BOLD + "you died and moved to observer mode");
                e.setCancelled(true);
                if (e.getDamager() instanceof Player) {
                    for(Player ps: Bukkit.getOnlinePlayers()) {
                        ps.sendMessage(ChatColor.GRAY + "" + ChatColor.BOLD + "player " + ChatColor.WHITE + "" + ChatColor.BOLD + p.getName() + ChatColor.GRAY + "" + ChatColor.BOLD + " killed by " + ChatColor.WHITE + "" + ChatColor.BOLD + k.getName());
                        Location  loc = p.getLocation();
                        p.getWorld().strikeLightningEffect(loc);
                        p.getWorld().playSound(loc,Sound.ENTITY_LIGHTNING_BOLT_IMPACT, 1, 0);
                    }
                }
                if (!(e.getDamager() instanceof Player)) {
                    for(Player ps: Bukkit.getOnlinePlayers()) {
                        ps.sendMessage(ChatColor.GRAY + "" + ChatColor.BOLD + "player " + ChatColor.WHITE + "" + ChatColor.BOLD + p.getName() + ChatColor.GRAY + "" + ChatColor.BOLD + " killed by " + ChatColor.WHITE + "" + ChatColor.BOLD + e.getDamager().getType());
                        Location  loc = p.getLocation();
                        p.getWorld().strikeLightningEffect(loc);
                        p.getWorld().playSound(loc,Sound.ENTITY_LIGHTNING_BOLT_IMPACT, 1, 0);
                    }
                }
                int size = 0;
                ArrayList<Player> PE = new ArrayList<Player>();
                ArrayList<Player> PL = new ArrayList<Player>();
                PL.addAll(Bukkit.getOnlinePlayers());
                for(Player pe : PL) {
                    if(pe.getGameMode().equals(GameMode.SURVIVAL)) {
                        PE.add(pe);
                        size = PE.size();
                    }
                }
                if(size < 2) {
                    if(status == 1) {
                        for(Player ps: Bukkit.getOnlinePlayers()) {
                            // top1
                        }
                    }
                }
            }
        }

Начиная с изначального вопроса - твой вариант должен сработать, если ты всё сделал правильно. Иначе же ты можешь проверить это через (e.getDamager().getType() == EntityType.PLAYER).

Ты в начале своего кода сразу рассчитываешь, что и ударивший, и жертва - игроки.

Код:

            Player p = (Player) e.getEntity();
            Player k = (Player) e.getDamager();

Это в итоге приведет к ошибке, если кто-то из них не игрок. Прежде всего убедись, что жертва - игрок.

Код:

if (e.getEntity().getType() != EntityType.PLAYER) return;
Player p = (Player) e.getEntity();

Вот здесь можно упростить.

Код:

if ((((Damageable) e.getEntity()).getHealth() - e.getFinalDamage() <= 0))

до

Код:

if (p.getHealth() <= e.getFinalDamage())

Дальше - у тебя два практически идентичных блока кода. К тому же ты кидаешь молнию и звук от каждого игрока, т.е. если на сервере 100 игроков - 100 молний ударят в одно и то же место. Также отправку сообщения всем игрокам можно сделать без цикла через Bukkit.broadcastMessage(String)

Код:

                if (e.getDamager() instanceof Player) {
                    for(Player ps: Bukkit.getOnlinePlayers()) {
                       ps.sendMessage(ChatColor.GRAY + "" + ChatColor.BOLD + "player " + ChatColor.WHITE + "" + ChatColor.BOLD + p.getName() + ChatColor.GRAY + "" + ChatColor.BOLD + " killed by " + ChatColor.WHITE + "" + ChatColor.BOLD + k.getName());
                        Location  loc = p.getLocation();
                        p.getWorld().strikeLightningEffect(loc);
                        p.getWorld().playSound(loc,Sound.ENTITY_LIGHTNING_BOLT_IMPACT, 1, 0);
                    }
                }
                if (!(e.getDamager() instanceof Player)) {
                    for(Player ps: Bukkit.getOnlinePlayers()) {
                       ps.sendMessage(ChatColor.GRAY + "" + ChatColor.BOLD + "player " + ChatColor.WHITE + "" + ChatColor.BOLD + p.getName() + ChatColor.GRAY + "" + ChatColor.BOLD + " killed by " + ChatColor.WHITE + "" + ChatColor.BOLD + e.getDamager().getType());
                        Location  loc = p.getLocation();
                        p.getWorld().strikeLightningEffect(loc);
                        p.getWorld().playSound(loc,Sound.ENTITY_LIGHTNING_BOLT_IMPACT, 1, 0);
                    }
                }

Сделай из двух блоков один и исправь косяки. Ради приличия ещё заменил все ChatColor одним методом, но это необязательно - медленнее на 1-2 миллисекунды хд

Код:

Entity damager = e.getDamager()
String name = damager.getType() == EntityType.PLAYER ? ((Player) damager).getName() : damager.getType().name();
Bukkit.broadcastMessage(ChatColor.translateAlternateColorCodes('&', "&7&lPlayer &f&l" p.getName() "&7&l killed by &f&l" + name));
Location  loc = p.getLocation();
p.getWorld().strikeLightningEffect(loc);
p.getWorld().playSound(loc,Sound.ENTITY_LIGHTNING_BOLT_IMPACT, 1, 0);

Здесь я тебе уже показывал, как можно одним циклом без лишних списков посчитать количество игроков по определенному критерию.

Код:

                int size = 0;
                ArrayList<Player> PE = new ArrayList<Player>();
                ArrayList<Player> PL = new ArrayList<Player>();
                PL.addAll(Bukkit.getOnlinePlayers());
                for(Player pe : PL) {
                    if(pe.getGameMode().equals(GameMode.SURVIVAL)) {
                        PE.add(pe);
                        size = PE.size();
                    }
                }

Код:

int size = 0;
for (Player player : Bukkit.getOnlinePlayer())
    if (player.getGameMode() == GameMode.SURVIVAL) size++;

Писал с тлф, потому неуверен, что весь код сработает без поправок

alexandrage · 15 сен 2020

Нахера тебе ChatColor? С таким же успехом можно написать сразу § не? Это же не чат где параграф писать нельзя.

imDaniX · 15 сен 2020

alexandrage сказал(а):
Нахера тебе ChatColor? С таким же успехом можно написать сразу § не? Это же не чат где параграф писать нельзя.

Можно, но лично мне зачастую просто лениво каждый раз гуглить или искать этот символ в коде. Хотя можно забиндить на какую-нибудь комбинацию к клавиатуре.

deadanykey · 15 сен 2020

imDaniX сказал(а):
Можно, но лично мне зачастую просто лениво каждый раз гуглить или искать этот символ в коде. Хотя можно забиндить на какую-нибудь комбинацию к клавиатуре.

Alt+0167

SaMEC · 15 сен 2020

deadanykey сказал(а):
Alt+0167

Еще проще alt+21

alexandrage · 15 сен 2020

imDaniX сказал(а):
Можно, но лично мне зачастую просто лениво каждый раз гуглить или искать этот символ в коде. Хотя можно забиндить на какую-нибудь комбинацию к клавиатуре.

Ну ты и курица, alt+21 че там запоминать?

AguRChek_TV · 15 сен 2020

imDaniX сказал(а):
Начиная с изначального вопроса - твой вариант должен сработать, если ты всё сделал правильно. Иначе же ты можешь проверить это через (e.getDamager().getType() == EntityType.PLAYER).

Ты в начале своего кода сразу рассчитываешь, что и ударивший, и жертва - игроки.

Код:

Player p = (Player) e.getEntity(); Player k = (Player) e.getDamager();

Это в итоге приведет к ошибке, если кто-то из них не игрок. Прежде всего убедись, что жертва - игрок.

Код:

if (e.getEntity().getType() != EntityType.PLAYER) return; Player p = (Player) e.getEntity();

Вот здесь можно упростить.

Код:

if ((((Damageable) e.getEntity()).getHealth() - e.getFinalDamage() <= 0))

до

Код:

if (p.getHealth() <= e.getFinalDamage())

Дальше - у тебя два практически идентичных блока кода. К тому же ты кидаешь молнию и звук от каждого игрока, т.е. если на сервере 100 игроков - 100 молний ударят в одно и то же место. Также отправку сообщения всем игрокам можно сделать без цикла через Bukkit.broadcastMessage(String)

Код:

if (e.getDamager() instanceof Player) { for(Player ps: Bukkit.getOnlinePlayers()) { ps.sendMessage(ChatColor.GRAY + "" + ChatColor.BOLD + "player " + ChatColor.WHITE + "" + ChatColor.BOLD + p.getName() + ChatColor.GRAY + "" + ChatColor.BOLD + " killed by " + ChatColor.WHITE + "" + ChatColor.BOLD + k.getName()); Location loc = p.getLocation(); p.getWorld().strikeLightningEffect(loc); p.getWorld().playSound(loc,Sound.ENTITY_LIGHTNING_BOLT_IMPACT, 1, 0); } } if (!(e.getDamager() instanceof Player)) { for(Player ps: Bukkit.getOnlinePlayers()) { ps.sendMessage(ChatColor.GRAY + "" + ChatColor.BOLD + "player " + ChatColor.WHITE + "" + ChatColor.BOLD + p.getName() + ChatColor.GRAY + "" + ChatColor.BOLD + " killed by " + ChatColor.WHITE + "" + ChatColor.BOLD + e.getDamager().getType()); Location loc = p.getLocation(); p.getWorld().strikeLightningEffect(loc); p.getWorld().playSound(loc,Sound.ENTITY_LIGHTNING_BOLT_IMPACT, 1, 0); } }

Сделай из двух блоков один и исправь косяки. Ради приличия ещё заменил все ChatColor одним методом, но это необязательно - медленнее на 1-2 миллисекунды хд

Код:

Entity damager = e.getDamager() String name = damager.getType() == EntityType.PLAYER ? ((Player) damager).getName() : damager.getType().name(); Bukkit.broadcastMessage(ChatColor.translateAlternateColorCodes('&', "&7&lPlayer &f&l" p.getName() "&7&l killed by &f&l" + name)); Location loc = p.getLocation(); p.getWorld().strikeLightningEffect(loc); p.getWorld().playSound(loc,Sound.ENTITY_LIGHTNING_BOLT_IMPACT, 1, 0);

Здесь я тебе уже показывал, как можно одним циклом без лишних списков посчитать количество игроков по определенному критерию.

Код:

int size = 0; ArrayList<Player> PE = new ArrayList<Player>(); ArrayList<Player> PL = new ArrayList<Player>(); PL.addAll(Bukkit.getOnlinePlayers()); for(Player pe : PL) { if(pe.getGameMode().equals(GameMode.SURVIVAL)) { PE.add(pe); size = PE.size(); } }

Код:

int size = 0; for (Player player : Bukkit.getOnlinePlayer()) if (player.getGameMode() == GameMode.SURVIVAL) size++;

Писал с тлф, потому неуверен, что весь код сработает без поправок

огромное спасибо братик

Помогите EntityDamageByEntityEvent проверка на Не игрока

AguRChek_TV

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

imDaniX

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

_MRDen3000YT_

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

AguRChek_TV

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

alexandrage

Старожил

AguRChek_TV

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

imDaniX

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

alexandrage

Старожил

imDaniX

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

deadanykey

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

SaMEC

Старожил

alexandrage

Старожил

AguRChek_TV

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