Jump to content

Working with capabilities correctly.


TestingSubject002

Recommended Posts

I can't understand how to correctly work with capabilities. For example, I have a Tile Entity which has IItemHandler Capability (1 slot is ok).

@CapabilityInject(IItemHandler.class)
public static final Capability<IItemHandler> ITEM_HANDLER_CAPABILITY = null;

private IItemHandler inventory =
        ITEM_HANDLER_CAPABILITY.getDefaultInstance(); 
// ... hasCapability, getCapability and other stuff

This code was written using McHorse tutorial, is it correct?

 

If yes, then why are we using getDefaultInstance method of ITEM_HANDLER_CAPABILITY which might be null?

 

Sorry for my English.

Link to comment
Share on other sites

2 hours ago, TestingSubject002 said:

@CapabilityInject(IItemHandler.class) public static final Capability<IItemHandler> ITEM_HANDLER_CAPABILITY = null;

You don't need to get an instance of the capability like this for the IItemHandler since forge provides one for free at CapabilityItemHandler.ITEM_HANDLER_CAPABILITY. You can though, obtaining capability references using CapabilityInject is the way to do this.

 

2 hours ago, TestingSubject002 said:

private IItemHandler inventory = ITEM_HANDLER_CAPABILITY.getDefaultInstance();

Don't do this for inventories. This method is acceptable for capabilities provided by other mods if you just need the default instance but an inventory capability holds a certain amount of slots, which you declare in it's constructor. Just instantinate ItemStackHandler.

 

15 minutes ago, TestingSubject002 said:

So?

This is a forum, not a chat room, please be patient.

  • Thanks 1
Link to comment
Share on other sites

4 minutes ago, V0idWa1k3r said:

You don't need to get an instance of the capability like this for the IItemHandler since forge provides one for free at CapabilityItemHandler.ITEM_HANDLER_CAPABILITY. You can though, obtaining capability references using CapabilityInject is the way to do this.

 

Don't do this for inventories. This method is acceptable for capabilities provided by other mods if you just need the default instance but an inventory capability holds a certain amount of slots, which you declare in it's constructor. Just instantinate ItemStackHandler.

 

This is a forum, not a chat room, please be patient.

Sorry, this won't repeat again.

 

And what about other capabilities? For example, if I create my own capability, would it be safe to call CAPABILITY.getDefaultInstance() this way or should I do this inside of function annotated @CapabilityInject?

Link to comment
Share on other sites

17 minutes ago, TestingSubject002 said:

if I create my own capability, would it be safe to call CAPABILITY.getDefaultInstance() this way or should I do this inside of function annotated @CapabilityInject?

If it's your own capability then you can just instantinate it directly. Use the getDefaultInstance to grab a default instance for a modded capability that you need. In general if you can instantinate it - you probably should. 

I would also recommend reading about capabilities on the official docs, they might answer some of your questions. Feel free to ask if you don't understand something.

  • Thanks 1
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.