我在弄清楚如何编写一个漂亮的多态退火器类时遇到了一些麻烦。我确信我做错了一些事情,而且这是一个重复的问题,但我已经搜索了很长时间但没有找到任何东西。
我想获取一个指向抽象类型 State(称为 pState)实例的指针,并使用纯虚函数 Perturb()(它随机地改变状态)和 Cost()(它评估函数)来找到最小化成本函数的状态。
我需要跟踪此函数中的多个子类实例,并且我相信这些变量的声明或赋值会导致问题。
void Annealer::Minimise(State *pState){
//Set up
....
....
State *state;
State *newState;
*state = *pState;
*newState = *state;
//Evaluate the initial cost
pState->Cost(); //Works just fine
double cost = state->Cost(); //Segfault
....
}
在尝试复制成功之前对子类指针调用 Cost。我没有包含子类,因为它非常长且复杂。
编辑:
状态定义如下:
class State {
public:
virtual void Perturb()=0;
virtual double Cost()=0;
};
最佳答案
问题出在这里:
*state = *pState;
*newState = *state;
您正在尝试将 pState 复制到状态变量指向的任何位置,但该状态变量尚未设置。如果要复制它,您需要分配内存:
virtual State* State::clone() = 0;
State *state;
State *newState;
state = pState->clone();
newState = state->clone();
// do some stuff
delete state;
delete newState;
或将其保留为指针:
State *state;
State *newState;
state = pState;
newState = state;
或者这两个选项的混合,在简单地复制以进行更改并复制到主变量(如果有效且没有错误)的情况下:
virtual State* State::clone() = 0;
virtual void State::copy(State* cpy) = 0;
State *state;
State *newState;
state = pState;
newState = state->clone();
... // do stuff to newState
state->copy(newState); // copy changes to input
delete newState;
编辑: 由于多态性和复制构造函数的问题,您可以实现纯虚拟克隆方法,但这会导致非堆栈分配,并且您需要删除或使用自动指针。
关于c++ - 调用子类方法时出现段错误,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/35852171/