首页 > IT职业 > IT生活 > 重构《Repeater控件动态变更列信息》
早上有分享《Repeater控件动态变更列(Header,Item和Foot)信息》 ,是动态变更一些列的内容。
虽然它算不上是完全动态化,但它已经达到初期想要的效果。如果你稍有仔细,也能轻易把它弄出来。现另开一篇,Insus.NET不是重新另外写,而是想重构cs的代码,因为前一篇的代码虽然简单,但代码冗余过多。
重构开始:
首先把这五个变量删除,因为在重构过程中,已经不需要这五个变量了。
//宣告5个变量,将用来存储那5个月份每个部分的数量decimal c1, c2, c3, c4, c5;
接下来需要改动的是宣告一个常量,很多地方使用到它:
const int dynamicColumns = 5;
把程序中的下面这句
objPrintLog.Months = 5; //最近连续5个月份
改为:
objPrintLog.Months = dynamicColumns;
也就是说,使用常量的变量去替代旧代码的"5"。
接下来,我们重构Repwater控件的Header的代码,为了好对比,Insus.NET把上一篇对应的图片引用在这里:

重构如下:

protected void RepeaterLFMS_ItemDataBound(object sender, RepeaterItemEventArgs e) { if (e.Item.ItemType == ListItemType.Header) { if (e.Item.FindControl("LabelH0") != null && e.Item.FindControl("LabelH1") != null && e.Item.FindControl("LabelH2") != null && e.Item.FindControl("LabelH3") != null && e.Item.FindControl("LabelH4") != null && e.Item.FindControl("LabelH5") != null) { for (int i = 0; i 
只要一对比,就可以明了看到变代码中的代码。下面是Repwater控件Item 部分:

旧代码重构之后的代码,第16行代码,是判断第一列,因为它是字符串,因此单独排除。第23行,使用ViewState来替代旧程序的5个变量。

1 if (e.Item.ItemType == ListItemType.AlternatingItem || e.Item.ItemType == ListItemType.Item) 2 { 3 DataRowView drv = (DataRowView)e.Item.DataItem; 4 if (e.Item.FindControl("LabelI0") != null 5 && e.Item.FindControl("LabelI1") != null 6 && e.Item.FindControl("LabelI2") != null 7 && e.Item.FindControl("LabelI3") != null 8 && e.Item.FindControl("LabelI4") != null 9 && e.Item.FindControl("LabelI5") != null)10 {11 12 for (int j = 0; j 
最后是Foot的重构:

Foot重构好的代码,第14行是判断是否为第一列,第17行,是把ViewState的值赋给Label。

1 if (e.Item.ItemType == ListItemType.Footer) 2 { 3 if (e.Item.FindControl("LabelF0") != null 4 && e.Item.FindControl("LabelF1") != null 5 && e.Item.FindControl("LabelF2") != null 6 && e.Item.FindControl("LabelF3") != null 7 && e.Item.FindControl("LabelF4") != null 8 && e.Item.FindControl("LabelF5") != null) 9 {10 for (int k = 0; k 
重构是在程序功能要求不变的情况之下,减少冗余的代码。
来自 “ ITPUB博客 ” ,链接:https://blog.itpub.net/1414/viewspace-2812537/ ,如需转载,请注明出处,否则将追究法律责任。