使用HashMap为Minecraft玩家分配Bukkit Runnable任务时的变量初始化问题

首先，你遇到的the local variable task may not have been initialised错误，原因是Lambda表达式在task变量被runTaskTimer赋值前就已经定义，而Java要求Lambda中引用的局部变量必须是已初始化的（或有效不可变的）。你在Lambda里直接调用task.getTaskId()，但此时task还没被赋值，所以编译器报错。

除此之外，你的代码逻辑还有核心问题：runTaskTimer是每隔指定时间执行一次任务，但你在任务里写了while (counter <=4)，这会导致任务第一次执行时就一次性给玩家加完4个物品，完全不符合“每分钟加一个，持续4分钟”的需求。

修复后的完整代码 #

import org.bukkit.Bukkit;
import org.bukkit.Material;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.player.PlayerInteractEvent;
import org.bukkit.event.player.PlayerQuitEvent;
import org.bukkit.inventory.ItemStack;
import org.bukkit.scheduler.BukkitTask;

import java.util.HashMap;
import java.util.Map;
import java.util.UUID;

public class ExampleListener implements Listener {
    // 存储玩家UUID -> 对应任务ID
    private final Map<UUID, Integer> playerTaskMap = new HashMap<>();
    // 存储玩家UUID -> 当前已添加物品的次数（计数器）
    private final Map<UUID, Integer> playerCounterMap = new HashMap<>();
    // 要发放的物品列表
    private final ItemStack[] rewardItems = {
            new ItemStack(Material.WATER),
            new ItemStack(Material.COBWEB),
            new ItemStack(Material.CAKE),
            new ItemStack(Material.RED_WOOL)
    };

    @EventHandler
    public void onPlayerQuit(PlayerQuitEvent event) {
        Player player = event.getPlayer();
        UUID uuid = player.getUniqueId();
        
        // 取消玩家的任务
        Integer taskId = playerTaskMap.remove(uuid);
        if (taskId != null) {
            Bukkit.getScheduler().cancelTask(taskId);
        }
        // 移除计数器
        playerCounterMap.remove(uuid);
    }

    @EventHandler
    public void onPlayerInteract(PlayerInteractEvent event) {
        ItemStack item = event.getItem();
        if (item == null || item.getType() != Material.WHITE_WOOL) {
            return;
        }

        Player player = event.getPlayer();
        UUID uuid = player.getUniqueId();
        
        // 如果玩家已有任务，不重复创建
        if (playerTaskMap.containsKey(uuid)) {
            return;
        }

        // 先初始化task为null，满足Lambda的引用要求
        BukkitTask task = null;
        // 初始化玩家计数器为0
        playerCounterMap.put(uuid, 0);

        task = Bukkit.getServer().getScheduler().runTaskTimer(this, () -> {
            // 任务执行时获取当前计数器
            int counter = playerCounterMap.getOrDefault(uuid, 0);
            
            // 检查计数器是否达到上限（4次）
            if (counter >= rewardItems.length) {
                // 取消任务
                if (task != null) {
                    task.cancel();
                }
                // 清理数据
                playerTaskMap.remove(uuid);
                playerCounterMap.remove(uuid);
                return;
            }

            // 给玩家添加对应物品
            Player targetPlayer = Bukkit.getPlayer(uuid);
            if (targetPlayer != null && targetPlayer.isOnline()) {
                targetPlayer.getInventory().addItem(rewardItems[counter]);
            }

            // 更新计数器
            playerCounterMap.put(uuid, counter + 1);
        }, 0, 20 * 60); // 延迟0 tick执行，之后每60秒（20*60 tick）执行一次

        // 将任务ID存入映射
        playerTaskMap.put(uuid, task.getTaskId());
    }
}

关键修改说明 #

1. 解决变量初始化问题：

   • 先声明BukkitTask task = null;，这样Lambda可以引用这个有效不可变的变量，之后再用runTaskTimer的返回值给它赋值。

2. 修正任务执行逻辑：

   • 移除了任务内的while循环，改用playerCounterMap存储每个玩家的独立计数器，每次任务执行时只处理一次物品发放，符合“每分钟一个”的需求。
   • 使用runTaskTimer的参数0, 20*60，表示立即执行第一次，之后每60秒执行一次（Bukkit中1秒=20 tick）。

3. 优化数据管理：

   • 拆分了原有的map为playerTaskMap（玩家-任务ID）和playerCounterMap（玩家-计数器），职责更清晰。
   • 玩家退出时，不仅移除映射数据，还主动取消任务，避免内存泄漏。
   • 增加了重复创建任务的判断，防止玩家多次点击羊毛创建多个任务。

4. 空指针安全处理：

   • 检查event.getItem()是否为null，避免空指针异常。
   • 任务执行时检查玩家是否在线，防止操作离线玩家的背包导致错误。

内容的提问来源于stack exchange，提问作者caledonian26