編集履歴

質問編集履歴

書式の改善

2021/06/26 07:27

投稿

退会済みユーザー

スコア0

title CHANGED Viewed

File without changes

body CHANGED Viewed

@@ -235,8 +235,9 @@
 ### 試したこと
-![![イメージ説明](ac0a681f26f67b8e07964a086b803ca8.png)](6cd8d382d3f3084a1fd6e86ede1bce7c.png)
+![![イメージ説明](ac0a681f26f67b8e07964a086b803ca8.png)
 【回答を追記・編集しました】
 ユニットはランダムに移動しているのですが、ユニットの「不満値」というパラメータが１００になると、他のユニットの隣接に移動して、攻撃する、という実装にしようと思っていました。
-ユニットが移動するスクリプトはUnit_RandomWalk.csというスクリプトです。すべてのユニットにこのスクリプトがアタッチされています。
+ユニットが移動するスクリプトはUnit_RandomWalk.csというスクリプトです。すべてのユニットにこのスクリプトがアタッチされています。
+目的のユニットが移動するたびに経路が再計算されるため、上記の図のように、ぐらぐらと揺れながら移動してしまっています。

スクリプトの追加・説明の編集

2021/06/26 07:27

投稿

退会済みユーザー

スコア0

title CHANGED Viewed

File without changes

body CHANGED Viewed

@@ -8,19 +8,20 @@
 using System.Collections;
 using System.Collections.Generic;
 using UnityEngine;
-using Module_of_OverWorld;
-using Module_of_Unit;
+using System.Linq;
 namespace UtilityModule
 {
     public class Moving_from_PositionA_to_PositionB : MonoBehaviour
     {
-        private int map_Width = 250;
+        private int map_Width = 100;
-        private int map_Height = 250;
+        private int map_Height = 100;
         private ChessboardPathFinder pathFinder;
         public List<Vector2> moveList = new List<Vector2>();
-        private float speed = 4;
+        private float speed = 2.5f;
         public bool stoped_Movement;
+        //始動したコルーチンを覚えておくためのフィールドを追加し...
+        private Coroutine movementAction;
         void Awake()
         {
@@ -36,17 +37,34 @@
         public void Start_MovementAction()
         {
+            //コルーチン始動時にそれを保存
-            StartCoroutine(Move());
+            movementAction = StartCoroutine(Move());
         }
+        //そして覚えておいたコルーチンを停止するためのメソッドを用意
+        public void Stop_MovementAction()
+        {
+            if (movementAction != null)
+            {
+                StopCoroutine(movementAction);
+                moveList.Clear();
+            }
+        }
         IEnumerator Move()
         {
-            foreach (Vector2Int position in pathFinder.Path)
+            foreach (Vector2Int position in pathFinder.Path.Skip(1))
             {
                 Vector2 pos = new Vector3(position.x, position.y);
                 moveList.Add(pos);
             }
+            Vector2 unitPos = new Vector2(gameObject.transform.position.x, gameObject.transform.position.y);
+            if(moveList.Contains(unitPos))
+            {
+                moveList.Remove(unitPos);
+            }
             int count = 0;
             while (true)
@@ -73,7 +91,6 @@
         }
     }
 }
 ```
 ```C#
 using System.Collections;
@@ -89,13 +106,12 @@
         private Unit_RandomWalk unit_RandomWalk;
         private Unit gameObjectUnit;
         private Moving_from_PositionA_to_PositionB moving_From_PositionA_To_PositionB;
         private Unit_RandomWalk unit_RandomWalk_SelectedUnit;
         private GameObject selectedUnit;
         private bool get_Distance;
         private bool switched_Berserkermode;
+        //前回のUpdateで調べたターゲットの位置を保存するためのフィールドを追加
+        private Vector2Int latestTargetPos;
         private void Awake()
         {
@@ -107,7 +123,7 @@
         public void Action()
         {
             //不満値が１００％になったら、ランダム移動を停止する
-            unit_RandomWalk.stoped_RandomWalk = true;
+            unit_RandomWalk.Stop_RandomWalk();
             Invoke("Go", 1);
         }
@@ -133,7 +149,7 @@
                 float distanse = Vector2.Distance(gameObject.transform.position, selectedUnit.transform.position);
                 if (distanse <= 3)
                 {
-                    unit_RandomWalk_SelectedUnit.stoped_RandomWalk = true;
+                    unit_RandomWalk_SelectedUnit.Stop_RandomWalk();
                     int x = Mathf.RoundToInt(unit_RandomWalk_SelectedUnit.transform.position.x);
                     int y = Mathf.RoundToInt(unit_RandomWalk_SelectedUnit.transform.position.y);
                     unit_RandomWalk_SelectedUnit.transform.position = new Vector3(x, y, 0);
@@ -141,15 +157,37 @@
             }
             //対象のユニットが存在していた座標に到達したら、また移動する
+            /*
             if (switched_Berserkermode && gameObject.transform.position == (Vector3)moving_From_PositionA_To_PositionB.moveList[moving_From_PositionA_To_PositionB.moveList.Count - 1])
             {
                 Go_To_TargetUnit();
             }
+            */
+            if (switched_Berserkermode)
+            {
+                //現在のターゲットの位置を調べ...
+                Vector2Int targetPos = new Vector2Int((int)selectedUnit.transform.position.x, (int)selectedUnit.transform.position.y);
+                //敵の位置に変化があったら...
+                if (targetPos != latestTargetPos)
+                {
+                    //現在動作中の移動コルーチンは中止し、目的地への経路を再計算
+                    latestTargetPos = targetPos;
+                    Stop_Movement();
+                    Go_To_TargetUnit();
+                }
+            }
         }
+        void Stop_Movement()
+        {
+            moving_From_PositionA_To_PositionB.Stop_MovementAction();
+        }
         void Go_To_TargetUnit()
         {
-            moving_From_PositionA_To_PositionB.moveList.Clear();
+            //moving_From_PositionA_To_PositionB.moveList.Clear();
             Vector2Int startPos = new Vector2Int((int)gameObject.transform.position.x, (int)gameObject.transform.position.y);
             Vector2Int goalPos = new Vector2Int((int)selectedUnit.transform.position.x, (int)selectedUnit.transform.position.y);
@@ -193,11 +231,12 @@
         }
     }
 }
 ```
 ### 試したこと
-[Unity A*(A-Star)アルゴリズム](https://teratail.com/questions/283731)を参考にコードを書きました。
+![![イメージ説明](ac0a681f26f67b8e07964a086b803ca8.png)](6cd8d382d3f3084a1fd6e86ede1bce7c.png)
+【回答を追記・編集しました】
-今のスクリプトでは、対象が存在していた座標に向かって移動するだけで、移動したあとでは、その対象はその場所にはいません。どのように書き換えればよいでしょうか？　回答よろしくお願い致します。
+ユニットはランダムに移動しているのですが、ユニットの「不満値」というパラメータが１００になると、他のユニットの隣接に移動して、攻撃する、という実装にしようと思っていました。
+ユニットが移動するスクリプトはUnit_RandomWalk.csというスクリプトです。すべてのユニットにこのスクリプトがアタッチされています。