Need help with NPC plugin

Okay so this is my Human.class

Code:java

package NPCSpawning;
 
import java.lang.reflect.Field;
import java.util.ArrayList;
import java.util.List;
import java.util.Random;
import java.util.UUID;
 
import net.minecraft.server.v1_6_R3.DataWatcher;
import net.minecraft.server.v1_6_R3.EntityHuman;
import net.minecraft.server.v1_6_R3.Packet17EntityLocationAction;
import net.minecraft.server.v1_6_R3.Packet18ArmAnimation;
import net.minecraft.server.v1_6_R3.Packet20NamedEntitySpawn;
import net.minecraft.server.v1_6_R3.Packet29DestroyEntity;
import net.minecraft.server.v1_6_R3.Packet33RelEntityMoveLook;
import net.minecraft.server.v1_6_R3.Packet40EntityMetadata;
import net.minecraft.server.v1_6_R3.Packet5EntityEquipment;
 
import org.bukkit.Bukkit;
import org.bukkit.Color;
import org.bukkit.EntityEffect;
import org.bukkit.Location;
import org.bukkit.Server;
import org.bukkit.World;
import org.bukkit.craftbukkit.v1_6_R3.entity.CraftPlayer;
import org.bukkit.entity.Entity;
import org.bukkit.entity.EntityType;
import org.bukkit.entity.Player;
import org.bukkit.event.entity.EntityDamageEvent;
import org.bukkit.event.player.PlayerTeleportEvent.TeleportCause;
import org.bukkit.inventory.ItemStack;
import org.bukkit.metadata.MetadataValue;
import org.bukkit.plugin.Plugin;
import org.bukkit.util.Vector;
 
 
public class Human implements Entity{
        String name;
        Location world;
        int id;
        Location l;
        int itemInHand;
 
        private List<Integer> ids = new ArrayList<Integer>();
 
        public Human(Location location, String name, int id, Location l, int itemInHand) {
                this.name = name;
                this.world = location;
                this.id = id;
                this.l = l;
                this.itemInHand = itemInHand;
                Packet20NamedEntitySpawn spawn = new Packet20NamedEntitySpawn();
                spawn.a = id;
                spawn.b = name;
                spawn.c = l.getBlockX() * 32;
                spawn.d = l.getBlockY() * 32;
                spawn.e = l.getBlockZ() * 32;
                spawn.f = getCompressedAngle(l.getYaw());
                spawn.g = getCompressedAngle(l.getPitch());
                spawn.h = itemInHand;
                DataWatcher d = new DataWatcher();
                d.a(0, (Object) (byte) 0);
                d.a(1, (Object) (short) 0);
                d.a(8, (Object) (byte) 0);
                try
                {
                    Field nameField = Packet20NamedEntitySpawn.class.getDeclaredField("i");
                    nameField.setAccessible(true);
                    nameField.set(spawn, d);
                } catch(Exception e) {
                        e.printStackTrace();
                }
                for(Player p : Bukkit.getOnlinePlayers()) {
                        ((CraftPlayer) p).getHandle().playerConnection.sendPacket(spawn);
                }
                ids.add(id);
        }
 
        public Human(EntityHuman h) {
                Packet20NamedEntitySpawn spawn = new Packet20NamedEntitySpawn(h);
                int id = new Random().nextInt(5000-1000)+1000;
                spawn.a = id;
                this.id = id;
                Packet5EntityEquipment armor1 = new Packet5EntityEquipment(id, 1, h.inventory.getArmorContents()[0]);
                Packet5EntityEquipment armor2 = new Packet5EntityEquipment(id, 2, h.inventory.getArmorContents()[1]);
                Packet5EntityEquipment armor3 = new Packet5EntityEquipment(id, 3, h.inventory.getArmorContents()[2]);
                Packet5EntityEquipment armor4 = new Packet5EntityEquipment(id, 4, h.inventory.getArmorContents()[3]);
                Packet5EntityEquipment sword = new Packet5EntityEquipment(id, 0, h.inventory.getItem(0));
                for(Player p : Bukkit.getOnlinePlayers()) {
                        ((CraftPlayer) p).getHandle().playerConnection.sendPacket(spawn);
                        ((CraftPlayer) p).getHandle().playerConnection.sendPacket(armor1);
                        ((CraftPlayer) p).getHandle().playerConnection.sendPacket(armor2);
                        ((CraftPlayer) p).getHandle().playerConnection.sendPacket(armor3);
                        ((CraftPlayer) p).getHandle().playerConnection.sendPacket(armor4);
                        ((CraftPlayer) p).getHandle().playerConnection.sendPacket(sword);
                }
        }
 
