Don't Know Error

Hello i just made a event on Left_Click_Block but i'm getting an error. The error is when i hit a block with nothing in my hand. I have code so when i have specific item with a display it creates a double chest, this works. But when i try to hit the block without an item in my hand it gives an error. If i hit the ground with a shovel or something other then the specific item it works fine just when i hit the ground with nothing.

Here is the error:

Code:

C:\Users\Timmy\Documents\Files\Local Host>java -Xms1024M -Xmx1024M -jar craftbuk
kit.jar -o true
229 recipes
27 achievements
17:27:34 [INFO] Starting minecraft server version 1.5.2
17:27:34 [INFO] Loading properties
17:27:34 [INFO] Default game type: CREATIVE
17:27:34 [INFO] Generating keypair
17:27:34 [INFO] Starting Minecraft server on *:25565
17:27:34 [INFO] This server is running CraftBukkit version git-Bukkit-1.5.1-R0.2
-14-g7d19b67-b2767jnks (MC: 1.5.2) (Implementing API version 1.5.2-R0.1-SNAPSHOT
)
17:27:34 [INFO] [BetterCast] Loading BetterCast v1.3
17:27:34 [INFO] [UltimateRPG] Loading UltimateRPG v1.0
17:27:34 [INFO] [ScoreboardMOTD] Loading ScoreboardMOTD v1.0
17:27:34 [INFO] [RpgFriends] Loading RpgFriends v1.0
17:27:34 [INFO] Preparing level "world"
17:27:35 [INFO] Preparing start region for level 0 (Seed: -2819586645922029152)
17:27:35 [INFO] ----- Bukkit Auto Updater -----
17:27:35 [INFO] It appears that you're running a Development Build, when you've
specified in bukkit.yml that you prefer to run Recommended Builds.
17:27:35 [INFO] If you would like to be kept informed about new Development Buil
d releases, it is recommended that you change 'preferred-channel' in your bukkit
.yml to 'dev'.
17:27:35 [INFO] With that set, you will be told whenever a new version is availa
ble for download, so that you can always keep up to date and secure with the lat
est fixes.
17:27:35 [INFO] If you would like to disable this warning, simply set 'suggest-c
hannels' to false in bukkit.yml.
17:27:35 [INFO] ----- ------------------- -----
17:27:36 [INFO] Preparing spawn area: 91%
17:27:36 [INFO] Preparing start region for level 1 (Seed: -2819586645922029152)
17:27:36 [INFO] [BetterCast] Enabling BetterCast v1.3
17:27:36 [INFO] [BetterCast] Enabled!
17:27:36 [INFO] [UltimateRPG] Enabling UltimateRPG v1.0
17:27:36 [INFO] Plugin enabled.
17:27:36 [INFO] [ScoreboardMOTD] Enabling ScoreboardMOTD v1.0
17:27:36 [INFO] [ScoreBoard MOTD] Enabled!
17:27:36 [INFO] [RpgFriends] Enabling RpgFriends v1.0
17:27:36 [INFO] [RpgFriends] Rpg Friends has been enabled!
17:27:36 [INFO] Server permissions file permissions.yml is empty, ignoring it
17:27:36 [INFO] Done (1.851s)! For help, type "help" or "?"
17:27:36 [INFO] Starting GS4 status listener
17:27:36 [INFO] Starting remote control listener
17:27:36 [INFO] Query running on 0.0.0.0:25565
17:27:36 [WARNING] No rcon password set in 'C:\Users\Timmy\Documents\Files\Local
Host\server.properties', rcon disabled!
17:27:42 [INFO] xXFuZioN360Xx[/127.0.0.1:63509] logged in with entity id 191 at
([world] 301.4507849689614, 63.0, -181.04911382764018)
17:27:51 [SEVERE] Could not pass event PlayerInteractEvent to UltimateRPG v1.0
org.bukkit.event.EventException
        at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.ja
va:427)
        at org.bukkit.plugin.RegisteredListener.callEvent(RegisteredListener.jav
a:62)
        at org.bukkit.plugin.SimplePluginManager.fireEvent(SimplePluginManager.j
ava:477)
        at org.bukkit.plugin.SimplePluginManager.callEvent(SimplePluginManager.j
ava:462)
        at org.bukkit.craftbukkit.v1_5_R3.event.CraftEventFactory.callPlayerInte
