作者darnoc (流浪)
看板OOAD
標題[設計] 重構去除未使用參數的方式
時間Sun May 9 20:56:26 2010
大家好：
最近遇到一個問題，就是要重構某個 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
1F：推 legendmtg :聽不太懂你的意思...感覺像是D2的Foo改成 05/09 21:47
2F：→ legendmtg :virtual void Foo(int a, int) 就好了... 05/09 21:47
3F：推 dk3208 :把多出來的參數變成D1的member然後在建構子傳入如何? 05/09 22:55
4F：→ dk3208 :或者如果參數a,b有相關性的話合成一個物件 05/09 22:57
5F：→ darnoc :1F的作法只是把這個warning"藏"起來，並沒有解決問題 05/09 23:11
6F：→ darnoc :dk的第二個建議，看起來可能要引入parameter object 05/09 23:13
7F：→ darnoc :似乎可以改善這個問題，謝謝了 :) 05/09 23:14
8F：→ darnoc :有沒有可能把原本的參數列，改為傳入heterogeneous 05/09 23:15
9F：→ darnoc :container? 在不同的實做版本中自行拆解出所須變數? 05/09 23:16
10F：→ qrtt1 :這沒有需要去除，它又不是用不到的暫時變數。 05/09 23:33
11F：→ rofellosx :我不懂為什麼要除去.. 05/12 20:05
12F：→ darnoc :唔，難道大家開發專案，沒有把warning level調到最高 05/14 00:04
13F：→ darnoc :順便把 treat warnings as errors 打開嗎？ 05/14 00:04
14F：→ darnoc :其實比起去除warning，我更在意的是，這樣的設計，在 05/14 00:05
15F：→ darnoc :某方面來說，總覺得有些壞味道，意味著設計不良？ 05/14 00:05
16F：→ qrtt1 :我們無法由單純的結構來判別bad smell 05/14 09:23
17F：→ qrtt1 :如果你覺得是設計不良, 那最好與設計者討論語意 05/14 09:24
18F：→ qrtt1 :單純就一個 library user 來說, 他沒用到那個參數 05/14 09:24
19F：→ qrtt1 :而懷疑總體的設計是很詭異的 05/14 09:25
20F：→ qrtt1 :因為一個只看到自己的需要, 一個要顧及總體需求 05/14 09:25
21F：→ qrtt1 :我不並不執著你現在用的是int來做例子 05/14 09:26
22F：→ qrtt1 :它可以是更複雜的類別. 05/14 09:26
23F：→ qrtt1 :不過您覺得它是用不到的變數, 但重構只要求我們針對 05/14 09:29
24F：→ qrtt1 :用不到的暫時性變數(通常是local)，看結構比較接近的 05/14 09:29
25F：→ qrtt1 :被拒絕的遺贈(refused bequest). 也許你可以參考一下 05/14 09:31