在可扩展类层次结构中实现单一责任原则的技术/模式

Question

单一职责原则说明，例如，Invoice类不应包含自行打印的代码。打印应该分成不同的类别。

但是假设你有Invoice类的软件的不同层的层次结构：

namespace CoreLayer { 
    public class Invoice { 
     public virtual void Print() { 
      ... 
     } 
    } 
} 

namespace CustomizedLayer { 
    public class LaborInvoice : Invoice { 
     public override void Print() { 
      ... 
     } 
    } 

    public class AccountInvoice : Invoice { 
     public override void Print() { 
      ... 
     } 
    } 
} 

什么技术或设计模式可用于分离出印刷reponsibility？

思路：

• 与测试为Invoice每个子类，并运行相应的印码一个伟大的大if声明一个单独的类。这似乎是错误的。
• 访客模式。问题在于访问者接口需要存在于核心层中，并且引用了定制层中的类。我希望能够在修改Core层的Customized图层中添加新的子类。

Answer 1

您可能要考虑Acyclic Visitor (PDF)。

Answer 2

您确实需要子类别发票吗？除了印刷以外，发票有什么不同？如果没有，有没有需要有不同类型的Invoice，你只需要不同类型的InvoicePrinter传递给Invoice实例：

namespace CoreLayer 
{ 
    public class IInvoicePrinter 
    { 
     void Print(Invoice invoice); 
    } 

    public class Invoice 
    { 
    } 
} 

namespace CustomizedLayer 
{ 
    public class LaborInvoicePrinter : IInvoicePrinter 
    { 
     public void Print(Invoice invoice) 
     { 
      ... 
     } 
    } 

    public class AccountInvoicePrinter : IInvoicePrinter 
    { 
     public void Print(Invoice invoice) 
     { 
      ... 
     } 
    } 
} 

你应该有某种的IoC为您提供适当的InvoicePrinter实例。

Answer 3

我认为bellow解决方案对于C＃很有用，它没有外部参数if，据我所知，不推荐使用visitor模式。

public class InvoicePrinterManager 
{ 
    public void Print(AccountInvoice invoice) 
    { 
     AccountInvoicePrinter p1 = new AccountInvoicePrinter(invoice); 
     p1.print(); 
    } 

    public void Print(LaborInvoice invoice) 
    { 
     LaborInvoicePrinter p1 = new LaborInvoicePrinter(invoice); 
     p1.print(); 
    } 
} 

public class InvoicePrinter<T> where T : Invoice, new() 
{ 
    T instance; 

    public InvoicePrinter(T invoice) 
    { 
     if (invoice != null) 
     { 
      this.instance = invoice; 
     } 
     else 
      instance = new T(); 
    } 

    public virtual void Print() 
    { 
     /// Arrange objects as you want and print them. 
    } 
} 

public class AccountInvoicePrinter : InvoicePrinter<AccountInvoice> 
{ 
    public AccountInvoicePrinter(AccountInvoice invoice) 
     : base(invoice) 
    { 
    } 

    public override void Print() 
    { 
     /// todo 
    } 
} 

public class LaborInvoicePrinter : InvoicePrinter<LaborInvoice> 
{ 
    public LaborInvoicePrinter(LaborInvoice invoice) 
     : base(invoice) 
    { 
    } 
    public override void Print() 
    { 
     /// todo: use instance 
    } 
} 

public class Test 
{ 
    public void TestPrint() 
    { 
     LaborInvoice li = new LaborInvoice(); 
     InvoicePrintManager printerManager = new InvoicePrintManager(); 
     printerManager.Print(li); 
    } 
}