代碼重構

      作者： 黎承湘

      不知從什麼時候起，發現自己越來越來不喜歡看到有重複的代碼。這段時間完成了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; }

 

 看了這段代碼，覺得比前面的好多了。那就這樣吧。

 

回想剛寫代碼的時候。天天學“代碼重構”，也沒有覺得出重構有什麼樂趣。代碼重構，從重構中提高自己，從重構中找到樂趣