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

前端之家收集整理的这篇文章主要介绍了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.f2er.com/csharp/97775.html

猜你在找的C#相关文章