Jump to content

[1.9.4]Adding custom attribute to EntityPlayer


Zeurkous

Recommended Posts

Hey guys,

 

I'm really new to modding so please go easy on me ;) I'm currently making a mod purely for educational purposes.

 

So my question:

 

I'd like to add a custom attribute (which i already made) to the player. How would I go about this?

Here is the attribute

public static final IAttribute MONEY_ATT = (new RangedAttribute((IAttribute)null, "money", 1000,0,1000000000)).setDescription("Money").setShouldWatch(true);

 

It's really something stupid but i'm trying to learn.

I've found how to add attributes to new entities and modify existing onces but this i can't find.

I hope someone can help me out.

 

Greetings;

Zeurkous

Link to comment
Share on other sites

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.

Link to comment
Share on other sites

Thought so, so this is what I have found. But it gives an error when the world loads.

 

public class CapabilityMoney {

@CapabilityInject(IMoney.class)
public static final Capability<IMoney> MONEY_CAPABILITY = null;

public static final EnumFacing DEFAULT_FACING = null;

public static final ResourceLocation ID = new ResourceLocation(Main.MODID, "Money");

public static void register() {
	CapabilityManager.INSTANCE.register(IMoney.class, new Capability.IStorage<IMoney>() {
		@Override
		public NBTBase writeNBT(Capability<IMoney> capability, IMoney instance, EnumFacing side) {
			return new NBTTagInt(instance.getMoney());
		}

		@Override
		public void readNBT(Capability<IMoney> capability, IMoney instance, EnumFacing side, NBTBase nbt) {
			instance.setMoney(((NBTTagInt) nbt).getInt());
		}
	},() -> new Money(null));

	MinecraftForge.EVENT_BUS.register(new EventHandler());
}

public static IMoney getMoney(EntityPlayer player) {
	return CapabilityUtils.getCapability(player, MONEY_CAPABILITY, DEFAULT_FACING);
}

public static ICapabilityProvider createProvider(IMoney money) {
	return new SimpleCapabilityProvider<>(MONEY_CAPABILITY, DEFAULT_FACING, money);
}


public static class EventHandler {

	@SubscribeEvent
	public void attachCapabilities(AttachCapabilitiesEvent.Entity event) {
		if (event.getEntity() instanceof EntityPlayer) {
			final Money money = new Money((EntityPlayer) event.getEntity());
			event.addCapability(ID, createProvider(money));
		}
	}

	@SubscribeEvent
	public void playerClone(PlayerEvent.Clone event) {
		final IMoney oldMoney = getMoney(event.getOriginal());
		final IMoney newMoney = getMoney(event.getEntityPlayer());

		if (newMoney != null && oldMoney != null) {
			newMoney.setMoney(oldMoney.getMoney());
		}
	}
}

}

 

public interface IMoney {
int getMoney();

void setMoney(int moneyToSet);

void addMoney(int moneyToAdd);
}

 

public class Money implements IMoney{

protected static final UUID MODIFIER_ID = UUID.fromString("d5d0d878-b3c2-469b-ba89-ac01c0635a9c");

protected static final String MODIFIER_NAME = "Money";

protected static final int MIN_AMOUNT = 0;

private final EntityPlayer player;


private int money;

public Money(EntityPlayer player) {
	this.player = player;
}
    public static final IAttribute MONEY_ATT = (new RangedAttribute((IAttribute)null, "money", 1000,0,1000000000)).setDescription("Money").setShouldWatch(true);
    private final IAttributeInstance dummyMoneyAttribute = new AttributeMap().registerAttribute(MONEY_ATT);

@Override
public final int getMoney() {		
	return money;

}

/**
 * Set the bonus max health.
 *
 * @param bonusMaxHealth The bonus max health
 */
@Override
public final void setMoney(int money) {
	this.money = money;
onMoneyChanged();
}

@Override
public final void addMoney(int moneyToAdd) {
	setMoney(getMoney() + moneyToAdd);

}

protected AttributeModifier createModifier() {
	return new AttributeModifier(MODIFIER_ID, MODIFIER_NAME, getMoney(), Constants.ATTRIBUTE_MODIFIER_OPERATION_ADD);
}
protected void onMoneyChanged() {
	if (player == null) return;
	final IAttributeInstance playerMoneyAttribute;
	playerMoneyAttribute = player.getEntityAttribute(MONEY_ATT);

	dummyMoneyAttribute.getModifiers().stream().forEach(dummyMoneyAttribute::removeModifier);

	dummyMoneyAttribute.setBaseValue(playerMoneyAttribute.getBaseValue());
	playerMoneyAttribute.getModifiers().stream().filter(modifier -> !modifier.getID().equals(MODIFIER_ID)).forEach(dummyMoneyAttribute::applyModifier);

	AttributeModifier modifier = createModifier();
	dummyMoneyAttribute.applyModifier(modifier);


	final AttributeModifier oldModifier = playerMoneyAttribute.getModifier(MODIFIER_ID);
	if (oldModifier != null) {
		playerMoneyAttribute.removeModifier(oldModifier);
	} else {
	}

	playerMoneyAttribute.applyModifier(modifier);
}
}

 

This I got from a tutorial from Choonster.

I understand most of it (not the dummy part) but it give as error at playerMoneyAttribute.getBaseValue(); (Class Money.java)

player.getEntityAttribute(MONEY_ATT) return null appaerently so .getBaseValue() gives null pointer.

 

Not sure if I'm doing something wrong or if I'm not doing something I should.

 

Link to comment
Share on other sites

Either create an attribute or capability for your money, you don't need both. My max health capability only exists because I need to manage an attribute modifier for an existing attribute.

 

If you're using an attribute, call

AbstractAttributeMap#registerAttribute

once when an entity first joins the world to create the attribute instance. You can then use

AbstractAttributeMap#getAttributeInstance

to get the attribute instance and set its current value. You don't need to use attribute modifiers.

 

If you're using a capability, just store the money value in the capability instance and sync it to the client when necessary. Don't mess around with attributes.

Please don't PM me to ask for help. Asking your question in a public thread preserves it for people who are having the same problem in the future.

Link to comment
Share on other sites

Ofcourse :) Thanks.

Almost getting it to work as it should.

I now have 2 more problems.

[*]Whenever I rejoin my world the client resets the value to it's base while the server keeps the correct value. How do I fix that? (Packets and Pakcet handling?)

[*]I made a RightClickBlock event but every now and again it decides to fire twice on the same right click (both for client and server). Any ideas on that?

 

 

Link to comment
Share on other sites

Single player is just a server with one player logged in. If you write your code properly, you don't need to care about which type of server is running.

Please don't PM me to ask for help. Asking your question in a public thread preserves it for people who are having the same problem in the future.

Link to comment
Share on other sites

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.



×
×
  • Create New...

Important Information

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