Bukkit插件内存泄漏

Question

在我的Spigot插件中，我使用重复任务每隔0.5秒更新一次玩家的库存。长时间运行代码后，我会得到java.lang.OutOfMemoryError:GC开销限制超过。

我做了一些调试并使用JProfiler来查看导致内存泄漏的原因，并注意到ItemStacks的数量不断增加。堆遍历器显示这是由更新库存的代码引起的。这是当前代码：

public void updateInventory(Collection<? extends Player> players) {
    for (int i = 0; i < guiSize; i++) {
        ItemStack itemStack;
        if (serverItems.containsKey(i)) {
            itemStack = serverItems.get(i);
        } else {
            short color = GLASS_PANE_COLORS[RANDOM.nextInt(GLASS_PANE_COLORS.length - 1)];
            itemStack = new ItemStack(Material.STAINED_GLASS_PANE, 1, color);
            ItemMeta itemMeta = itemStack.getItemMeta();
            itemMeta.setDisplayName(ChatColor.RESET + "");
            itemStack.setItemMeta(itemMeta);
        }

        for (Player player : players) {
            player.getOpenInventory().getTopInventory().setItem(i, itemStack);
        }
    }
}

经过更多的调试，我发现删除了player.getOpenInventory（）.getTopInventory（。setItem（i，itemStack）将停止增加的itemStack数量。

什么可能导致内存泄漏？

编辑：

我已经更改了代码，不再为每个玻璃窗格制作新的ItemStack。相反，它现在将所有颜色存储在地图中，并从该地图中获取一个ItemStack，以放入库存。这仍然不起作用，服务器将在一段时间后崩溃，并出现OOME。

Answer 1

给定的代码看起来是正确的，因为它不会导致观察到的内存泄漏。虽然没有显示，但是对预生成ItemStacks的后续修改(一个更好的解决方案)也不会有所贡献。

但是，您是否知道getOpenRetentory（）返回玩家当前正在查看的库存，如果没有正在查看，则返回玩家的默认视图，即他们的制作库存？您的方法是否有一个先决条件，即给定集合中的所有玩家都打开了某个库存？如果一个或多个玩家当前正在查看烤箱、分配器等的库存，会发生什么？如果您想始终访问玩家库存，您应该使用getRetentory（）。

我问是因为否则将物品设置为不同库存的不存在插槽应该会导致其他异常而不是您的 OOM，尽管我不会在 Spigot 或 Minecraft 中为库存边界检查提供凭证。想到可能是这样，我简要地浏览了最新的Spigot-Server代码。没有什么突出会导致内存泄漏。

测试时是否加载了其他插件？如果是，请禁用所有插件，除了您的插件可能依赖的插件。

我希望我能提供更多的洞察力，但我只需要一个代码片段就可以做不了多少事情，这不会花费更长的时间来研究。