プレイヤーが敵から離れたときや敵が倒れた時に自動でロックオンを非表示にする方法

Question

カメラのスクリプトは
using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using System.Linq;

public class CameraControllerVer : MonoBehaviour
{
public GameObject player;
public GameObject mainCamera;
public float rotate_speed;
private const int ROTATE_BUTTON = 2;
private const float ANGLE_LIMIT_UP = 60f;
private const float ANGLE_LIMIT_DOWN = -5f;
private LockOnTargetDetector lockOnTargetDetector;
[SerializeField]
public GameObject lockOnTarget;
protected LockonCursor lockonCursor;

void Start()
{
    mainCamera = Camera.main.gameObject;
    player = GameObject.FindGameObjectWithTag("Player");
    lockOnTargetDetector = player.GetComponentInChildren<LockOnTargetDetector>();
    lockonCursor = GameObject.FindObjectOfType<LockonCursor>();
}

void LateUpdate()
{
    transform.position = player.transform.position;
    rotateCmaeraNaturalAngle();
    if (Input.GetMouseButton(ROTATE_BUTTON))
    {
        rotateCmaeraAngle();
    }
    if (Input.GetButtonDown("Fire2"))
    {
        if (lockOnTarget == null)
        {
            // ロックオンターゲット取得
            GameObject target = SetTargetClosestScreenCenter();

            if (target != null)
            {
                lockOnTarget = target;
                lockonCursor.OnLockonStart(target.transform);                  
            }
            else
            {
                // 視点リセット
                iTween.RotateTo(gameObject, iTween.Hash(
                    "rotation", player.transform.eulerAngles,
                    "time", 0.5f
                ));
            }
        }
        else
        {
            // 既にターゲットが設定されていた場合は解除
            lockOnTarget = null;
            lockonCursor.OnLockonEnd();
        }
    }

    if (lockOnTarget)
    {
        lockOnTargetObject(lockOnTarget);
    }
    else
    {
        rotateCmaeraNaturalAngle();
        if (Input.GetMouseButton(ROTATE_BUTTON))
        {
            rotateCmaeraAngle();
        }

    }
    float angle_x = 180f <= transform.eulerAngles.x ? transform.eulerAngles.x - 360 : transform.eulerAngles.x;
    transform.eulerAngles = new Vector3(
        Mathf.Clamp(angle_x, ANGLE_LIMIT_DOWN, ANGLE_LIMIT_UP),
        transform.eulerAngles.y,
        transform.eulerAngles.z
    );

}
private void rotateCmaeraNaturalAngle()
{
    Vector3 angle = new Vector3(
        Input.GetAxis("Angles X") * rotate_speed,
        Input.GetAxis("Angles Y") * rotate_speed,
        0
    );

    transform.eulerAngles += new Vector3(angle.y, angle.x);
}
private void rotateCmaeraAngle()
{
    Vector3 angle = new Vector3(
        Input.GetAxis("Mouse X") * rotate_speed,
        Input.GetAxis("Mouse Y") * rotate_speed,
        0
    ) ;

    transform.eulerAngles += new Vector3(angle.y, angle.x);
}
private void lockOnTargetObject(GameObject target)
{
    transform.LookAt(target.transform, Vector3.up);
}

protected GameObject SetTargetClosestScreenCenter()
{
    float search_radius = 15f;

    var hits = Physics.SphereCastAll(
        player.transform.position,
        search_radius,
        player.transform.forward,
        0.01f,
        LayerMask.NameToLayer("LockonTarget")
    ).Select(h => h.transform.gameObject).ToList();

    hits = FilterTargetObject(hits);

    if (0 < hits.Count())
    {
        float min_target_distance = float.MaxValue;
        GameObject target = null;

        foreach (var hit in hits)
        {
            Vector3 targetScreenPoint = Camera.main.WorldToViewportPoint(hit.transform.position);
            float target_distance = Vector2.Distance(
                new Vector2(0.5f, 0.5f),
                new Vector2(targetScreenPoint.x, targetScreenPoint.y)
            );
            Debug.Log(hit.gameObject + ": " + target_distance);

            if (target_distance < min_target_distance)
            {
                min_target_distance = target_distance;
                target = hit.transform.gameObject;
            }
        }

        return target;
    }
    else
    {
        return null;
    }
    
}

protected List<GameObject> FilterTargetObject(List<GameObject> hits)
{
    return hits
        .Where(h => {
            Vector3 screenPoint = Camera.main.WorldToViewportPoint(h.transform.position);
            return screenPoint.x > 0 && screenPoint.x < 1 && screenPoint.y > 0 && screenPoint.y < 1;
        })
        .Where(h => {
            foreach (var component in h.GetComponents<Component>())
            {
                if (component is Enemy)
                {
                    return true;
                }
            }
            return false;
        })
        .ToList();
}

}
です。
ちなみに敵の位置を設定し距離でif文をかいたところNUllエラーになります。
どうすれば自動でロックオンを非表示にできるでしょうか？
回答よろしくおねがいします！