        private byte getCompressedAngle(float value) {
                return (byte) (value * 256.0F / 360.0F);
        }
 
        public void remove() {
                Packet29DestroyEntity packet = new Packet29DestroyEntity(id);
                for(Player p : Bukkit.getOnlinePlayers()) {
                        ((CraftPlayer) p).getHandle().playerConnection.sendPacket(packet);
                }
        }
 
        @SuppressWarnings("deprecation")
        public void setItemInHand(ItemStack i) {
                Packet29DestroyEntity packet = new Packet29DestroyEntity(id);
                for(Player p : Bukkit.getOnlinePlayers()) {
                        ((CraftPlayer) p).getHandle().playerConnection.sendPacket(packet);
                }
                Packet20NamedEntitySpawn spawn = new Packet20NamedEntitySpawn();
                spawn.a = id;
                spawn.b = name;
                spawn.c = l.getBlockX() * 32;
                spawn.d = l.getBlockY() * 32;
                spawn.e = l.getBlockZ() * 32;
                spawn.f = getCompressedAngle(l.getYaw());
                spawn.g = getCompressedAngle(l.getPitch());
                spawn.h = i.getTypeId();
                itemInHand = i.getTypeId();
                DataWatcher d = new DataWatcher();
                d.a(0, (Object) (byte) 0);
                d.a(1, (Object) (short) 0);
                d.a(8, (Object) (byte) 0);
                try
                {
                    Field nameField = Packet20NamedEntitySpawn.class.getDeclaredField("i");
                    nameField.setAccessible(true);
                    nameField.set(spawn, d);
                } catch(Exception e) {
                        e.printStackTrace();
                }
                for(Player p : Bukkit.getOnlinePlayers()) {
                        ((CraftPlayer) p).getHandle().playerConnection.sendPacket(spawn);
                }
        }
 
        @Deprecated
        public void setName(String s) {
                DataWatcher d = new DataWatcher();
                d.a(0, (Object) (byte) 0);
                d.a(1, (Object) (short) 0);
                d.a(8, (Object) (byte) 0);
                d.a(10, (Object) (String) s);
                //d.a(11, (Object) (byte) 0);
                Packet40EntityMetadata packet40 = new Packet40EntityMetadata(id, d, true);
                for(Player p : Bukkit.getOnlinePlayers()) {
                        ((CraftPlayer) p).getHandle().playerConnection.sendPacket(packet40);
                }
        }
 
        public void hideForPlayer(Player p) {
                DataWatcher d = new DataWatcher();
                d.a(0, (Object) (byte) 32);
                d.a(1, (Object) (short) 0);
                d.a(8, (Object) (byte) 0);
                Packet40EntityMetadata packet40 = new Packet40EntityMetadata(id, d, true);
                ((CraftPlayer) p).getHandle().playerConnection.sendPacket(packet40);
        }
 
        public void showForPlayer(Player p) {
                DataWatcher d = new DataWatcher();
                d.a(0, (Object) (byte) 0);
                d.a(1, (Object) (short) 0);
                d.a(8, (Object) (byte) 0);
                Packet40EntityMetadata packet40 = new Packet40EntityMetadata(id, d, true);
                ((CraftPlayer) p).getHandle().playerConnection.sendPacket(packet40);
        }
 
        public void addPotionColor(Color r) {
                int color = r.asBGR();
                final DataWatcher dw = new DataWatcher();
                dw.a(7, Integer.valueOf(color));
                Packet40EntityMetadata packet40 = new Packet40EntityMetadata(id, dw, true);
                for(Player p : Bukkit.getOnlinePlayers()) {
                        ((CraftPlayer) p).getHandle().playerConnection.sendPacket(packet40);
                }
        }
 
