Solved NPE Errors

This code is giving me two issues actually:

Code:java

}else if(event.getPlayer().getItemInHand().getType() == Material.WOOD_SWORD){
                if(player.hasPermission("magictools.woodsword")){
                    if(event.getPlayer().getItemInHand().getItemMeta().getDisplayName().equalsIgnoreCase(ChatColor.GREEN + "Sword of Repelling")){
                        //Block block = (Block) player.getTargetBlock(null, 50);
                        Block block = (Block) player.getEyeLocation();
                        block.setType(Material.COBBLESTONE);
                    }else{
                        return;
                    }
                }else{
                    player.sendMessage(ChatColor.AQUA + "[MagicTools] " + ChatColor.BLUE + "You don't have permission to use the wood sword ability. The admin may have disabled it to prevent world destruction.");
                }
            }else{
                return;
            }

First off, it's giving me an NPE error on the line that checks the display name of the item. Secondly, how can I set a block 1 block above the target block location of the player? I can figure out the last one on my own if I tried, but I can't find out why the first one is giving me errors. Also, the sword is just named "Wooden Sword" because it is the default item name. Not changed at all. I need to make sure that the cobble isn't summoned if the name doesn't equal what is set.

 

If there's no display name then getDisplayName() is returning null, therefore causing an NPE. You can get the block on top of another block by doing block.getRelative(BlockFace.UP)

 

The block.getRelative worked, but I'm still getting an NPE, even after checking to make sure the Display Name didn't equal null. Updated code:

Code:java

}else if(event.getPlayer().getItemInHand().getType() == Material.WOOD_SWORD){
                if(player.hasPermission("magictools.woodsword")){
                    if(event.getPlayer().getInventory().getItemInHand().getItemMeta().getDisplayName().equalsIgnoreCase(ChatColor.GREEN + "Sword of Repelling") && !(event.getPlayer().getInventory().getItemInHand().getItemMeta().getDisplayName().equalsIgnoreCase("Wood Sword"))){
                        Block block = (Block) player.getTargetBlock(null, 50).getRelative(BlockFace.UP);
                        block.setType(Material.WATER);
                    }
                }else{
                    player.sendMessage(ChatColor.AQUA + "[MagicTools] " + ChatColor.BLUE + "You don't have permission to use the wood sword ability. The admin may have disabled it to prevent world destruction.");
                }
            }else{
                return;
            }

It's still giving me the same NPE Error on the line that checks the display name. I checked if it was null before, but that didn't work. Any help? Stealth2800

 

nateracecar5 The item will only return a display name if there is one set, otherwise it returns null. On that line, you're checking if the display name is equal to "Wood Sword", which I assume is your way to check if it has the default name. Use hasDisplayName() instead to see if the item has a display name before doing anything.

 

Thanks. Will try that.

 

nateracecar5
Make sure you check if it is null before checking what it equals to.

 

Solved. Thanks! Also, the plugin is finished thanks to you so I will post a link to it later. AoH_Ruthless Stealth2800