一个实际项目的重构例子

为了加深大家对重构的直观认识,我在这里贴上自己开发项目的一个实际例子.
如果能让大家对重构这项优化代码的技术多一点认同,心满意足.

代码下载:here

重构前:

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Drawing;
using System.Data;
using System.Text;
using System.Windows.Forms;

using Pinnacle.Prison.Model;
using Pinnacle.Prison.Client;
using Pinnacle.Prison.IServer;

namespace Pinnacle.Prison.PrisonWin
{
   public partial class Logout : UserControl
   {
      String icCardStr = null;
      TICCard icCard = new TICCard();
      GroupInfo groupTemp = null;
      List<VisitorInfo> listVisitor = null;

      public Logout()
      {
         InitializeComponent();
         btnPass.Enabled = false;
      }

        /// <summary>
        /// 全组人资料处理
        /// </summary>
        /// <param name="group"></param>
        internal void HandleGroup(GroupInfo group)
        {
            if (group != null)
            {
                this.icCardStr = group.ICCard.ICCardValue;
                this.groupTemp = group;
                if (group.ICCard.ICCardOwner == ICCardOwner.GuiderPliceMan)
                    listVisitor = group.Visitors;

                ShowInfo(group);

            }
        }

      // 验证，注销IC卡状态
      private void btnPass_Click( object sender, EventArgs e )
      {
         Boolean icCardStatus = false;

         if(groupTemp.ICCard.ICCardOwner == ICCardOwner.Visitor && groupTemp.ICCard.Status == ICCardStatus.OutPrison)//访问者
         {
            if(!string.IsNullOrEmpty( txtIDNumber.Text ))//判断带头警员有没打卡,如果有。。。
            {
               for(int i = 0 ; i < listVisitor.Count ; i++)
               {
                  if(dgVisitorList.Rows[i].Cells[3].Value.ToString() != groupTemp.Visitors[0].IdNumber)//判断打卡人在不在访问人员列表中,如果不在。。。
                  {
                     lblICCardMsg.Text = "验证失败！查无此人！！！";
                     btnPass.Enabled = false;
                  }
                  if(dgVisitorList.Rows[i].Cells[3].Value.ToString() == groupTemp.Visitors[0].IdNumber)//判断打卡人在不在访问人员列表中，如果在。。。
                  {
                     icCardStatus = icCard.UpICCardStatus( icCardStr );//发送验证注销请求
                     if(icCardStatus == true)//如果成功。。。
                     {
                        listVisitor.Remove( listVisitor[i] );
                        i--;
                        BindingSource bindingsource = new BindingSource();
                        bindingsource.DataSource = listVisitor;
                        dgVisitorList.AutoGenerateColumns = false;
                        dgVisitorList.DataSource = bindingsource;
                        clearInfo( groupTemp );
                        return;
                     }
                  }
               }
            }
                else//判断带头警员有没打卡,如果没有。。。
            {
               lblICCardMsg.Text = "错误！请带头警员先打卡！！！";
               btnPass.Enabled = false;
            }
         }
         else if(groupTemp.ICCard.ICCardOwner == ICCardOwner.CommonPliceMan && groupTemp.ICCard.Status == ICCardStatus.OutPrison)//普通警员
         {
            icCardStatus = icCard.UpICCardStatus( icCardStr );
            if(icCardStatus == true)
               clearInfo( groupTemp );
            else
            {
               lblICCardMsg.Text = "验证失败！请重试！！！";
               btnPass.Enabled = true;
            }
         }
         else
         {
            lblICCardMsg.Text = "验证失败！请重试！！！";
            btnPass.Enabled = true;
         }//end if
      }

