NavMeshAgent not fully reaching destination, thus it won't delete itself when it gets there

Questions & Answers
1

Hi guys,

I have some code that spawns a bunch of NavMeshAgents (they’re cars with NavMeshAgent components) and sends them to destinations that are specified on the spawnpoints. Each car has the following code in its Update() function:

`
...
// Get new navpoint if previous target is reached
	
	if (transform.position == target.transform.position || transform.position.z == target.transform.position.z -1 || transform.position.z == target.transform.position.z + 1 || transform.position == (target.transform.position - Vector3(1, 1, 1)) || transform.position == (target.transform.position - Vector3(-1, -1, -1)))
	{
		if(target.tag == "Car End Point")
		{
			Debug.Log("At endpoint, destory");
			Destroy(this.gameObject);
			NotificationCenter.DefaultCenter().PostNotification(this, "CarDeleted");
		}
		else
		{
			//Debug.Log("At navpoint, deciding where to go next...");
		
			if (targetScript.connections.Length > 0)
			{
				var element : int = Random.Range(0, targetScript.connections.Length);
			
					target = targetScript.connections[element];
				
				//Debug.Log("Connection[" + element + "] chosen.");
			
				GetComponent(NavMeshAgent).destination = target.transform.position;
			}
			else
			{	
				GetComponent(NavMeshAgent).destination = target.transform.position + Vector3(30, 0, 0);
			
				//Debug.Log("No connections found, proceeding straight as an arrow...");
			}
		}
	}
...
`

I put the || operators because I wanted to make it remove my cars if they come within a 1m of their targets (because they all seem to stop moving when they get to a distance of less than 1m from their targets. The original if statement looked like this:

`
if (transform.position == target.transform.position)
`

How do I make my NavMeshAgents go all the way to their targets, and/or make them delete themselves if they get within 1m of their targets?

MachCUBED

2

It’s unlikely you’re going to hit right on a point with your movement logic. Had this problem with a loader last night going from 0 to 1. It was at 0.9999… for a while and wouldn’t quite get there.

I’ve written something that’s kinda psuedocode. I haven’t really tested it, but this is essentially what you need to do for your checks. First off, use Vector3.Distance. It’s there to help you. If that’s below a threshold( hard coded to 1 in my example), then go to the next target position. If you’re at the last position you needed to go to, you “DestroyImmediate(this.gameObject);”

Hope this helps. It’s C#, which I strongly recommend you start working towards coding in all of the time.

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

public class MagnitudeTest : MonoBehaviour {

    Vector3 currentTargetPos;

    int posIndex = 0;

    public List<Transform> positions = new List<Transform>();
    
    void Awake() {
        currentTargetPos = positions[posIndex].position;
    }
    
    //DO NOT keep using GetComponent, store a reference in your class
    
    // Update is called once per frame
    void Update () {
        //your distance threshold should probably be smaller than 1 meter if you want them to not snap to this spot
        if( Vector3.Distance(transform.position, currentTargetPos) <= 1 ) {
            Debug.Log("We're at our desintation");
            transform.position = currentTargetPos; //set to make sure it's actually at the destination instead of 1m off
            ++posIndex;
            
            if( posIndex != positions.Count ) {
                //assign next target as new endPos target, do all of this over again
                currentTargetPos = positions[posIndex].position;
            }
            else {
                //we're at the end destination for this GameObject
                DestroyImmediate(this.gameObject);
            }
        }
        else {
            //do your movement logic towards your target however you wish
        }	
    }
}