为了获得Customer和Employee的Display()方法,我实际上开始用Person抽象类替换我的IPerson接口.@H_502_3@
但后来我停下来,记得一个播客,其中鲍勃叔叔告诉Scott Hanselman关于单一责任原则,你应该有很多小班,每个人做一件特定的事情,即一个Customer类不应该有一个Print()并保存()和CalculateSalary()方法,但您应该有CustomerPrinter类和CustomerSaver类以及CustomerSalaryCalculator类.@H_502_3@
这似乎是一种奇怪的编程方式.然而,摆脱我的界面也感觉不对(因为很多IoC容器和DI示例本身使用它们)所以我决定尝试单一责任原则.@H_502_3@
所以下面的代码与我过去编程的不同(我会用Display()方法创建一个抽象类并摆脱界面)但是基于我所听到的关于解耦和S.O.L.I.D的内容.原则,这种新的编码方式(接口和PersonDisplayer类)我认为这是正确的方法.@H_502_3@
我想听听其他人是否在这个问题上采用相同的方式,或者是否经历过这种情况的正面或负面影响(例如,每个人做一件特别的事情都是笨拙的,等等).@H_502_3@
using System; namespace TestGeneric33 { class Program { static void Main(string[] args) { Container container = new Container(); Customer customer1 = container.InstantiateType<Customer>("Jim","Smith"); Employee employee1 = container.InstantiateType<Employee>("Joe","Thompson"); Console.WriteLine(PersonDisplayer.SimpleDisplay(customer1)); Console.WriteLine(PersonDisplayer.SimpleDisplay(employee1)); Console.ReadLine(); } } public class Container { public T InstantiateType<T>(string firstName,string lastName) where T : IPerson,new() { T obj = new T(); obj.FirstName = firstName; obj.LastName = lastName; return obj; } } public interface IPerson { string FirstName { get; set; } string LastName { get; set; } } public class PersonDisplayer { private IPerson _person; public PersonDisplayer(IPerson person) { _person = person; } public string SimpleDisplay() { return String.Format("{1},{0}",_person.FirstName,_person.LastName); } public static string SimpleDisplay(IPerson person) { PersonDisplayer personDisplayer = new PersonDisplayer(person); return personDisplayer.SimpleDisplay(); } } public class Customer : IPerson { public string FirstName { get; set; } public string LastName { get; set; } public string Company { get; set; } } public class Employee : IPerson { public string FirstName { get; set; } public string LastName { get; set; } public int EmployeeNumber { get; set; } } }
解决方法
你的类非常简单,并且非常适合于一个带有你所提到的Print()和Save()函数的抽象类.我倾向于将这种设计保留在你当前的设计之上.@H_502_3@
但是,如果打印和保存是可能以不同方式执行的更复杂的任务,那么将保证专用的打印机或Saver类,因为现在该责任更加复杂.制作新课程的“复杂性”阈值非常主观,并且取决于具体情况,但最终,代码只是我们低级人类理解的抽象,因此使其成为最直观的.@H_502_3@
你容器类有点误导.它实际上并不包含任何东西.它实际上实现了Factory Method Pattern,并将从命名工厂中受益.@H_502_3@
此外,您的PersonDisplayer永远不会实例化,并且可以通过静态方法提供其所有功能,那么为什么不将其设置为静态类呢?对于诸如打印机或储蓄器之类的utility classes而言,这种情况并不常见.除非您需要具有不同属性的打印机的单独实例,否则请将其保持静态.@H_502_3@