        //显示IC卡和身份证资料界面
        private void ShowInfo(GroupInfo group)
        {
            if (group.ICCard.ICCardOwner == ICCardOwner.Visitor && group.ICCard.Status == ICCardStatus.OutPrison)//访问者
            {
                showVisitorInfo(group);

                for (int i = 0; i < dgVisitorList.RowCount; i++)
                {
                    if (dgVisitorList.Rows[i].Cells[0].Value.ToString() == txtVisitorName.Text)
                        dgVisitorList.Rows[i].DefaultCellStyle.BackColor = Color.Green;
                }
                txtProperty.Text = "访问者";
                lblVisitorMsg.Text = "";
                lblICCardMsg.Text = "资料读取成功！请验证！";
                txtState.Text = "使用中";
                txtLogOut.Text = "未注销";
                btnPass.Enabled = true;
            }
            else if (group.ICCard.ICCardOwner == ICCardOwner.CommonPliceMan && group.ICCard.Status == ICCardStatus.OutPrison)//普通警员
            {
                showPliceManInfo(group);

                txtProperty.Text = "工作警员";
                txtState.Text = "使用中";
                txtLogOut.Text = "未注销";
                btnPass.Enabled = true;
            }
            else if (group.ICCard.ICCardOwner == ICCardOwner.GuiderPliceMan && group.ICCard.Status == ICCardStatus.OutPrison)//带头警员
            {
                showPliceManInfo(group);

                dgVisitorList.AutoGenerateColumns = false;
                dgVisitorList.DataSource = listVisitor;

                grpIDData.Text = "带头警员身份证资料";
                txtProperty.Text = "带头警员";
                lblVisitorMsg.Text = "其他访问人员请打卡！";
                txtState.Text = "使用中";
                txtLogOut.Text = "未注销";
                lblVisitorsMsg.Text = "";
                btnPass.Enabled = false;

                clearVisitroInfo();
            }//end if
            lblICCardMsg.Text = "";
        }
        //显示警员身份资料
        private void showPliceManInfo(GroupInfo group)
        {
            txtPliceNum.Text = group.PliceMan.PliceManId.ToString();
            txtName.Text = group.PliceMan.Name;
            txtSex.Text = group.PliceMan.Sex;
            txtFolk.Text = group.PliceMan.Folk;
            txtYear.Text = group.PliceMan.Birthday;
            txtAddress.Text = group.PliceMan.Address;
            txtIDNumber.Text = group.PliceMan.IdNumber;
            txtDepartment.Text = group.PliceMan.SignDepartment;
            txtValidTerm.Text = group.PliceMan.ValidTerm;
            imgPhoto.Image = group.PliceMan.Photo;
        }
        //显示访问者身份资料
        private void showVisitorInfo(GroupInfo group)
        {
            txtVisitorName.Text = group.Visitors[0].Name;
            txtVisitorSex.Text = group.Visitors[0].Sex;
            txtVisitorFolk.Text = group.Visitors[0].Folk;
            txtVisitorYear.Text = group.Visitors[0].Birthday;
            txtVisitorAddress.Text = group.Visitors[0].Address;
            txtVisitorIDNumber.Text = group.Visitors[0].IdNumber;
            txtVisitorDepartment.Text = group.Visitors[0].SignDepartment;
            txtVisitorValidTerm.Text = group.Visitors[0].ValidTerm;
            imgVisitorPhoto.Image = group.Visitors[0].Photo;
        }


      // 清除资料界面显示
        private void clearInfo(GroupInfo group)
        {
            if (group.ICCard.ICCardOwner == ICCardOwner.Visitor)//访问者
            {
                clearVisitroInfo();
                lblICCardMsg.Text = "验证通过！";
                if (dgVisitorList.Rows.Count != 0)
                    lblICCardMsg.Text = "验证通过！请其他访问人员继续打卡！";
            }
            else//警员
            {
                clearIDInfo();
                lblICCardMsg.Text = "验证通过！";
            }
            txtProperty.Text = "";
            txtState.Text = "未使用";
            txtLogOut.Text = "已注销";
            btnPass.Enabled = false;
        }
        //清除警员身份资料
        private void clearIDInfo()
        {
            txtPliceNum.Text = "";
            txtName.Text = "";
            txtSex.Text = "";
            txtFolk.Text = "";
            txtYear.Text = "";
            txtAddress.Text = "";
            txtIDNumber.Text = "";
            txtDepartment.Text = "";
            txtValidTerm.Text = "";
            imgPhoto.Image = null;
        }
        //清除访问者身份资料
        private void clearVisitroInfo()
        {
            txtVisitorName.Text = "";
            txtVisitorSex.Text = "";
            txtVisitorFolk.Text = "";
            txtVisitorYear.Text = "";
            txtVisitorAddress.Text = "";
            txtVisitorIDNumber.Text = "";
            txtVisitorDepartment.Text = "";
            txtVisitorValidTerm.Text = "";
            imgVisitorPhoto.Image = null;
        }
   }
}

重构后:

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Drawing;
using System.Data;
using System.Text;
using System.Windows.Forms;