ractEvent(CraftEventFactory.java:186)
        at net.minecraft.server.v1_5_R3.PlayerInteractManager.dig(PlayerInteract
Manager.java:108)
        at net.minecraft.server.v1_5_R3.PlayerConnection.a(PlayerConnection.java
:541)
        at net.minecraft.server.v1_5_R3.Packet14BlockDig.handle(SourceFile:46)
        at net.minecraft.server.v1_5_R3.NetworkManager.b(NetworkManager.java:292
)
        at net.minecraft.server.v1_5_R3.PlayerConnection.d(PlayerConnection.java
:109)
        at net.minecraft.server.v1_5_R3.ServerConnection.b(SourceFile:35)
        at net.minecraft.server.v1_5_R3.DedicatedServerConnection.b(SourceFile:3
0)
        at net.minecraft.server.v1_5_R3.MinecraftServer.r(MinecraftServer.java:5
81)
        at net.minecraft.server.v1_5_R3.DedicatedServer.r(DedicatedServer.java:2
26)
        at net.minecraft.server.v1_5_R3.MinecraftServer.q(MinecraftServer.java:4
77)
        at net.minecraft.server.v1_5_R3.MinecraftServer.run(MinecraftServer.java
:410)
        at net.minecraft.server.v1_5_R3.ThreadServerApplication.run(SourceFile:5
73)
Caused by: java.lang.NullPointerException
        at plugins.xxfuzion360xx.UltimateRPG.UltimateRPG.onPlayerInteract(Ultima
teRPG.java:487)
        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
        at java.lang.reflect.Method.invoke(Unknown Source)
        at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.ja
va:425)
        ... 16 more
>

Here is where it is in the code:

