重构《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 <= dynamicColumns; i++)                {                    Label lh = (Label)e.Item.FindControl("LabelH" + i.ToString());                    lh.Text = objDt.Columns[i].ColumnName;                }            }        }

只要一对比，就可以明了看到变代码中的代码。下面是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 <= dynamicColumns; j++)13                 {14                     Label li = (Label)e.Item.FindControl("LabelI" + j.ToString());15 16                     if (j == 0) 17                         li.Text = drv[objDt.Columns[0].ColumnName].ToString();18                     else19                     {20                         decimal v = string.IsNullOrEmpty(drv[objDt.Columns[j].ColumnName].ToString()) ? 0 : Convert.ToDecimal(drv[objDt.Columns[j].ColumnName].ToString());21                         li.Text = v.ToString();22 23                         ViewState["c" + j.ToString()] = ViewState["c" + j.ToString()] == null ? 0 : Convert.ToDecimal(ViewState["c" + j.ToString()]) + v;24                     }25                 }26             }27         }

最后是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 <= dynamicColumns; k++)11                 {12                     Label lf = (Label)e.Item.FindControl("LabelF" + k.ToString());13 14                     if (k == 0)15                         lf.Text = "Total";16                     else17                         lf.Text = ViewState["c" + k.ToString()] == null ? "0" : ViewState["c" + k.ToString()].ToString();18                 }19             }20         }21     }

 重构是在程序功能要求不变的情况之下，减少冗余的代码。