c++ - 使用派生类时出现段错误

Question

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

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

包装机器人.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

这是我的输入:

40单位10 20

程序输出:#0:单位，包装 $3.50段错误(核心已转储)

Answer 1

virtual ~WrapperRobot() {delete this;}

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

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