Solved dropped item hit detection

Hi, does anyone know how to register hit detection from when a dropped item thrown by a player (tomahawk) and then deal damage to them?
- Loogeh
Thanks

 

Dropped? And deal damage to "them"? When you're launching a projectile at someone and it hits them you mean?

 

No, like. I throw the item in my hand by world.dropItem(stuffhere) and the make it go further with grenade.setVelocity(player.getLocation.getDirection)
So I need to register when it hits a player and then deal a specified amount of damage to them.

 

So you throw an item, give it a higher vector, and want it to deal damage to whoever it hits? Well, you could check the item's position every server tick and check if it's inside a player. Would be a little buggy though since items doesn't really "hit" players, just travels straight through them. If you then threw a little of math into it, you could make it work better.

 

That's a good idea.
Would you be able to provide a little example? If that's possible.

 

I can give you a few hints: Use a scheduledSyncTask to check the distance to every nearby player every server tick. If the distance is below some specific number, you can assume it "hit them" and kill it's velocity. You can save the items thrown in a set or something, and remove it and cancle the task when it has hit someone.

 

Okay, I'm not really sure what i'm doing as I had no idea how to do that lol. I gave it a shot though with what i know but it didn't work

Code:

 @EventHandler
public void tomahawk(PlayerInteractEvent event) {
final Player player = event.getPlayer();
final World world = player.getWorld();
if(player.hasPermission("mci.grenade")) {
if(player.getItemInHand().getType() == Material.STONE_AXE) {
if(event.getAction() == Action.LEFT_CLICK_AIR) {
final Item grenade = world.dropItem(player.getEyeLocation(), new ItemStack(Material.MAGMA_CREAM));
grenade.setVelocity(player.getEyeLocation().getDirection());
int amount = player.getPlayer().getItemInHand().getAmount();
  player.getItemInHand().setAmount(amount-1);
  plugin.getServer().getScheduler().scheduleAsyncRepeatingTask(plugin, new Runnable() {
 
  @Override
  public void run() {
  for(Player pls : utilServer.getPlayers()) {
  if(grenade.getLocation().equals(pls.getLocation())) {
  Bukkit.getServer().broadcastMessage("HIT DETECTED!");
 
  }
  }
  }
  }, 1L, 1L);
}
}
}
}

I don't think I even got close to what I need.

 

How I would have done it:

Code:JAVA

// Store a predefined key, and the scheduler key
final HashMap<Integer, Integer> grenades = new HashMap<Integer, Integer>();
 
@EventHandler
public void tomahawk(PlayerInteractEvent event) {
 
final Player player = event.getPlayer();
 
if (!(player.hasPermission("mci.grenade") || player.isOp()))
return;
if (player.getItemInHand().getType() != Material.STONE_AXE)
return;
if (event.getAction() != Action.LEFT_CLICK_AIR)
return;
 
final Item grenade = player.getWorld().dropItem(player.getEyeLocation(), new ItemStack(Material.MAGMA_CREAM));
grenade.setVelocity(player.getEyeLocation().getDirection());
int amount = player.getPlayer().getItemInHand().getAmount();
player.getItemInHand().setAmount(amount - 1);
 
int tKey = 0;
while (grenades.containsKey(tKey))
tKey++;
final int pKey = tKey;
 
// Use scheduleSyncRepeatingTask instead of scheduleAsyncRepeatingTask
int sKey = PLUGIN.getServer().getScheduler().scheduleSyncRepeatingTask(PLUGIN, new Runnable() {
 
public void run() {
 
if (grenade.isDead()) {
PLUGIN.getServer().getScheduler().cancelTask(grenades.get(pKey));
} else {
World w = grenade.getWorld();
Location l1 = grenade.getLocation();
for (Player pls : Bukkit.getOnlinePlayers()) {
 
if (pls == player || pls.getWorld() != w)
continue;
 
Location l2 = pls.getLocation();
 
double d = (l1.getX() > l2.getX() ? l1.getX() - l2.getX() : l2.getX() - l1.getX());
if (d > .5)
continue;
d = (l1.getZ() > l2.getZ() ? l1.getZ() - l2.getZ() : l2.getZ() - l1.getZ());
if (d > .5)
continue;
d = l1.getY() - l2.getY();
if (d > 2 || d < 0)
continue;
 
grenade.getWorld().createExplosion(grenade.getLocation(), .5F);
PLUGIN.getServer().getScheduler().cancelTask(grenades.get(pKey));
break;
}
}
}
}, 1L, 1L);
 
grenades.put(pKey, sKey);
}

One annoying bug though, the player you're trying to hit might pick the item up before it hits him.

 

Look at those "Item display" plugins (those that create floating unpickable items somewhere) and see how they work.

 

Fixing it isn't harder than extending the no-pickup timer or listening for when the item is being picked up.

 

Thank you so much!

 

Oh, one thing: I don't think line 19 of the code I provided will do you much good.