C++编译结构体数组代码出现[int]数组下标无效错误排查

你遇到的Invalid types ‘[int]’ for array subscript报错，以及后续数组存取异常，核心是内存操作和指针使用逻辑错误导致的，具体问题点和修复方案如下：

核心问题排查 #

• 野指针访问：类成员array_of_points是Point3D*类型指针，所有构造函数均未通过new为其分配实际堆内存，直接解引用写入属于未定义行为，会导致指针指向非法内存区域，触发编译器的下标类型校验错误。
• 指针偏移计算错误：C/C++中对指向具体类型的指针做算术运算时，编译器会自动按照指向类型的大小计算步长，无需手动乘sizeof值。你在setAddPoint中写的*(array_of_points + sizeof(point)*number_of_points)会让偏移量放大24倍（Point3D由3个double组成，单实例占24字节），直接跳到数组外的非法地址。
• 成员变量未初始化：带参构造函数未给number_of_points成员赋值，点计数混乱会导致后续遍历越界。
• 缺失动态扩容逻辑：初始数组仅能容纳固定数量的点，新增点时没有重新分配更大的内存块，写入新点必然越界。
• 遍历边界错误：printObj中循环条件写为x<=Obj.getNumber()会多访问1个不存在的数组元素，数组有效下标范围是0 ~ 元素个数-1。

修复后核心代码 #

以下是元素设置、输出相关逻辑的修复实现，修复后指针类型合法，下标访问可正常运行：

#include <iostream>
#include <string.h>
using namespace std;

typedef struct Point3D{
    double x;
    double y;
    double z;
}Point3D;

Point3D createPoint3D(double xnum, double ynum, double znum){
    Point3D point;
    point.x = xnum;
    point.y = ynum;
    point.z = znum;
    return point;
}

void PrintPoint(Point3D point){
    cout << "(" << point.x << "," << endl;
    cout << point.y << "," << endl;
    cout << point.z << ";" << "\n" << endl;
}

class Obj3D{
private:
    Point3D *array_of_points;
    int number_of_points;
public:
    void setAddPoint(Point3D point){
        // 扩容：新数组大小为当前点数+1
        Point3D* temp = new Point3D[number_of_points + 1];
        // 拷贝原有数组数据
        for(int i = 0; i < number_of_points; i++){
            temp[i] = array_of_points[i];
        }
        // 释放旧数组内存
        delete[] array_of_points;
        array_of_points = temp;
        // 直接用下标访问即可，等价于指针偏移，无需手动计算sizeof
        array_of_points[number_of_points] = point;
        number_of_points++;
    }

    int getNumber(){
        return number_of_points;
    }

    Point3D *getArray(){
        return array_of_points;
    }

    Obj3D(Point3D point, int number){
        // 分配指定大小的数组内存，初始化计数
        array_of_points = new Point3D[number];
        number_of_points = number;
        array_of_points[number - 1] = point;
    }

    Obj3D(){
        // 初始分配1个元素的内存，解决野指针问题
        array_of_points = new Point3D[1];
        array_of_points[0] = createPoint3D(0.0, 0.0, 0.0);
        number_of_points = 1;
    }

    ~Obj3D(){
        // 析构释放堆内存，避免内存泄漏
        delete[] array_of_points;
    }
};

void printObj(Obj3D Obj){
    // 修正循环边界，避免越界
    for(int x = 0; x < Obj.getNumber(); x++){
        PrintPoint(Obj.getArray()[x]);
    }
}

补充说明：如果对性能有要求，可以不用每次加1个元素就扩容，而是预留容量（比如每次扩容为原来的2倍）减少内存拷贝次数，属于优化项不影响功能正确性。

内容的提问来源于stack exchange，提问作者Danil Dubok