C++读取PPM文件后在关闭文件前崩溃，如何排查修复？

问题定位 #

• 核心崩溃原因：二维像素数组越界访问
  你定义的r是Pixel**类型，第一维仅分配了height个行指针，仅支持索引范围0 ~ height-1。但你在循环中直接用范围在0 ~ width*height-1的i作为r的第一维索引，当i >= height时就会访问非法内存，触发内存访问错误导致崩溃。
  同时你使用->访问成员也是错误的：r[行索引]拿到的是该行的Pixel数组首地址，访问具体像素需要再加列索引，最终得到的Pixel结构体对象要用.访问成员，而非箭头。
• 潜在触发问题：PPM文件头读取逻辑错误
  PPM ASCII格式（P3）的文件头允许存在任意数量以#开头的注释行，你硬编码读4次头部字段的逻辑，遇到带注释的PPM文件时会把像素值误读为头部字段，导致后续读像素时偏移，也可能触发越界。

修复方案 #

1. 修复像素数组访问逻辑 #

将线性索引转换为行、列二维索引后再访问像素，示例代码如下：

for (int i = 0; i < width * height; i++)
{
    unsigned int row = i / width;
    unsigned int col = i % width;
    readFile >> Red;
    r[row][col].R = stoi(Red);
    readFile >> Green;
    r[row][col].G = stoi(Green);
    readFile >> Blue;
    r[row][col].B = stoi(Blue);
}

也可以直接用双层循环遍历行和列，逻辑更直观：

for (unsigned int row = 0; row < height; row++)
{
    for (unsigned int col = 0; col < width; col++)
    {
        readFile >> Red;
        r[row][col].R = stoi(Red);
        readFile >> Green;
        r[row][col].G = stoi(Green);
        readFile >> Blue;
        r[row][col].B = stoi(Blue);
    }
}

2. 优化PPM头读取逻辑 #

读取头部时跳过所有注释行，避免读取错误，示例逻辑：

string tmp;
// 读魔数
readFile >> tmp;
// 跳过注释
while (readFile.peek() == '#' || isspace(readFile.peek()))
{
    if (readFile.peek() == '#') getline(readFile, tmp);
    else readFile.ignore();
}
// 读宽高
readFile >> width >> height;
// 跳过注释
while (readFile.peek() == '#' || isspace(readFile.peek()))
{
    if (readFile.peek() == '#') getline(readFile, tmp);
    else readFile.ignore();
}
// 读最大亮度值
readFile >> maxValue;

额外优化建议 #

• 每次执行readFile >>操作后判断流状态，避免文件损坏或长度不足时继续读取
• 校验stoi转换后的数值范围，确保在0 ~ maxValue之间
• 类析构中添加r的内存释放逻辑，避免内存泄漏

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