[設計] 重構去除未使用參數的方式

[darnoc (流浪)]

        大家好：

            最近遇到一個問題，就是要重構某個 class 階層中，
        未使用到的 function parameters，但卻不知該如何下手
        才好。原本的 code 示意如下：

        (程式碼以C++舉例，但實際的應用可能是其他語言，例如
        不支援多重繼承的Java)

        class BaseClass
        {
        public:
            virtual void Foo(int a, int b) = 0;
        };

        class DerivedOne : public BaseClass
        {
        public:
            virtual void Foo(int a, int b)
            {
                cout << a << b << endl;
            }
        };

        class DerivedTwo : public BaseClass
        {
        public:
            virtual void Foo(int a, int b)
            {
                // b is unreferenced
                cout << a << endl;
            }
        };

        如上所示，當我期望透過多型的方式，一視同仁的操作
        實際上指向不同類別的物件時，有的子類別會用到某個
        參數(例子中的b) ，但某些子類別則否，這時候該怎麼
        去除這樣的 unreferenced parameters比較好呢？

        如：
            BaseClass *ptrA = new DerivedOne;
            ptrA->Foo(0, 1);
            BaseClass *ptrB = new DerivedTwo;
            ptrB->Foo(0, 1);  // 1 is unreferenced

        不知道大家有沒有什麼比較好的想法？謝謝。

--
記憶是不由人的，它想來，才會來，
它不想走，妳怎麼也躲不過。
                                －朱少麟．燕子－

--
※ 發信站: 批踢踢實業坊(ptt.cc) 
◆ From: 114.44.2.245
推 legendmtg   :聽不太懂你的意思...感覺像是D2的Foo改成
→ legendmtg   :virtual void Foo(int a, int) 就好了...
推 dk3208      :把多出來的參數變成D1的member然後在建構子傳入如何?
→ dk3208      :或者如果參數a,b有相關性的話合成一個物件
→ darnoc      :1F的作法只是把這個warning"藏"起來，並沒有解決問題
→ darnoc      :dk的第二個建議，看起來可能要引入parameter object
→ darnoc      :似乎可以改善這個問題，謝謝了 :)
→ darnoc      :有沒有可能把原本的參數列，改為傳入heterogeneous
→ darnoc      :container? 在不同的實做版本中自行拆解出所須變數?
→ qrtt1       :這沒有需要去除，它又不是用不到的暫時變數。
→ rofellosx   :我不懂為什麼要除去..
→ darnoc      :唔，難道大家開發專案，沒有把warning level調到最高
→ darnoc      :順便把 treat warnings as errors 打開嗎？
→ darnoc      :其實比起去除warning，我更在意的是，這樣的設計，在
→ darnoc      :某方面來說，總覺得有些壞味道，意味著設計不良？
→ qrtt1       :我們無法由單純的結構來判別bad smell
→ qrtt1       :如果你覺得是設計不良, 那最好與設計者討論語意
→ qrtt1       :單純就一個 library user 來說, 他沒用到那個參數
→ qrtt1       :而懷疑總體的設計是很詭異的
→ qrtt1       :因為一個只看到自己的需要, 一個要顧及總體需求
→ qrtt1       :我不並不執著你現在用的是int來做例子
→ qrtt1       :它可以是更複雜的類別.
→ qrtt1       :不過您覺得它是用不到的變數, 但重構只要求我們針對
→ qrtt1       :用不到的暫時性變數(通常是local)，看結構比較接近的
→ qrtt1       :被拒絕的遺贈(refused bequest). 也許你可以參考一下
→ qrtt1       :http://tinyurl.com/ydxzsz9 摘自重構試閱電子檔