使用派生类时出现分段错误

时间:2016-03-24 00:04:12

标签: c++ segmentation-fault

我正在编写一个简单的程序来演示我对派生类的了解。

我正在设置派生类的不同变量,然后将这些变量输出到屏幕并循环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)

1 个答案:

答案 0 :(得分:1)

virtual ~WrapperRobot() {delete this;}

永远不要从析构函数中执行delete this;。当其他人delete是您的对象时,或者在您的情况下,一旦其范围结束,它被分配到堆栈后自动销毁,就会调用析构函数。如果您不需要释放任何内存或释放析构函数中的任何资源,只需将其主体留空即可。

我真的很惊讶你有一个段错误而不是堆栈溢出,因为析构函数中的delete this;应该触发无限递归。 编辑: 或者至少我是,直到我用Google搜索this question