Minecraft Forums

Are you certain the ItemStack[] is empty? I mean, maybe the server knows the items but the client doesn't, and the client is the one with the GUI. Add some printouts to be sure.

First, properly learn Java. "Hacking" MCreator to do this will only result in horrible, messy, disastrous code.

Second, always state which version of Minecraft you are modding for. In 1.7.10 and 1.8.x, use IEntityExtendedProperties to save a value to a specific player. In 1.9 and beyond, use the Capabilities system. You do not have to give the player an item to use these systems, so don't worry about that.

Third, post code on Pastebin or Gist and link to it here instead of posting directly on the forums. Make sure it has proper formatting and the .java extension for proper syntax highlighting.

Fourth, learn how to work with Forge before you try this. You will just get confused if you dive into modding too fast. Once you know Java, practice by making basic blocks, items, event handlers, etc. until you know how to navigate the IDE to find what you need.

Quote from jit8»

There is no sideHit in my objectMouseOver :/

ModelLoader.setCustomModelResourceLocation should be called from preInit, not init.

Quote from jit8»

i get what you mean but i don't know how to use it doe :/ like, how can i check it?

Exactly like you would check any equality:

if(objectMouseOver.sideHit == EnumFacing.WEST)

Or, since you want any horizontal to work:

if(objectMouseOver.sideHit != EnumFacing.UP && objectMouseOver.sideHit != EnumFacing.DOWN)

First off, if you insist on using the Minecraft class (mc) put everything inside a statement if(world.isRemote) or something similar. You should be able to access the working world object without the Minecraft class, or it kind of defeats the purpose.

Second, I think the objectMouseOver is a RayTrace or MovingObjectPosition object, correct? Just check that objectMouseOver.sideHit is equal to EnumFacing.WEST

To have your code run if it is any horizontal side, make sure it is not equal to EnumFacing.UP or EnumFacing.DOWN.

Or, check that EnumFacing.HORIZONTAL contains the sideHit.

Quote from Alpvax»

There is actually an EnumSet EnumFacing.HORIZONTAL, just check that the hit side is in that set.

Your code will not work on a server, as you are using the Minecraft class, which only exists on the client. That won't be a problem if you just want it to affect rendering or show a message in the player's chat.

Look at the code that calls Block.onRightClick (I don't think that is the correct method name, but you should be able to work out which method I mean).

See where the EnumFacing parameter comes from.

Add the "particle" tag with your texture

{
    "parent": "block/cube_all",
    "textures": {
        "all": "minecraft:blocks/dirt",
        "particle": "minecraft:blocks/dirt"
    }
}

I have a tutorial on Complex Structure Generation that may be useful depending on the size of your structure. It requires an array of "relative" locations that are passed into a helper method -- that helper method makes each BlockPos for you.

Either way, you will end up creating a lot of temporary BlockPos instances... there's not really a way to avoid that, so don't worry too much.

Each chunk can only handle around 1000 block updates per second, so if you set more than a hundred blocks you should consider using a Timer. That is, generate segments of your structure every few hundred milliseconds, which gives the game time to update the client.

Quote from Melonslise»

What do you mean field boundingbox is public? What field? How do I fix this?

I think you can use player.boundingBox . If it complains about "not visible" then boundingBox is not public, but I think it will work.

Quote from sky_01»

My impression was that for each entity, it fires a
RenderLivingEvent.Pre, then a RenderLivingEvent.Post, then moves on to
the next Entity. What's going on?

Well yes, that is what happen, and what you see in your log.
I guess you didn't realize rendering is really fast. Many many times a second.
And when an entity doesn't render, it doesn't reach RenderLivingEvent.Pre nor RenderLivingEvent.Post events once.
You should only register client events through your client proxy.
Also, since only one entity is rendered at a time you didn't need the list.
Finally, you could simply do this in your custom entity renderer. That will do the same, without requiring the events.

The effect you wanted could be done roughly like this:

//In you entity class
private boolean isRendered;//Server-tick accuracy
private int timeSinceLastRender;

public void onUpdate(){
//Some other code here
if(timeSinceLastRender > 0)
setRendered(false);
timeSinceLastRender++;
}

//Call setRendered(true) in your rendering method/event
public void setRendered(boolean render){
if(render)
timeSinceLastRender = 0;
isRendered = render;
}