责任链模式引发的思考

背景：

在业务频繁迭代的时候，有时候一个菜单页面对应的后台代码会越来越凌乱，一个方法本来就做A、B事情，后面迭代，变成会做A、B、C、D、E事情，且A、B又细分出a1、a2、a3、a4、b1、b2、b3。一个菜单页开始设计代码时，通常写法都较为面向对象，后面经多个人，可能是不同的人迭代后就越来越面向流程。

为什么后面越来越面向流程呢？大约是因为方便吧。如果只是迭代一个小逻辑，一旦要抽出一些通用性的东西，面向对象，写出好扩展的代码，要考虑的情况可能涉及：之前的业务，牵涉到的上下文变量，跟UI的交互（例如弹窗提示报错，询问用户是否继续某操作等等），把之前上线过的东西抽出来放到其他地方又增加了测试的地方等等，太难了。

但代码还能能简单改造一下，这种业务流程像是可以通过责任链设计模式来重构代码。但是这种模式有点隆重感，建好多类，又要抽象类，好像把一个菜单的后台代码复杂化了。但是它的本质，职责层层传递确实又让扩展容易，阅读代码容易。

代码重构过程：

重构前：

 private void buttonOld_Click(object sender, EventArgs e)  {      string fundId = "2323112";   string fundCode = ""; //跨a3、b1使用   string poolID = "";//跨a1、a2使用   string fundName = "";    //A业务   //a1验证   //...   poolID = "787908AA";   //...   //a2验证   int count = 0;   //...   if (count == 0)   {    MessageBox.Show($"{fundId}不在配置数据里;");    LogHelper("....");    return;   }   //a3验证   //....   if (poolID == "33333")   {    //...   }   fundName = "新能源混合";   fundCode = "908898";   //...   //B业务   //b1验证   List<string> list = new List<string>();   if (list.Contains(fundCode))    {    //...     if (MessageBox.Show($"{fundName}确认发送给某某用户", "提示", MessageBoxButtons.YesNo) != DialogResult.Yes)    {     return;    }   }  }

View Code

重构后：

using System;using System.Collections.Generic;using System.Data;using System.Windows.Forms;namespace WindowsFormsApp2{ public partial class Form1 : Form {  private DataTable ConfigTable;  private Dictionary<string, string> LimitDict;  public Form1()  {   InitializeComponent();  }  private void Form1_Load(object sender, EventArgs e)  {   ConfigTable = new DataTable();   //...   LimitDict = new Dictionary<string, string>();   //...  }    private void buttonNew_Click(object sender, EventArgs e)  {   string checkErrMesg = "";   string fundID = "2323112";   string fundCode;   string fundName;   //...   CheckBXFund checkBXFund = new CheckBXFund(LogHelper,ConfigTable, LimitDict);      if(checkBXFund.Handle(fundID,ref checkErrMesg) == false)   {    MessageBox.Show(checkErrMesg);    return;   }   fundCode = checkBXFund.OutFundCode;   fundName = checkBXFund.OutFundName;   //...   CheckRisk checkRisk = new CheckRisk();   //   if(checkRisk.Handle(fundCode, ref checkErrMesg)==false)   {    MessageBox.Show(checkErrMesg);    return;   }   //...  }  private void LogHelper(string Message)  {     } } /// <summary> /// 假设是整个A业务 /// </summary> internal class CheckBXFund {  //应当返回出去的值 1、基金名称 2 基金代码  public string OutFundName;  public string OutFundCode;  //外面传进来协助验证的一些变量  private DataTable InConfigTable;  private Dictionary<string, string> InRepeatDict;  private Action<string> LogAction; //有一些操作可能要用外面的窗体实例,例如只有窗体可以Log：this.Log()  //方法之间传递的变量，方便跨方法调用  private string PoolID;  //参数变量，不必重复获取的一些变量  public CheckBXFund(Action<string> logAction, DataTable dt, Dictionary<string, string> dict)  {   InConfigTable = dt;   InRepeatDict = dict;   LogAction = logAction;  }       public bool Handle(string fundId, ref string errorMessage)  {   if(IsRepeat(fundId,ref errorMessage) == false)   {    return false;   }   if (IsInConfig(fundId, ref errorMessage) == false)   {    return false;   }   if (IsCanGetName(fundId, ref errorMessage) == false)   {    return false;   }   return true;  }  /// <summary>  /// 假设是a1步骤  /// </summary>  /// <param name="fundId"></param>  /// <param name="errorMessage"></param>  /// <returns></returns>  pri......
原文转载：http://www.shaoqun.com/a/841211.html
跨境电商：https://www.ikjzd.com/
网上1号店：https://www.ikjzd.com/w/2263
 
promotion：https://www.ikjzd.com/w/127
 
打折网站：https://www.ikjzd.com/w/74
 
背景：在业务频繁迭代的时候，有时候一个菜单页面对应的后台代码会越来越凌乱，一个方法本来就做A、B事情，后面迭代，变成会做A、B、C、D、E事情，且A、B又细分出a1、a2、a3、a4、b1、b2、b3。一个菜单页开始设计代码时，通常写法都较为面向对象，后面经多个人，可能是不同的人迭代后就越来越面向流程。为什么后面越来越面向流程呢？大约是因为方便吧。如果只是迭代一个小逻辑，一旦要抽出一些通用性的东西
c2c模式：https://www.ikjzd.com/w/1576
做亚马逊建立品牌官网，如何注册域名？：https://www.ikjzd.com/articles/112186
亚马逊如何打造高效引流标题？：https://www.ikjzd.com/articles/112187
良心分析！速卖通自己运营和找代运营有什么区别？：https://www.ikjzd.com/articles/112188
三年亚马逊老司机教你如何安全快速留评打造爆款！：https://www.ikjzd.com/articles/112189
和黑人老外性经历口述 初体验黑人40厘米巨大连接：http://lady.shaoqun.com/a/247108.html
女人喜欢男人㖭 女人喜欢被㖭是什么感觉：http://www.30bags.com/m/a/249812.html
老师张开腿让你桶个够 昏暗公车被直接进入：http://www.30bags.com/m/a/249959.html
女生第一次被男朋友扒光衣服是什么体验：http://lady.shaoqun.com/a/397669.html
亚马逊卖家请注意：这30种产品上架有新规：https://www.ikjzd.com/articles/146305
沃尔玛"Prime Day"战报出炉 与亚马逊"对战"谁赢了?：https://www.ikjzd.com/articles/146307
出现大规模封号，账号关联是主因，请自检：https://www.ikjzd.com/articles/146306