敵の行動と索敵範囲をリンクさせる

　敵の行動と索敵範囲について実装をしてきました。ですがまだこのままでは「索敵範囲内のPlayerに対して攻撃行動を行う」といったロジックは組めていません。
何故ならば、実装した行動と索敵範囲とがリンクしていないため、それぞれがバラバラの動きになっているためです。

　本来の設計は、索敵範囲を基準として敵の行動を決定していました。現在はバラバラですが、それらをつなぎ合わせてあげることで
本来考えていた設計の実装が可能になります。その手順を考えてみましょう。

索敵範囲を基準としたロジックを考える

　敵の行動は、いまの所は「待機」→「移動」→「攻撃」→「待機」のサイクルで動いています。
これを索敵範囲を基準としたサイクルに置き換えていきます。

　移動と攻撃ですが、こちらは、索敵範囲を基準にして、どちらの行動を選択するかを判別させるようにします。
ここでいう索敵範囲とは、敵の視覚情報の範囲であると考えてください。つまり、視認出来ている場合にのみ、プレイヤーを追いかけるというようにしたいですね。
今回はわかりやすく、プレイヤーが敵の索敵範囲外の場合には移動を、索敵範囲内の場合には攻撃を選択できるようにプログラムを考えましょう。

①移動について

　いままでは敵の向いている方向に前進をさせていました。とはいえ、常に前進してしまうとわかってしまうとプレイヤーは敵を脅威と感じません。
かといって、索敵範囲外であったのに、毎回プレイヤーの方向を自動的に向き直すのも、おかしな挙動になります。（プレイヤーが見えていないはずなので）

　つまり「索敵範囲外にプレイヤーがいる場合には、敵は移動をする。移動する場合にはランダムな方向を向いて、その方向に前進させる」というような内容であれば
「敵は、自分の目にプレイヤーが映らない場合には、プレイヤーを探してウロウロしている」という状態を表現できそうです。　
　これを「移動」と定義しましょう。

②攻撃について

　索敵範囲内の場合には攻撃を選択するようにしてあげたいのですが、索敵範囲は結構な範囲があります。
つまり、範囲のギリギリにプレイヤーがいてもそれは索敵範囲内ですから発見されたものとなります。そのため
索敵範囲内だからすぐに攻撃、というようにプログラムを組んでしまうと、ものすごくプレイヤーが遠いにも関わらず
その場で攻撃を行ってしまうことになり、これも非常に滑稽かつ、脅威と感じることはないでしょう。

　そのため、「索敵範囲内にプレイヤーがいる場合には、まずプレイヤーの位置まで移動を行って、その後、攻撃を行う」という内容であれば
プレイヤーに対して向かってきてから攻撃する挙動になりますから、敵として申し分ない行動といえるでしょう。

　これらの点を踏まえて、EnemyControlerクラスを修正していきます。

　<= ＋ボタンを押すと開きますので、自分なりの実装を行った上で確認をしてみましょう。また確認後、不要なDebug.Logはコメントアウトするか削除しましょう。

using System.Collections;
using System.Collections.Generic;
using UnityEngine;

public class EnemyController : MonoBehaviour {

    private string weapon = "Weapon";   // タグ指定用

    [Header("現在のHP")]
    public int hp;

    [Header("クリティカルの発生確率")]
    public int criticalSuccessRate;

    [Header("クリティカル時の攻撃力倍率")]
    public float criticalMultiplier;

    [Header("移動速度の最小値")]
    public float minMoveSpeed;
    [Header("移動速度の最大値")]
    public float maxMoveSpeed;

    [Header("攻撃力の最小値")]
    public int minAttackPower;
    [Header("攻撃力の最大値")]
    public int maxAttackPower;

    private float moveSpeed;　　// 適用する移動速度
    public int attackPower;　　// 適用する攻撃力

    public enum ENEMY_STATE {
        SET_UP,
        WAIT,
        MOVE,
        ATTACK,
        READY
    }
    public ENEMY_STATE enemyState;

    private float actionTime;
    private Rigidbody rb;
    private Animator anim;

    public SearchArea searchArea;
    private Vector3 destinationPos;   // 索敵時の移動の目的地(Playerの位置情報)
    Vector3 direction;                // 移動する際の方向

    void Start() {
        // 移動速度などの値を設定する
        SetUpEnemyParameter();
    }

    /// <summary>
    　　/// 敵のパラメータ(移動速度や攻撃力など)と状態を設定
    　　/// </summary>
    private void SetUpEnemyParameter() {
        enemyState = ENEMY_STATE.SET_UP;
        Debug.Log(enemyState);

        moveSpeed = Random.Range(minMoveSpeed, maxMoveSpeed);
        attackPower = Random.Range(minAttackPower, maxAttackPower);
        actionTime = Random.Range(3, 6);

        rb = GetComponent<Rigidbody>();
        anim = GetComponent<Animator>();

        enemyState = ENEMY_STATE.WAIT;
        
        Debug.Log(enemyState);
        Debug.Log("MoveSpeed : " + moveSpeed);
        Debug.Log("AttakPower : "+ attackPower);
    }

