C++抽象类对象指针调用函数时触发abort()错误排查

你的程序在调用pend->period()时崩溃，大概率是因为访问了无效指针，结合代码细节来看，主要有这几个核心问题：

1. findMaxPeriod函数缺少数组长度参数，循环依赖未明确的n #

你的findMaxPeriod只接收了指针数组a，但没有传入数组的实际长度。如果这里的n是全局变量，它的值很可能和你实际创建的数组长度不匹配：

• 若n大于实际数组长度，循环会越界访问数组外的随机内存（野指针），调用period()时必然触发崩溃；
• 若n小于实际长度，会漏掉部分元素，导致结果不准确。

修复方案：给函数添加数组长度参数，同时在循环前做指针有效性检查：

void findMaxPeriod(pendulum **a, int arrayLength) {
    pendulum *maxPend = nullptr; // 初始化为nullptr更安全，避免内存泄漏
    double maxPeriod = 0;
    double period1;
    for (int i = 0; i < arrayLength; i++) {
        pendulum *pend = a[i];
        // 先跳过空指针/无效指针，避免崩溃
        if (!pend) continue;
        
        period1 = pend->period();
        if (period1 > maxPeriod) {
            maxPend = a[i];
            maxPeriod = period1;
        }
    }
    // 最后检查是否找到有效摆，避免空指针输出
    if (maxPend) {
        cout << "max period pendulum is " << maxPend << ". It's period is " << maxPend->period() << endl;
    } else {
        cout << "No valid pendulum exists in the array." << endl;
    }
}

2. 指针数组未初始化，存在野指针 #

你创建pendulum** pendArray = new pendulum*[n]后，数组内的元素默认是未初始化的野指针。如果填充数组时，没有给所有n个元素都赋值为math或spring的对象指针，残留的野指针会导致调用period()时崩溃。

修复方案：

• 先将数组所有元素初始化为nullptr：

pendulum** pendArray = new pendulum*[n];
// 初始化所有指针为nullptr，避免野指针
for (int i = 0; i < n; i++) {
    pendArray[i] = nullptr;
}

• 填充对象指针时，确保每个需要使用的元素都通过new math()或new spring()正确赋值，不要留下未初始化的位置。

3. 抽象类的虚析构函数缺失（潜在风险） #

虽然这不是当前崩溃的直接原因，但如果你的pendulum类没有定义虚析构函数，后续用基类指针释放派生类对象时会导致内存泄漏或未定义行为。建议补充：

class pendulum {
public:
    virtual double period() = 0; // 纯虚函数
    virtual ~pendulum() = default; // 虚析构函数，确保派生类对象能被正确销毁
};

内容的提问来源于stack exchange，提问作者Евгений Романюк