C语言动态分配数组中mean函数无输出及add指针警告问题求助

问题现象 #

1. 调用add函数时出现指针类型不兼容警告：期望Statistician（即int *）但实参类型为int **
2. mean函数调用后无输出，程序存在未定义行为

原代码片段：

typedef int* Statistician;

void add(Statistician answer, int *count, int *SIZE, int item);
// 警告：传递add函数的第1个实参时出现不兼容指针类型 [-Wincompatible-pointer-types]

int main() {
int SIZE;
Statistician *answer;
int count;
int item;

add(answer, count, SIZE, item);

printf("\nThe mean is: %.2f", mean(answer, SIZE));

return 0;
}

核心问题分析 #

1. 指针传递与类型不匹配 #

• typedef int* Statistician 定义后，Statistician *answer 等价于 int **answer，但add函数的第一个参数是Statistician（即int*），类型不匹配导致警告。
• 更严重的是：add函数中对answer的赋值是值传递，修改的是函数内部的副本，main中的answer指针不会被更新，最终mean函数访问的是未初始化的野指针，导致无输出或崩溃。
• 额外问题：add函数声明的参数int *count、int *SIZE，但调用时传递的是count、SIZE的值（而非地址），参数类型完全不匹配，无法正确传递元素个数。

2. add函数的内存分配错误 #

• 错误判断内存分配结果：if(item == NULL)，item是int类型，NULL是指针常量，类型不匹配，正确判断应为if(answer == NULL)。
• realloc时内存大小计算错误：追加元素后，新内存大小应为(item + m) * sizeof(int)，原代码用m * sizeof(int)会导致内存不足，数组越界。
• 元素个数未同步回main：add函数中修改的item是局部变量，main中的SIZE/count无法获取实际元素数量，导致mean函数循环次数错误。

3. mean函数的逻辑错误 #

• 变量类型错误：mean定义为int，执行mean / cnt是整数除法，丢失小数部分，最终返回的int转float无法正确表示平均值。
• 冗余变量cnt：count已表示元素个数，循环中cnt++完全多余，若count为无效值（如未初始化的SIZE），会导致循环不执行或除以0。

修正后的完整代码 #

main函数 #

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

typedef int* Statistician;

void add(Statistician *answer, int *count, int item);
float mean(Statistician answer, int count);

int main() {
    int count = 0; // 存储实际元素个数
    Statistician answer = NULL; // 初始化为空指针
    int item;

    add(&answer, &count, item);

    if (answer != NULL && count > 0) {
        printf("\nThe mean is: %.2f", mean(answer, count));
        free(answer); // 释放动态分配的内存，避免泄漏
    } else {
        printf("\nNo valid data to calculate mean.");
    }

    return 0;
}

add函数 #

void add(Statistician *answer, int *count, int item) {
    int i;
    printf("Enter n: ");
    scanf("%d", &item);

    // 分配初始内存，通过指针修改main中的answer
    *answer = (int*)malloc(item * sizeof(int));
    if(*answer == NULL) {
        printf("Memory not allocated.\n");
        exit(EXIT_FAILURE);
    }

    printf("Enter %d elements: ", item);
    for(i = 0; i < item; ++i) {
        scanf("%d", &(*answer)[i]);
    }

    printf("Elements of array are: ");
    for(i = 0; i < item; i++) {
        printf("%d ", (*answer)[i]);
    }

    if(item == 10) {
        int m;
        printf("\nAppend array (enter number of elements to add): ");
        scanf("%d", &m);
            
        // 重新分配内存，大小为原元素数+追加元素数
        int *temp = realloc(*answer, (item + m) * sizeof(int));
        if(temp == NULL) {
            printf("Realloc failed.\n");
            free(*answer);
            exit(EXIT_FAILURE);
        }
        *answer = temp;

        printf("Enter %d elements: ", m);
        for(i = item; i < item + m; i++) {
            scanf("%d", &(*answer)[i]);
        }
        item += m;

        // 排序逻辑保留
        int temp_val, j;
        for(i = 0; i < item; i++) {
            for(j = 0; j <= i; j++) {
                if((*answer)[i] < (*answer)[j]) {
                    temp_val = (*answer)[i];
                    (*answer)[i] = (*answer)[j];
                    (*answer)[j] = temp_val;
                }
            }
        }
        
        printf("\nFinal array: \n");
        for(i = 0; i < item; ++i) { 
            printf("%d ", (*answer)[i]);
        }
    }

    // 将实际元素个数同步回main
    *count = item;
}

mean函数 #

float mean(Statistician answer, int count) {
    if(count <= 0) {
        return 0.0f; // 避免除以0
    }
    float total = 0.0f;
    for(int i = 0; i < count; i++){
        total += answer[i];
    }
    return total / count; // 浮点数除法保留小数
}

关键修复点总结 #

• 调整add函数参数为Statistician *，实现指针的引用传递，让main能获取动态分配的内存地址。
• 修正内存分配结果的判断逻辑，确保malloc/realloc失败时正确处理。
• 同步元素个数到main的count变量，保证mean函数能获取正确的循环次数。
• 修改mean函数的变量类型为float，使用浮点数除法计算平均值。
• 添加内存释放逻辑，避免内存泄漏。

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