From 7d5637e4ecb1a52493993a0c36aa2a743f7a002d Mon Sep 17 00:00:00 2001
From: Samantha Sullivan <samantha@sullivans.id.au>
Date: Mon, 11 Sep 2023 11:59:09 +1000
Subject: [PATCH] mobs will attack with melee if within range

---
 .../game/components/TouchAttackComponent.java |  2 +-
 .../game/components/tasks/MobAttackTask.java  | 34 ++++++++++++-------
 2 files changed, 22 insertions(+), 14 deletions(-)

diff --git a/source/core/src/main/com/csse3200/game/components/TouchAttackComponent.java b/source/core/src/main/com/csse3200/game/components/TouchAttackComponent.java
index 01ed762a4..6a7e7e2e1 100644
--- a/source/core/src/main/com/csse3200/game/components/TouchAttackComponent.java
+++ b/source/core/src/main/com/csse3200/game/components/TouchAttackComponent.java
@@ -64,7 +64,7 @@ public void create() {
     hitboxComponent = entity.getComponent(HitboxComponent.class);
   }
 
-  private void onCollisionStart(Fixture me, Fixture other) {
+  public void onCollisionStart(Fixture me, Fixture other) {
     if (hitboxComponent.getFixture() != me) {
       // Not triggered by hitbox, ignore
       return;
diff --git a/source/core/src/main/com/csse3200/game/components/tasks/MobAttackTask.java b/source/core/src/main/com/csse3200/game/components/tasks/MobAttackTask.java
index 703024c8d..a46c6f430 100644
--- a/source/core/src/main/com/csse3200/game/components/tasks/MobAttackTask.java
+++ b/source/core/src/main/com/csse3200/game/components/tasks/MobAttackTask.java
@@ -12,6 +12,7 @@
 import com.csse3200.game.physics.BodyUserData;
 import com.csse3200.game.physics.PhysicsEngine;
 import com.csse3200.game.physics.PhysicsLayer;
+import com.csse3200.game.physics.components.HitboxComponent;
 import com.csse3200.game.physics.components.PhysicsMovementComponent;
 import com.csse3200.game.physics.raycast.RaycastHit;
 import com.csse3200.game.rendering.AnimationRenderComponent;
@@ -33,6 +34,8 @@ public class MobAttackTask extends DefaultTask implements PriorityTask {
   private static final String FIRING = "shootStart";
   private static final String IDLE = "idleStart";
 
+  private Fixture target;
+
   private final int priority;
   private final float maxRange;
   private Vector2 mobPosition = new Vector2(10f,10f);
@@ -125,16 +128,20 @@ public void updateMobState() {
           mobState = STATE.STOW;
         } else {
           if (this.meleeOrProjectile() instanceof Melee) {
-
+            System.out.println("Melee attack");
+            TouchAttackComponent attackComp = owner.getEntity().getComponent(TouchAttackComponent.class);
+            HitboxComponent hitboxComp = owner.getEntity().getComponent(HitboxComponent.class);
+            attackComp.onCollisionStart(hitboxComp.getFixture(), target);
+          } else {
+            Entity newProjectile = ProjectileFactory.createMobBall(PhysicsLayer.HUMANS, new Vector2(0, owner.getEntity().getPosition().y), new Vector2(2f,2f));
+            newProjectile.setPosition((float) (owner.getEntity().getPosition().x), (float) (owner.getEntity().getPosition().y));
+            newProjectile.setScale(-1f, 0.5f);
+            ServiceLocator.getEntityService().register(newProjectile);
+
+//            System.out.printf("ANIMATION: " + owner.getEntity().getComponent(AnimationRenderComponent.class).getCurrentAnimation() + "\n");
+            owner.getEntity().getEvents().trigger(FIRING);
+            mobState = STATE.STOW;
           }
-          Entity newProjectile = ProjectileFactory.createMobBall(PhysicsLayer.HUMANS, new Vector2(0, owner.getEntity().getPosition().y), new Vector2(2f,2f));
-          newProjectile.setPosition((float) (owner.getEntity().getPosition().x), (float) (owner.getEntity().getPosition().y));
-          newProjectile.setScale(-1f, 0.5f);
-          ServiceLocator.getEntityService().register(newProjectile);
-
-          System.out.printf("ANIMATION: " + owner.getEntity().getComponent(AnimationRenderComponent.class).getCurrentAnimation() + "\n");
-          owner.getEntity().getEvents().trigger(FIRING);
-          mobState = STATE.STOW;
         }
         owner.getEntity().getComponent(PhysicsMovementComponent.class).setEnabled(true);
 
@@ -220,12 +227,13 @@ private boolean isTargetVisible() {
    * returns the Weapon (Melee or Projectile) the mob will use to attack the target. null if immune target or no target
    * */
   private Weapon meleeOrProjectile() {
-    Vector2 newVector = new Vector2(owner.getEntity().getPosition().x - 10f, owner.getEntity().getPosition().y - 2f);
-    Fixture hitraycast = physics.raycastGetHit(owner.getEntity().getPosition(), newVector, TARGET);
+//    Vector2 newVector = new Vector2(owner.getEntity().getPosition().x - 10f, owner.getEntity().getPosition().y - 2f);
+//    Fixture hitraycast = physics.raycastGetHit(owner.getEntity().getPosition(), newVector, TARGET);
+    setTarget();
     TouchAttackComponent comp = owner.getEntity().getComponent(TouchAttackComponent.class);
     Weapon chosenWeapon = null;
     if (comp != null) {
-      chosenWeapon = comp.chooseWeapon(hitraycast);
+      chosenWeapon = comp.chooseWeapon(target);
     }
 
     return chosenWeapon;
@@ -233,6 +241,6 @@ private Weapon meleeOrProjectile() {
 
   private void setTarget() {
     Vector2 newVector = new Vector2(owner.getEntity().getPosition().x - 10f, owner.getEntity().getPosition().y - 2f);
-    Fixture hitraycast = physics.raycastGetHit(owner.getEntity().getPosition(), newVector, TARGET);
+    target = physics.raycastGetHit(owner.getEntity().getPosition(), newVector, TARGET);
   }
 }