How to get progress of attack indicator

[Wolfboycoolkid]

I want to get the progress of the attack indicator.

I have a weapon that has a lot of knockback, I want to balance by applying the knockback based on how much progress the attack indicator has.

 

If you don't know what I'm talking about have a look at the video, it's the long sword in the middle.

 

[warjort]

You can see how that is rendered in Gui.renderCrossHair() using Player.getAttackStrengthScale()

[Wolfboycoolkid]

Thank you!

[Wolfboycoolkid]

I'm trying to do this but it always returns 0.0, there's no dynamic change. Increasing it gives weird numbers till a max of 66.6668.

Player player = (Player) entity;

float testVar = player.getAttackStrengthScale(0.0f);

[warjort]

Do you really think this question is in anyway answerable?

It's 2 lines of code without any context.

 

My guess is you have the same or a similar problem to this: https://forums.minecraftforge.net/topic/114943-make-an-enchantment-specifically-for-a-shovel/#comment-509407

[Wolfboycoolkid]

  On 9/3/2022 at 11:25 PM, warjort said:

Do you really think this question is in anyway answerable?

It's 2 lines of code without any context.

 

My guess is you have the same or a similar problem to this: https://forums.minecraftforge.net/topic/114943-make-an-enchantment-specifically-for-a-shovel/#comment-509407

Expand  

Looking at it yep, I do have that problem. Seeing as getAttackStrengthScale resets to 0 every attack I'm not sure how to get it before it resets, how would I do that?

Also if there's any more context I need to give tell me, I'm new to this

My code:

public static void onEntityAttacked(LivingHurtEvent event) {
		if (event != null && event.getEntity() != null && event.getSource().getEntity() != null) {

			// Attacking entity
			LivingEntity entity = (LivingEntity) event.getSource().getEntity();
			// Item in main-hand of entity
			Item item = entity.getMainHandItem().getItem();
			Item amethystBroadsword = SwordtemberModItems.AMETHYST_BROADSWORD.get();
			boolean yeetTarget = false;

			// Is Amethyst Broadsword
			if (item instanceof AmethystBroadswordItem) {
				// If they're a player, wait for the max cooldown to attack
				// Else just attack
				if (entity instanceof Player) {
					Player player = (Player) entity;
					float testVar = player.getAttackStrengthScale(0.0f);
					
					// Nope float cooldownPercent = player.getCooldowns().getCooldownPercent(amethystBroadsword, 0.1f);
					
					// Debug
              	 	if (!entity.level.isClientSide()) {
						MinecraftServer server = ServerLifecycleHooks.getCurrentServer();
						if (server != null) {
							
							server.getPlayerList().broadcastMessage(new TextComponent(Float.toString(testVar)), ChatType.SYSTEM, Util.NIL_UUID);
						}
					}
					
				} else {
					yeetTarget = true;
					
				}
			}

			// Yeet the target
			if (yeetTarget == true) {
				float knockbackAmount = 6.5f;
                double knockbackX = 0 - entity.getLookAngle().x();
                double knockbackZ = 0 - entity.getLookAngle().z();
                event.getEntityLiving().knockback(knockbackAmount, knockbackX, knockbackZ);
			}
		}
	}

[warjort]

