我正在编写一个简单的程序来演示我对派生类的了解。
我正在设置派生类的不同变量,然后将这些变量输出到屏幕并循环numRobots
次。但是,在成功输出第一次迭代后,程序输出segmentation fault (core dumped)
。我无法弄清楚原因。
当我在第一次迭代中使用它们时,所有派生类都有效。程序只是不循环到第二次迭代。
这是我的代码:
#include <iostream>
#include <iomanip>
#include "abstractrobot.h"
#include "wrapperrobot.h"
#include "benderrobot.h"
#include "welderrobot.h"
using namespace std;
int main()
{
cout.setf(ios_base::fixed,ios_base::floatfield);
cout.precision(2);
int numRobots;
int robotType;
string robotName;
cin >> numRobots;
for (int i = 0; i < numRobots; i++)
{
cin >> robotType;
cin >> robotName;
switch (robotType)
{
case 0:
{
WrapperRobot a_wrapper(robotName);
int numHours, numPackages;
cin >> numHours;
a_wrapper.set_numHours(numHours);
cin >> numPackages;
a_wrapper.set_numPackages(numPackages);
cout << "#" << i << ": ";
cout << a_wrapper.getName() << ", " << a_wrapper.getType()
<< " $" << a_wrapper.getPay() << endl;
break;
}
case 1:
{
BenderRobot a_bender(robotName);
int numReg, numExtra, numBends;
cin >> numReg;
a_bender.set_numReg(numReg);
cin >> numExtra;
a_bender.set_numExtra(numExtra);
cin >> numBends;
a_bender.set_numBends(numBends);
cout << "#" << i << ": ";
cout << a_bender.getName() << ", " << a_bender.getType()
<< " $" << a_bender.getPay() << endl;
break;
}
case 2:
{
WelderRobot a_welder(robotName);
int numReg, numExtra, numRad;
cin >> numReg;
a_welder.set_numReg(numReg);
cin >> numExtra;
a_welder.set_numExtra(numExtra);
cin >> numRad;
a_welder.set_numRad(numRad);
cout << "#" << i << ": ";
cout << a_welder.getName() << ", " << a_welder.getType()
<< " $" << a_welder.getPay() << endl;
break;
}
default:
cout << "Invalid type" << endl;
}
cout << "hello" << endl; // tester; the program never outputs this
}
return 0;
}
这是0型机器人的头文件:
#ifndef WRAPPERROBOT_H
#define WRAPPERROBOT_H
#include <string>
#include "abstractrobot.h"
using namespace std;
class WrapperRobot : public AbstractRobot
{
private:
float numHours;
float numPackages;
protected:
string c_name;
public:
const float c_rate = 0.25;
const float c_bonus = 0.05;
const string c_type = "Wrapper";
WrapperRobot(string n) : AbstractRobot(n) {c_name = n;}
virtual float getPay() const {return ((c_rate * numHours) +
(c_bonus * numPackages));}
virtual const string& getName() const {return c_name;}
virtual const string& getType() const {return c_type;}
virtual ~WrapperRobot() {delete this;}
void set_numHours(int n);
int get_numHours() {return numHours;}
void set_numPackages(int n);
int get_numPackages() {return numPackages;}
};
#endif
wrapperrobot.cpp
#include "wrapperrobot.h"
void WrapperRobot::set_numHours(int n)
{
numHours = n;
return;
}
void WrapperRobot::set_numPackages(int n)
{
numPackages = n;
return;
}
抽象类:
#ifndef ABSTRACTROBOT_H
#define ABSTRACTROBOT_H
#include <string>
using namespace std;
class AbstractRobot
{
protected:
string c_name;
public:
AbstractRobot(string n)
{
c_name = n;
}
virtual float getPay() const = 0;
virtual const string& getName() const = 0;
virtual const string& getType() const = 0;
virtual ~AbstractRobot() {}
};
#endif
这是我的意见:
4
0
Unit
10 20
该计划输出:
#0: Unit, Wrapper $3.50
Segmentation fault (core dumped)
答案 0 :(得分:1)
virtual ~WrapperRobot() {delete this;}
永远不要从析构函数中执行delete this;
。当其他人delete
是您的对象时,或者在您的情况下,一旦其范围结束,它被分配到堆栈后自动销毁,就会调用析构函数。如果您不需要释放任何内存或释放析构函数中的任何资源,只需将其主体留空即可。
我真的很惊讶你有一个段错误而不是堆栈溢出,因为析构函数中的delete this;
应该触发无限递归。 编辑: 或者至少我是,直到我用Google搜索this question。