Jump to content

[Solved] Increase the use time of a tool.


CommandCore

Recommended Posts

Hey all, I'm trying to find a way to decrease the interval in which onItemUse() is fired on an item. For my application, I'm trying to increase the use time of my overwritten vanilla shears on a certain block that yields drops after you shear it. I want it to shear as fast as someone would punch out a bunch of tall grass.

 

Here's my code in attempt to set the swinging time to zero when detecting the block being right clicked in the RightClickBlock event, sadly it doesn't work:

@SubscribeEvent
    public void farmBlock(PlayerInteractEvent.RightClickBlock event)
    {
        if(event.getPlayer().getHeldItem(event.getHand()).getItem().equals(Items.SHEARS) && event.getWorld().getBlockState(event.getPos()).equals(RegistryHandler.MY_BLOCK.get().getDefaultState())){
            System.out.println("My block was right clicked!"); //Testing purposes.
            event.getPlayer().swingProgressInt = 1;
        }
    }

 

I also have code in the onItemUse() function in my new vanilla shears class that allows the player to shear my block (turns it into another block) much like using an axe to right click will shave wood. Everything works the way that it should, I just want to decrease the time in which the item is used. :)

 

Like said before, I can change the shears however I want, as long as this works. Any pointers?

 

Thanks a ton!

Edited by CommandCore
Issue Still Exists.
Link to comment
Share on other sites

1 hour ago, diesieben07 said:

I think this is Minecraft#rightClickDelayTimer. You can subscribe to TickEvent.ClientTickEvent, check for Phase.START and set it to 0 to forcibly disable this cooldown.

Okay, so I'll set Minecraft#rightClickDelayTimer to 0 through reflection, but won't this set the variable globally? Because TickEvent.ClientTickEvent doesn't have a "player" variable or "getPlayer()" method. 

Link to comment
Share on other sites

2 minutes ago, diesieben07 said:

Minecraft#player is the player that you are "controlling".

Okay cool, I did the reflection straight in the PlayerInteractEvent.RightClickBlock method though, and everything's working A-Okay. Thanks for the help man. Here's my code for anyone that views this thread:

 

Actual Method:

@SubscribeEvent
    public void farmBlock(PlayerInteractEvent.RightClickBlock event) throws IllegalAccessException, NoSuchFieldException, ClassNotFoundException {
        if(event.getPlayer().getHeldItem(event.getHand()).getItem().equals(Items.SHEARS) && event.getWorld().getBlockState(event.getPos()).equals(RegistryHandler.MY_BLOCK.get().getDefaultState())){
            JavaStuff.setInstanceValue(Minecraft.getInstance(), "rightClickDelayTimer", 0);
        }
    }

 

And since I see a lot of people that don't know how to do reflection and need it done, here you go. Click the link in the comment for more ways to do it, like with static variables and stuff. Chances are you'll only be doing it with non-static variables though, since literally Minecraft itself is an instance in this situation lol:

public static void setInstanceValue(final Object classInstance, final String fieldName, final Object newValue) throws SecurityException,
            NoSuchFieldException, ClassNotFoundException, IllegalArgumentException, IllegalAccessException {
        // Get the private field
        final Field field = classInstance.getClass().getDeclaredField(fieldName);
        // Allow modification on the field
        field.setAccessible(true);
        // Sets the field to the new value for this instance
        field.set(classInstance, newValue);

    }

    // Via https://blog.sevagas.com/Modify-any-Java-class-field-using-reflection
    //Thanks Sevagas!

 

Link to comment
Share on other sites

31 minutes ago, diesieben07 said:

You cannot do that. That is a server-side event.

 

Do not get the Field instance every time, get it once and store it in a static final field. Then also note that Minecraft fields will be obfuscated outside the development environment, so you need to use ObfuscationReflectionHelper and give it the SRG name instead of plain reflection.

 

I see. Okay, so I did this, and it works, but am I doing it wrong still?:

 

private static boolean brokeMyBlock = false;

    @SubscribeEvent
    public void onClickTickEvent(TickEvent.ClientTickEvent event){

        PlayerEntity player = Minecraft.getInstance().player;

        if(brokeMyBlock){
            System.out.println("Sheared Block");
            ObfuscationReflectionHelper.setPrivateValue(Minecraft.class, Minecraft.getInstance(), 0, "rightClickDelayTimer");
            brokeMyBlock = false;
        }
    }

    @SubscribeEvent
    public void farmMyBlock(PlayerInteractEvent.RightClickBlock event) {
        if(event.getPlayer() == Minecraft.getInstance().player){
            if(event.getPlayer().getHeldItem(event.getHand()).getItem().equals(Items.SHEARS) && event.getWorld().getBlockState(event.getPos()).equals(RegistryHandler.MY_BLOCK.get().getDefaultState())){
                brokeMyBlock = true;
            }
        }
    }

 

32 minutes ago, diesieben07 said:

Do not get the Field instance every time, get it once and store it in a static final field.

Couldn't see how to fit this in, because from what I saw, ObfuscationReflectionHelper didn't have a variation that uses a Field.

 

But I did put some code for it, don't know how to put the puzzles pieces together though with what I did above:

private static final Field field = rightClickDelayTimerField();

    private static Field rightClickDelayTimerField() {
        try {
            return Minecraft.getInstance().getClass().getDeclaredField("rightClickDelayTimer");
        } catch (NoSuchFieldException e) {
            return null;
        }
    }

 

Link to comment
Share on other sites

1 hour ago, diesieben07 said:

You cannot do this. The player can and will change.

 

