防止/サブクラスがオーバーライド部材

Question

以下の擬似C＃コードを考える呼び出し場合に警告を生成する：（Work()メソッドをオーバーライド）派生クラスがWork()メソッドを呼び出すことは、この設計で

public abstract class Worker 
{ 
    public void DoWork() 
    { 
     Prepare(); 
     Work(); 
     Cleanup(); 
    } 

    protected abstract void Work(); 
} 

、それが可能です直接。

私はどのようにサブクラスがこの呼び出しを行うか、または少なくともコンパイラの警告を生成しないようにするためのヒントを探しています。

なぜ私はこれをしたいですか？ ワーカーは実際にWork()を別のスレッドにディスパッチします。 DoWork()はロックに注意します。

---

私は知っている、これは奇妙です。 Work()は、DoMyWork()へのメソッド呼び出しで構成され、そのオブジェクトはDoMyWork()を別の場所に呼び出す可能性があります。それを傍受することは不可能です。

とにかく。私はその問題に関する考えを感謝します。

Answer 1

エリスのおかげとコーリー私はそれを考え出したと思う。この方法はどうですか：

public abstract class Worker 
{ 
    public void DoWork() 
    { 
     if (Work != null) 
     { 
      Prepare(); 
      Work(); 
      Cleanup(); 
     } 
    } 

    protected Action Work { private get; set; } 
} 

public class ImplWorker { 
    public ImplWorker() { 
     Work = //whatever 
    } 
} 

まだ彼は彼が本当に他の場所やものに保存することができたら。しかし、それは私が想像できる限り遠くに行きます。

Answer 2

作業は、リソース（例えば、リモート・サーバー、データベース接続）を使用して行われる、またはあなたがこのようにそれを行うことができ、そのように変更することができます場合は、次の

public interface IResource 
{ 
} 

public abstract class Worker 
{ 
    private class TheResource : IResource 
    { 
     // implement 
    } 

    public void DoWork() 
    { 
     Prepare(); 

     var resObj = new TheResource(); 
     Work(resObj); 
     // if needed, cleanup/dispose resObj here 

     Cleanup(); 
    } 

    protected abstract void Work(IResource resourceObj); 
} 

Answer 3

これはややハックですが、メソッドを仮想に変更し、新しいメソッドを呼び出すボディを追加してからObsoleteAttributeを追加することができます。

protected void DoWork() 
    { 
     // New method 
    } 

    [Obsolete("This method is deprecated.")] 
    protected virtual void Work() 
    { 
     DoWork(); 
    } 

このメソッドを仮想としてマークすると、以前はWork（）をオーバーライドする必要があったため、既存の子クラスが破損することはありません。