在包含抽象方法的抽象类中重构具体方法

时间:2015-05-29 13:04:24

标签: c# design-patterns refactoring abstract-class

考虑以下代码,

abstract class AbstractClass
{
 public abstract void AbstractMethodA();
 public void ConcreteMethodA()
 {
  //Some operation
  ConcreteMethodB();
 }
}

 public void ConcreteMethodB()
 {
  //Huge code unrelated to this class
  AbstractMethodA();
 }
}

class DerivedClass : AbstractClass
{
public void AbstractMethodA()
{
//Some operation
}
}

现在我希望将ConcreteMethodB()移动到单独的类中,并从抽象类中的方法ConcreteMethodA()调用它。但是由于ConcreteMethodB()使用DerivedClass中实现的抽象方法AbstractMethodA(),我无法从新类访问AbstractMethodA()方法?有关如何解决此问题的想法吗?

2 个答案:

答案 0 :(得分:7)

你为什么不这样做

static class Helper {
 public static void ConcreteMethodB(AbstractClass caller)
 {
      //Huge code unrelated to this class
      caller.AbstractMethodA();
 }
}

然后在AbstractClass

abstract class AbstractClass
{
 public abstract void AbstractMethodA();
 public void ConcreteMethodA()
 {
  //Some operation
  Helper.ConcreteMethodB(this);
 }
}

编辑包括David Arno的基于界面的解耦建议:

static class Helper {
 public static void ConcreteMethodB(IAbstractClass caller)
 {
      //Huge code unrelated to this class
      caller.AbstractMethodA();
 }
}
interface IAbstractClass {
     void AbstractMethodA();
}

然后在AbstractClass

abstract class AbstractClass
{
 public abstract void AbstractMethodA();
 public void ConcreteMethodA()
 {
  //Some operation
  Helper.ConcreteMethodB(this);
 }
}

这为您提供了更好的图层隔离。当然,大卫在他提到使用“行动”和传递方法作为参数的帖子中提出的解决方案也值得考虑。

答案 1 :(得分:4)

要完全与两者分离,你可以采用"功能性"路线:

static class Helper 
{
    public static void ConcreteMethodB(Action caller)
    {
        //Huge code unrelated to this class
        caller();
    }
}

将AbstractClass更改为:

abstract class AbstractClass
{
    public abstract void AbstractMethodA();
    public void ConcreteMethodA()
    {
        Helper.ConcreteMethodB(AbstractMethodA);
    }
}