在 Update() 中创建新类会导致 StackOverflow 异常 Unity

Question

我有一个 IPawnInput 接口，它有一个 OnInput(object sender) 方法用于输入。之前，我只是通过“主”类 - Player 为其他组件（PlayerMovement、PlayerCamera）调用此方法。很快，我在将所有必要的组件连接到主要组件时遇到了问题。我决定有一个输入管理器，在所有必需的组件上调用此方法。因为该管理器本身应该从上面获取输入命令，所以我创建了一个 PlayerInputManagerSender 类，其中包含有关输入请求的原始发送者的信息。直接发送原始发送者不是一个选择，因为某些组件需要知道它们是通过管理器使用的。由于某种原因，创建此类会导致 Stackoverflow 错误。我知道每帧调用方法并创建一个类并不是最好的解决方案，但我没有其他解决方案，因为某些组件中的输入必须每帧读取。

using System.Collections.Generic;
using UnityEngine;

namespace LOK1game
{
    public class PlayerInputManager : MonoBehaviour, IPawnInput
    {
        [HideInInspector] public List<IPawnInput> PawnInputs = new List<IPawnInput>();

        [SerializeField] private List<MonoBehaviour> _actors;

        private void Awake()
        {
            foreach (var actor in _actors)
            {
                if(actor is PlayerInputManager)
                {
                    throw new System.Exception("Input manager can not calls itself!");
                }

                PawnInputs.Add(actor.GetComponent<IPawnInput>());
            }
        }

        private void Update()
        {
            //Test temp solution
            OnInput(this);
        }

        public void OnInput(object sender)
        {
            var managerSender = new PlayerInputManagerSender(sender, this);

            foreach (var pawn in PawnInputs)
            {
                pawn.OnInput(managerSender);
            }
        }
    }

    public struct PlayerInputManagerSender
    {
        public object OriginSender;
        public object ActualSender;

        public PlayerInputManagerSender(object originSender, object actualSender)
        {
            OriginSender = originSender;
            ActualSender = actualSender;
        }
    }
}

我刚刚尝试将类更改为结构

Answer 1

我的猜测是你有循环引用。 PlayerInputManager 是一个IPawnInput，并拥有一个IPawnInput 列表。因此，其他一些 IPawnInput 对象可能包含对所属对象的引用。本质上形成一个图表。因此，当您调用 OnInput 时，您最终会陷入无限递归，并最终耗尽堆栈。

要检查这一点，您可以迭代图表并检查是否有任何对象出现两次：

public static bool HasCycle<T>(T self, Func<T, IEnumerable<T>> selector, IEqualityComparer<T> comparer = default)
{
    var stack = new Stack<T>();
    stack.Push(self);
    comparer ??= EqualityComparer<T>.Default;
    var visited = new HashSet<T>(comparer);
    while (stack.Count > 0)
    {
        var current = stack.Pop();
        if (!visited.Add(current))
        {
            return true;
        }
        foreach (var child in selector(current))
        {
            stack.Push(child);
        }
    }
    return false;
}

调用方式

var result = HasCycle(this, p => p.PawnInputs);

您可以修改上面的迭代代码以给出出现两次的对象，或者在每次添加 pawn 时调用它以查找循环的位置创建并解决问题。

您还可以使用类似的方法来忽略图中的任何重复项，以防您实际上打算有循环，但希望处理其中的每个项一次。

Answer 2

StackOverflow异常通常是由无限递归引起的。您的一个函数正在调用另一个函数，该函数以某种方式回调最初调用的函数。在 OnInput() 和 Update() 函数中使用断点/写入行，以确保它们被调用的次数符合您的预期。

Answer 3

您的类 PlayerInputManager 正在实现 IPawnInput 本身，因此当您这样做时

foreach (var actor in _actors)
{
    if(actor is PlayerInputManager)
    {
        throw new System.Exception("Input manager can not calls itself!");
    }

    PawnInputs.Add(actor.GetComponent<IPawnInput>());
}

，actor 本身很可能不是 PlayerInputManager组件，但另一个附加到同一 GameObject 的组件。

=>不会阻止 PlayerInputManager 将其放入 PawnInputs！

您应该像 after GetComponent 那样进行检查

foreach (var actor in _actors)
{
    if(actor.TryGetComponent<IPawnInput>(out var pawnInput))
    { 
        if(pawnInput is PlayerInputManager)
        {
            // personally I would just ignore it and not throw an exception
            // that seems a bit overkill ;)
            continue;
        }

        PawnInputs.Add(pawnInput);
    }
}