Answer 1

文の最後に
////////←のついている箇所が主な修正部分です

//Edited by kanotoP (2019/10/16-17:09)
using System;////////
using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using System.Linq; 

public class CameraControllerVer : MonoBehaviour{
	public GameObject player;
	public GameObject mainCamera;
	public Single rotate_speed;
	const Int32 ROTATE_BUTTON = 2;
	const Single ANGLE_LIMIT_UP = 60f;
	const Single ANGLE_LIMIT_DOWN = -5f;
	const Single EYESIGHT = 10f;////////離れられる距離の最大
	private LockOnTargetDetector lockOnTargetDetector;
	[SerializeField]public GameObject lockOnTarget;
	protected LockonCursor lockonCursor; 
	
	void Start(){
		mainCamera = Camera.main.gameObject;
		player = GameObject.FindGameObjectWithTag("Player");
		lockOnTargetDetector = player.GetComponentInChildren<LockOnTargetDetector>();
		lockonCursor = GameObject.FindObjectOfType<LockonCursor>();
	} 
	
	///<summary>Updateの後に呼び出されるメソッドです。呼び出し間隔に規定はありません</summary>
	void LateUpdate(){
		transform.position = player.transform.position;
		rotateCmaeraNaturalAngle();
		if (Input.GetMouseButton(ROTATE_BUTTON)){
			rotateCmaeraAngle();
		}
		
		////////lockOnTargetの座標を弄りたいならこの場所で
		////////lockOnTarget.transform.position = ≪新しい座標≫;
		////////としてください
		
		if (lockOnTarget == null && Input.GetButtonDown("Fire2")){////////
			// ロックオンターゲット取得
			GameObject target = SetTargetClosestScreenCenter(); 
			if (target != null){
				lockOnTarget = target;
				lockonCursor.OnLockonStart(target.transform);
			}
			else{
				// 視点リセット
				iTween.RotateTo(gameObject, iTween.Hash("rotation", player.transform.eulerAngles, "time", 0.5f));
			}
		}
		if (lockOnTarget != null && (Input.GetButtonDown("Fire2") || Vector3.Distance(lockOnTarget.transform.position, transform.position) > EYESIGHT /*|| "lockOnTargetが死んでいるか"の真偽値*/ )){////////
			// ロックオンターゲット解除
			lockOnTarget = null;
			lockonCursor.OnLockonEnd();
		}
		if (lockOnTarget/*提示ソースには書かれていませんでしたが多分lockOntarget!=nullなのではないですか?*/)　lockOnTargetObject(lockOnTarget);////////
		else{
			rotateCmaeraNaturalAngle();
			if (Input.GetMouseButton(ROTATE_BUTTON)) rotateCmaeraAngle();
		}
		Single angle_x = 180f <= transform.eulerAngles.x ? transform.eulerAngles.x - 360 : transform.eulerAngles.x;
		transform.eulerAngles = new Vector3( Mathf.Clamp(angle_x, ANGLE_LIMIT_DOWN, ANGLE_LIMIT_UP), transform.eulerAngles.y, transform.eulerAngles.z); 
	}
	
	private void rotateCmaeraNaturalAngle(){
		Vector3 angle = new Vector3(Input.GetAxis("Angles X") * rotate_speed, Input.GetAxis("Angles Y") * rotate_speed, 0); 
		transform.eulerAngles += new Vector3(angle.y, angle.x);
	}
	
	private void rotateCmaeraAngle(){
		Vector3 angle = new Vector3(Input.GetAxis("Mouse X") * rotate_speed, Input.GetAxis("Mouse Y") * rotate_speed,　0) ; 
transform.eulerAngles += new Vector3(angle.y, angle.x);
	}
	
	private void lockOnTargetObject(GameObject target){
		transform.LookAt(target.transform, Vector3.up);
	} 
	