Code:java

 
if ((event.getAction() == Action.LEFT_CLICK_BLOCK) && 
 

here is that whole if statement:

Code:java

 
if ((event.getAction() == Action.LEFT_CLICK_BLOCK) && 
        (player.getItemInHand().getItemMeta().hasDisplayName()) && 
        (player.getItemInHand().getItemMeta().getDisplayName().equals(ChatColor.DARK_PURPLE + "" + ChatColor.BOLD +"Hearthstone"))) {
      event.getClickedBlock().getLocation().add(0,1,0).getBlock().setType(Material.CHEST);
      event.getClickedBlock().getLocation().add(1,1,0).getBlock().setType(Material.CHEST);
          player.sendMessage(ChatColor.GREEN + "Shop Created!");
          event.setCancelled(true);
        }
 

here is the whole event:

Code:java

 
@SuppressWarnings({ "unchecked", "deprecation" })
@EventHandler(priority=EventPriority.LOW)
  public void onPlayerInteract(PlayerInteractEvent event)
  {
    Player player = event.getPlayer();
 
    if ((event.getAction() == Action.RIGHT_CLICK_AIR) && 
      (player.getItemInHand().getItemMeta().hasDisplayName()) && 
      (player.getItemInHand().getItemMeta().getDisplayName().equals(ChatColor.DARK_PURPLE + "" + ChatColor.BOLD +"Hearthstone"))) {
      try {
        player.teleport(getHearthstone(player));
        player.sendMessage("You have been teleported to your Hearthstone's location.");
      } catch (Exception e) {
        player.sendMessage("You might not have set your Hearthstone location.");
      }
 
    }
 
    if ((player.getItemInHand() == null) && 
        (event.getAction() == Action.LEFT_CLICK_BLOCK)){
        event.setCancelled(true);
   return;
    }
 
    if ((event.getAction() == Action.LEFT_CLICK_BLOCK) && 
        (player.getItemInHand().getItemMeta().hasDisplayName()) && 
        (player.getItemInHand().getItemMeta().getDisplayName().equals(ChatColor.DARK_PURPLE + "" + ChatColor.BOLD +"Hearthstone"))) {
      event.getClickedBlock().getLocation().add(0,1,0).getBlock().setType(Material.CHEST);
      event.getClickedBlock().getLocation().add(1,1,0).getBlock().setType(Material.CHEST);
          player.sendMessage(ChatColor.GREEN + "Shop Created!");
          event.setCancelled(true);
        }
 
    if (event.getClickedBlock() == null) return;
    Block block = event.getClickedBlock();
 
    if (event.getAction() == Action.RIGHT_CLICK_BLOCK) {
      if (block.getType() == Material.ENDER_CHEST) { 
        player.sendMessage("Bank chest opened.");
        int s = 0;
 
        event.setCancelled(true);
        Inventory inv = Bukkit.createInventory(player, getConfig().getInt("Options.default-bankslots"), player.getName() + "'s Bank");
        if (getConfig().isSet("Players." + event.getPlayer().getName() + ".bankinv")) {
          for (@SuppressWarnings("unused") ItemStack i : inv.getContents()) {
            if (getConfig().getString("Players." + event.getPlayer().getName() + ".bankinv.slot" + s) != "nothing") {
              String[] d = getConfig().getString("Players." + event.getPlayer().getName() + ".bankinv.slot" + s).split(",");
              if (d[0].startsWith("Bank Cheque")) {
                ItemStack is = new ItemStack(Material.PAPER, Integer.parseInt(d[1]));
                ItemMeta im = is.getItemMeta();
                im.setDisplayName(d[0]);
                @SuppressWarnings("rawtypes")
ArrayList lore = new ArrayList();
                lore.add(ChatColor.GOLD + "This can be cashed in for Gold");
                lore.add(ChatColor.GOLD + "by left clicking on a bank sign.");
                im.setLore(lore);
                is.setItemMeta(im);
                inv.setItem(s, is);
              } else {
                inv.setItem(s, returnItem(d[0], Integer.parseInt(d[1]), Short.parseShort(d[2])));
              }
              s++;
            }
          }
        }
        player.openInventory(inv);
      }
      if (block.getType() == Material.CHEST) {
        if (!getConfig().isSet("Players." + player.getName() + ".Looted")) {
          upd("Players." + player.getName() + ".Looted", "0,0,0,world_");
        }
        Chest chest = (Chest)block.getState();
        Inventory inv = chest.getInventory();
        Location loc = block.getLocation();
        String blockloc = loc.getBlockX() + "," + loc.getBlockY() + "," + loc.getBlockZ() + "," + loc.getWorld().getName();
        if (!getConfig().getString("Players." + player.getName() + ".Looted").contains(blockloc + "_")) {
          if (getConfig().isSet("Loot." + blockloc))
            for (String item : getConfig().getString("Loot." + blockloc).split(","))
              try {
                inv.addItem(new ItemStack[] { returnItem(item, 1, Short.parseShort("0")) });
              }
              catch (Exception localException1) {
              }
          else for (String item : getConfig().getString("Loot.defaultloot").split(","))
              try {
                inv.addItem(new ItemStack[] { returnItem(item, 1, Short.parseShort("0")) });
              }
              catch (Exception localException2)
              {
              } upd("Players." + player.getName() + ".Looted", getConfig().getString(new StringBuilder("Players.").append(player.getName()).append(".Looted").toString()) + blockloc + "_");
        }
      }
      if (block.getType() == Material.ANVIL) {
        event.setCancelled(true);
        if (money(player) >= getConfig().getInt("Options.Anvil.Price")) {
          rmoney(player, getConfig().getInt("Options.Anvil.Price"));
          player.getItemInHand().setDurability((short)(player.getItemInHand().getDurability() - getConfig().getInt("Options.Anvil.DuraToAdd")));
          player.sendMessage("Weapon durability increase by " + getConfig().getInt("Options.Anvil.DuraToAdd") + " for " + getConfig().getInt("Options.Anvil.Price") + " Gold.");
        } else {
          player.sendMessage("This weapon can't be fixed because you don't have enough Gold.");
          player.sendMessage("It costs " + getConfig().getInt("Options.Anvil.Price") + " Gold to repair.");
        }
      }
      if ((block.getType() == Material.WALL_SIGN) || (block.getType() == Material.SIGN_POST)) {
        Sign sign = (Sign)block.getState();
        if (sign.getLine(0).equalsIgnoreCase("[buy]")) {
          if (money(player) >= Integer.parseInt(sign.getLine(2)))
            try {
              rmoney(player, Integer.parseInt(sign.getLine(2)));
              player.getInventory().addItem(new ItemStack[] { returnItem(sign.getLine(1), 1, Short.parseShort("0")) });
              player.sendMessage("You bought " + sign.getLine(1) + " for " + sign.getLine(2) + " Gold.");
            } catch (Exception e) {
              player.sendMessage("There was problem trying to buy this item.");
            }
          else {
            player.sendMessage("You don't have enough to buy this item.");
          }
        }
        if (sign.getLine(0).equalsIgnoreCase("[hearthstone]")) {
          try {
            player.sendMessage("You set Hearthstone to " + sign.getLine(1) + ".");
            Location loc = player.getLocation();
            String loca = loc.getWorld().getName() + "," + loc.getBlockX() + "," + loc.getBlockY() + "," + loc.getBlockZ();
            upd("Players." + player.getName() + ".hearthstone", loca);
          } catch (Exception e) {
            player.sendMessage("There was a problem setting your Hearthstone here.");
          }
        }
        if ((sign.getLine(0).equalsIgnoreCase("[bank]")) && 
          (sign.getLine(1).equalsIgnoreCase("bank cheque"))) {
          if (money(player) >= Integer.parseInt(sign.getLine(2))) {
            rmoney(player, Integer.parseInt(sign.getLine(2)));
            ItemStack is = new ItemStack(Material.PAPER, 1);
            ItemMeta im = is.getItemMeta();
            im.setDisplayName("Bank Cheque of " + sign.getLine(2) + " Gold");
            @SuppressWarnings("rawtypes")
ArrayList lore = new ArrayList();
            lore.add(ChatColor.GOLD + "This can be cashed in for Gold");
            lore.add(ChatColor.GOLD + "by left clicking on a bank sign.");
            im.setLore(lore);
            is.setItemMeta(im);
            player.getInventory().addItem(new ItemStack[] { is });
            player.sendMessage("Bank Cheque created and stored in your inventory.");
            player.updateInventory();
          } else {
            player.sendMessage("You don't have " + sign.getLine(2) + " Gold, so you can't create a Cheque.");
          }
        }
      }
 
    }
 
    if ((event.getAction() == Action.LEFT_CLICK_BLOCK) && (
      (block.getType() == Material.WALL_SIGN) || (block.getType() == Material.SIGN_POST))) {
      Sign sign = (Sign)block.getState();
      if ((sign.getLine(0).equalsIgnoreCase("[bank]")) && 
        (sign.getLine(1).equalsIgnoreCase("bank cheque")) && 
        (player.getItemInHand() != null))
        try {
          if ((player.getItemInHand().getItemMeta().hasDisplayName()) && 
            (player.getItemInHand().getItemMeta().getDisplayName().startsWith("Bank Cheque"))) {
            String[] d = player.getItemInHand().getItemMeta().getDisplayName().split(" ");
            try {
              amoney(player, Integer.parseInt(d[3]));
              if (player.getItemInHand().getAmount() > 1)
                player.getItemInHand().setAmount(player.getItemInHand().getAmount() - 1);
              else {
                player.setItemInHand(new ItemStack(Material.AIR));
              }
              player.updateInventory();
              player.sendMessage("Thank you cashing this cheque.");
            } catch (Exception e) {
              player.sendMessage("There was an error trying to cash your cheque.");
            }
          }
        }
        catch (Exception localException3)
        {
        }
    }
  }
 

know one knows?

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

FuZioN720 It could be because you're using "player.getItemInHand().getItemMeta().hasDisplayName()" without checking to see if the item in hand is null or not.
To fix this, you could try putting if (player.getItemInHand() != null) { } around that statement.

 

Polaris29 Thanks but it is still saying that there is an error on that same line.

 

Line 487. What is it?

 

Why are you using Try Catch blocks without catching?

 

Jamesthatguy line 487 is

Code:java

(player.getItemInHand().getItemMeta().hasDisplayName()) &&

TheGreenGamerHD What do you mean Where do i do that?

 

line 178 is your catch

 

TheGreenGamerHD I am catching it fine that part of the code works its just the part were the error is something is null

 

If you don't catch anything don't bother using try catch

 

So your saying get rid of the try and catch where it catch's

Code:java

catch (Exception localException3)

 

Try putting the item in hand to an itemstack and phrase the statement like so

Code:

if ([itemstacknamehere].hasItemMeta() == true){

 

Jamesthatguy So you mean

Code:

if([ChatColor.DARK_PURPLE + "" + ChatColor.BOLD +"Hearthstone"].hasItemMeta() == true){

or

Code:

if(Material.NETHER_STAR.hasItemMeta() == true){

 

No, I would recommend using an item stack for efficiency. For instance, when the event is triggered I would get item in hand and if the item is a nether star I would then cast it to an item stack that I will call "j" for example purposes. I would then to

Code:

 if (j.hasItemMeta() == true){}

None of the methods you purposed would work because your not casting it to anything. On the first one your technically saying if the chat color is dark purple and its contains "Hearthstone" string and has an itemmeta? You would only get errors. Similar problem with the second. Use my method.

 

@jamesthathguy Ok so is this what you mean i now have this:

Code:java

 
if ((event.getAction() == Action.RIGHT_CLICK_AIR)){
   ItemStack item = player.getItemInHand();
   if(item.getType().equals(Material.NETHER_STAR)){
   if (item.hasItemMeta() == true){
   if (item.getItemMeta().getDisplayName().equals(ChatColor.DARK_PURPLE + "" + ChatColor.BOLD +"Hearthstone")){
//code
}
}
} 
 

Jamesthatguy Thanks now i fixed it.

EDIT by Moderator: merged posts, please use the edit button instead of double posting.