C语言结构体Set存储int数组：可变参数读取失败求助

我帮你梳理下代码里的几个关键问题，这些都是导致数字读取失败、打印无输出的核心原因：

1. 结构体传值调用导致修改无效 #

你当前的read_set函数参数是set s，这是传值调用——函数里操作的是原结构体的副本，所有修改都不会影响全局的SETA。必须改成传指针，才能直接修改原结构体的数组。

2. 可变参数列表的错误使用 #

• 不能直接用*ap != -1判断终止条件：va_list是C标准库的内部指针类型，直接解引用它的行为是未定义的，正确做法是先通过va_arg取出当前参数，再判断是否为-1。
• 终止逻辑顺序错误：你需要先取出值、判断不是-1后再存入数组，否则会把终止符-1也误存进去。

3. 数组索引与打印的错误 #

在read_set里，你执行s.arr[i++]=va_arg(ap,int);后，立刻用printf("%d was entered\n",s.arr[i]);——此时i已经自增，访问的是下一个未赋值的元素（默认初始化为0），应该用i-1来访问刚存入的元素。

4. print_set函数的遍历逻辑错误 #

你的遍历代码while(*iptr++){ printf("%d ",*iptr); }有两个问题：

• 第一次判断*iptr（第一个元素）后，指针先自增再打印，导致第一个元素永远不会被输出；
• 如果数组第一个元素是0，循环会直接终止，后续元素都无法被处理。
  建议在结构体里增加一个count字段记录元素数量，用数量来遍历，既安全又直观。

修正后的完整代码 #

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

#define MAX 100

// 优化结构体，增加元素个数字段，方便安全遍历
typedef struct set {
    int arr[MAX];
    int count; // 记录实际存储的元素数量
} set;

set SETA;

// 修改为传指针，确保能修改原结构体
int read_set(set *s, ...);
void print_set(set s);

int main(){
    int x;
    // 传入SETA的地址，让函数直接操作原结构体
    x = read_set(&SETA, 2, 3, 4, -1);
    printf("%d numbers were read\n", x);
    print_set(SETA);
    return 0;
}

void print_set(set s){
    // 用count字段遍历，避免0元素导致的提前终止
    for(int i = 0; i < s.count; i++){
        printf("%d ", s.arr[i]);
    }
    printf("\n");
}

int read_set(set *s, ...){
    va_list ap;
    int i = 0;
    int num;

    va_start(ap, s); // 最后一个固定参数是s（指针）

    // 先取出参数，判断不是-1再存入数组
    while((num = va_arg(ap, int)) != -1){
        if(i >= MAX){ // 防止数组越界，增加容错
            printf("Warning: 超过最大容量%d，停止读取\n", MAX);
            break;
        }
        s->arr[i] = num;
        printf("%d was entered\n", s->arr[i]); // 直接用i访问刚存入的元素
        i++;
    }
    va_end(ap);

    s->count = i; // 更新结构体里的元素个数
    return i;
}

运行输出 #

2 was entered
3 was entered
4 was entered
3 numbers were read
2 3 4 

这样就完美解决了你的问题：数字正确存入数组，print_set能正常输出所有元素，读取的数量也准确无误。

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