Right Click Furnace

Im trying to make a merchant on a furnace, but ins't working, can anyone help me? here is my code:

Code:java

@EventHandler
public void MerchantT1(PlayerInteractEvent e) {
    int gemt1 = 3 + (int)(Math.random() * 9.0D);
    Player p = (Player)e.getPlayer();
    if (e.getAction() == Action.RIGHT_CLICK_BLOCK)
    if (e.getClickedBlock().equals(Material.FURNACE))
    if (p.getItemInHand().getType() == Material.WOOD_SWORD)
      e.setCancelled(true);
      p.getItemInHand().setType(Material.AIR);
      amoney(p, gemt1);
      p.sendMessage(ChatColor.GREEN + "Voce trocou uma " + ChatColor.RED + "Espada de Madeira" + ChatColor.GREEN + "por gemas.");
 
    }

 

What exactly isn't working?
The money part? The message?
Player p = (Player)e.getPlayer();
You don't have to cast to Player on this.

 

Thanks, the error was in the (Player)

But now everything i click on the furnace it give me the money, i just want to give the money if the player is holding a wood sword

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

 

Profan
if (e.getClickedBlock().equals(Material.FURNACE))
if (p.getItemInHand().getType() == Material.WOOD_SWORD)

Why don't you have brackets on any of these o.o
Switch those two above so wood sword comes before Furnace

 

I maked it , and now nothing works...

 

Profan

what exactly do you mean with a Merchant since an Merchant is a inventory type to so I'm sort of confused.
also whats the real problem, do you want the item to be removed from the players hand?

 

Profan The problem is that you're trying to compare aBlock object to a Material enum, which will never return true.

e.getClickedBlock().getType() returns the Material enum for any given Block object.