c# – 这是单一责任原则的一个例子吗？

我做了以下代码示例,以了解如何使用泛型方法签名.

为了获得Customer和Employee的display()方法,我实际上开始用Person抽象类替换我的IPerson接口.

但后来我停下来,记得一个播客,其中鲍勃叔叔告诉Scott Hanselman关于单一责任原则,你应该有很多小班,每个人做一件特定的事情,即一个Customer类不应该有一个Print()并保存()和CalculateSalary()方法,但您应该有CustomerPrinter类和CustomerSaver类以及CustomerSalaryCalculator类.

这似乎是一种奇怪的编程方式.然而,摆脱我的界面也感觉不对(因为很多IoC容器和DI示例本身使用它们)所以我决定尝试单一责任原则.

所以下面的代码与我过去编程的不同(我会用display()方法创建一个抽象类并摆脱界面)但是基于我所听到的关于解耦和S.O.L.I.D的内容.原则,这种新的编码方式(接口和Persondisplayer类)我认为这是正确的方法.

我想听听其他人是否在这个问题上采用相同的方式,或者是否经历过这种情况的正面或负面影响(例如,每个人做一件特别的事情都是笨拙的,等等).

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; }
    }
}

解决方法

我喜欢将 Single Responsibility Principle视为 separation of duties的实现.在我开始分割我的课程之前,我试着想一下每个课程应该负责什么.

你的类非常简单,并且非常适合于一个带有你所提到的Print()和Save()函数的抽象类.我倾向于将这种设计保留在你当前的设计之上.

但是,如果打印和保存是可能以不同方式执行的更复杂的任务,那么将保证专用的打印机或Saver类,因为现在该责任更加复杂.制作新课程的“复杂性”阈值非常主观,并且取决于具体情况,但最终,代码只是我们低级人类理解的抽象,因此使其成为最直观的.

你容器类有点误导.它实际上并不包含任何东西.它实际上实现了Factory Method Pattern,并将从命名工厂中受益.

此外,您的Persondisplayer永远不会实例化,并且可以通过静态方法提供其所有功能,那么为什么不将其设置为静态类呢？对于诸如打印机或储蓄器之类的utility classes而言,这种情况并不常见.除非您需要具有不同属性的打印机的单独实例,否则请将其保持静态.

原文地址：https://www.jb51.cc/csharp/97775.html

c# – 这是单一责任原则的一个例子吗？

解决方法

相关推荐