Jump to content

Recommended Posts

Posted

I'm curious about how this works.

 

I am "monitoring" when mobs dies, and I want to add drops to their list. This is what I'm doing, the "problem" is that I'm not sure, will this guarantee the drop 100% the time? Is there a way to pass a % chance of dropping the added item.

 

PS: Would it be ok to spawn the item on the world, so I can add a chance % there?

 

	@SubscribeEvent
public void onLivingDropsEvent(LivingDropsEvent event) {

	Entity entity = event.getEntity();
	World world = event.getEntity().getEntityWorld();
	BlockPos pos = event.getEntity().getPosition();

	if(entity.getClass().equals(MobsReference.VANILLA_ZOMBIE)){

		EntityItem itemDropX = new EntityItem(world, pos.getX()+.5, pos.getY()+.5, pos.getZ()+.5, ItemsReference.IRON_ORE.copy());

//			world.spawnEntityInWorld(itemDropX);

		event.getDrops().add(itemDropX);

	}

}

Posted

Well, if you add drop to drops - it will be dropped (unless someone would catch event after you and remove drops), simple as that.

The % chance can be done by you (Random) inside event. (e.g: 50% to add some item to drop list).

1.7.10 is no longer supported by forge, you are on your own.

Posted

if(entity.getClass().equals(MobsReference.VANILLA_ZOMBIE)){

Oh god why? Please use
entity instanceof EntityZombie

- it will be more performant and also not exclude mod-mobs that are zombies.

 

I did that so I can also reference Mobs from other mods in the near future.

 

Is there a better way? That's what worked for me on a 1.7.10 Mod I "did" (Continued the updates) time ago.

 

Example: If I use instanceof I wont be able to reference "KingSlime" A mob from the Aether mod, or "demonmobs.rahovart" a Boss Mob from Lycanites Mobs.

Posted

Entity entity = event.getEntity();

World world = event.getEntity().getEntityWorld();

 

"Eh, just fuck that reference we made. Call that method again! "

  • Like 1

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.

 

Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.

 

DO NOT PM ME WITH PROBLEMS. No help will be given.

Posted

Entity entity = event.getEntity();

World world = event.getEntity().getEntityWorld();

 

"Eh, just fuck that reference we made. Call that method again! "

 

Whut???

 

Entity entity = event.getEntity();

World world = entity .getEntityWorld();

 

I see that, would it really matter?

 

What reference you talking about?

Posted

Sure you would?

So I don't know how to use instanceof in this case... Been:

 

MobsReference

public static final Class VANILLA_ZOMBIE = (Class) EntityList.NAME_TO_CLASS.get("Zombie");

 

How would I go about getting the entity to see if it's an instanceof the entity on the die event later on?

 

And why would it be better to use instanceof? I lose control as you said, and any mod that extends a Minecraft Zombie will also drop my items, using the equals gives me control to set a different drop for that mod mob.

Posted

So I can reference the mobs with the names on the registry, even if I don't know the class they belong to. At least I think that's why it was done like that on the old 1.7.10 mod I told you.

Posted

Entity entity = event.getEntity();

World world = event.getEntity().getEntityWorld();

 

"Eh, just fuck that reference we made. Call that method again! "

 

Whut???

 

Entity entity = event.getEntity();

World world = entity .getEntityWorld();

 

I see that, would it really matter?

 

What reference you talking about?

 

Entity entity

This is a reference.

 

If you don't use it, you're wasting space in the JVM stack.  By calling

getEntity()

over and over again, you're wasting execution speed on bytes that are irrelevant (because the result is already stored in a local variable).

 

Try to pretend that

getEntity()

takes 2 days to execute.  You don't mind your program taking a day or two to execute, but a week would be too long.

 

How many times would you call

getEntity()

then?

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.

 

Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.

 

DO NOT PM ME WITH PROBLEMS. No help will be given.

Posted

Entity entity = event.getEntity();

World world = event.getEntity().getEntityWorld();

 

"Eh, just fuck that reference we made. Call that method again! "

 

Whut???

 

Entity entity = event.getEntity();

World world = entity .getEntityWorld();

 

I see that, would it really matter?

 

What reference you talking about?

 

Entity entity

This is a reference.

 

If you don't use it, you're wasting space in the JVM stack.  By calling

getEntity()

over and over again, you're wasting execution speed on bytes that are irrelevant (because the result is already stored in a local variable).

 

Try to pretend that

getEntity()

takes 2 days to execute.  You don't mind your program taking a day or two to execute, but a week would be too long.

 

How many times would you call

getEntity()

then?

 

So the best practice would be:

 

if(event.getEntity().getClass() == MobsReference.VANILLA_ZOMBIE)

 

and...

 

doDrop(event, event.getEntity().getEntityWorld(), itemDropX, ChancesControl.ZOMBIE_BY_HUMAN, ChancesControl.ZOMBIE_BY_OTHER, killedByPlayer);

 

And not create "entity" and "world"?

Posted

And not create "entity" and "world"?

 

Jesus fucking christ.  The opposite of this.

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.

 

Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.

 

DO NOT PM ME WITH PROBLEMS. No help will be given.

Posted

And not create "entity" and "world"?

 

Jesus fucking christ.  The opposite of this.

Now I'm just more confuse, I really don't understand what you pointing at, and what you think I'm doing wrong / should do different.

 

 

Posted

Step 1: make a local variable (you skipped this in your last post)

Step 2: fucking use it (you skipped this in your first post)

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.

 

Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.

 

DO NOT PM ME WITH PROBLEMS. No help will be given.

Posted

Step 1: make a local variable (you skipped this in your last post)

Step 2: fucking use it (you skipped this in your first post)

That's why I'm confuse, I did used the variable on my first post.

 

Entity entity = event.getEntity();
	World world = event.getEntity().getEntityWorld();
	BlockPos pos = event.getEntity().getPosition();

 

I did used entity, world and pos. None of them are unused.

 

If you mean that what I should have created is a Class class = event.getEntity().getClass(); That's something I did missed.

Posted

How about

entity.getEntityWorld()

so your actually using the local variable instead of calling getEntity again.  :|

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.

 

Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.

 

DO NOT PM ME WITH PROBLEMS. No help will be given.

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Unfortunately, your content contains terms that we do not allow. Please edit your content to remove the highlighted words below.
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Announcements



×
×
  • Create New...

Important Information

By using this site, you agree to our Terms of Use.