避免調用基類的受保護成員

Question

下面是樹結構的一個愚蠢示例，其中每個節點都執行不同類型的操作，但只有一個節點（任何節點）必須在開始和結束時執行一些常見工作的項目。

public abstract class Employee 
{ 
    public void StartProject() 
    { 
     AnnounceProjectToNewspapers(); 
     DoActualWork(); 
     PutProductOnMarket(); 
    } 

    protected abstract void DoActualWork(); 

    private void AnnounceProjectToNewspapers() { } 
    private void PutProductOnMarket() { } 
} 

public class Engineer : Employee 
{ 
    protected override void DoActualWork() 
    { 
     // Build things. 
    } 
} 

public class Salesman : Employee 
{ 
    protected override void DoActualWork() 
    { 
     // Design leaflets. 
    } 
} 

public class Manager : Employee 
{ 
    protected override void DoActualWork() 
    { 
     // Make gantt charts. 

     // Also delegate. 
     foreach (var subordinate in subordinates) 
      // ...but then compiler stops you. 
      subordinate.DoActualWork(); 
    } 

    private List<Employee> subordinates; 
} 

問題是，您無法在基類上調用受保護的方法DoActualWork()。

兩個解決方案，我看到的是：

• 讓DoActualWork()公衆。但是，這將允許任何人在沒有AnnounceProjectToNewspapers()或PutProductOnmarket()的情況下致電。
• 使DoActualWork()內部。但是這會阻止其他組件使用「管理系統」。

是否有一個標準的變通方法來避免這種限制？

Answer 1

我有幾種方法來實現你所需要的。

1. 第一個也是顯而易見的方法是從Manager而不是員工繼承Engineer和Salesman。但是這使得繼承聽起來很荒謬。每個工程師是一個管理器（我希望這是真的。）

2. 正如你所提到的，

讓DoActualWork（）公共

這並不工作。爲了防止任何其他方法調用DoActualWork（），您可以使用reflection/stacktrace找到類的類型並阻止無效類型。請參閱here

但這兩種方法都對我感到嚴峻。應該有一種方法來設計類來獲得你所需要的。

Answer 2

我想出了這樣一個解決方案（這還沒有經過測試）

public interface IEmployee { 
    void StartProject(); 
    void DoWork(); 
} 

public abstract class EmployeeBase : IEmployee { 
    public void StartProject() { 
     AnnounceProjectToNewspapers(); 
     DoActualWork(); 
     PutProductOnMarket(); 
    } 

    void IEmployee.DoWork() { 
     // Maybe set a flag to see whether the work has been already done by calling StartProject(). 
     this.DoActualWork(); 
    } 

    protected abstract void DoActualWork(); 

    private void AnnounceProjectToNewspapers() { } 
    private void PutProductOnMarket() { } 
} 

public class Employee : EmployeeBase { 
    protected override void DoActualWork() { 
     // Log system Action 
    } 
} 

public class Engineer : Employee { 
    protected override void DoActualWork() { 
     // Build things. 
    } 
} 

public class Salesman : Employee { 
    protected override void DoActualWork() { 
     // Design leaflets. 
    } 
} 

public class Manager : Employee { 
    protected override void DoActualWork() { 
     DoWorkInternal(); 
    } 

    private void DoWorkInternal() { 
     foreach (var subordinate in subordinates) 
      subordinate.DoWork(); 
    } 

    private List<IEmployee> subordinates; 
} 

Answer 3

所有響應思考，一個妥協只是穿過我的腦海：通過內部方法使用一個間接調用。換句話說，一個窮人的朋友通過非正式的評論聲明。

下面是相關位：

public abstract class Employee 
{ 
    // Meant to be called by Manager class only. 
    internal void FollowTheLeader() 
    { 
     DoActualWork(); 
    } 
} 

public class Manager : Employee 
{ 
    protected override void DoActualWork() 
    { 
     // Make gantt charts. 

     // Also delegate. 
     foreach (var subordinate in subordinates) 
      // ...but then compiler stops you. 
      subordinate.FollowTheLeader(); 
    } 
} 

從其他組件派生類將能夠自定義工作員工和經理做的，但它們將不能夠讓任何人做的工作沒有適當的上下文。

程序集中的類將能夠做任何事情，但讓我們假設您可以信任開發框架的人員按照註釋中的說明操作。