重构《Repeater控件动态变更列(Header,Item和Foot)信息》

早上有分享《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                     else
19                     {
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                     else
17                         lf.Text = ViewState["c" + k.ToString()] == null ? "0" : ViewState["c" + k.ToString()].ToString();
18                 }
19             }
20         }
21     }


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

posted @ 2013-03-22 16:52  Insus.NET  阅读(2583)  评论(0)  编辑  收藏  举报