Merge pull request #3 from N3ROO/MC_1.16.2_dev

Mc 1.16.2 dev

Author: lilgallon

.github/images/1_2_0_before_update.gif

@@ -1,3 +1,23 @@
+## Version 1.2.0
+
+- Smart block aim assistance
+
+**BEFORE:**
+![before](.github/images/1_2_0_before_update.gif)
+
+**AFTER:**
+
+It will recognize the block that you're trying to break.
+![smart aim](.github/images/1_2_0_smart_aim.gif)
+
+But, if you want to change the block that you're breaking, just move your mouse.
+![smart aim](.github/images/1_2_0_smart_aim_captions.gif)
+
+- Improved attack assistance (it puts your crosshair higher than before)
+- Increased attack assistance duration (1 sec -> 1.7 sec)
+- The assistance won't be running while you're in GUIs
+
+
 ## Version 1.1.0
 
 - Built for forge 1.16.2 (also works with forge 1.16.1)

.github/images/1_2_0_smart_aim.gif

@@ -13,7 +13,7 @@ apply plugin: 'net.minecraftforge.gradle'
 apply plugin: 'eclipse'
 apply plugin: 'maven-publish'
 
-version = '1.1.0-MC1.16.2'
+version = '1.2.0-MC1.16.2'
 group = 'dev.nero.aimassistance' // http://maven.apache.org/guides/mini/guide-naming-conventions.html
 archivesBaseName = 'aimassistance'
 
