作者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