Entity (CraftItem) Help

hey guys, I'm trying to get the entities around an entity. although, this method wont work

PHP:

<entity>.getNearbyEntity(5,5,5).contains(264);

. Help please.

 

you are checking it for an int?

This will give of a list of the entitys, log it to see ^^

For example, i use this in a plugin;

Code:java

Entity chicken = block.getWorld().spawnCreature(
block.getLocation(), EntityType.CHICKEN);
 
 
int limit = 0;
while (chicken == null && limit < 5){
chicken = block.getWorld().spawnCreature(
block.getLocation(), EntityType.CHICKEN);
limit++;
}
 
 
List<Entity> list = chicken.getNearbyEntities(
radius, radius, radius);
chicken.remove();
 
int test01 = 0;
int test02 = list.size();
 
while (test02 > test01) {
 
Entity entity = list.get(test01);
if (entity instanceof Player) {
((Player) entity).getPlayer().sendMessage(
ChatColor.GOLD
+ "5 minutes left! - " + ChatColor.RED + "Max!");
 
}
 
test01++;
}

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

 

the methode is working exactly as it designed to do, return true if the specified object is in the list, and because there are no integers at the list, it return false

 

.getNearbyEntities(5,5,5);
returns a List<Entity> all the entities around the player, 5 blocks in every direction (up, down, left, right, forward and backward)
so, it wont help to check if it contains an integer

 

// Edit... Ok, i logged it, and it returns CraftItems.. How can i cast them to a item such as diamond?

ferrybig stelar7

 

.getNearbyEntity(5,5,5)
actually returns an Collection<Entity> you can see if an Entity is an item whit instanceof , and call the getItem() on it for an item stack, on an itemStack, you can call getType, to see if its diamond

 

Example please. i don't understand.

 

Code:

List<Entity> entities = player.getNearbyEntities(5,5,5); // Get entities in a 5x5x5 box size around the player

for(Entity ent : entities) // loop through each entitiy
{
    if(ent instanceof CraftItem) // if is that Entity an instance of CraftItem (which extends Entity)
    {
        CraftItem itemEnt = (CraftItem)ent; // cast Entity to CraftItem

        // now see what methods are available, it should provide something to get the type ID or Material for that dropped item.
    }
}

 

Alright, What I'm trying to achieve..

• Listen for 2 diamond being thrown
• Listen for a stick being thrown

If Both requirements are met, remove them both, and Spawn a Diamond sword, I've done alot of messing about with things, and none of them work, Can someone please set the code up for me? For future reference too (Notes if possible, I should be able to figure it out without them, but still)

Digi
V10lator
ferrybig
stelar7

I've tagged you because you seem to have some knowledge on this. Sorry if you are not interested.

If there is anyone else that is interested, Please go ahead and do this.

Thankyou again, Husky.

 

Items are entities?

 

I see some bad points at this, if you want to craft an iron sword, you need to thhrow the stick first, else you make shears. basicly, you need an list of the recipes, and try to match it whit the items laying on the ground

 

It's Just going to be for swords, armour, tools. Nothing else.. ;D

 

you need to have an set/list/array for al the recipes, when there is an item ropped, you need to check if you can make an recipe whit those items (loop them al) and make the recipe
look out whit armor, they have the same base materials, only at diferent amounts

 

Would you mind doing this? I dont know how this would operate

Like, i can do armour, no problem. But pickaxes and other tools would be an issue..

 

Actually armor could be a problem, because you'll have to drop exacly the amount of ingots it requires to craft that armor, if you drop 1 at a time it will craft boots... unless you make a command or something to trigger the craft manually.

And yes, pickaxe and axe is a problem because you need a shape... I have no ideea how to diferentiate between those... you could change the recipe to use 3 sticks and 2 iron.

Code:

S I
S I
S

looks like an axe to me but I get that you don't need it in the workbench, this is just to vizualize that it could be changed to that.

Still, what are you mainly trying to achieve ?

 

Well, i am making a plugin, theres classes, and one is blacksmith, so you need to make an anvil, (all this code is done AND works)
Then you need to throw materials onto the "anvil" to craft forged items (pickaxe, armour)

Could you make the plugin wait 10 seconds before performing anything? (crafting anything)

That could also add a realistic touch to the plugin, because it needs time to form,

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

 

Waiting can be done with a scheduler.

If you want really realistic blacksmithing, I suggest you search on youtube how smelting is done in the Dark Messiah and Gothic 1 games.

 

Yeah, I am using flint to scratch the sword sharper, but can't get my head around this, or I extremely over think :/

Are you able to do something like this for me (just code, not compiled)


Edit - if you want to change recipes (and your willing to do this) go ahead.

 

I don't really know your plan for the crafting steps so I can't really help with any code... but you really should try to do it yourself first and then post if you have any difficulties :}

 

I've tried coding it a few times, I will give it One last hoorah, then post my code, thanks for your help Digi

