HashMap值為true時不會取消任務

Question

我目前正在學習計划任務。

基本上，我正在制作一個Bukkit插件，您可以在其中啟用或禁用PVP。 當播放器鍵入/ pvp on或/ pvp off時，他們不得移動5秒鍾，但是任務取消似乎沒有執行。

主班：

package me.mortadelle2.pvptoggle;

import java.util.ArrayList;
import java.util.HashMap;

import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
import org.bukkit.plugin.java.JavaPlugin;
import org.bukkit.util.Vector;

public class PVPToggle extends JavaPlugin {

    ArrayList<String> noPVP = new ArrayList<String>();

    HashMap<String, Boolean> hasMoved = new HashMap<String, Boolean>();

    Player p;

    public void onEnable() {

        new PlayerDamage(this);
        getLogger().info("PVPToggle toggled!");

    }

    public void onDisable() {

        getLogger().info("PVPToggle disabled!");

    }

    public boolean onCommand(CommandSender sender, Command cmd, String label,
            String[] args) {

        p = (Player) sender;

        if (cmd.getName().equalsIgnoreCase("pvp")) {

            if (args.length == 0) {

                p.sendMessage(ChatColor.RED + "Invalid usage! /pvp [on or off]");
                return true;

            }

            if (args.length == 1) {

                if (args[0].equalsIgnoreCase("on")) {

                    p.sendMessage(ChatColor.YELLOW
                            + "PVP will be turned on in 5 seconds! Don't move!");
                    hasMoved.remove(p.getName());
                    hasMoved.put(p.getName(), false);

                    int turnOn = this.getServer().getScheduler()
                            .scheduleSyncDelayedTask(this, new Runnable() {

                                @Override
                                public void run() {

                                    if (hasMoved.get(p.getName()) == false) {
                                        noPVP.remove(p.getName());
                                        p.sendMessage(ChatColor.YELLOW
                                                + "You have turned PVP on!");
                                    }

                                }
                            }, 100L);

                    if (hasMoved.get(p.getName()) == true) {
                        p.sendMessage(ChatColor.RED
                                + "You moved so the action was cancelled.");
                        this.getServer().getScheduler().cancelTask(turnOn);
                    }

                    return true;

                }

                if (args[0].equalsIgnoreCase("off")) {

                    p.sendMessage(ChatColor.YELLOW
                            + "PVP will be turned off in 5 seconds. Don't move!");

                    hasMoved.remove(p.getName());
                    hasMoved.put(p.getName(), false);

                    int turnOff = this.getServer().getScheduler()
                            .scheduleSyncDelayedTask(this, new Runnable() {

                                @Override
                                public void run() {

                                    if (hasMoved.get(p.getName()) == false) {
                                        noPVP.add(p.getName());
                                        p.sendMessage(ChatColor.YELLOW
                                                + "You have turned PVP off!");
                                    }

                                }

                            }, 100L);

                    if (hasMoved.get(p.getName()) == true) {
                        p.sendMessage(ChatColor.RED
                                + "You moved so the action was cancelled.");
                        this.getServer().getScheduler().cancelTask(turnOff);
                    }

                    return true;
                }

            }

        }

        return false;
    }

}

偵聽器類：

    package me.mortadelle2.pvptoggle;

import org.bukkit.Bukkit;
import org.bukkit.entity.Arrow;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.entity.EntityDamageByEntityEvent;
import org.bukkit.event.entity.EntityShootBowEvent;
import org.bukkit.event.player.PlayerJoinEvent;
import org.bukkit.event.player.PlayerMoveEvent;
import org.bukkit.util.Vector;

public class PlayerDamage implements Listener{

    PVPToggle getter;

    public PlayerDamage(PVPToggle plugin) {

        plugin.getServer().getPluginManager().registerEvents(this, plugin);

        this.getter = plugin;

    }

    @EventHandler
    public void playerJoins(PlayerJoinEvent e){

        Player p = e.getPlayer();
        getter.hasMoved.put(p.getName(), false);

    }

    @EventHandler
    public void playerDamages(EntityDamageByEntityEvent e){

        Player victim = (Player) e.getEntity();
        Player killer = (Player) e.getDamager();

        victim.sendMessage("hey");

        if (getter.noPVP.contains(victim.getName()) || getter.noPVP.contains(killer.getName())){
            e.setCancelled(true);
        }

    }

    @EventHandler
    public void playerShootsPlayer(EntityShootBowEvent e){

        Arrow a = (Arrow) e.getEntity();

        Player shooter = (Player) a.getShooter();

        if (getter.noPVP.contains(a.getName()) || getter.noPVP.contains(shooter.getName())){
            e.setCancelled(true);
        }



    }

    @EventHandler
    public void playerMoves(PlayerMoveEvent e) {

        final Player p = e.getPlayer();

        final Vector pVel = p.getVelocity();

        getter.getServer().getScheduler()
                .scheduleSyncDelayedTask(getter, new Runnable() {

                    @Override
                    public void run() {


                        if (pVel.getX() != 0 || pVel.getBlockY() != 0
                                || pVel.getBlockZ() != 0) {
                            getter.hasMoved.remove(p.getName());
                            getter.hasMoved.put(p.getName(), true);
                        } else {
                            getter.hasMoved.remove(p.getName());
                            getter.hasMoved.put(p.getName(), false);
                        }

                    }
                }, 5L);

    }

}

Answer 1

無需取消任務，因為它已經結束。 我的代碼中還有一些問題。 首先，可能是最嚴重的是，您在PlayerMoveEvent中安排任務，這不僅如此，而且也不必要。 使用該事件時，應始終格外小心。 每當玩家移動任何東西（包括他們的頭部）時，都會調用它。 因此，該事件每秒可被稱為數百次或數千次。 如果不是絕對必要，則應始終進行檢查，以確保播放器移至新塊以減少代碼運行的次數。 一種更簡單的方法是執行以下操作：

@EventHandler
public void playerMoves(PlayerMoveEvent e) {
    Player p = e.getPlayer();

    //Check if the player moved to a new block
    if (e.getTo().getBlockX() != e.getFrom().getBlockX() || e.getTo().getBlockY() != e.getFrom().getBlockY() || e.getTo().getBlockZ() != e.getFrom().getBlockZ()) {
        //If they have, set their value in the hashmap to true
        getter.hasMoved.put(p.getName(), true);
    }
}

還知道您的調度程序僅在滴答滴答過去后才被調用一次，其余程序繼續進行。 您可以檢查播放器是否已移入調度程序

Bukkit.getScheduler().scheduleSyncDelayedTask(this, new Runnable() {

    @Override
    public void run() {

        if (hasMoved.get(p.getName()) == false) {
            noPVP.remove(p.getName());
            p.sendMessage(ChatColor.YELLOW + "You have turned PVP on!");
        } else {
            p.sendMessage(ChatColor.RED + "You moved so the action was cancelled.");
        }
    }
}, 100L);

還有一個簡短的注意事項：每次更改播放器的值時，都不需要將其從HashMap中刪除。 HashMap.put（）將覆蓋現有值。

編輯（另一個建議）：這可能是最好的方法，並且如果您想獲得在他們移動后立即將取消消息發送給播放器的效果，則可以從playerMoves方法內部取消調度程序，而不是添加HashMap的播放器。 這樣，甚至不需要HashMap，它只會提高整體代碼質量。 您確實應該盡量避免使用HashMaps，通常有一種更好的處理方式。