using Pinnacle.Prison.Model;
using Pinnacle.Prison.Client;
using Pinnacle.Prison.IServer;

namespace Pinnacle.Prison.PrisonWin
{
   public partial class Logout : UserControl
   {
      String icCardStr = null;
      TICCard icCard = new TICCard();
      GroupInfo groupTemp = null;
      List<VisitorInfo> listVisitor = null;

      public Logout()
      {
         InitializeComponent();
         btnPass.Enabled = false;
         dgVisitorList.AutoGenerateColumns = false;
      }

      /// <summary>
      /// 全组人资料处理
      /// </summary>
      /// <param name="group"></param>
      internal void HandleGroup( GroupInfo group )
      {
         if(group != null)
         {
            this.icCardStr = group.ICCard.ICCardValue;

            this.groupTemp = group;

            if(group.ICCard.ICCardOwner == ICCardOwner.GuiderPliceMan)
               listVisitor = group.Visitors;

            ShowInfo( group );
         }
      }





      


      // 验证，注销IC卡状态
      private void btnPass_Click( object sender, EventArgs e )
      {    
         //访问者
         if(IsVisitor( groupTemp.ICCard ))
         {
            //带头警员没打卡,
            if(txtProperty.Text != "带头警员")
            {
               ShowMsgHandle( "错误！请带头警员先打卡！！！", false );
               return;
            }

            int visitorIndex = 0;

            //如果带头警员有打卡
            for(int i = 0 ; i < listVisitor.Count ; i++)
            {
               //打卡人在访问人员列表中
               if(dgVisitorList.Rows[i].Cells[5].Value.ToString() == groupTemp.Visitors[0].IdNumber)
               {
                  visitorIndex = i;
               }
            }//--------end for-------------

            //如果打卡人不在访问人员列表中
            if(visitorIndex == 0)
            {
               ShowMsgHandle( "验证失败！查无此人！！！", false );
               return;
            }

            //----a.打卡人在访问人员列表中
            //验证注销失败
            if(icCard.UpICCardStatus( icCardStr ) == false)
            {
               ValidateFailed();
               return;
            }

            //验证成功
            listVisitor.Remove( listVisitor[visitorIndex] );

            //重新绑定数据
            ReBindGridView( listVisitor );

            //清除
            ClearInfo( groupTemp.ICCard.ICCardOwner );           

            //----a

         }
         else if(IsCommonPliceMan( groupTemp.ICCard ))//普通警员
         {
            //验证注销失败
            if(icCard.UpICCardStatus( icCardStr ) == false)
            {
               ValidateFailed();
               return;
            }

            //清除
            ClearInfo( groupTemp.ICCard.ICCardOwner );

         }
         else
         {
            ValidateFailed();
         }//end if
      }

      //显示IC卡和身份证资料界面
      private void ShowInfo( GroupInfo group )
      {
         if(IsVisitor( group.ICCard ))//访问者
         {
            ShowVisitorInfo( group.Visitors[0] );

            for(int i = 0 ; i < dgVisitorList.RowCount ; i++)
            {
               if(dgVisitorList.Rows[i].Cells[0].Value.ToString() == txtVisitorName.Text)
                  dgVisitorList.Rows[i].DefaultCellStyle.BackColor = Color.Green;
            }

            ShowMsgHandle( "资料读取成功！请验证！", true );
         }
         else if(IsCommonPliceMan( group.ICCard ))//普通警员
         {
            ShowPliceManInfo( group.PliceMan );            
            txtProperty.Text = "普通警员";
            txtState.Text = "使用中";
            btnPass.Enabled = true;
         }
         else if(IsGuiderPliceMan( group.ICCard ))//带头警员
         {
            ShowPliceManInfo( group.PliceMan );
            txtProperty.Text = "带头警员";
            txtState.Text = "使用中";
            grpIDData.Text = "带头警员信息";
         
            dgVisitorList.DataSource = listVisitor; 

            ShowMsgHandle( "其他访问人员请打卡！", false );

            ClearVisitroInfo();
         }//end if
      }

