共享健康插件布基特

Question

在for循环中，我的目标是破坏其他没有被破坏的玩家。所以，我做了一个新的事件，它的原因是自杀，这样当它破坏玩家时，这个方法就不会被再次调用，最终杀死玩家。我做了我的研究，发现了调用一个新的实体损坏事件EntityDamageEve=new EntityDamageEv（p，DamageCause.SUICIDE，（int）event.getDamage（））；Bukkit.getServer（）. getPluginManager（）. all事件（e）；来损坏玩家，但它似乎不起作用。我需要一种替代的方法来损坏我的玩家，但不要一次又一次地调用这个事件。

＜code＞p.Damage（event.getDamage）放置在内部时，事件循环。

@EventHandler
    public void onPlayerDamage(EntityDamageEvent event) {
        if(event.getEntity() instanceof Player && event.getCause() != DamageCause.SUICIDE ) {
            Player damaged =  (Player) event.getEntity();
            for(Player p :Bukkit.getOnlinePlayers()) {
                if(!p.getName().equalsIgnoreCase(damaged.getName())) {
                    EntityDamageEvent e = new EntityDamageEvent(p, DamageCause.SUICIDE,(int)event.getDamage());
                    Bukkit.getServer().getPluginManager().callEvent(e);
                    
                    //double cHealth =p.getHealth()-event.getDamage();
                    //p.setHealth(cHealth);
                }
            }
            
            
        }

注释代码有效，但当它达到负数时，它不会杀死玩家，它只是将玩家的生命值设置为负数。

Answer 1

将玩家存储到一个集合是一个好主意，但我认为你错过了一些东西。

假设:参与人1 =甜甜圈；玩家2 =煎饼

假设甜甜圈受到损害:

damaged = donut
if(!donut.equals(donut))
false
if(!donut.equals(pancake))
true 
   if(!playerSet.contains(pancake))
   true
      playerset.add(pancake);
      pancake.damage(event.getDamage());

我们现在损坏了煎饼并将其存储在playerSet中，因为煎饼损坏了，此事件将再次调用

damaged = pancake
if (!pancake.equals(donut))
true
   if (!playerSet.contains(donut))
   true 
      playerSet.add(donut);
      donut.damage(event.getDamage());

然后它继续。。。

由于损坏的播放器被损坏了两次，因此需要更好的解决方案。

我偶然发现了这个解决方案，添加了playerSet.add（损坏）

@EventHandler
    public void onPlayerDamage(EntityDamageEvent event) {
        if(event.getEntity() instanceof Player) {
            Player damaged =  (Player) event.getEntity();
            playerSet.add(damaged);
            for(Player p :Bukkit.getOnlinePlayers()) {
                if(!damaged.equals(p)) {
                    if(!playerSet.contains(p)) {
                        playerSet.add(p);
                        p.damage(event.getDamage());
                    }
                    else {
                        playerSet.remove(p);
                    }
                }
            }   
        }
    }

我仍然没有检查此代码中是否有错误，因此请随时指出谢谢。

Answer 2

调用伤害事件不会伤害玩家。事件用于在发生某些事情时通知插件，事件本身不会使任何事情发生。

Damagable#setHealth方法不会调用损坏事件，因此您可以在不重复调用事件的情况下使用它。作为javadocs状态，如果健康小于0，该方法将抛出IllegalArgumentExc的异常，因此只需添加检查即可。

double cHealth = p.getHealth() - event.getDamage();
p.setHealth(cHealth < 0 ? 0 : cHealth);

然而，我不建议使用setHealth方法，因为它不会考虑装甲。要使用Damageable#damage方法而不重复调用损坏事件，您可以将玩家添加到集合中，并检查该玩家是否在集合中。

private Set<Player> playerSet = new HashSet<>();

@EventHandler
public void onPlayerDamage(EntityDamageEvent event) {
    if (event.getEntity() instanceof Player) {
        Player damaged = (Player) event.getEntity();
        if (!playerSet.contains(damaged)) {
            for (Player p : Bukkit.getOnlinePlayers()) {
                if (!damaged.equals(p)) { 
                    playerSet.add(p);
                    p.damage(event.getDamage());
                }
            }
        } else {
            playerSet.remove(damaged);
        }
    }
}