如何用C语言求所有命令行参数的和？程序报错求助

我来帮你搞定这个问题，你的代码出现段错误和计算逻辑错误主要有几个明显的问题，咱们一步步修正：

问题分析 #

• 循环越界导致段错误：命令行参数数组argv的索引范围是0到argc-1（argv[0]是程序本身的名称，argv[argc]是一个空指针）。你写的循环条件counter <= argc会让counter最终等于argc，此时访问argv[argc]就是访问空指针，直接触发Segmentation fault。
• 错误使用atoi函数：atoi(sum)完全是误用——atoi需要接收字符串类型的参数（char*），但sum是int类型的变量，这里根本不需要对sum做转换，直接累加转换后的参数值就行。
• 缺少必要的头文件：atoi函数的声明在<stdlib.h>里，虽然部分编译器可能允许隐式调用，但标准C要求必须显式包含这个头文件，否则会有未定义行为。

修正后的完整代码 #

/* Printing sum of all command line arguments */
#include <stdio.h>
#include <stdlib.h>  // 必须包含这个头文件才能使用atoi

int main(int argc, char *argv[]) {
    int sum = 0, counter;
    // 循环条件改为counter < argc，避免访问argv[argc]空指针
    for (counter = 1; counter < argc; counter++) {
        // 直接累加转换后的参数值，不需要对sum用atoi
        sum += atoi(argv[counter]);
    }
    // 注意：这里要输出的是"传入的参数个数"，argc包含了程序名，所以应该是argc-1
    printf("Sum of %d command line arguments is: %d\n", argc - 1, sum);
    return 0;  // 标准main函数应该返回int值
}

额外说明 #

• 我还修改了printf里的参数个数：argc的值是程序名+传入参数的总个数，所以实际用户传入的参数个数是argc-1，这样输出更准确。
• 给main函数加上了return 0;，符合C标准的要求（虽然有些编译器允许省略，但加上更规范）。

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