        public void addPotionColor(int color) {
                final DataWatcher dw = new DataWatcher();
                dw.a(7, Integer.valueOf(color));
                Packet40EntityMetadata packet40 = new Packet40EntityMetadata(id, dw, true);
                for(Player p : Bukkit.getOnlinePlayers()) {
                        ((CraftPlayer) p).getHandle().playerConnection.sendPacket(packet40);
                }
        }
 
        public void walk(double a, double b, double c) {
                //teleports, not walks
                byte x = (byte) a;
                byte y = (byte) b;
                byte z = (byte) c;
                Packet33RelEntityMoveLook packet = new Packet33RelEntityMoveLook();
                packet.a = id;
                packet.b = x;
                packet.c = y;
                packet.d = z;
                packet.e = getCompressedAngle(l.getYaw());
                packet.f = getCompressedAngle(l.getPitch());
                for(Player p : Bukkit.getOnlinePlayers()) {
                        ((CraftPlayer) p).getHandle().playerConnection.sendPacket(packet);
                }
                l.setX(x);
                l.setY(y);
                l.setZ(z);
        }
 
        public void setInvisible() {
                DataWatcher d = new DataWatcher();
                d.a(0, (Object) (byte) 32);
                d.a(1, (Object) (short) 0);
                d.a(8, (Object) (byte) 0);
                Packet40EntityMetadata packet40 = new Packet40EntityMetadata(id, d, true);
                for(Player p : Bukkit.getOnlinePlayers()) {
                        ((CraftPlayer) p).getHandle().playerConnection.sendPacket(packet40);
                }
        }
 
        public void setCrouched() {
                DataWatcher d = new DataWatcher();
                d.a(0, (Object) (byte) 2);
                d.a(1, (Object) (short) 0);
                d.a(8, (Object) (byte) 0);
                Packet40EntityMetadata packet40 = new Packet40EntityMetadata(id, d, true);
                for(Player p : Bukkit.getOnlinePlayers()) {
                        ((CraftPlayer) p).getHandle().playerConnection.sendPacket(packet40);
                }
        }
 
        public void reset() {
                DataWatcher d = new DataWatcher();
                d.a(0, (Object) (byte) 0);
                d.a(1, (Object) (short) 0);
                d.a(8, (Object) (byte) 0);
                Packet40EntityMetadata packet40 = new Packet40EntityMetadata(id, d, true);
                for(Player p : Bukkit.getOnlinePlayers()) {
                        ((CraftPlayer) p).getHandle().playerConnection.sendPacket(packet40);
                }
        }
 
        public void sprint() {
                DataWatcher d = new DataWatcher();
                d.a(0, (Object) (byte) 8);
                d.a(1, (Object) (short) 0);
                d.a(8, (Object) (byte) 0);
                Packet40EntityMetadata packet40 = new Packet40EntityMetadata(id, d, true);
                for(Player p : Bukkit.getOnlinePlayers()) {
                        ((CraftPlayer) p).getHandle().playerConnection.sendPacket(packet40);
                }
        }
 
        @Deprecated
        public void block() {
                DataWatcher d = new DataWatcher();
                d.a(0, (Object) (byte) 16);
                d.a(1, (Object) (short) 0);
                d.a(6, (Object) (byte) 0);
                Packet40EntityMetadata packet40 = new Packet40EntityMetadata(id, d, true);
                for(Player p : Bukkit.getOnlinePlayers()) {
                        ((CraftPlayer) p).getHandle().playerConnection.sendPacket(packet40);
                }
        }
 
        public void damage() {
                Packet18ArmAnimation packet18 = new Packet18ArmAnimation();
                packet18.a = id;
                packet18.b = 2;
                for(Player p : Bukkit.getOnlinePlayers()) {
                        ((CraftPlayer) p).getHandle().playerConnection.sendPacket(packet18);
                }
        }
 
        public void swingArm() {
                Packet18ArmAnimation packet18 = new Packet18ArmAnimation();
                packet18.a = id;
                packet18.b = 1;
                for(Player p : Bukkit.getOnlinePlayers()) {
                        ((CraftPlayer) p).getHandle().playerConnection.sendPacket(packet18);
                }
        }
 
        @Deprecated
        public void eatInHand() {
                Packet18ArmAnimation packet18 = new Packet18ArmAnimation();
                packet18.a = id;
                packet18.b = 5;
                for(Player p : Bukkit.getOnlinePlayers()) {
                        ((CraftPlayer) p).getHandle().playerConnection.sendPacket(packet18);
                }
        }
 
