Solved Properly launching TNT on left click..

I can't seem to get the launching of the TNT correct, it spawns, and launches a TINY bit forward, but that's all I can get in the code.. Should I set the velocity to like, 20 or something? Anyone help a little on this? Also the removing of the ammo, wont work, I have debugged it, ( shown ) and its still not removing or updating the inventory.

Code:java

    @SuppressWarnings("deprecation")
    @EventHandler
    public void onTnTThrow(PlayerInteractEvent e){
        if(e.getPlayer().getItemInHand().getType() == Material.TNT
                && e.getAction() == Action.LEFT_CLICK_AIR || e.getAction() == Action.LEFT_CLICK_BLOCK){
            e.getPlayer().sendMessage("1");
            ItemStack ammo = new ItemStack(Material.TNT, 2);
            if(e.getPlayer().getItemInHand().hasItemMeta()){
                e.getPlayer().sendMessage("2");
                if(e.getPlayer().getItemInHand().getItemMeta().getDisplayName().equals("§rBombs")){
                    e.getPlayer().sendMessage("3");
                    Player p = e.getPlayer();
                    Entity tnt = p.getWorld().spawn(p.getLocation().add(0.65,0, 0), TNTPrimed.class);
                    ((TNTPrimed)tnt).setVelocity(tnt.getVelocity().multiply(3));
                    ((TNTPrimed)tnt).setFuseTicks(40);
                    p.getInventory().remove(ammo);
                    e.getPlayer().sendMessage("4");
// this line doesnt work \/\/\/ 
                    p.updateInventory();
                    e.getPlayer().sendMessage("5");
                }
            }
        }
    }

 

If you want to launch it where they are looking, use this:

Code:java

((TNTPrimed)tnt).tnt.setVelocity(player.getLocation().getDirection().multiply(1.5));

 

chasechocolate It only shooting outwards when a player is flying, and when they are on the ground, it spawns on their feet..

 

I m using this in my TF2 plugin here the code, just remove cooldown.get and cooldown.add cos thats just something im using for reloading
also play around with the package, cos right now its my TF2 package hope this helps

code:

package me.Shadow.TF2;

import org.bukkit.ChatColor;
import org.bukkit.Material;
import org.bukkit.entity.Player;
import org.bukkit.entity.TNTPrimed;
import org.bukkit.inventory.ItemStack;
import org.bukkit.inventory.meta.ItemMeta;

public class GLauncher {
@SuppressWarnings("deprecation")
public static void shoot(Player player){
if(Cooldown.get(player)==false){
ItemStack grenade = new ItemStack(Material.TNT,1);
ItemMeta meta=grenade.getItemMeta();
meta.setDisplayName("Grenades");
grenade.setItemMeta(meta);
if(player.getInventory().containsAtLeast(grenade,1)){
TNTPrimed tnt = player.getWorld().spawn(player.getEyeLocation(), TNTPrimed.class);
tnt.setVelocity(player.getLocation().getDirection().multiply(1.5));
tnt.setFuseTicks(30);
player.getInventory().removeItem(grenade);
player.updateInventory();
Cooldown.add(player, "reloading");
}else{
player.sendMessage(ChatColor.DARK_GRAY+"You need 'Grenades'");
}
}else{
player.sendMessage("Your "+player.getItemInHand().getItemMeta().getDisplayName()+" is still reloading ("+Numbers.round(Cooldown.time)+")");
}
}
}

 

XxShadow_NinjaxxX That kinda just screwed it up, some how, now it is not shooting at all..

Code:java

    @SuppressWarnings("deprecation")
    @EventHandler
    public void onTnTThrow(PlayerInteractEvent e){
        Player p = e.getPlayer();
        if(e.getAction() == Action.LEFT_CLICK_AIR || e.getAction() == Action.LEFT_CLICK_BLOCK &&
                p.getItemInHand().getType() == Material.TNT){
            p.sendMessage("1");
            ItemStack ammo = new ItemStack(Material.TNT, 2);
            if(p.getInventory().containsAtLeast(ammo, 1)){
                p.sendMessage("2");
                if(p.getItemInHand().equals(ammo)){
                    if(p.getItemInHand().getItemMeta().getDisplayName().equals("§rBombs")){
                        p.sendMessage("3");
                        Entity tnt = p.getWorld().spawn(p.getLocation().add(0.65, 0, 0), TNTPrimed.class);
                        tnt.setVelocity(p.getLocation().getDirection().multiply(1.5));
                        ((TNTPrimed) tnt).setFuseTicks(40);
                        p.getInventory().removeItem(ammo);
                        p.sendMessage("4");
                        p.updateInventory();
                        p.sendMessage("5");
                    }
                }
            }
        }
    }

