我有一个需要大量属性进行初始化的类:
Hand hand = new Hand();
hand->skinColor(Color(0.5, 0.5, 0.2));
hand->indexFingerLength(0.5);
hand->middleFingerLength(0.6);
hand->ringFingerLength(0.55);
...
hand->init(); // Builds the hand model
这些属性仅在调用init()之前有效修改,init()使用它们来构建类。问题是在调用init()之后仍然可以修改这些属性,这个界面可能会误导用户认为它们会产生影响。
除了将所有属性移动到init()的参数列表中之外,还有更好的方法来重构吗?
答案 0 :(得分:6)
struct structHandParms
{
param1;
param2;
param3;
...
};
structHandParms stParams;
Hand hand = new Hand(stParams)
答案 1 :(得分:3)
您可以添加传递给构造函数的“Hand_Init”结构:
Hand_Init hand_init;
hand_init.skinColor = Color(0.5, 0.5, 0.2);
hand_init.indexFingerLength = 0.5;
hand_init.middleFingerLength = 0.6;
hand_init.ringFingerLength = 0.55;
Hand hand = new Hand(hand_init);
这样一旦构造函数调用完成,手就会被初始化,并且你没有一堆可能导致混淆的匿名参数。
答案 2 :(得分:3)
如果您要创建多个Hand
,那么可以使用类似Builder Pattern的内容。
// set properties
HandBuilder hb;
hb.skinColor(Color(0.5, 0.5, 0.2));
hb.indexFingerLength(0.5);
hb.middleFingerLength(0.6);
hb.ringFingerLength(0.55);
// create object
Hand hand = hb.buildHand();
答案 3 :(得分:2)
这完全是在构造函数中使用逻辑而不是init之后的单独方法。然后,完全明显(并由编译器强制执行)以后无法更改它们。如果有很多参数,请将它们全部分组到一个结构中,该结构被填充并传递给构造函数而不是25个单独的参数。
答案 4 :(得分:1)
“对init()的调用,它使用它们来构建类”真的构建了一个类,还是初始化了这个类的实例?
为什么不在ctor中进行整个初始化并将所需的值作为参数传递?
答案 5 :(得分:0)
我认为这是维护项目文档和用户手册的一个很好的理由,或者您可以更改为您的班级发送描述符。然后,创建描述符,用配置填充它,并立即将所有信息发送给构造函数。
答案 6 :(得分:0)
您可以通过抛出异常来解决init()修改后的问题。这将阻止用户调用其中一个设置功能并期望它执行某些操作。
作为替代方法,您可以创建一个HandParameter类/结构。该对象负责收集有关Hand对象的所有数据。接下来,您可以使Hand对象的init()函数将HandParameter作为其唯一参数。
struct HandParam
{
// Whatever settings you need
};
class Hand
{
bool initialized;
// other private data
public:
Hand();
Hand( const HandParam& params ); // Calls the init() funciton with params
void init( const HandParam& params ); // sets up hand and sets initialized to true
// Whatever else you need in your class
};
祝你好运!