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

看板OOAD作者 (流浪)時間14年前 (2010/05/09 20:56), 編輯推噓2(2024)
留言26則, 5人參與, 最新討論串1/1
大家好: 最近遇到一個問題,就是要重構某個 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

05/09 21:47, , 1F
聽不太懂你的意思...感覺像是D2的Foo改成
05/09 21:47, 1F

05/09 21:47, , 2F
virtual void Foo(int a, int) 就好了...
05/09 21:47, 2F

05/09 22:55, , 3F
把多出來的參數變成D1的member然後在建構子傳入如何?
05/09 22:55, 3F

05/09 22:57, , 4F
或者如果參數a,b有相關性的話合成一個物件
05/09 22:57, 4F

05/09 23:11, , 5F
1F的作法只是把這個warning"藏"起來,並沒有解決問題
05/09 23:11, 5F

05/09 23:13, , 6F
dk的第二個建議,看起來可能要引入parameter object
05/09 23:13, 6F

05/09 23:14, , 7F
似乎可以改善這個問題,謝謝了 :)
05/09 23:14, 7F

05/09 23:15, , 8F
有沒有可能把原本的參數列,改為傳入heterogeneous
05/09 23:15, 8F

05/09 23:16, , 9F
container? 在不同的實做版本中自行拆解出所須變數?
05/09 23:16, 9F

05/09 23:33, , 10F
這沒有需要去除,它又不是用不到的暫時變數。
05/09 23:33, 10F

05/12 20:05, , 11F
我不懂為什麼要除去..
05/12 20:05, 11F

05/14 00:04, , 12F
唔,難道大家開發專案,沒有把warning level調到最高
05/14 00:04, 12F

05/14 00:04, , 13F
順便把 treat warnings as errors 打開嗎?
05/14 00:04, 13F

05/14 00:05, , 14F
其實比起去除warning,我更在意的是,這樣的設計,在
05/14 00:05, 14F

05/14 00:05, , 15F
某方面來說,總覺得有些壞味道,意味著設計不良?
05/14 00:05, 15F

05/14 09:23, , 16F
我們無法由單純的結構來判別bad smell
05/14 09:23, 16F

05/14 09:24, , 17F
如果你覺得是設計不良, 那最好與設計者討論語意
05/14 09:24, 17F

05/14 09:24, , 18F
單純就一個 library user 來說, 他沒用到那個參數
05/14 09:24, 18F

05/14 09:25, , 19F
而懷疑總體的設計是很詭異的
05/14 09:25, 19F

05/14 09:25, , 20F
因為一個只看到自己的需要, 一個要顧及總體需求
05/14 09:25, 20F

05/14 09:26, , 21F
我不並不執著你現在用的是int來做例子
05/14 09:26, 21F

05/14 09:26, , 22F
它可以是更複雜的類別.
05/14 09:26, 22F

05/14 09:29, , 23F
不過您覺得它是用不到的變數, 但重構只要求我們針對
05/14 09:29, 23F

05/14 09:29, , 24F
用不到的暫時性變數(通常是local),看結構比較接近的
05/14 09:29, 24F

05/14 09:31, , 25F
被拒絕的遺贈(refused bequest). 也許你可以參考一下
05/14 09:31, 25F

05/14 09:33, , 26F
http://tinyurl.com/ydxzsz9 摘自重構試閱電子檔
05/14 09:33, 26F
文章代碼(AID): #1Bvh3-ng (OOAD)
文章代碼(AID): #1Bvh3-ng (OOAD)