        public void sleep() {
                Packet17EntityLocationAction packet17 = new Packet17EntityLocationAction();
                packet17.a = id;
                packet17.b = (int) getX();
                packet17.c = (int) getY();
                packet17.d = (int) getZ();
                packet17.e = 0;
                for(Player p : Bukkit.getOnlinePlayers()) {
                        ((CraftPlayer) p).getHandle().playerConnection.sendPacket(packet17);
                }
        }
 
        public double getX() {
                return l.getX();
        }
 
        public double getY() {
                return l.getY();
        }
 
        public double getZ() {
                return l.getZ();
        }
 
        @Override
        public List<MetadataValue> getMetadata(String arg0) {
            // TODO Auto-generated method stub
            return null;
        }
 
        @Override
        public boolean hasMetadata(String arg0) {
            // TODO Auto-generated method stub
            return false;
        }
 
        @Override
        public void removeMetadata(String arg0, Plugin arg1) {
            // TODO Auto-generated method stub
 
        }
 
        @Override
        public void setMetadata(String arg0, MetadataValue arg1) {
            // TODO Auto-generated method stub
 
        }
 
        @Override
        public boolean eject() {
            // TODO Auto-generated method stub
            return false;
        }
 
        @Override
        public int getEntityId() {
            // TODO Auto-generated method stub
            return 0;
        }
 
        @Override
        public float getFallDistance() {
            // TODO Auto-generated method stub
            return 0;
        }
 
        @Override
        public int getFireTicks() {
            // TODO Auto-generated method stub
            return 0;
        }
 
        @Override
        public EntityDamageEvent getLastDamageCause() {
            // TODO Auto-generated method stub
            return null;
        }
 
        @Override
        public Location getLocation() {
            // TODO Auto-generated method stub
            return null;
        }
 
        @Override
        public Location getLocation(Location arg0) {
            // TODO Auto-generated method stub
            return null;
        }
 
        @Override
        public int getMaxFireTicks() {
            // TODO Auto-generated method stub
            return 0;
        }
 
        @Override
        public List<Entity> getNearbyEntities(double arg0, double arg1,
                double arg2) {
            // TODO Auto-generated method stub
            return null;
        }
 
        @Override
        public Entity getPassenger() {
            // TODO Auto-generated method stub
            return null;
        }
 
        @Override
        public Server getServer() {
            // TODO Auto-generated method stub
            return null;
        }
 
        @Override
        public int getTicksLived() {
            // TODO Auto-generated method stub
            return 0;
        }
 
        @Override
        public EntityType getType() {
            // TODO Auto-generated method stub
            return null;
        }
 
        @Override
        public UUID getUniqueId() {
            // TODO Auto-generated method stub
            return null;
        }
 
        @Override
        public Entity getVehicle() {
            // TODO Auto-generated method stub
            return null;
        }
 
        @Override
        public Vector getVelocity() {
            // TODO Auto-generated method stub
            return null;
        }
 
        @Override
        public World getWorld() {
            // TODO Auto-generated method stub
            return null;
        }
 
        @Override
        public boolean isDead() {
            // TODO Auto-generated method stub
            return false;
        }
 
        @Override
        public boolean isEmpty() {
            // TODO Auto-generated method stub
            return false;
        }
 
        @Override
        public boolean isInsideVehicle() {
            // TODO Auto-generated method stub
            return false;
        }
 
        @Override
        public boolean isOnGround() {
            // TODO Auto-generated method stub
            return false;
        }
 
        @Override
        public boolean isValid() {
            // TODO Auto-generated method stub
            return false;
        }
 
        @Override
        public boolean leaveVehicle() {
            // TODO Auto-generated method stub
            return false;
        }
 
        @Override
        public void playEffect(EntityEffect arg0) {
            // TODO Auto-generated method stub
 
        }
 
        @Override
        public void setFallDistance(float arg0) {
            // TODO Auto-generated method stub
 
        }
 
        @Override
        public void setFireTicks(int arg0) {
            // TODO Auto-generated method stub
 
        }
 
        @Override
        public void setLastDamageCause(EntityDamageEvent arg0) {
            // TODO Auto-generated method stub
 
        }
 