Yes, you have the same problem as the other post (assuming you don't have a recent version of forge).

 

This is something that was very recently changed:

https://maven.minecraftforge.net/net/minecraftforge/forge/1.19.2-43.1.15/forge-1.19.2-43.1.15-changelog.txt

- 43.1.6  [1.19.x] Moved Player.resetAttackStrengthTicker() to the end of Player.attack() (#9000)

https://github.com/MinecraftForge/MinecraftForge/pull/9000

 

You can try the latest version of forge to see if your code now works.

 

  Quote

Also if there's any more context I need to give tell me, I'm new to this

Expand  

Being able to answer this question (rather than guessing) required knowing;

* where you are using the method

* which version of forge you are using

These are just part of the https://en.wikipedia.org/wiki/Five_Ws used to give a complete description of a problem.

[Wolfboycoolkid]

  On 9/4/2022 at 3:40 PM, warjort said:

Yes, you have the same problem as the other post (assuming you don't have a recent version of forge).

 

This is something that was very recently changed:

https://maven.minecraftforge.net/net/minecraftforge/forge/1.19.2-43.1.15/forge-1.19.2-43.1.15-changelog.txt

- 43.1.6  [1.19.x] Moved Player.resetAttackStrengthTicker() to the end of Player.attack() (#9000)

https://github.com/MinecraftForge/MinecraftForge/pull/9000

 

You can try the latest version of forge to see if your code now works.

 

Being able to answer this question (rather than guessing) required knowing;

* where you are using the method

* which version of forge you are using

These are just part of the https://en.wikipedia.org/wiki/Five_Ws used to give a complete description of a problem.

Expand  

Alright, I've update my forge to the latest, although from what I can tell the change was done on 1.19, which isn't useful for me as I'm using 1.18.2.

 

Also I'm now using Forge 40.1.76 - 1.18.2

I'm not sure how I can show where I'm using the method precisely without you having to search through the code.

Also for this I'm using an actual IDE (Intellij) so my code has changed slightly, as well I've changed some stuff that I think is better.

package com.shadowdragon.swordtember.events;

import com.shadowdragon.swordtember.advanceditems.AmethystBroadswordItem;
import com.shadowdragon.swordtember.registry.Items;
import net.minecraft.Util;
import net.minecraft.network.chat.ChatType;
import net.minecraft.network.chat.TextComponent;
import net.minecraft.server.MinecraftServer;
import net.minecraft.world.entity.LivingEntity;
import net.minecraft.world.entity.player.Player;
import net.minecraft.world.item.Item;
import net.minecraftforge.event.entity.living.LivingHurtEvent;
import net.minecraftforge.eventbus.api.SubscribeEvent;
import net.minecraftforge.fml.common.Mod;
import net.minecraftforge.server.ServerLifecycleHooks;

@Mod.EventBusSubscriber
public class LivingEntityHurt {
    @SubscribeEvent
    public static void onEntityAttacked(LivingHurtEvent event) {
        if (event != null && event.getEntity() != null && event.getSource().getEntity() != null) {

            // Attacking entity
            LivingEntity entity = (LivingEntity) event.getSource().getEntity();
            // Item in main-hand of entity
            Item item = entity.getMainHandItem().getItem();
            Item amethystBroadsword = Items.amethystBroadsword.get();
            boolean yeetTarget = false;

            // Is Amethyst Broadsword
            if (item instanceof AmethystBroadswordItem) {
                // If they're a player, wait for the max cooldown to attack
                // Else just attack
                if (entity instanceof Player player) {
                    float testVar = player.getAttackStrengthScale(0.0f); // <--- HERE

                    // Nope float cooldownPercent = player.getCooldowns().getCooldownPercent(amethystBroadsword, 0.1f);

                    // Debug
                    if (!entity.level.isClientSide()) {
                        MinecraftServer server = ServerLifecycleHooks.getCurrentServer();
                        if (server != null) {

                            server.getPlayerList().broadcastMessage(new TextComponent(Float.toString(testVar)), ChatType.SYSTEM, Util.NIL_UUID);
                        }
                    }

                } else {
                    yeetTarget = true;
                }
            }

            // Yeet the target
            if (yeetTarget) {
                float knockbackAmount = 6.5f;
                double knockbackX = 0 - entity.getLookAngle().x();
                double knockbackZ = 0 - entity.getLookAngle().z();
                event.getEntityLiving().knockback(knockbackAmount, knockbackX, knockbackZ);
            }
        }
    }
}

[warjort]

The change has not been applied to 1.18.2, it was only added to 1.19.2 yesterday. 🙂 

https://github.com/MinecraftForge/MinecraftForge/commit/2a4230868ffaf5c364cfbea8c66e6c12176bd1c4

If you want to speed up the process, you can submit a PR for 1.18.2.

This change was regarded as a feature so it's not guaranteed to get backported to earlier versions unless somebody steps to do it.

[Wolfboycoolkid]

Alright, thank you!

[Wolfboycoolkid]

After trying to figure this out I'm confused.

I'm pretty confident I know what I have to change & where

But then there's these lines of code that I have a vague idea of what they do but no idea how to reproduce them.

@@ -1200,7 +_,7 @@

Edited September 5, 2022 by Wolfboycoolkid

[warjort]

https://docs.minecraftforge.net/en/latest/forgedev/#getting-started and

https://docs.minecraftforge.net/en/latest/forgedev/prguidelines/

[warjort]

If you are really interested in what that means: https://www.gnu.org/software/diffutils/manual/html_node/Detailed-Unified.html#Detailed-Unified

[Wolfboycoolkid]

  Quote

Build the test classes by selecting the src/main/test directory in your project view and then run Build -> Build module 'Forge_test' from the menu bar.

Expand  

I only see src/main