Need help on Right Click Air Event (ERROR)

I wrote But it only spams in the chat "You do not have enough ammo"

Code:java

 @EventHandler
public void onPlayerInteract(PlayerInteractEvent event) {
Player player = event.getPlayer();
if(event.getAction().equals(Action.RIGHT_CLICK_AIR)){
if (player.getItemInHand().getType() == Material.WOOD_HOE) {
if (event.getPlayer().getInventory().contains(Material.SEEDS, 1)) {
player.getInventory().remove(new ItemStack(Material.SEEDS, 1));
player.launchProjectile(Arrow.class);
}else
 
player.sendMessage(ChatColor.GOLD+"You do not have enough ammo");
 
 
 
 
}}}

I took out the ! but still gosh I use to be really good as this stuff then after 4 months of not programming u forget shit. And I did use your example Thank You.

 

WauloK I don't think hes trolling. I think hes got issues.

uyuyuy99 Okay. But I can't belive your still helping this guy?

thatdubstepgamer Look at your fricken logic. Your checking if the player DOESNT have seeds. (with the !).

I said this 400 times. USE THE CODE I GAVE YOU. Seriously. Come on!

This is redicolous. This is getting to the point were I don't even think you know java, I think your RELIYING on us!

 

I wrote But it only spams in the chat "You do not have enough ammo"

Code:java

 @EventHandler
public void onPlayerInteract(PlayerInteractEvent event) {
Player player = event.getPlayer();
if(event.getAction().equals(Action.RIGHT_CLICK_AIR)){
if (player.getItemInHand().getType() == Material.WOOD_HOE) {
if (event.getPlayer().getInventory().contains(Material.SEEDS, 1)) {
player.getInventory().remove(new ItemStack(Material.SEEDS, 1));
player.launchProjectile(Arrow.class);
}else
 
player.sendMessage(ChatColor.GOLD+"You do not have enough ammo");
 
 
 
 
}}}

I took out the ! but still gosh I use to be really good as this stuff then after 4 months of not programming u forget shit. And I did use your example Thank You.

 

thatdubstepgamer You probally were never good.

Your missing a { on the else statement. Your IDE should tell you that.

 

Hmmmmm I added one still

Code:java

    @EventHandler
    public void onPlayerInteract(PlayerInteractEvent event) {
        Player player = event.getPlayer();
        if(event.getAction().equals(Action.RIGHT_CLICK_AIR)){ 
            if (player.getItemInHand().getType() == Material.WOOD_HOE) {
                if (event.getPlayer().getInventory().contains(Material.SEEDS, 1)) {
                    player.launchProjectile(Arrow.class);
                player.getInventory().remove(new ItemStack(Material.SEEDS, 1));
 
            }else{
 
                player.sendMessage(ChatColor.GOLD+"You do not have enough ammo");
 
 
 
 
                }}}}

Ok its working when I have no seeds just my hoe but when I have seeds it does not take 1 every arrow

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

 

How many seeds are in your inventory when you test the right-click air with a HOE?

 

1 and when I right click the seed is still there but then i got to click again and it says "You do not have enough ammo"Do i need a loop?

 

thatdubstepgamer It should work. And again, combine 2 sentances together bro.

 

You are too, and given your reactions to his constant questions, that's more unbelievable

thatdubstepgamer In that case, adding the { didn't change anything, because there's only 1 line after it. Also, you may need to add a player.updateInventory() after removing the seeds to refresh the client's view of the inventory.

 

Maybe you need this?

Code:java

player.updateInventory();

It's deprecated but I think still works?

 

uyuyuy99 I can't argue with that haha

thatdubstepgamer Yeah the } didn't change anything. Mai bad.

You don't need a loop, since the events happen every time you click with the hoe

WauloK it doesn't work. I think bukkit removed it.

Are you even sure Seeds is a material?

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

 

hmmmmmmmmm

Code:java

    @EventHandler
    public void onPlayerInteract(PlayerInteractEvent event) {
 
        Player player = event.getPlayer();
        if(event.getAction().equals(Action.RIGHT_CLICK_AIR)){ 
            if (player.getItemInHand().getType() == Material.WOOD_HOE) {
                if (event.getPlayer().getInventory().contains(Material.SEEDS, 1)) {
                    player.launchProjectile(Arrow.class);
                player.getInventory().remove(new ItemStack(Material.SEEDS, 1));
 
            }else{
 
                player.sendMessage(ChatColor.GOLD+"You do not have enough ammo");
 
 
 
 
                }}}}

Ill try a different material because bukkit is allowing it

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

 

thatdubstepgamer It should work. Does the event actually fire?

If bukkit is allowing it then it should work.

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

 

Nope I tried stone still same issue

Yes the event is actually firing

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

 

thatdubstepgamer Thats wierd.

Go in debug mode. After every statement sed the player a message, then determine which one is screwing up the code.

 

thatdubstepgamer Did you try what me and WauloK said?

And JPG, that won't help, because as he said, the arrow is actually firing, which is the deepest part of his code.

 

ok

When i try player.updateInventory(); it wants Suppressions Warnings

 

thatdubstepgamer That doesn't matter. It's just a warning, not an error.

 

But then it puts a black line through it dosnt work

 

Code:java

  if (event.getPlayer().getInventory().contains(Material.SEEDS, 1)) {

Does this need a ,1 in it?

 

It has that Waulok

 

thatdubstepgamer Yes, as I said, ignore the black line through it. Just try it.

 

I did try it

 

Does the if statement I posted require a ,1 in it?

 

It does not require but will still not take

 

thatdubstepgamer I just copied the code you gave me into a plugin.

It works perfectly.

 

JPG2000 Are you sure it's actually removing the seeds, though?

 

At JPG2000 earlier; You should calm down. The OP is clearly trying and he tried to implement it on his own. It's not possible for everyone to learn by copy-pasting. The OP could learn through personal and manual implementation, but was confused.

Thanks.

 

I still need help

 

Jade Yeah. I agree. I was very tired, and had a long day. D:

thatdubstepgamer Tell me exactly what happens when you click the seeds. Does it work?

Also, send me the code.

uyuyuy99 And yes, its removing the seeds. But my IDE gave me a error, that I fixed. I forgot the error.