Only the first debug message comes up.

 

kreashenz

1. What number does it get up to?
2. Your code will only currently work if the player has exactly 2 TNT items in their hand

 

MrBluebear3 It only goes up to the first debug message, and how do I fix the second problem?

 

Code:

 @SuppressWarnings("deprecation")
    @EventHandler
    public void onTnTThrow(PlayerInteractEvent e){
        Player p = e.getPlayer();
        if(e.getAction() == Action.LEFT_CLICK_AIR || e.getAction() == Action.LEFT_CLICK_BLOCK &&
                p.getItemInHand().getType() == Material.TNT){
            p.sendMessage("1");
            ItemStack ammo = new ItemStack(Material.TNT, 2);
            if(p.getItemInHand().getAmount()>=2) {
                    if(p.getItemInHand().getItemMeta().getDisplayName().equals("§rBombs")){
                        p.sendMessage("3");
                        Entity tnt = p.getWorld().spawn(p.getLocation().add(0.65, 0, 0), TNTPrimed.class);
                        tnt.setVelocity(p.getLocation().getDirection().multiply(1.5));
                        ((TNTPrimed) tnt).setFuseTicks(40);
                        p.getInventory().removeItem(ammo);
                        p.sendMessage("4");
                        p.updateInventory();
                        p.sendMessage("5");
                    }
                }
        }
    }

That should work

Actually wait, found another problem!

Code:

@SuppressWarnings("deprecation")
    @EventHandler
    public void onTnTThrow(PlayerInteractEvent e){
        Player p = e.getPlayer();
        if(e.getAction() == Action.LEFT_CLICK_AIR || e.getAction() == Action.LEFT_CLICK_BLOCK &&
                p.getItemInHand().getType() == Material.TNT){
            p.sendMessage("1");
            ItemStack ammo = new ItemStack(Material.TNT, 2);
            if(p.getItemInHand().getAmount()>=2) {
                    if(p.getItemInHand().getItemMeta().getDisplayName().equals("§rBombs")){
                        p.sendMessage("3");
                        Entity tnt = p.getWorld().spawn(p.getLocation().add(0.65, 0, 0), TNTPrimed.class);
                        tnt.setVelocity(p.getLocation().getDirection().multiply(1.5));
                        ((TNTPrimed) tnt).setFuseTicks(40);
                        if(p.getItemInHand().getAmount()>2) {p.getItemInHand().setAmount(p.getItemInHand().getAmount()-2)}else{p.setItemInHand(new ItemStack(Material.AIR, 0))}
                        p.sendMessage("4");
                        p.updateInventory();
                        p.sendMessage("5");
                    }
                }
        }
    }

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

 

MrBluebear3 It's not removing the items, now :/ (Causing an infinite bug).. This time its sending all the debug messages.
[EDIT] When I posted this, your comment popped up. Thanks for this

 

Then remove the items...?

 

MrBluebear3 When the item gets to 2, it doesn't remove anything. Should I add an if statement, for when the player has 2 of the item, then set the slot to air?

ohtwo

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

 

Code:

@SuppressWarnings("deprecation")
    @EventHandler
    public void onTnTThrow(PlayerInteractEvent e){
        Player p = e.getPlayer();
        if(e.getAction() == Action.LEFT_CLICK_AIR || e.getAction() == Action.LEFT_CLICK_BLOCK &&
                p.getItemInHand().getType() == Material.TNT){
            p.sendMessage("1");
            ItemStack ammo = new ItemStack(Material.TNT, 2);
            if(p.getItemInHand().getAmount()>=2) {
                    if(p.getItemInHand().getItemMeta().getDisplayName().equals("§rBombs")){
                        p.sendMessage("3");
                        Entity tnt = p.getWorld().spawn(p.getLocation().add(0.65, 0, 0), TNTPrimed.class);
                        tnt.setVelocity(p.getLocation().getDirection().multiply(1.5));
                        ((TNTPrimed) tnt).setFuseTicks(40);
                        if(p.getItemInHand().getAmount()>2) {p.getItemInHand().setAmount(p.getItemInHand().getAmount()-2)}else{p.setItemInHand(new ItemStack(Material.AIR, 0))}
                        p.sendMessage("4");
                        p.updateInventory();
                        p.sendMessage("5");
                    }
                }
        }
    }

Try changing it to this

 

ya you need to change somethings like you can remove if(p.getItemInHand.getItemMeta,getDisplayName.equals(Bombs))
bcause i have custom names for my ammo and stuff