MissingReferenceException：“攻击者”类型的对象已被破坏，但您仍在尝试访问它

Question

我收到上述错误，我尝试使用 debug.log 打印错误所在。 我正在创建一种坦克 obj。这会触发一些实例化的 obj。实例化的对象是攻击者。

在更新中，我使用 foreach 循环遍历所有实例化对象。如果找到并且如果对象在射程内。

void Update () {


        if (attacker!= null  )
        {
            //Debug.Log("inside att");
            attacker = FindObjectsOfType<Attacker>();
        }

        // fire only if attacker is in range 

        if (IfInRange() && attacker!= null && running)
        {

            faceTowardTarget();
            Fire();
        }


    }
    bool IfInRange()
    {// run through all the instantiated attacker
        foreach (Attacker currentAttacker in attacker)

这工作正常，但上面给出了一些东西。在控制台的最后，循环继续进行，并且 currentAttacker 最终为空。我试图在控制台中打印它。但它不会出现在其他 if 语句中

{   //if attacker is in range

            if (attacker != null )
                {

                    Debug.Log(currentAttacker.GetComponent<Health>().isAlive);

                    if (Vector2.Distance(transform.position, currentAttacker.transform.position) < minDistance)               
                    {

                        attackerPos = currentAttacker.transform;

                        return true;
                    }
                }
                if (currentAttacker == null)
                {
                    Debug.Log("curre Attacker null");
                    running = false;
                    return false;
                }

             }return false;
        }

攻击者有一个简单的生命值脚本，可以处理被射弹击中时造成的伤害。

Void update()
    {
     if (health <= 0)
            {
**is there any better way to destroy an obj. If i destroy gameObject that error appear and it get stuck in foreach loop**
               // Destroy(gameObject);
                noOfKilled++;
                isAlive = false;
                scoreHolder.addScore(scoreValue);
            }
    }

非常感谢您的帮助。我尝试搜索但无法解决此问题。

Answer 1

解决此问题的快速而肮脏的方法是使用DestroyImmediate 函数而不是Destroy 函数。使用DestroyImmediate(gameObject) 将破坏该帧中的对象，FindObjectsOfType<Attacker>() 找不到它，因此不会在foreach 循环中访问它。

---

正确的方法是在主代码中创建一个List 来保存实例化的Attacker 脚本：

public GameObject attackerPrefab;
public List<Attacker> attacker = new List<Attacker>();

当您实例化预制件时，添加Attacker 脚本到List：

//Instantiate
GameObject obj = Instantiate(attackerPrefab);
//Get Attacker component 
Attacker tempAttacker = obj.GetComponent<Attacker>();
//Add Attacker component to List
attacker.Add(tempAttacker);

最后，当您想在运行状况脚本中销毁攻击者对象时，将其从List 中删除，然后将其销毁。通过在销毁之前将其从List 中删除，您将不会访问标记为已销毁的对象。

//Get the script that stores the List of Attacker
Test otherScript = GameObject.Find("NameOfGameObjectYourScriptIsAttachedTo").GetComponent<Test>();
//Remove from the List
otherScript.attacker.Remove(gameObject.GetComponent<Attacker>());
//Now, destroy this gameObject
Destroy(gameObject);

Answer 2

有很多方法可以解决这个问题。一个选项是在您编辑循环时不要使用 foreach 循环。但是当你想保留 foreach 循环时，只需将对象保存在另一个列表中并在 foreach 循环之后销毁它们。

示例1

for(int i = 0; i < attackers.Count; i++)
            {
                //if in range
                attackers.RemoveAt(i);
                i--;
            }

示例2

List<string> attackers = new List<string>();
        List<string> _shootAt = new List<string>();
        foreach(string attacker in attackers)
        {
            //if in range
            _shootAt.Add(attacker);
        }

        foreach (string attacker in _shootAt)
        {
            //if in range
            attackers.Remove(attacker);
        }

希望对你有帮助

Answer 3

我假设，在第一个坦克被摧毁后实例化另一个坦克时会出现错误？你所有的坦克都是 tank1 的克隆，所以当 tank1 被销毁时，你会得到一个空指针，因为它试图实例化一个不再存在的对象。

2 个解决方案...

A) UGLY：将 tank1 的初始位置更改为永远不会被摧毁的位置，例如 5000,5000,5000。由于坦克不能被摧毁，它永远不会为空

B) 聪明、正确的方法：制作预制件。创建一个名为 prefabs 的文件夹，将 tank1 拖入其中。选择生成坦克的脚本，然后将 tank1 的 PREFAB 副本拖入其中。现在你总是有一个 tank1 的实例，你的空指针不见了！

我不能足够强调正确使用预制件的重要性，不仅仅是为了性能和可靠性，还有理智......

Answer 4

FindObjectsOfType<Attacker>(); //returns an array

将返回一个数组，而不是一个对象，使用攻击者[0]作为第一个或使用

FindObjectOfType<Attacker>()

（将返回第一个找到的）。

使用 objectsOfType 你总是有一个空数组，所以

if (attacker==null)

当然会返回 false，因为数组不是空的，它只是空的。这就是为什么它没有出现在您的第二个 if 语句中；

还有

void update()
        {
         if (health <= 0)
                {
    **is there any better way to destroy an obj. If i destroy gameObject that error appear and it get stuck in foreach loop**

                    noOfKilled++;
                    isAlive = false;
                    scoreHolder.addScore(scoreValue);
                    Destroy(gameObject);// destroy it after at the end, not before it have something else to do
                }
        }