        @Override
        public boolean setPassenger(Entity arg0) {
            // TODO Auto-generated method stub
            return false;
        }
 
        @Override
        public void setTicksLived(int arg0) {
            // TODO Auto-generated method stub
 
        }
 
        @Override
        public void setVelocity(Vector arg0) {
            // TODO Auto-generated method stub
 
        }
 
        @Override
        public boolean teleport(Location arg0) {
            // TODO Auto-generated method stub
            return false;
        }
 
        @Override
        public boolean teleport(Entity arg0) {
            // TODO Auto-generated method stub
            return false;
        }
 
        @Override
        public boolean teleport(Location arg0, TeleportCause arg1) {
            // TODO Auto-generated method stub
            return false;
        }
 
        @Override
        public boolean teleport(Entity arg0, TeleportCause arg1) {
            // TODO Auto-generated method stub
            return false;
        }
 
}
 

I want it if a player damages the entity three times it dissapears and drops a players inventory. The NPC will also only be there for 10 seconds.
Here is a part of my other code:

Code:java

human.showForPlayer(otherplayers);
                Thread.sleep(1000);
                human.hideForPlayer(otherplayers);
                return true;

I dont think the Thread.sleep(100) works.. ;(

 

kevinspl2000
You need to use a scheduler for that

 

kevinspl2000 Thread.sleep(1000) will freeze the server for 10 seconds, take a look at scheduler programming

 

CaptainBern
mike546378
Okay I had a look at that, now how would I make the NPC damagable?

 

kevinspl2000 The best way to do so is to add a health variable and set that to 20.0, and then each time someone clicks the npc, play hurt animation, set health to -1.0 or so...

 

CaptainBern
How would I check if a player has left clicked the NPC?

 

kevinspl2000
You can't check if you hit something that doesn't exist.

 

xTrollxDudex
So how would I do this? o.o

 

kevinspl2000
I'm just saying. Try EntityDamageEvent

 

xTrollxDudex Actually you can. kevinspl2000 You have to use protocolLib for this. You will need to intercept the incoming Packet7UseEntity. In this packet there are 3 fields, a, b and c. Field a = entityId of the player that send the packet, this is ingored by minecraft. Field b = the entityId of the entity interacted with and Field c, this is the action, it's an integer, when the integer is 1, then it was a left click, if the integer is 0 then it was a right click. xTrollxDudex The EntityDamageEvent will ofcourse not work since it only get's fired by bukkit when an entity get's damaged and bukkit notices this. But since the client thinks there is an entity, it will send the server a packet saying he interacted with that entity. Ofcourse the server doesn't find this entity because it doesn't know about it so the server just ignores it.

 

CaptainBern
Exactly. It actually doesn't exist server side, technically speaking

 

WOW that is a big class haha. Split up your code between classes

 

CaptainBern
Thanks really explained carefully and overall really detailed answer.

 

kevinspl2000 You're welcome , happy coding!

 

I wondered if it was possible to do this but it hasn't come up where I needed to yet so I never inquired into it

 

CaptainBern
Two more questions. What do you mean by intercept and also where would I add all the code? In the Human.class and if is it where all the public void's go? So would I do something like

Code:java

        public void setHealth(Player p) {
            DataWatcher d = new DataWatcher();
            d.a(); // I don't know what to do here? Sorry no I didn't find any tutorials on the net :(
            d.b(); // What in here?
            d.c(); // What in these brackets?
            Packet40EntityMetadata packet40 = new Packet40EntityMetadata(); // What should I type in these brackets? (id, d, (byte) 0) or what? Or what packet would I even send???
                ((CraftPlayer) p).getHandle().playerConnection.sendPacket(packet40);
        }

 

kevinspl2000 You do not need the setHealth method, just create a new variable in the human class that contains the health. Also by intercept I mean you need to detect the incoming packets.

 

CaptainBern
I don't get it :/

 

kevinspl2000 To detect when there is interaction with an npc, you need to intercept all the incoming Packet7UseEntity packets.

kevinspl2000 http://forums.bukkit.org/threads/in...pplied-to-an-npc-created-with-packets.188454/

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

 

CaptainBern
I dont understand a word you are saying ;(. And I dont know what packets to use and how to detect a packet