malloc分配的一维模拟多维数组scanf写入崩溃问题及实现最佳实践

咱们直奔主题——你代码崩溃的核心原因是input函数里的scanf调用用错了指针层级！

你看，arr是int**类型，它指向的是你malloc出来的一维数组的指针（也就是*arr）。而你现在写的arr[x * *len_y + y]，相当于直接对arr这个二级指针本身做偏移计算，这完全不是你要访问的那块连续数组内存！这种错误的内存访问直接触发了越界，程序自然崩溃。

你的核心思路（用连续一维数组模拟多维数组）是完全正确的——这种方式内存连续、缓存友好，还比指针数组省内存，没问题！只是指针操作时没搞清楚层级。

先把input函数里的错误改掉，同时咱们加上malloc失败的检查（这是好习惯，别忽略），还有完善函数返回值、补充内存释放：

#include <stdio.h>
#include <stdlib.h>

// 返回0表示成功，非0表示错误
int input(int **arr, int *len_x, int *len_y);
void output(int *arr, int len_x, int len_y); // 直接传长度值更清晰

int main() {
    int *array = NULL;
    int length_x, length_y;
    
    if (input(&array, &length_x, &length_y) != 0) {
        fprintf(stderr, "输入或内存分配失败\n");
        return 1;
    }
    
    output(array, length_x, length_y);
    free(array); // 别忘了释放内存！你原代码漏了这个
    return 0;
}

int input(int **arr, int *len_x, int *len_y) {
    if (scanf("%d %d", len_x, len_y) != 2) {
        return 1; // 输入格式错误
    }
    
    // 计算总元素数，分配内存
    size_t total = (size_t)*len_x * *len_y;
    *arr = malloc(total * sizeof(int));
    if (*arr == NULL) {
        return 2; // 内存分配失败
    }
    
    // 访问*arr指向的一维数组，用(*arr)+偏移量或&(*arr)[下标]
    for (int x = 0; x < *len_x; x++) {
        for (int y = 0; y < *len_y; y++) {
            // 两种写法二选一，选你看着舒服的
            scanf("%d", (*arr) + x * *len_y + y);
            // scanf("%d", &(*arr)[x * *len_y + y]);
        }
    }
    return 0;
}

void output(int *arr, int len_x, int len_y) {
    for (int x = 0; x < len_x; x++) {
        for (int y = 0; y < len_y; y++) {
            printf("%d ", arr[x * len_y + y]); // 加空格让输出更美观
        }
        printf("\n");
    }
}

1. 指针层级要拎清

   • int** arr：是指向int*的指针，它存的是一维数组的地址
   • *arr：才是你malloc出来的一维数组的首地址，所有数组元素都在这个指针指向的连续内存里

2. 连续存储模拟多维数组的优势

   • 内存连续，CPU缓存命中率更高，访问速度更快
   • 只需要一次malloc和一次free，内存管理更简单
   • 比“指针数组+多个malloc”的方式少占用内存（不需要存储额外的指针）

3. 必做的健壮性检查

   • 永远检查malloc的返回值，避免空指针访问
   • 检查scanf的返回值，确保输入符合预期
   • 记得释放分配的内存，避免内存泄漏

4. 代码可读性优化

   • 可以把索引计算封装成宏，比如：

     #define INDEX(x, y, len_y) ((x) * (len_y) + (y))
     

     这样写(*arr)[INDEX(x, y, *len_y)]就不容易写错了
   • 输出函数里的长度不需要传指针，直接传值更清晰，因为你不需要修改它们

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