代码重构

      作者： 黎承湘

      不知从什么时候起，发现自己越来越来不喜欢看到有重复的代码。这段时间完成了CMS系统开发的第一期，有一点点空闲时间，查看一下代码。

     /// <summary> /// 产品分类 /// </summary> public class ProductCategory { public ProductCategory() { } public int ID { get; set; } public string Name { get; set; } public int ParentID { get; set; } } /// <summary> /// 文章分类 /// </summary> public class ArticleCategory { public ArticleCategory() { } public int ID { get; set; } public string Name { get; set; } public int ParentID { get; set; } }

      在对文章，产品，的列表显示，添加，修改的时候，都需要用到“分类下拉框”的显示。每次都得进行同一种开式的的数据绑定。

      手痒了。重构吧，6次重复，那还得了。

      第一次重构，用到了反射。

     /// <summary> /// 分类列表绑定到DropDownList /// </summary> /// <typeparam name="T"></typeparam> /// <param name="list">分类列表</param> /// <param name="parentID">父级ID</param> /// <returns>一组ListItem</returns> public static List<ListItem> BindData<T>(this List<T> list, int parentID) { return BindData(list, parentID, "--", "Name", "ID", "ParentID"); } /// <summary> /// 分类列表绑定到DropDownList上 /// </summary> /// <typeparam name="T"></typeparam> /// <param name="list">分类列表</param> /// <param name="parentID">父级ID</param> /// <param name="cateStr">区别分类级别</param> /// <param name="cateName">分类名的属性</param> /// <param name="cateID">分类ID的属性</param> /// <param name="cateParent">分类父级的属性</param> /// <returns>返回一组ListItem</returns> public static List<ListItem> BindData<T>(this List<T> list, int parentID, string cateStr, string cateName, string cateID, string cateParent) { List<ListItem> litems = new List<ListItem>(); var subList = list.Where(c => (int)c.GetType().GetProperty(cateParent).GetValue(c, null) == parentID).ToList(); foreach (T t in subList) { litems.Add(new ListItem(cateStr + t.GetType().GetProperty(cateName).GetValue(t, null).ToString(), t.GetType().GetProperty(cateID).GetValue(t, null).ToString())) litems = litems.Concat(BindData(list, (int)t.GetType().GetProperty(cateID).GetValue(t, null), cateStr + cateStr, cateName, cateID, cateParent)).ToList(); } return litems; }

      运行一下，OK，达到效果，但认真看了一下代码，觉得还是不够好。假如调用方法的时候，传错了 "cateName","cateID","cateParnet" 这几个参数，这代码就得报错了，虽然，能在代码块中进行判断。

 

觉得不够好，就还得改。那就用接口吧！（我以前刚写代码时，最讨厌接口，因为代码经常找不到....)

 

/// <summary> /// 产品分类 /// </summary> public class ProductCategory : ICategory { public ProductCategory() { } public int ID { get; set; } public string Name { get; set; } public int ParentID { get; set; } } /// <summary> /// 文章分类 /// </summary> public class ArticleCategory : ICategory { public ArticleCategory() { } public int ID { get; set; } public string Name { get; set; } public int ParentID { get; set; } } public interface ICategory { int ID { get; set; } string Name { get; set; } int ParentID { get; set; } } public static List<ICategory> BindData(List<ICategory> list, int parentID, string cateStr) { List<ICategory> newList = new List<ICategory>(); var subList = list.Where(c => c.ParentID == parentID).ToList(); foreach (ICategory c in subList) { c.Name = cateStr + c.Name; newList.Add(c); newList = newList.Concat(BindData(list, c.ID, cateStr + cateStr)).ToList(); } return newList; }

 

 看了这段代码，觉得比前面的好多了。那就这样吧。

 

回想刚写代码的时候。天天学“代码重构”，也没有觉得出重构有什么乐趣。代码重构，从重构中提高自己，从重构中找到乐趣