Minecraft Spigot/Bukkit 插件在相等性检查时抛出 NullPointerException

Question

我正在尝试创建一个菜单，我想确保所有玩家都有加入的指南针。

我的代码不起作用——如果你的库存中有一个没有命名的指南针，它就不能给你指南针。如果我没有检查这些==null，我的代码会给出一个带有上述条件的NullPointerException。

Eclipse不显示错误。我做错了什么？

package me.psrcek.compassMenu.listeners;

import org.bukkit.ChatColor;
import org.bukkit.Material;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.player.PlayerJoinEvent;
import org.bukkit.inventory.ItemStack;
import org.bukkit.inventory.meta.ItemMeta;

public class PlayerJoinListener implements Listener {

    @EventHandler
    public void onJoin(PlayerJoinEvent e) {
        ItemStack[] inv = e.getPlayer().getInventory().getContents();
        
        for (ItemStack item : inv) {
            if (item == null) continue;
            
            if (item.getType().equals(Material.COMPASS)) {
                if (item.getItemMeta().getDisplayName() == null) continue;
                
                if (item.getItemMeta().getDisplayName().equals(ChatColor.RESET + "" + ChatColor.AQUA + "Menu")) {
                    e.getPlayer().sendMessage("found proppertly named compass");
                    return;
                }
            }
        }
        
        ItemStack compass = new ItemStack(Material.COMPASS);
        ItemMeta compassMeta = compass.getItemMeta();
        
        compassMeta.setDisplayName(ChatColor.RESET + "" + ChatColor.AQUA + "Menu");
        
        compass.setItemMeta(compassMeta);
        
        e.getPlayer().getInventory().addItem(compass);
        e.getPlayer().updateInventory();

    }

}

Answer 1

<代码>。equals方法和< code>==运算符不是一回事。

.equals 比较对象的值。

==比较对象的引用（内存中的位置）。

例子：

Integer x = 10;
Integer y = 10;

// is false
System.out.println(x == y)

// is true
System.out.println(x.equals(y)) 

Answer 2

我看到有几个地方可能会出现NullPointer异常。首先，您正在检查项目是否为空，如果为空，则检查它是否是指南针。但是如果为空，它就不能也是指南针。试试这个：

if(item != null){
    if(item.getType() == Material.COMPASS){

    }
}

我马上看到的第二个点是在同一个for循环中，当你得到指南针的项目元。嗯，Bukkit不喜欢说itemStack有一个itemMeta，直到你给它分配了一个，所以试试这个：

if(item.getType() == Material.COMPASS){
    if(item.hasItemMeta()){
        if(item.getItemMeta().getDisplayName().contains(ChatColor.AQUA + "Menu"){
            p.sendMessage("found properly named compass");
            return;
        }
    }
}

如果您想要更简单的方法来解决这个问题，我建议使用以下示例代码：

ItemStack compass = new ItemStack(Material.COMPASS);
ItemMeta compassMeta = compass.getItemMeta();

//No need to use ChatColor.RESET before the text has a color
compassMeta.setDisplayName(ChatColor.AQUA + "Menu");

compass.setItemMeta(compassMeta);

for(ItemStack item:e.getPlayer().getInventory().getContents()){
    if(item != null){
        if(item.equals(compass)){
            p.sendMessage("found properly named compass");
            return;
        }
    }
}

希望这对你有帮助。

Answer 3

我不知道为什么这有效，但改变

item.getType().equals(Material.COMPASS)

到

item.getType() == Material.COMPASS

成功了。我一直认为这是一回事。