对设计及重构的一点反思

本文是我对一个项目中一个小功能点的演进及重构过程的一点反思与心得。

成都创新互联专注于印台企业网站建设,成都响应式网站建设,商城网站定制开发。印台网站建设公司,为印台等地区提供建站服务。全流程按需网站策划，专业设计，全程项目跟踪，成都创新互联专业和态度为您提供的服务

背景：

本项目是一个电子商务类的网站，其中有个功能是在订单状态改变到某种状态后向客户发送通知短信的功能，短信及网关功能均已封装为组建的方式，我们直接调用即可。

为更清晰明白地说明与本主题相关的功能，在此我以一个控制台的程序方式说明代码的演进过程。

重构的演进过程：

最初我们是如大多数项目一样，为在规定的时间内完成相关功能点而努力奋斗着，这个功能点的主要代码如下：

 
 
 
    
  
  
  
  1. v1  
    2. 
  
  
  
  2. 
  
  
  
  3.         static void SendSMS_V1(DataTable dt) 
    4. 
  
  
  
  4.         { 
    5. 
  
  
  
  5.             if (null == dt) return; 
    6. 
  
  
  
  6. 
  
  
  
  7.             for (int i = 0; i < dt.Rows.Count; i++) 
    8. 
  
  
  
  8.             { 
    9. 
  
  
  
  9.                 var row = dt.Rows[i]; 
    10. 
  
  
  
  10.                 OrderStateEnum state = (OrderStateEnum)((int)row["OrderState"]); 
    11. 
  
  
  
  11.                 string template = string.Empty; 
    12. 
  
  
  
  12.                 switch (state) 
    13. 
  
  
  
  13.                 { 
    14. 
  
  
  
  14.                     case OrderStateEnum.UnConfirmed: 
    15. 
  
  
  
  15.                         template = "尊敬的{0}，你好！你的订单已成功下达，请尽快付款以便配送。"; 
    16. 
  
  
  
  16.                         break; 
    17. 
  
  
  
  17.                     case OrderStateEnum.Confirmed: 
    18. 
  
  
  
  18.                         template = "尊敬的{0}，你好！你的订单(订单号:{1})已被确认，请耐心等待。"; 
    19. 
  
  
  
  19.                         break; 
    20. 
  
  
  
  20.                     case OrderStateEnum.Cancel: 
    21. 
  
  
  
  21.                         template = "尊敬的{0}，你好！你的订单(订单号:{1})已被取消，具体原因请上网查看。"; 
    22. 
  
  
  
  22.                         break; 
    23. 
  
  
  
  23.                     case OrderStateEnum.Finish: 
    24. 
  
  
  
  24.                         template = "尊敬的{0}，你好！你的订单(订单号:{1})已完成，网站感谢您的支持与配合，欢迎再次光临。"; 
    25. 
  
  
  
  25.                         break; 
    26. 
  
  
  
  26.                     default: 
    27. 
  
  
  
  27.                         break; 
    28. 
  
  
  
  28.                 } 
    29. 
  
  
  
  29. 
  
  
  
  30.                 string content = string.Format(template, row["CustomerName"], row["OrderID"]); 
    31. 
  
  
  
  31.                 SendSMS.Send(row["phone"].ToString(), content); 
    32. 
  
  
  
  32.             } 
    33. 
  
  
  
  33.         }
    34.

在项目上线初期，这段代码工作良好。