You cannot do this. You are reaching across logical sides here.

 

ObfuscationReflectionHelper.findField

 

I believe this new version covers everything you said. I used an attached player capability for the boolean.

private static final Field field = ObfuscationReflectionHelper.findField(Minecraft.class, "rightClickDelayTimer");

    @SubscribeEvent
    public void onClientTickEvent(TickEvent.ClientTickEvent event) throws IllegalAccessException {

        if(event.phase == TickEvent.Phase.START)
        {
            try {

                if(CustomCapabilities.getPlayer(Minecraft.getInstance().player).getVariable("string", "HasBrokenMyBlock").equals("true")){
                    field.setInt(Minecraft.getInstance(), 0);
                    CustomCapabilities.getPlayer(Minecraft.getInstance().player).setVariable("HasBrokenMyBlock", "false");
                }

            } catch (NullPointerException e) {}
        }
    }

    @SubscribeEvent
    public void farmBlock(PlayerInteractEvent.RightClickBlock event) {
        if(event.getPlayer() == Minecraft.getInstance().player){
            if(event.getPlayer().getHeldItem(event.getHand()).getItem().equals(Items.SHEARS) && event.getWorld().getBlockState(event.getPos()).equals(RegistryHandler.MY_BLOCK.get().getDefaultState())){
                CustomCapabilities.getPlayer(event.getPlayer()).setVariable("HasBrokenMyBlock", "true");
            }
        }
    }

 

Everthing works, but a server instance isn't running because it doesn't like the ClientTickEvent: "Attempted to load class net/minecraft/client/entity/player/ClientPlayerEntity for invalid dist DEDICATED_SERVER"

 

Tried to use regular TickEvent and it gave me the same error. :/

Link to comment
Share on other sites

36 minutes ago, diesieben07 said:

You need to use the SRG name, this won't work outside the development environment.

 

What...? What... Just... what is this?

 

This will crash on a server, as the server has no idea about the Minecraft class.

 

Client-only events must be in a separate @EventBusSubscriber class with Dist.CLIENT passed to @EventBusSubscriber.

Alright, put this in a class that's only for client events.

 

private static final Field field = ObfuscationReflectionHelper.findField(Minecraft.class, "field_71467_ac");

    @SubscribeEvent

    public static void onClientTickEvent(TickEvent.ClientTickEvent event) throws IllegalAccessException {

        if(event.phase == TickEvent.Phase.START)
        {
            try {

                if(CustomCapabilities.getPlayer(Minecraft.getInstance().player).getVariable("string", "HasBrokenMyBlock").equals("true")){
                    field.setInt(Minecraft.getInstance(), 0);
                    CustomCapabilities.getPlayer(Minecraft.getInstance().player).setVariable("HasBrokenMyBlock", "false");
                }

            } catch (NullPointerException e) {}
        }
    }

 

I removed

38 minutes ago, diesieben07 said:

if(event.getPlayer() == Minecraft.getInstance().player){

here is the new code that stayed in that class:

 

@SubscribeEvent
    public void farmBlock(PlayerInteractEvent.RightClickBlock event) {
        if(event.getPlayer().getHeldItem(event.getHand()).getItem().equals(Items.SHEARS) && event.getWorld().getBlockState(event.getPos()).equals(RegistryHandler.MY_BLOCK.get().getDefaultState())){
            CustomCapabilities.getPlayer(event.getPlayer()).setVariable("HasBrokenMyBlock", "true");
        }
    }

 

And

39 minutes ago, diesieben07 said:

CustomCapabilities.getPlayer(Minecraft.getInstance().player).getVariable("string", "HasBrokenMyBlock").equals("true")

is the custom capability system of mine that I was talking about. You can think of it as setting a value on a capability that is attached to PlayerEntitys. The real variable-setting stuff is buried under a bunch of static methods for convenience.

 

This seems to be the only way to put a boolean value on my player that really just functions to link the functions of the RightClickBlock event to the ClientTickEvent. I suppose if there was a better way to link them, I wouldn't have to do this.

 

Everything works, even when the client logged into the server.

Link to comment
Share on other sites

Got it. Moved it to the client event class. Everything seems to be working.

 

5 minutes ago, diesieben07 said:

You don't need this "capability" stuff... If you do need to store data on players, use actual Forge capabilities.

It is a Forge capability, with an interface, class, provider, and storage, along with a AttachCapabilitiesEvent<Entity> called in a class to attach it to PlayerEntity's. Sorry if that wasn't clear in my last post. Unless you're talking about a capability that is already in Forge?

Link to comment
Share on other sites

Just now, diesieben07 said:

Okay so it is a normal capability. Then why are you not just making normal fields in it that you put values in?

Because I wanted the ability to add custom values when setting them. For example, I wouldn't have to go into the capability and add a new variable, I can just say "Set the variable 'Working' to 'true'."

And if its trying to get a variable that hasn't been set yet, it'll return 0 or "" depending on what kind of variable I set it as, (Notice the "string" argument.)

 

Its sort of mimicking the stuff you can do with getPersistentData() on an entity. TLDR; Convenience. ¯\_(ツ)_/¯

Link to comment
Share on other sites

2 minutes ago, diesieben07 said:

So you are bypassing the nice typesystem that Java gives you, for... convenience.

This is a pretty bad antipattern.

 

https://wiki.c2.com/?StringlyTyped

Ah, I see. So unnecessary casting and assigning of variables via strings is the culprit of this bad practice? Does this contribute to inefficiency? From that site you posted, that seems to be the case.

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.

Announcements



×
×
  • Create New...

Important Information

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