	protected GameObject SetTargetClosestScreenCenter(){
		Single search_radius = 15f; 
		var hits = Physics.SphereCastAll(player.transform.position,　search_radius,　player.transform.forward,　0.01f,　LayerMask.NameToLayer("LockonTarget")).Select(h => h.transform.gameObject).ToList(); 
		hits = FilterTargetObject(hits); 
		if (0 < hits.Count()){
			Single min_target_distance = Single.MaxValue;
			GameObject target = null; 
			foreach (var hit in hits){
				Vector3 targetScreenPoint = Camera.main.WorldToViewportPoint(hit.transform.position);
				Single target_distance = Vector2.Distance(new Vector2(0.5f, 0.5f),　new Vector2(targetScreenPoint.x, targetScreenPoint.y));
				Debug.Log(hit.gameObject + ": " + target_distance); 
				if (target_distance < min_target_distance){
					min_target_distance = target_distance;
					target = hit.transform.gameObject;
				}
			} 
			return target;
		}
		else{
			return null;
		} 
	} 
	
	protected List<GameObject> FilterTargetObject(List<GameObject> hits){
		return hits.Where(h => {
			Vector3 screenPoint = Camera.main.WorldToViewportPoint(h.transform.position);
			return screenPoint.x > 0 && screenPoint.x < 1 && screenPoint.y > 0 && screenPoint.y < 1;
		}).Where(h => {
			foreach (var component in h.GetComponents<Component>()){
				if (component is Enemy){
					return true;
				}
			}
			return false;
		}).ToList();
	} 
}

Answer 2

提示コードが長すぎるので、やり方だけ書いておきます。

LateUpdateメソッド内に

//前略

bool WasLiving=false;

//中略

LateUpdate(){
if(!/*ターゲットが生きているかどうかのフラグ*/&&WasLiving){
//ロックオンを解除する処理
}//前回は生きていたのに、今は生きていない→つまり死んだときの処理

if(lockOnTarget!=null&&(Vector3.Distance(player,lockOnTarget)>/*強制的にロックオンを解除するプレイヤーとターゲットの距離(定数)*/)){
//ロックオンを解除する処理
}//距離が離れ過ぎたときの処理

//中略

WasLiving=/*ターゲットが生きているかどうかのフラグ*/;
}

これで大丈夫だと思いますが、もしダメなら
・ソースコードは「`」×3で囲み、インデントを入れてください。
・メソッドや単位処理にそれぞれ何をしているのかコメントをもっと付けてください。
・ターゲットが倒れるというのは具体的に、”どの変数がどうなることを指すのか”を教えてください。

Comments

[退会済みユーザー]

とりあえず先に距離設定を行いたかったので設定しようとしたら
Vector3　tmp=GameObject.Find("lockOnTarget").transform.position;
LateUpdate()
{
if (lockOnTarget != null && Vector3.Distance(tmp, transform.position) > 10f)
{
lockOnTarget = null;
lockonCursor.OnLockonEnd();
}
}
を追加しましたが
NullReferenceException: Object reference not set to an instance of an object
がでます。

[KanazawaKureha]

エラーメッセージにはエラー箇所も記載されていると思いますが、どの箇所にエラーが発生しているのでしょうか？

[退会済みユーザー]

Vector3 tmp=GameObject.Find("lockOnTarget").transform.position;
ところです

[KanazawaKureha]

その箇所を
Vector3 tmp;
lockOnTarget=GameObject.Find(“lockOnTarget”);
if(lockOnTarget!=null)tmp=lockOnTarget.transform.position;
else tmp=null;
と書き換えれば、その部分は解決です。
しかし、また他のところでエラーが発生するかも知れませんから、エラー箇所をif(≪nullになってそうなフィールド≫!=null){...}で囲んでみて下さい。

[退会済みユーザー]

Cannot convert null to 'Vector3' because it is a non-nullable value type
というエラーがでました
else tmp = null;というところです

[KanazawaKureha]

すみません、こちらのミスです。tmp=Vector3.zero;としたら動くと思います。(もし動作したら念のため座標0,0,0に異常がないか確認して下さい)

[退会済みユーザー]

すみませんもう一回まとめてもらってもよろしいでしょうか？
実はロックオンターゲット取得が消えたりして、こんがらがっています。

[KanazawaKureha]

新たな回答を追加しました。
繰り返し言うようですが、インデントは入れて、```で文を囲みましょう