在运营一段时间后运营部门同事陆续提出要在某些地方将产品名称给加上去，在这一改动过程中，我发现代码没有和数据相分离，再者如要增加一个订单状态后增加相应的短信提示或者取消某一个状态的短信提示，这个改动过程有点麻烦。于是初步想到将短信的内容放到配置文件中，在调用的时候读取订单状态对应的配置文件然后格式化即可。如读取的内容为空或者该文件不存在则跳过不发送。主要代码如下：

 
 
 
    
  
  
  
  1. v2 内容和数据分离  
    2. 
  
  
  
  2. 
  
  
  
  3.         #region v2 内容和数据分离  
    4. 
  
  
  
  4.         static void SendSMS_V2(DataTable dt) 
    5. 
  
  
  
  5.         { 
    6. 
  
  
  
  6.             if (null == dt) return; 
    7. 
  
  
  
  7. 
  
  
  
  8.             for (int i = 0; i < dt.Rows.Count; i++) 
    9. 
  
  
  
  9.             { 
    10. 
  
  
  
  10.                 var row = dt.Rows[i]; 
    11. 
  
  
  
  11.                 string path = Path.Combine(AppDomain.CurrentDomain.BaseDirectory, "SMSTemplates", 
    12. 
  
  
  
  12.                     "V2", row["OrderState"].ToString() + ".txt"); 
    13. 
  
  
  
  13. 
  
  
  
  14.                 var template = FileHelper.Read(path); 
    15. 
  
  
  
  15.                 if (!string.IsNullOrEmpty(template)) { 
    16. 
  
  
  
  16.                     string content = string.Format(template, row["CustomerName"], row["OrderID"], row["ProductName"]); 
    17. 
  
  
  
  17.                     SendSMS.Send(row["phone"].ToString(), content); 
    18. 
  
  
  
  18.                 } 
    19. 
  
  
  
  19.             } 
    20. 
  
  
  
  20.         } 
    21. 
  
  
  
  21.         #endregion
    22.

模版

 
 
 
    
  
  
  
  1. 尊敬的{0}，你好！你的订单(订单号:{1})已被确认，请耐心等待。
    2.

随着运营进行，运营部门不断地提出将某些字段在某些地方显示， string.Format 后的参数不断地增加，每次改后都要将整个过程重新测试一通，很是让人头疼！我的想法是开发这边提供一个数据标签列表，同时在后台提供操作界面将短信管理模板让运营同事自己去修改，不用每次都找我们？有了以上想法，使用一个自定义的 formatter ：

 
 
 
    
  
  
  
  1. 自定义参数格式化  
    2. 
  
  
  
  2. 
  
  
  
  3.     public class IndexerNamedFormatter : IFormatProvider, ICustomFormatter 
    4. 
  
  
  
  4.     { 
    5. 
  
  
  
  5.         public IndexerNamedFormatter() { } 
    6. 
  
  
  
  6. 
  
  
  
  7.         public object GetFormat(Type formatType) 
    8. 
  
  
  
  8.         { 
    9. 
  
  
  
  9.             if (formatType == typeof(ICustomFormatter)) 
    10. 
  
  
  
  10.                 return this; 
    11. 
  
  
  
  11. 
  
  
  
  12.             throw new TypeAccessException("不匹配的类型。"); 
    13. 
  
  
  
  13.         } 
    14. 
  
  
  
  14. 
  
  
  
  15.         public string Format(string format, object arg, IFormatProvider formatProvider) 
    16. 
  
  
  
  16.         { 
    17. 
  
  
  
  17.             if (null == arg) 
    18. 
  
  
  
  18.                 throw new ArgumentNullException("参数 arg 不能为 null"); 
    19. 
  
  
  
  19. 
  
  
  
  20.             int indexer = 0; 
    21. 
  
  
  
  21.             bool isIndexed = int.TryParse(format, out indexer); 
    22. 
  
  
  
  22. 
  
  
  
  23.             //如是 datarow  
    24. 
  
  
  
  24.             if (arg is System.Data.DataRow) 
    25. 
  
  
  
  25.             { 
    26. 
  
  
  
  26.                 return GetStringFromDataRow(format, arg, indexer, isIndexed); 
    27. 
  
  
  
  27.             } 
    28. 
  
  
  
  28. 
  
  
  
  29.             //如是 datareader 之类的 
    30. 
  
  
  
  30.             if (arg is System.Data.IDataRecord) 
    31. 
  
  
  
  31.             { 
    32. 
  
  
  
  32.                 GetStringFromIDataRecord(format, arg, indexer, isIndexed); 
    33. 
  
  
  
  33.             } 
    34. 
  
  
  
  34. 
  
  
  
  35.             return string.Empty; ; 
    36. 
  
  
  
  36. 
  
  
  
  37.         } 
    38. 
  
  
  
  38. 
  
  
  
  39.         private static void GetStringFromIDataRecord(string format, object arg, int indexer, bool isIndexed) 
    40. 
  
  
  
  40.         { 
    41. 
  
  
  
  41.             var dr = (System.Data.IDataRecord)arg; 
    42. 
  
  
  
  42.             string.Format("{0}", isIndexed ? dr[indexer] : dr[format]); 
    43. 
  
  
  
  43.         } 
    44. 
  
  
  
  44. 
  
  
  
  45.         string GetStringFromDataRow(string format, object arg, int indexer, bool isIndexed) 
    46. 
  
  
  
  46.         { 
    47. 
  
  
  
  47.             var row = (System.Data.DataRow)arg; 
    48. 
  
  
  
  48.             return string.Format("{0}", isIndexed ? row[indexer] : row[format]); 
    49. 
  
  
  
  49.         }
    50. 
 
 
 
 
 
 
    
  
  
  
  1. v3 使用自定义标签  
    2. 
  
  
  
  2. 
  
  
  
  3. #region v3 使用自定义标签  
    4. 
  
  
  
  4.         static void SendSMS_V3(DataTable dt) 
    5. 
  
  
  
  5.         { 
    6. 
  
  
  
  6.             if (null == dt) return; 
    7. 
  
  
  
  7. 
  
  
  
  8.             IndexerNamedFormatter formatter = new IndexerNamedFormatter(); 
    9. 
  
  
  
  9.             for (int i = 0; i < dt.Rows.Count; i++) 
    10. 
  
  
  
  10.             { 
    11. 
  
  
  
  11.                 var row = dt.Rows[i]; 
    12. 
  
  
  
  12.                 string path = Path.Combine(AppDomain.CurrentDomain.BaseDirectory, "SMSTemplates", 
    13. 
  
  
  
  13.                     "V3", row["OrderState"].ToString() + ".txt"); 
    14. 
  
  
  
  14. 
  
  
  
  15.                 var template = FileHelper.Read(path); 
    16. 
  
  
  
  16.                 if (!string.IsNullOrEmpty(template)) 
    17. 
  
  
  
  17.                 { 
    18. 
  
  
  
  18.                     string content = string.Format(formatter, 
    19. 
  
  
  
  19.                         template, row); 
    20. 
  
  
  
  20.                     SendSMS.Send(row["phone"].ToString(), content); 
    21. 
  
  
  
  21.                 } 
    22. 
  
  
  
  22.             } 
    23. 
  
  
  
  23.         } 
    24. 
  
  
  
  24.         #endregion
    25.

模版

 
 
 
    
  
  
  
  1. 尊敬的{0:CustomerName}，你好！你的订单(订单号:{0:OrderID}，产品：产品{0:productname})已被确认，请耐心等待。
    2.

将数据字段取出来后写成一个标签列表文档提供给运营人员后，从此关于这一块的修改要求安静了。

以上只是一个小小设计技巧，这也让我明白需求的准确把握与挖掘是何等地重要！往往客户今天说要这样，明天要那样，大概很多人都在抱怨：你们真麻烦！但在需求不断地出现时我们是不是在修改的时候也反思下是不是我们未准确把握他们所想要的功能而让功能设计出了点问题？当然我也不崇尚一开始就大谈设计，过度设计要付出大量的时间成本和可能导致实现的复杂度的增加。

一点疑问：

通过查阅文档我一直感觉 arg is System.Data.DataRow 这种实现方式很是别扭，为什么索引器不定义为一个接口 ？有谁有更好的实现方法麻烦告知，谢谢！

感谢您的阅读！文中所涉及到的代码可从此下载。

新闻名称：对设计及重构的一点反思
网站地址：http://www.shufengxianlan.com/qtweb/news49/151449.html

网站建设、网络推广公司-创新互联，是专注品牌与效果的网站制作，网络营销seo公司；服务项目有等

广告

声明：本网站发布的内容（图片、视频和文字）以用户投稿、用户转载内容为主，如果涉及侵权请尽快告知，我们将会在第一时间删除。文章观点不代表本网站立场，如需处理请联系客服。电话：028-86922220；邮箱：631063699@qq.com。内容未经允许不得转载，或转载时需注明来源： 创新互联

猜你还喜欢下面的内容

微信小程序知识

行业网站建设

安娜尚美　　 　重庆网站营销　　 　郫县明安消防器材　　 　成都网站推广　　 　一元广告　　 　成都广告设计　　 　成都网站建设哪家好　　 　成都网络公司　　 　邛崃网站建设　　 　虚拟主机购买　　 　成都网站设计　　 　网站定制　　 　四川成都网站制作　　 　门户网站建设方案　　 　微信小程序开发　　 　网络营销公司　　 　400电话办理　　 　成都标志设计　　 　网站建设方案　　 　成都茶叶销售