Question

继承还是新手，所以我需要一些帮助。目前，我有一项任务是创建一个使用3武器继承的基本游戏。我的基类：

public class Weapons : MonoBehaviour
{
    public int rateOfFire;
    public string myName;

    public Weapons(string Name)
    {
        myName = Name;
    }
}

其中一个子课程：

public class Rifle : Weapons
{
    public Rifle(string Name) 
        : base(Name)
    {
        myName = Name;
        rateOfFire = 5;
    }
}

我的第一个问题是，即使我这样做也是如此吗？

我的第二个问题是如何将这些实例化为我的Unity场景？我已经编写了一种切换武器的方法，这是我的播放器脚本上的一个数组，所以我会在这里插入一个实例化的行还是创建另一种方法？缺少一些线条，但这只是它的正义：

public GameObject[] weapons;
public int currentweapon = 0;
private int invweap;
void Start()
{
    //Managing weapons
    invweap = weapons.Length;
    SwitchWeapon(currentweapon);
}
void SwitchWeapon(int index)
{
    for (int i = 0; i<invweap;i++)
    {
        if (i == index)
        {
            weapons[i].gameObject.SetActive(true);
        }
        else
        {
            weapons[i].gameObject.SetActive(false);
        }
     }
}

Answer 1

我的第一个问题是，如果我这样做的话，那就是事实吗？

非常接近。我们来看看。

public class Weapons : MonoBehaviour
{

我认为这是Unity的限制，这种类型来自给定的基类型。通常人们永远不会说武器是一种“单一行为”，不管是什么。

是否有任何武器不是更具体的武器？不，所以这应该是一个抽象的类。

除非类本身代表一组事物，否则应将类命名为单数。这应该是“武器”，而不是“武器”。

abstract class Weapon: MonoBehaviour 
{

继续前进。

public int rateOfFire;

我们已经遇到麻烦了。公共字段在C＃中是一种糟糕的代码味道。如果你想要一个公共财产，那就把它变成一个财产：

public int RateOfFire { get; set; }

更多关于鼻子：火速仅适用于远程武器。你在基类中有这个，暗示它适用于所有武器。你不开剑。

public string myName;

再次，公共领域，不好。把它变成一个财产。不要叫它myName。称之为它：Name：

public string Name { get; set; }

继续前进。

public Weapons(string Name)
{
    myName = Name;
}

使用camelCase作为参数，使用PascalCase作为属性：

public Weapon(string name)
{
    Name = name;
}

姓名会改变吗？如果没有，请把它变成一个只有房产。

public string Name { get; }

（取决于您使用的C＃版本可能不合法。如果不起作用，请尝试{ get; private set; }。）

可以在ctor中设置名称意味着武器的每个实例都有一个名称。是对的吗？这似乎不对。有没有名为Bob's Sword的剑，以及名为“Stormbringer”的剑和诸如此类的剑？我原以为名称会与类型相关联，而不是与给定的实例相关联。我希望该名称是一个抽象属性，由派生类型覆盖。

public abstract string Name { get; }

转到派生类型：

public Rifle(string Name) 
    : base(Name)
{
    myName = Name;
    rateOfFire = 5;
}

这搞砸了。您已经在基类构造函数中设置了名称;不要再设置了！为什么名称由基础ctor设定，但由衍生的ctor设定的射速？火灾率应该是派生类型的属性，或者应该与名称一起传递。

让我们摆脱它们，只是在派生类中重写fire的名称和速率抽象属性。让我们改进类型层次结构，以便武器基类不包括仅对远程武器通用的东西。

所以，把它们放在一起：

public abstract class Weapon : MonoBehaviour
{
  public abstract string Name { get; }
}

public abstract class RangedWeapon : Weapon
{
  public abstract int RateOfFire { get; }
}

public class Rifle : RangedWeapon
{
  public override string Name { get { return "rifle"; } }
  public override int RateOfFire { get { return 5; } }
}

public class Sword : Weapon
{
  public override string Name { get { return "sword"; } }
}

有意义吗？

我的第二个问题是如何将这些实例化为我的Unity场景？

最好在StackOverflow上针对每个问题提出一个问题，因为通常情况是第二个问题无法得到解答。就像我在这里没有回答的那样。

实例化一个继承的对象

1 个答案: