重构嵌套循环后出现段错误，C语言音频分帧功能故障排查

你优化后的单循环存在3个致命问题，直接导致段错误：

• 分配判断逻辑写反：if (j == 0 && !!(afp->frames[i] = calloc(n, sizeof(double)))) 里的!!完全颠倒了判断逻辑：原本应该是「分配失败（返回空指针）时进入错误处理」，现在变成了「分配成功时进入错误处理」，直接把已分配的结构体内存释放，返回空指针，后续访问返回值必然触发段错误。
• 循环提前终止导致帧未分配：原嵌套循环的逻辑是先为所有num_frames个帧完成内存分配，再逐元素填充，哪怕当前帧的数据超出输入长度也不会打断分配流程；但单循环中只要某一次(i*hop)+j >= sz就直接break终止整个循环，后续还没来得及分配内存的帧对应的frames[i]是未初始化的野指针，访问最后一帧时刚好触发野指针访问。
• 循环变量被污染：错误处理分支的释放循环直接修改了外层的循环变量i，哪怕前面没有问题，也会导致外层循环逻辑完全错乱，出现越界访问。

修复上述问题后的单循环实现如下：

for (size_t i = 0, j = 0, k; i < num_frames; (j == n - 1) ? (j = 0,i++) : ++j) {
    // 修正分配判断逻辑，仅分配失败时进入错误处理
    if (j == 0 && !(afp->frames[i] = calloc(n, sizeof(double)))) {
        double* p = afp->frames[i];
        // 用独立变量遍历释放，不要修改外层循环的i
        for (long ii = (long)i - 1; 0 <= ii; ii--) {
            free(afp->frames[ii]);
        }
        free(afp);
        return p;
    }
    // 不要break循环，仅超出输入长度时跳过赋值即可，保证所有帧分配完成
    k = i * hop + j;
    if (k < sz) {
        afp->frames[i][j] = x[k];
    }
}

• 合并内存分配：不要逐帧调用calloc，一次性计算所有帧需要的总内存大小 total_size = num_frames * n * sizeof(double)，单次分配一块连续内存，再将frames[i]指向内存块的i * n偏移位置，既减少了内存分配的系统调用次数，也消除了逐帧分配的错误分支，仅需要判断一次总分配是否成功。
• 提前消除边界判断：先预计算所有帧的起始偏移存入临时数组，循环填充时直接复用偏移量，同时利用calloc自动初始化0的特性，不需要在循环内判断是否超出输入长度，超出位置默认保留0即可。
• 避免浮点运算：帧数量、跳步长度的计算可以全部替换为整数运算，避免浮点精度误差导致的帧数量计算错误，也能提升计算效率。

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