@@ -34,7 +34,7 @@ minecraft {
             property 'forge.logging.console.level', 'debug'
 
             mods {
-                examplemod {
+                aimassistancemod {
                     source sourceSets.main
                 }
             }
@@ -43,5 +43,5 @@ minecraft {
 }
 
 dependencies {
-    minecraft 'net.minecraftforge:forge:1.16.2-33.0.21'
+    minecraft 'net.minecraftforge:forge:1.16.2-33.0.37'
 }

.github/images/1_2_0_smart_aim_captions.gif

@@ -1,7 +1,8 @@
 package dev.nero.aimassistance;
 
 import dev.nero.aimassistance.config.Config;
-import dev.nero.aimassistance.module.AimAssistance;
+import dev.nero.aimassistance.core.AimAssistance;
+import dev.nero.aimassistance.utils.MouseUtils;
 import dev.nero.aimassistance.utils.Wrapper;
 import net.minecraftforge.common.MinecraftForge;
 import net.minecraftforge.event.TickEvent;
@@ -55,6 +56,7 @@ public void onPlayerTick(TickEvent.PlayerTickEvent playerTickEvent) {
     @SubscribeEvent
     public void onClientTick(TickEvent.ClientTickEvent clientTickEvent) {
         if (Wrapper.playerPlaying()) {
+            MouseUtils.checkForMouseMove();
             aimAssistance.analyseEnvironment();
         }
     }

CHANGELOG.md

@@ -1,6 +1,7 @@
-package dev.nero.aimassistance.module;
+package dev.nero.aimassistance.core;
 
 import dev.nero.aimassistance.config.Config;
+import dev.nero.aimassistance.utils.MouseUtils;
 import dev.nero.aimassistance.utils.TimeHelper;
 import dev.nero.aimassistance.utils.Wrapper;
 import net.minecraft.entity.Entity;
@@ -33,7 +34,7 @@ public class AimAssistance {
     private final float INTERACTION_ATTACK_SPEED = 1f / 1000f; // (attacks per ms) user faster means user attacking
     private final int INTERACTION_ATTACK_DURATION = 2000; // (ms) duration after which we give up
     private final int INTERACTION_MINING_DURATION = 500; // (ms) duration the player needs to be mining to assist
-    private final int ASSISTANCE_ATTACK_DURATION = 1000; // (ms) duration during which the assistance will assist on mobs
+    private final int ASSISTANCE_ATTACK_DURATION = 1700; // (ms) duration during which the assistance will assist on mobs
     private final int ASSISTANCE_MINING_DURATION = 300; // (ms) duration during which the assistance will assist on blocks
     private final float RANGE_TO_SCAN = 5; // (blocks) range to scan from the player to find entities
     private final Class<? extends Entity> ENTITY_TYPE_TO_SCAN = MobEntity.class; // defines the type of entity to scan
@@ -109,6 +110,8 @@ else if (!this.miningTimer.isDelayComplete(this.INTERACTION_MINING_DURATION) &&
         // Else (means that the player is mining) if the player has been mining for the given duration, then they're
         // interacting
         else if (this.miningTimer.isDelayComplete(this.INTERACTION_MINING_DURATION) && Wrapper.attackKeyPressed()) {
+            this.attackTimer.stop();
+
             this.miningTimer.stop();
             this.interactionTimer.start(); // it will reset if already started, so we're all good
             this.interactingWith = TargetType.BLOCK;
@@ -141,6 +144,8 @@ else if (this.attackCount > 0 && playerAttacks) {
             // If player's attack speed is greater than the speed given to toggle the assistance, then we can tell to
             // the instance that the player is interacting
             if (speed > this.INTERACTION_ATTACK_SPEED) {
+                this.miningTimer.stop();
+
                 // We need to reset the variables that are used to define if the player is interacting because we know
                 // that the user is interacting right now
                 this.attackCount = 0;
@@ -191,7 +196,33 @@ public void assistIfPossible() {
                     aimForce, aimForce // forceX, forceY
             );
 
-            Wrapper.setRotations(rotations[0], rotations[1]);
+            boolean assist = true;
+
+            // We need to prevent focusing an other block while assisting if the player is not moving his mouse
+            if (this.interactingWith == TargetType.BLOCK && !MouseUtils.mouseMoved()) {
+                BlockRayTraceResult nextBlock = Wrapper.rayTrace(
+                        this.BLOCK_REACH, Wrapper.MC.player.getEyePosition(1.0F), rotations[0], rotations[1]
+                );
+
+                // If, after moving the mouse, an other block is focused, then don't assist
+                if (nextBlock != null && this.target.getTarget() != null) {
+                    if (this.target.getTarget() instanceof BlockRayTraceResult) {
+
+                        BlockPos next = nextBlock.getPos();
+                        BlockPos curr = ((BlockRayTraceResult) this.target.getTarget()).getPos();
+
+                        assist = (
+                                next.getX() == curr.getX() &&
+                                        next.getY() == curr.getY() &&
+                                        next.getZ() == curr.getZ()
+                        );
+                    } else {
+                        assist = false;
+                    }
+                }
+            }
+
+            if (assist) Wrapper.setRotations(rotations[0], rotations[1]);
         }
     }
 }

build.gradle

@@ -1,4 +1,4 @@
-package dev.nero.aimassistance.module;
+package dev.nero.aimassistance.core;
 
 import net.minecraft.entity.Entity;
 import net.minecraft.util.Direction;

src/main/java/dev/nero/aimassistance/AimAssistanceMod.java

@@ -1,4 +1,4 @@
-package dev.nero.aimassistance.module;
+package dev.nero.aimassistance.core;
 
 public enum TargetType {
     ENTITY, BLOCK, NONE

src/main/java/dev/nero/aimassistance/module/AimAssistance.java renamed to src/main/java/dev/nero/aimassistance/core/AimAssistance.java

@@ -0,0 +1,35 @@
+package dev.nero.aimassistance.utils;
+
+/**
+ * Need this class because forge did not implement the mouse move event. They did for the scroll and
+ * input events, but not when the mouse moves.
+ */
+public class MouseUtils {
+
+    private static boolean mouseMoved;
+
+    private static int delay = 2;
+    private static double prevX = -1;
+    private static double prevY = -1;
+
+    public static void checkForMouseMove() {
+        double currX = Wrapper.MC.mouseHelper.getMouseX();
+        double currY = Wrapper.MC.mouseHelper.getMouseY();
+
+        if (prevX != -1 && prevY != -1) {
+            mouseMoved = prevX != currX || prevY != currY;
+        }
+
+        if (delay == 0) {
+            prevX = currX;
+            prevY = currY;
+            delay = 2;
+        } else {
+            delay --;
+        }
+    }
+
+    public static boolean mouseMoved() {
+        return mouseMoved;
+    }
+}