Unity工厂方法模式实现枪械系统的使用与优化咨询

一、现有代码核心错误修正 #

首先你目前的实现存在两个基础错误：

1. M16、Uzi都是继承自MonoBehaviour的组件类，不能直接用new关键字创建，你现有具体Creator里return new M16()的写法运行会直接报错。
2. 枪械类的Shoot方法逻辑错误：CreateBullet已经返回了生成的子弹实例，但你直接把子弹预设传入ShootBullet，操作的是预设资产而非场景内生成的子弹实例，需要修正。

二、基于现有架构实现枪械Prefab生成的方案 #

修正Creator类逻辑 #

调整具体Creator的实现，改为持有对应枪械的Prefab引用（Prefab上已挂载对应枪械业务脚本），工厂方法负责实例化Prefab并返回枪械组件：

public class M16Creator : GunCreator // ConcreteCreator
{
    // Inspector面板中拖拽赋值已经挂载M16脚本的枪械Prefab
    [SerializeField] private GameObject m16Prefab;
    public override Guns GunFactory()
    {
        GameObject gunObj = Instantiate(m16Prefab);
        return gunObj.GetComponent<Guns>();
    }
}

客户端调用示例 #

public class GunManager : MonoBehaviour
{
    // Inspector面板中拖拽赋值对应Creator的实例
    [SerializeField] private GunCreator m16Creator;
    [SerializeField] private GunCreator uziCreator;

    public Guns GenerateGun(string gunType)
    {
        switch(gunType)
        {
            case "M16":
                return m16Creator.GunFactory();
            case "Uzi":
                return uziCreator.GunFactory();
            default:
                return null;
        }
    }

    // 测试调用
    private void Update()
    {
        if(Input.GetKeyDown(KeyCode.Alpha1))
        {
            Guns newM16 = GenerateGun("M16");
            newM16.shoot();
        }
    }
}

三、省略单独Creator类的简化方案 #

完全可以省略每款枪械对应一个Creator类的冗余设计，推荐两种常用优化方案：

方案1：泛型通用Creator #

仅写一个泛型Creator基类，新增枪械不需要编写额外Creator逻辑：

public abstract class GunCreatorBase<T> : GunCreator where T : Guns
{
    [SerializeField] private GameObject gunPrefab;
    public override Guns GunFactory()
    {
        GameObject gunObj = Instantiate(gunPrefab);
        return gunObj.GetComponent<T>();
    }
}

// 新增枪械仅需要声明对应封闭泛型类即可，无需写逻辑代码
public class M16Creator : GunCreatorBase<M16> {}
public class UziCreator : GunCreatorBase<Uzi> {}

方案2：替换为简单工厂模式（最简便，推荐） #

直接废弃抽象Creator层，用静态工厂统一管理所有枪械生成，新增枪械仅需要加枚举值和Prefab映射即可：

// 定义枪械类型枚举
public enum GunType
{
    M16,
    Uzi,
    AK47 // 后续新增枪械直接加枚举值即可
}

// 静态工厂类
public static class GunFactory
{
    // 游戏初始化时将所有枪械Prefab存入该字典，也可以用ScriptableObject做配置表
    private static Dictionary<GunType, GameObject> _gunPrefabDict = new Dictionary<GunType, GameObject>();

    // 游戏启动时调用一次完成初始化
    public static void Init(Dictionary<GunType, GameObject> prefabDict)
    {
        _gunPrefabDict = prefabDict;
    }

    public static Guns CreateGun(GunType gunType)
    {
        if(_gunPrefabDict.TryGetValue(gunType, out var prefab))
        {
            GameObject gunObj = Object.Instantiate(prefab);
            return gunObj.GetComponent<Guns>();
        }
        Debug.LogError($"未配置对应枪械Prefab：{gunType}");
        return null;
    }
}

客户端调用非常简洁：

// 直接生成指定类型枪械
Guns newUzi = GunFactory.CreateGun(GunType.Uzi);
newUzi.shoot();

四、原有Shoot方法Bug修正 #

public override void shoot()
{
    // 接收CreateBullet返回的生成后的子弹实例
    GameObject bullet = CreateBullet(m16bullet);
    // 传入实例而非预设进行移动操作
    ShootBullet(bullet, m16bulletdirection);
}

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