什么是分段错误。如何处理它[重复]

Question

问题：

• • 一名学生报名参加研讨会并希望参加最多
    没有两个车间重叠的车间数量。你必须这样做
    以下：实施结构：
    1. struct 具有以下成员的工作室：工作室的开始时间。研讨会的持续时间。研讨会的结束时间。
    2. struct Available_Workshops 具有以下成员： 整数，（学生报名参加的研讨会数量）。安
       类型为 Workshop 数组的数组，大小为 。实现功能：
    3. Available_Workshops* 初始化 (int start_time[], int duration[], int n) 创建一个 Available_Workshops 对象并 使用 和 参数中的元素初始化其元素 （两者都是大小）。这里， 和 是各自的开始时间和 研讨会的持续时间。这个函数必须返回一个指向 一个 Available_Workshops 对象。
    4. int CalculateMaxWorkshops(Available_Workshops* ptr) 返回学生可以参加的最大研讨会数 - 没有重叠。
       直到上一个工作坊才能参加下一个工作坊
       结束。注意：未知大小的数组 ( ) 应声明为
       如下： DataType* arrayName = new DataType[n];

  你的初始化函数必须返回一个指向 Available_Workshops 对象。您的 CalculateMaxWorkshops 函数
  必须返回学生的非重叠研讨会的最大数量
  可以参加。
  样本输入

6
1 3 0 5 5 8
1 1 6 2 4 1
样本输出

4

说明第一行表示，车间数量。下一行包含空格分隔的整数，其中整数 是研讨会的开始时间。下一行包含 空格分隔的整数，其中整数是车间的 期间。学生可以参加研讨会，而无需 重叠，因此 CalculateMaxWorkshops 返回到 main（然后 打印到标准输出）。

我的代码：

#include <iostream>

using namespace std;


class Workshop{
    public:
    int start_time{},duration{},end_time{};};

class Available_Workshops
{
    public:
     int n{};
    struct Workshop*arr=new struct Workshop[n];
    ~Available_Workshops()
    {
        delete [] arr;

    }
    void arr_sort();
    void arr_delete(int i);


};
////////////////////////////////////////////////////////////////////////////////////////////
Available_Workshops * initialize(int start_time[],int duration[],int n)
{
    Available_Workshops * x=new Available_Workshops{};
    x->n=n;
    for(int i=0;i<n;i++)
    {
     x->arr[i].start_time=start_time[i];
     x->arr[i].duration=duration[i];
     x->arr[i].end_time=start_time[i]+duration[i];
    }

    return x;

}
///////////////////////////////////////////////////////////////////////////////////////////
void Available_Workshops:: arr_delete(int i)
{

    n-=1;


    for(int j=i;j<n;j++)
    {
     arr[j]=arr[j+1];
    }


}
///////////////////////////////////////////////////////////////////////////////////////////
void Available_Workshops:: arr_sort()
{
for(int i=0;i<n;i++)
{
    for(int j=i+1;j<n;j++)
    {
        if(arr[i].start_time>arr[j].start_time)
            {
                struct Workshop temp=arr[i];
                arr[i]=arr[j];
                arr[j]=temp;
            }
    }
}


}
///////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
int CalculateMaxWorkshops(Available_Workshops * x)
{
    x->arr_sort();
    for(int i=0;i<x->n-1;i++)
    {
       for(int j=i+1;j<x->n;j++)
       {
           if(x->arr[i].end_time>x->arr[j].start_time)
           {
               if(x->arr[i].duration>=x->arr[j].duration)
                x->arr_delete(i);
               else x->arr_delete(j);
               j--;
           }
       }
    }
    int y=x->n;
    delete x;
   return y;

}

int main(int argc, char *argv[]) {
    int n; // number of workshops
    cin >> n;
    // create arrays of unknown size n
    int* start_time = new int[n];
    int* duration = new int[n];

    for(int i=0; i < n; i++){
        cin >> start_time[i];
    }
    for(int i = 0; i < n; i++){
        cin >> duration[i];
    }

    Available_Workshops * ptr;
    ptr = initialize(start_time,duration, n);

    cout << CalculateMaxWorkshops(ptr) << endl;
    return 0;
}

我的代码没有运行。它有分段错误。请帮我找出这个错误

Answer 1

你的bug可以从类声明中看出：

class Available_Workshops
{
public:
    int n{};
    struct Workshop* arr = new struct Workshop[n];

    ~Available_Workshops()
    {
        delete[] arr;

    }
    void arr_sort();
    void arr_delete(int i);


};

成员 n 被显式初始化为 0。然而，您的 initialize 函数会很乐意将更多元素填充到 arr（零元素数组）中，并导致各种未定义的行为。

您真的非常想要一个适合您的类的构造函数，而不是尝试内联初始化成员。

Available_Workshops(int size) :
n(size)
{
    arr = new Workshop[n];
}

另一个问题，虽然与您的崩溃无关，但在您的 arr_delete 函数中。

for (int j = i; j < n; j++)
{
    arr[j] = arr[j + 1];
}

当j == n-1在循环的最后一次迭代时，它将执行arr[n-1] = arr[n]。访问arr[n] 是未定义的行为，因为数组中唯一有效的索引来自[0..n-1]