Well, I need help, I've come to a halt, I need to get the entities Around the one entity that was just thrown... which is why i think i was stuck, i tried to get the amount of something in an itemstack, but it just returns '1', Even when i throw 60. :/ Help Digi?

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

 

I have no clue where you tried that and how you did it, so can you post the code ?

 

Code:java

public void onSmithCraft(PlayerDropItemEvent e) {
Player p = e.getPlayer();
List<Entity> nearby = p.getNearbyEntities(5,5,5);
Location l = p.getLocation();
if(api.getJob(p) == "Blacksmith") {
CraftItem c = null;
ItemStack i = null;
for(Entity ent : nearby) // loop through each entitiy
{
   if(ent instanceof CraftItem) // if is that Entity an instance of CraftItem (which extends Entity)
   {
      c = (CraftItem)ent;
       i = c.getItemStack();
 
   }
}
//
int a = i.getAmount();
if(a == 8) {
if(i.getType() == Material.IRON_INGOT) {
i.setAmount(0);
c.remove();
p.getWorld().dropItem(l, new ItemStack(307,1));
p.sendMessage(ChatColor.GREEN + "[hSkyrim] You have just crafted an iron chestplate.");
} else if(i.getType() == Material.GOLD_INGOT) {
 
} else if(i.getType() == Material.DIAMOND) {
 
}
}
}
}

 

You're not breaking the loop when you find an item, so the i can be re-asigned to an entity that was detected last but not necesarily dropped last.
Also, indent your code as it will be alot easier to read by everybody, including yourself. In Eclipse you can use Ctrl+i to do it for you.

 

Right i got some code that is working, although there is 1 bug.

When you throw an item, it then registers that. after that. it will pickup whatever you threw.

If you just throw the 8 iron (grouped) straight out, it wont work. here's the code. (indented)

Code:java

@EventHandler
public void onSmithCraft(PlayerDropItemEvent e) {
Player p = e.getPlayer();
List<Entity> nearby = p.getNearbyEntities(5,5,5);
Location l = p.getLocation();
if(api.getJob(p) == "Blacksmith") {
CraftItem c = null;
ItemStack i = null;
for(Entity ent : nearby) // loop through each entitiy
{
if(ent instanceof CraftItem) // if is that Entity an instance of CraftItem (which extends Entity)
{
c = (CraftItem)ent;
i = c.getItemStack();
if(i.getAmount() == 8) {
if(i.getTypeId() == 265) {
i.setAmount(0);
c.remove();
p.getWorld().dropItem(l, new ItemStack(307,1));
p.sendMessage(ChatColor.GREEN + "[hSkyrim] You have just crafted an iron chestplate.");
} else if(i.getType() == Material.GOLD_INGOT) {
 
} else if(i.getType() == Material.DIAMOND) {
 
}
}
break;
}
}
}
}

hah, Never mind, i will use an itemstack to check if it has the amount... it works this time,

Code:

Code:java

@EventHandler
public void onSmithCraft(PlayerDropItemEvent e) {
Player p = e.getPlayer();
Location l = p.getLocation();
if(api.getJob(p) == "Blacksmith") {
ItemStack i = e.getItemDrop().getItemStack();
int amount = 0;
i = e.getItemDrop().getItemStack();
amount = i.getAmount();
if(amount == 8) {
if(i.getTypeId() == 265) {
i.setAmount(0);
e.getItemDrop().remove();
p.getWorld().dropItem(l, new ItemStack(307,1));
p.sendMessage(ChatColor.GREEN + "[hSkyrim] You have just crafted an iron chestplate.");
} else if(i.getType() == Material.GOLD_INGOT) {
if(amount == 8) {
i.setAmount(0);
e.getItemDrop().remove();
p.getWorld().dropItem(l, new ItemStack(Material.GOLD_CHESTPLATE,1));
p.sendMessage(ChatColor.GREEN + "[hSkyrim] You have just crafted an gold chestplate.");
}
} else if(i.getType() == Material.DIAMOND) {
 
}
}
}
}

Can you do something similar for Say a Pickaxe Digi? I don't understand that.

// Edit - got it, thanks anyway Digi!

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

 

Indent your code already, code must be easily read by everybody, it shouldn't be stacked tightly, spacing is of no importance between expressions, use as much spacing as you need to make it easily readable !
And when you finally use Ctrl+i to make eclipse indent it for you, you'll be able to read it much easier.

Also, you don't understand your own code ? Apart from the inexistent indentation which makes it a pain to read xD

 

It's cool Digi, I got it. <3 Anyway, I did CTRL + I and it indented it, In eclipse then i paste it here and bam. :/

 

And bam what ? That's the result, unindented ? Use pastebin then
Or "bam I got it" as in you understand the code better ? :}

 

both.. ;3

 

if you use the [ syntax] tag, the formatting dissapears

 

I must say I had never problems whit the [ syntax] tag and line indents lost, as showing this example:

Code:YAML

 
commands:
    test:
        usage: /<command>