C语言整数拆分数组异常：多位数输入仅返回长度为2的数组

你的代码核心问题出在main函数中计算数组长度的方式，还有几处潜在隐患，下面逐一拆解说明：

1. 指针的sizeof计算完全错误 #

在main里，arr是int*类型的指针，不是数组。C语言中，对指针使用sizeof得到的是指针本身的内存大小（32位系统为4字节，64位系统为8字节），而非它指向的数组总大小。

你写的sizeof(arr) / sizeof(arr[0])，在64位系统下结果是8/4=2，循环条件k<=2会执行3次？不对，实际是你循环写了k <=这个值，导致只能遍历到前2个元素（加上k=0共3次，但因为static数组残留的问题，可能只显示有效元素中的2个）——这就是输入超过3位时，看起来数组长度只有2的根本原因。

2. static数组的隐患 #

你用了static int arr[5];，这个数组存在全局静态存储区，每次调用splitNumberIntoArr都会覆盖之前的内容，且未初始化的元素会保留垃圾值。比如第一次输入3位数，数组后2个元素是垃圾；第二次输入4位数，前4个元素被覆盖，但如果循环多执行，会输出无意义的垃圾值。

3. 未传递有效元素长度 #

你的函数只返回了数组指针，但调用者不知道到底拆分出了多少位数字，导致循环次数无法正确控制。

修复后的代码方案 #

这里提供两种实用的修复思路：

思路一：通过指针参数返回有效元素个数 #

#include <stdio.h>

// 用len指针传递有效元素的个数
int* splitNumberIntoArr(int num, int* len) {
    static int arr[5];
    int i = num;
    int j = 0;

    // 清空数组，避免残留垃圾值
    for (int k = 0; k < 5; k++) {
        arr[k] = 0;
    }

    // 加j<5防止数组越界
    while (i > 0 && j < 5) {
        arr[j] = i % 10;
        i = i / 10;
        j++;
    }
    *len = j; // 把有效元素个数存入len指向的变量
    return arr;
}

int main() {
    int num;
    printf("Provide a number: ");
    scanf("%d", &num);   

    int validLen;
    int* arr = splitNumberIntoArr(num, &validLen);

    // 只遍历有效元素
    for(int k = 0; k < validLen; k++) {
        printf("%d\n", arr[k]);
    }
    
    return 0;
}

思路二：先计算位数再动态分配数组（更灵活） #

这种方式避免了static数组的全局存储区问题：

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

// 计算数字的位数
int getDigitCount(int num) {
    if (num == 0) return 1;
    int count = 0;
    while (num > 0) {
        count++;
        num /= 10;
    }
    return count;
}

int* splitNumberIntoArr(int num, int len) {
    int* arr = (int*)malloc(len * sizeof(int));
    if (arr == NULL) {
        printf("内存分配失败\n");
        exit(1);
    }
    int j = 0;
    while (num > 0 && j < len) {
        arr[j] = num % 10;
        num /= 10;
        j++;
    }
    return arr;
}

int main() {
    int num;
    printf("Provide a number: ");
    scanf("%d", &num);   

    int digitCount = getDigitCount(num);
    int* arr = splitNumberIntoArr(num, digitCount);

    for(int k = 0; k < digitCount; k++) {
        printf("%d\n", arr[k]);
    }

    free(arr); // 记得释放动态分配的内存
    return 0;
}

额外注意点 #

• 你原代码的main函数中，for循环后面遗漏了闭合大括号，导致return 0;被包含在循环内部，这是语法错误，编译时会报错。
• 如果输入数字为0，原代码的while(i>0)不会执行，数组无有效元素，修复后的代码已经处理了这种边界情况。

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