Unity中通过字符串动态调用其他脚本方法的问题求助

错误根源 #

你犯了两个核心错误：

1. SendMessage传参错误：你把变量名"rewriteMethodName"当成字符串传了，而不是变量里存储的方法名字符串。
2. 反射调用错误：同样硬编码了"rewriteMethodName"字符串，没有使用变量值；另外代码里的实例名instanceManager和instanceMapManager属于拼写错误，应该统一成Manager类里的静态实例名。

方案1：修复SendMessage调用 #

直接修改PopUp脚本的DoThings方法，去掉引号，使用变量rewriteMethodName：

public void DoThings()
{
    // 提前获取MapManager引用，避免每次Find的性能消耗
    GameObject mapManager = GameObject.Find("MapManager");
    if(mapManager != null)
    {
        // 传递变量值，而非硬编码字符串
        mapManager.SendMessage(rewriteMethodName, SendMessageOptions.DontRequireReceiver);
    }
}

建议加上SendMessageOptions.DontRequireReceiver，避免找不到方法时抛出错误，更安全。

方案2：使用反射（更灵活的方式） #

先修正反射代码的错误，确保传递变量值而非硬编码字符串，同时修正实例名：

第一步：确保Manager类有正确的静态实例 #

public class Manager : MonoBehaviour
{
    // 统一静态实例名
    public static Manager instance;

    private void Awake()
    {
        instance = this;
    }

    public void ActionA()
    {
        // 执行操作
    }
    
    public void ActionB()
    {
        // 执行其他操作
    }
}

第二步：修正PopUp里的反射调用 #

public void DoThings()
{
    if(Manager.instance == null) return;

    // 使用变量rewriteMethodName，而非硬编码字符串
    MethodInfo method = Manager.instance.GetType().GetMethod(rewriteMethodName, 
        BindingFlags.NonPublic | BindingFlags.Public | BindingFlags.Instance);
    
    if(method != null)
    {
        method.Invoke(Manager.instance, null);
    }
    else
    {
        Debug.LogWarning($"找不到方法：{rewriteMethodName}");
    }
}

加上空值判断，避免空引用错误，同时打印警告方便调试。

额外优化建议 #

1. 避免重复查找对象：在PopUp的Awake或Start方法里提前获取MapManager的引用，不要每次点击都调用GameObject.Find，提升性能。
2. 类型安全替代方案：如果不想用字符串匹配，可以给每个GameObject绑定对应的委托或枚举，比如：

   // 在PopUpCaller里定义枚举
   public enum ActionType { ActionA, ActionB }
   public ActionType actionType;
   
   // 点击时传递枚举值给PopUp，而非字符串
   public void TaskOnClick()
   {
       popUp.SetActive(true);
       popUp.transform.position = Input.mousePosition;
       PopUp.instance.selectedAction = actionType;
   }
   
   // 在PopUp里根据枚举调用方法
   public ActionType selectedAction;
   public void DoThings()
   {
       switch(selectedAction)
       {
           case ActionType.ActionA:
               Manager.instance.ActionA();
               break;
           case ActionType.ActionB:
               Manager.instance.ActionB();
               break;
       }
   }
   

   这种方式完全避免字符串拼写错误，类型更安全，调试也更方便。

内容的提问来源于stack exchange，提问作者AndreyChe