为什么这段代码只在我穿靴子的时候输出“Test”，而在我脱靴子的时候却从来没有输出“no”？

Question

以下是代码：

@EventHandler
public void onPlayerMove(PlayerMoveEvent event) {
    Player p = event.getPlayer();
    ItemStack boots = p.getInventory().getBoots();
    if (!boots.getItemMeta().getDisplayName().equals("§3Elemental Boots")) {
        p.sendMessage("no");
        p.setAllowFlight(false);
    }
    if (boots.getItemMeta().getDisplayName().equals("§3Elemental Boots")) {
        p.setAllowFlight(true);
        p.sendMessage("Test");
    }

}

为了指定，当我穿上靴子并移动时，它会输出测试。当我脱下靴子时，它什么也不输出。请告诉我我做错了什么，谢谢！

Answer 1

因为你没有穿靴子。我会逐行分解：）

前三行很明显

@EventHandler
public void onPlayerMove(PlayerMoveEvent event) {
    Player p = event.getPlayer();

在这里，您创建了一个变量boots，并为其提供了库存价值。getBoots（）；如果玩家戴着它们，它将返回一个物品堆栈。如果查看服务器控制台，您可能会得到一个NullPointerException，因为如果玩家没有穿靴子，那么boots变量将为null。

    ItemStack boots = p.getInventory().getBoots();

现在你检查靴子的显示名称是否不等于“§3元素靴子”，如果玩家穿着靴子，这一切都很好。在你的情况下，当靴子关闭，引导变量执行为空时，你试图访问一个不存在的项目的显示名称，所以什么都不会发生（除了NPE错误）。（这一行还有另一个问题，我会在回复底部解释）

    if (!boots.getItemMeta().getDisplayName().equals("§3Elemental Boots")) {
        p.sendMessage("no");
        p.setAllowFlight(false);
    }

同样的问题也在这里发生

    if (boots.getItemMeta().getDisplayName().equals("§3Elemental Boots")) {
        p.setAllowFlight(true);
        p.sendMessage("Test");
    }
}

解决这个问题的方法是添加另一个if语句，以确保玩家穿着靴子。你可以这样做：

if(p.getInventory().getBoots().equals(null)) {
    //Gets called when the player isn't wearing boots
    p.sendMessage("You aren't wearing any boots!")
} else {
    //Gets called when the player is wearing boots
    ItemStack boots = p.getInventory().getBoots();
    //Rest of your code
}

上面的第二个问题是，使用Java时，文本中的颜色可能会很奇怪，使用时可能会遇到问题。等于（“§3Elemental Boots”）这里有两个选项，

• 选项1，使用：。equals（ChatColor.translateAlternateColorCodes（“§”和“§3Elemental Boots”）在生成任何带有颜色的字符串时，也应该使用ChatColor方法
• 选项2，使用：ChatColor。StripColor（boots.getItemMeta（）。getDisplayName（））。等于（“元素靴”）

我个人倾向于第二种选择

我希望这有帮助！