      //显示警员身份资料
      private void ShowPliceManInfo( PliceManInfo pliceMan )
      {
         txtName.Text = pliceMan.Name;
         txtSex.Text = pliceMan.Sex;
         txtFolk.Text = pliceMan.Folk;
         txtYear.Text = pliceMan.Birthday;
         txtAddress.Text = pliceMan.Address;
         txtIDNumber.Text = pliceMan.IdNumber;
         txtDepartment.Text = pliceMan.SignDepartment;
         txtValidTerm.Text = pliceMan.ValidTerm;
         imgPhoto.Image = pliceMan.Photo;
         txtPrison.Text = pliceMan.PrisonID.ToString();//显示警员的所在监区
      }

      //显示访问者身份资料
      private void ShowVisitorInfo( VisitorInfo visitor )
      {
         txtVisitorName.Text = visitor.Name;
         txtVisitorSex.Text = visitor.Sex;
         txtVisitorFolk.Text = visitor.Folk;
         txtVisitorYear.Text = visitor.Birthday;
         txtVisitorAddress.Text = visitor.Address;
         txtVisitorIDNumber.Text = visitor.IdNumber;
         txtVisitorDepartment.Text = visitor.SignDepartment;
         txtVisitorValidTerm.Text = visitor.ValidTerm;
         imgVisitorPhoto.Image = visitor.Photo;
         txtVisitorPrison.Text = visitor.PrisonID.ToString();//显示访问者访问的所在监区
         txtVisitorProperty.Text = "访问者";
         txtVisitorState.Text = "使用中";
      }


      // 清除资料界面显示
      private void ClearInfo( ICCardOwner owner )
      {
         String msg = "";
         if(owner == ICCardOwner.Visitor)//访问者
         {
            ClearVisitroInfo();
            txtVisitorProperty.Text = "";
            txtVisitorState.Text = "已注销";
            txtVisitorPrison.Text = "";
            msg = "验证通过！";
            if(dgVisitorList.Rows.Count != 0)
               msg = "验证通过！请其他访问人员继续打卡！";
         }
         else//警员
         {
            ClearIDInfo();
            msg = "验证通过！";
            txtProperty.Text = "";
            txtState.Text = "已注销";
            txtPrison.Text = "";
         }

         ShowMsgHandle( msg, false );         
      }

      //清除警员身份资料
      private void ClearIDInfo()
      {
         txtName.Text = "";
         txtSex.Text = "";
         txtFolk.Text = "";
         txtYear.Text = "";
         txtAddress.Text = "";
         txtIDNumber.Text = "";
         txtDepartment.Text = "";
         txtValidTerm.Text = "";
         imgPhoto.Image = null;
      }

      //清除访问者身份资料
      private void ClearVisitroInfo()
      {
         txtVisitorName.Text = "";
         txtVisitorSex.Text = "";
         txtVisitorFolk.Text = "";
         txtVisitorYear.Text = "";
         txtVisitorAddress.Text = "";
         txtVisitorIDNumber.Text = "";
         txtVisitorDepartment.Text = "";
         txtVisitorValidTerm.Text = "";
         imgVisitorPhoto.Image = null;
      }

      //是否合法状态
      private static bool IsRightStatus( ICCardStatus status )
      {
         return status == ICCardStatus.OutPrison;
      }

      //是否访问者
      private bool IsVisitor( ICCardInfo icCard )
      {
         return icCard.ICCardOwner == ICCardOwner.Visitor && IsRightStatus( icCard.Status );
      }

      //显示提示信息
      private void ShowMsg( string msg )
      {
         lblMsg.Text = msg;
      }

      //是否普通警员
      private bool IsCommonPliceMan( ICCardInfo icCard )
      {
         return icCard.ICCardOwner == ICCardOwner.CommonPliceMan && IsRightStatus( icCard.Status );
      }

      //是否带头警员
      private bool IsGuiderPliceMan( ICCardInfo icCard )
      {
         return icCard.ICCardOwner == ICCardOwner.GuiderPliceMan && IsRightStatus( icCard.Status );
      }

      //验证失败
      private void ValidateFailed()
      {
         ShowMsgHandle( "验证失败！请重试！！！", true );
      }

      //验证结果处理
      private void ShowMsgHandle( String msg, Boolean enablePassButton )
      {
         ShowMsg( msg );
         btnPass.Enabled = enablePassButton;
      }

      //重新绑定数据
      private void ReBindGridView( List<VisitorInfo> listVisitor )
      {
         BindingSource bindingsource = new BindingSource();
         bindingsource.DataSource = listVisitor;
         dgVisitorList.DataSource = bindingsource;
      }
   }
}