使用指针与strtok计算逗号分隔整数列表的算术均值遇问题

你的代码存在几个关键问题导致无法正常运行：

• 循环逻辑错误：第一次获取token后进入while循环，先调用strtok(NULL, sep)，当处理到最后一个token时，strtok会返回NULL，此时你仍然执行atoi(token)（传入NULL是未定义行为），并且还会错误地累加total和count，导致结果多算一次无效值。
• 未处理空输入：如果传入的list是空字符串，初始strtok返回NULL，直接执行atoi(token)会触发错误。
• 数据类型溢出风险：使用int8_t存储总和和计数，范围仅为-128到127，一旦列表中整数数值较大或数量较多，会立即溢出，导致计算结果错误。
• 整数除法丢失精度：total/count是整数除法，即使赋值给double变量，也会先丢弃小数部分，无法得到正确的算术均值。

修复后的代码 #

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

float mean(char *list);

float mean(char *list) {
    const char sep[] = ",";
    char *token;

    // 初始化总和与计数，用更大范围的类型避免溢出
    long total = 0;
    int count = 0;

    // 第一次获取token
    token = strtok(list, sep);
    while (token != NULL) {
        // 转换为整数并累加
        total += atoi(token);
        count++;
        // 获取下一个token
        token = strtok(NULL, sep);
    }

    // 处理空列表的情况，避免除以0
    if (count == 0) {
        return 0.0f; // 或根据需求返回特定值
    }

    // 先转浮点型再做除法，保留小数精度
    return (float)total / count;
}

关键修复点说明 #

• 调整循环逻辑：将token的获取和判断统一放在while条件中，确保每次处理的都是有效的token，避免处理NULL指针。
• 扩大数据类型：用long存储总和，int存储计数，避免溢出问题。
• 处理空列表：增加count为0的判断，防止除以0的运行时错误。
• 修正除法精度：将total强制转换为浮点型后再除以count，确保得到正确的浮点结果。
• 移除不必要的头文件：inttypes.h在当前代码中未被使用，可以移除。

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