實戰案例 - 資料對接工具程式碼重構 (3)

在經歷過上一篇的慘不忍睹程式碼之後，就開始思考：為什麼我用物件導向寫出來的程式會是這個樣子？為什麼我寫出來的東西，跟曾經看過的物件導向程式差這麼多？

幸好後來有一段蠻長的空檔，讓我有充裕的時間思考反省這個問題，更慶幸的是也有足夠時間去改正缺失。經過反思之後，終於發現程式慘不忍睹的原因：當時寫程式的第一個反應，幾乎都是「現在要寫哪個步驟的程式」。換句話來說就是有一點見招拆招、打游擊的感覺。

接著就趁那段蠻長的空檔，不斷反覆琢磨、回味之前經常在學習的理論，也就是「術語定義篇」裡的內容，然後調整自己寫程式的思考模式，並開始按照這樣的心法，來思考如何實現我要的功能和效果。也就是往後在寫程式的時候，應該要這樣思考：現在要實現的功能，有哪幾個要做互動的事物？這些事物各別有哪些特徵和行為？最後直接來看改進之後的 code-behind 程式碼。

using System;
using System.Windows.Forms;


namespace SybaseUtility
{
    public partial class fTransmitter : Form
    {
        private Transmitter tt = new Transmitter();
        private TransmitterOperation to;
        private ITransmitting t;

        public fTransmitter()
        {
            InitializeComponent();
        }

        private void fTransmitter_Load(object sender, EventArgs e)
        {
            to = new TransmitterOperation(new Button[] { btnSendAll, btnSendPick, btnRestoreAll, btnRestorePick }, labKeyword, dgvPreview, tsslbStatus);
            to.CreateComboBoxItem(cbxGenre);
        }

        private void rbUnSend_CheckedChanged(object sender, EventArgs e)
        {
            tt.PreviewState = PreviewState.UnSend;
            to.ButtonStateForUnSend();
        }

        private void rbSended_CheckedChanged(object sender, EventArgs e)
        {
            tt.PreviewState = PreviewState.Sended;
            to.ButtonStateForSended();
        }

        private void cbxGenre_SelectedIndexChanged(object sender, EventArgs e)
        {
            tt.Genre = cbxGenre.SelectedItem.ToString();

            switch (tt.Genre)
            {
                case "客户资料":
                    t = new CustomersTransmitting();
                    break;

                case "销售单":
                    t = new SaleListsTransmitting();
                    break;

                case "维修历史":
                    t = new RepairingTransmitting();
                    break;

                case "会员卡资料":
                    t = new VIPCardsTransmitting();
                    break;

                case "积分兑换项目":
                    t = new CreditConvertItemsTransmitting();
                    break;

            }

            to.ShowFilterText(t);
        }

        private void btnPreview_Click(object sender, EventArgs e)
        {
            Validating v = new Validating(cbxGenre, rbUnSend, rbSended);
            v.Validate();

            if (string.IsNullOrEmpty(v.errorMsgText))
            {
                to.Preview(t, tt);
            }
        }

        private void tbKeyword_TextChanged(object sender, EventArgs e)
        {
            Validating v = new Validating(cbxGenre, rbUnSend, rbSended);
            v.Validate();

            if (string.IsNullOrEmpty(v.errorMsgText))
            {
                to.Filtering(t, tt, tbKeyword.Text);
            }
        }

        private void btnSendAll_Click(object sender, EventArgs e)
        {
            tt.FinishState = FinishState.Updated;
            if (t != null) to.SendAll(t, tt);
        }

        private void btnSendPick_Click(object sender, EventArgs e)
        {
            tt.FinishState = FinishState.Updated;

            if (t != null) to.SendPick(t, tt);
        }

        private void btnRestoreAll_Click(object sender, EventArgs e)
        {
            tt.FinishState = FinishState.Restored;
            if (t != null) to.RestoreAll(t, tt);
        }

        private void btnRestorePick_Click(object sender, EventArgs e)
        {
            tt.FinishState = FinishState.Restored;
            if (t != null) to.RestorePick(t, tt);
        }
    }
}

很明顯的，慘不忍睹的版本，總共 689 行程式碼，而且是個大雜燴，經常害我寫程式寫到忘了自己是誰；重構之後的程式碼只有 114 行，而且層次清析，哪一段程式在做什麼都簡單明瞭。雖然這個版本的程式已經有很大的改進，但還是有個缺點，就是仍看得到 switch case 的程式片段，這在日後遇到需求變更時，一定得再回來改寫程式，這樣也不是很好的開發體驗。

我要的最終效果，是「不因需求變更而改動用戶端程式碼」，當然這個部份屬於下個階段的重構，因為會牽涉到設計模式，這又是比物件導向更抽象的東西，日後再發文分享心得。