內部類Constructer ...做這個工作嗎？

Question

好吧，不好意思，我知道你們會告訴我我需要搜索和搜索，但是我已經確定我是正確的，並且認爲這樣會按照我想要的方式工作，但是我想我會在這裏問一下，並嘗試在我的學習經驗上得到一些專業幫助，因爲統一答案並不是那麼好。

反正我想開始構建另一個MMORPG，我也在同一時間學習c sharp。我有一個課程，我想在創建玩家類的同時創建一個Vocation類（玩家的職業，像法師，騎士等），所以我需要用一個ID來決定哪個職業和什麼職業它們繼承的屬性值。

這就是我的，這項工作，我正在努力？還是我在做可怕的錯事...？

編輯

using UnityEngine; 
using System.Collections; 

//DONE: abstract: a personage can't be "Vocation", but Mage, Warrior, Archer... 
public abstract class Vocation 
{ 
    //DONE: just a readonly property 
    public int Vid {get; } 
    //DONE: just a readonly property 
    public string Name { get { return _Name; } } 
    protected string _Name = "None"; 

    //DONE: let's ensure the property to be overriden 
    public abstract HitPointsPerLevel { get; } 
    public abstract ManaPointsPerLevel { get; } 

    //DONE: you don't want this constructor to be public, but protected only 
    //DONE: Assign all the data in one place 
    protected Vocation(int vid) 
    { 
     Vid = vid; 
    } 
} 

//DONE: do not declare derived class as inner one 
internal class Mage : Vocation 
{ 
    sealed public override float HitPointsPerLevel { get { return 12f; } } 
    sealed public override string _Name = "Mage"; 

    //DONE: typo constructor should have been "Mage" 
    public Mage() : base(1) 
    { 
    } 
} 

如何做IT現在看GUYS？

Answer 1

我建議重新設計實施

using UnityEngine; 
using System.Collections; 

//DONE: abstract: a personage can't be "Vocation", but Mage, Warrior, Archer... 
public abstract class Vocation 
{ 
    //DONE: just a readonly property 
    public int Vid {get; } 
    //DONE: just a readonly property 
    public string Name {get; } 

    //DONE: let's ensure the property to be overriden 
    public abstract HitPointsPerLevel { get; } 

    //DONE: you don't want this constructor to be public, but protected only 
    //DONE: Assign all the data in one place 
    protected Vocation(int vid, string name) 
    { 
     if (string.IsNullOrEmpty(name)) 
      throw new ArgumentNullException("name"); 

     Vid = vid; 
     Name = name; 
    } 
} 

//DONE: do not declare derived class as inner one 
internal class Mage : Vocation 
{ 
    sealed public override float HitPointsPerLevel { get { return 12f; } } 

    //DONE: typo constructor should have been "Mage" 
    public Mage() : base(1, "Mage") 
    { 
    } 
} 

Answer 2

該代碼將工作，除了構造函數名稱（Warrior而不是Mage）。

我在想，爲什麼Mage必須是內部嵌套類Vocation。那有什麼用？

我可以理解你爲什麼要做它internal：你不想讓外部程序集創建一個實例，你可以通過工廠來實現。但是，沒有理由讓類嵌套。基類可以獨立運作。只要將嵌套類拉出來，就可以在基類上使用protected成員來在基類​​和派生類之間進行通信。

一個小側面說明：您可以在這裏使用this代替base：

public Mage(int vid) : base(vid) 
{ 
    this.Vid = 1; 
    this.Name = "Mage"; 
} 

Answer 3

很少有東西，除了其他的答案來發表評論，但他們都在代碼審查中。也許這在codereview.stackexchange.com更好。

首先（這歸結於風格） - 我儘量避免使用像Vid這樣的變量 - vocationId沒有什麼問題 - 描述性更好。

第二，更相關的問題，我不知道你Mage /人，語義上講，是一種職業 - 所以它不應該繼承它。也許這是一個有職業的人，但它本身不是一種職業 - 它可能不會延伸職業功能。這是繼承的唯一意圖。

您是否試圖在實踐中繼承傳統概念中的鞋號角？

而且，順便說一句，在C＃6你有表達濃郁的性能，讓您可以將您的代碼一點：

public string Name => "None"; 

雖然這些看起來像常量給我。這基本上需要重新設計，從OOP封裝的基本原理開始，並保持簡單。