    void Update() {
        // 準備状態。敵のパラメータの準備が終了していない場合にはUpdateを処理しない
        if (enemyState == ENEMY_STATE.SET_UP) {
            return;
        }

        actionTime -= Time.deltaTime;

        // 待機状態
        if (enemyState == ENEMY_STATE.WAIT) {
　　　　　　Debug.Log("待機中 あと : " + actionTime + " 秒");
            if (actionTime <= 0) {
                CheckNextAction();
                Debug.Log("待機終了");
                //NextAction(ENEMY_STATE.MOVE);
            }
        }

        // 移動状態
        if (enemyState == ENEMY_STATE.MOVE) {
            Move();
            Debug.Log("移動中 あと : " + actionTime + " 秒");
            
            if (actionTime <= 0) {
                NextWait();
            }
        }

        // 索敵範囲内で目的地(Playerの位置)がある場合、目的地まで移動する
        if (enemyState == ENEMY_STATE.ATTACK) {
            Debug.Log(Vector3.Distance(transform.position, destinationPos));
            if (Vector3.Distance(transform.position, destinationPos) > 0.8f) {
                Move();
                Debug.Log("移動攻撃中");

            } else {
                // 目的地に着いたら攻撃
                enemyState = ENEMY_STATE.READY;
                StartCoroutine(Attack());
                Debug.Log("攻撃");
                return;
            }
        }
    }

    /// <summary>
    /// 次の行動を確認する
    /// </summary>
    private void CheckNextAction() {
        // 索敵範囲内にPlayerがいる場合
        if (searchArea.isSearching) {
            if (searchArea.player != null) {
                // Playerの位置まで移動して攻撃をする
                MoveAttack(searchArea.player);
                Debug.Log("移動して攻撃する");
            }
        } else {
            // 索敵範囲内にPlayerがいない場合、移動の準備をする
            PraparateMove();
            Debug.Log("移動準備");
        }
    }

    /// <summary>
    /// 索敵範囲内のPlayerの位置を目的地とし、移動する
    /// </summary>
    /// <param name="player"></param>
    private void MoveAttack(GameObject player) {
        // 目的地を設定しておく
        destinationPos = player.transform.position;
        // 移動する方向を設定する
        direction = (transform.position - destinationPos).normalized;
        // 移動する方向を向く
        transform.LookAt(destinationPos);
        // UpdateのAttackの部分で移動させる
        enemyState = ENEMY_STATE.ATTACK;
        anim.SetBool("Run", true);
    }

    /// <summary>
    /// 移動の準備
    /// </summary>
    private void PraparateMove() {
        // ランダムな方向を向く
        transform.rotation = Quaternion.AngleAxis(Random.Range(0, 360), new Vector3(0, 1, 0));
        direction = transform.position.normalized;
        enemyState = ENEMY_STATE.MOVE;
        actionTime = Random.Range(3, 5);
        anim.SetBool("Run", true);
    }

    /// <summary>
    /// 次回の行動までの待機
    /// </summary>
    /// <param name="nextState">ENEMY_STATE</param>
    private void NextWait() {
        anim.SetBool("Run", false);
        // 待機
        actionTime = Random.Range(3, 6);
        enemyState = ENEMY_STATE.WAIT;
    }

    /// <summary>
    /// 移動の処理
    /// </summary>
    private void Move() {
        rb.AddForce(-direction * moveSpeed);
        Debug.Log(rb.velocity);
    }

    /// <summary>
    /// 攻撃の処理
    /// </summary>
    /// <returns></returns>
    private IEnumerator Attack() {
        anim.SetTrigger("Attack");
        yield return new WaitForSeconds(2.0f);
        NextWait();
    }

    private void OnTriggerEnter(Collider col) {
        // Weaponタグ以外は当たり判定として判定しない
        if (col.gameObject.tag != weapon) {
            return;
        }

        // 攻撃力への初期倍率
        float attackPowerRate = 1.0f;

        // クリティカルの判定を戻り値で行う。Trueが戻ってきたらクリティカルした判定
        if (JudgeCritialHit()) {
            // クリティカルした判定の場合、攻撃倍率を変更
            attackPowerRate = criticalMultiplier;
        }

        // AttackPowerを取得したいため、まずはWeaponゲームオブジェクトのWeaponControllerから、PlayerControllerを取得
        MoveSample playerController = col.gameObject.GetComponent<WeaponController>().playerController;

        // PlayerControllerクラスが変数に代入できているか確認
        Debug.Log(playerController);

        // 最終的な攻撃力を算出(小数点は切り捨て)　右辺はFloat型同士での計算式のため計算できるが、計算結果がFloatになるため、そのままだと左辺のInt型と型が合わない。
        // 左辺と合わせるために、計算後にInt型に明示的に型の変換をしている(これをキャストといいます)
        int damage = (int)Mathf.Floor(playerController.attackPower * attackPowerRate);
        Debug.Log(damage);

        // HPからdamage分だけ減算
        hp -= damage;
        Debug.Log("残りHP : " + hp);

        // 残りHPの確認
        if (hp <= 0) {
            Debug.Log("敵を倒した");
            Destroy(this.gameObject);
        }
    }

    /// <summary>
    /// クリティカルの判定
    /// </summary>
    /// <returns></returns>
    private bool JudgeCritialHit() {
        // 判定用の変数。Trueならクリティカルした判定、Falseならクリティカルしていない判定
        bool isCritical = false;

        // 乱数を１つ取得
        int randomValue = Random.Range(0, 100);

        // 乱数とクリティカルの確率を比べ、クリティカル発生確率よりも乱数の方が低いならクリティカルしたことにする
        if (randomValue <= criticalSuccessRate) {
            Debug.Log("Critical!!");
            return true;
        }
        return false;
    }
}