递归函数中结构体指针语法问题：查找指定ID位置编译报错

编译错误核心原因 #

• 参数类型不匹配：
  1. 调用idPosition时，第一个参数传了*fp（实际是FILE对象，函数需要FILE*类型），应直接传fp；
  2. 第二个参数传了*cli（单个struct client对象），但函数参数定义为int* arr，实际需要struct client*类型，类型不匹配导致后续访问.id成员报错。
• 返回值类型冲突：函数声明返回int，但代码里返回NULL（指针类型），C语言不允许跨类型返回，应使用-1这类无效索引标记未找到。
• 结构体访问错误：arr被定义为int*，却尝试访问(*(arr+i)).id，int类型没有id成员，本质是参数类型定义错误，arr应为struct client*。

逻辑错误问题 #

• 递归终止条件错误：用idWanted == top作为终止条件完全不符合逻辑，正确的终止条件是遍历索引i >= top（超出数组范围）。
• 冗余文件读取：注释说明cli数组已初始化id={1,2,3}，但函数每次递归都调用fread覆盖已有数据，逻辑矛盾。
• 资源泄漏与输出缺失：main函数未检查fopen是否成功，未关闭文件，也未输出查询结果。

修正后的代码 #

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

struct client {
    int id;
} cli[10] = {{1}, {2}, {3}}; // 显式初始化前三个元素的id

// 修正参数类型，返回-1表示未找到
int idPosition(struct client *arr, int idWanted, int i, int top);

int main() {
    int id, position;
    FILE *fp = fopen("clients.txt","rb+");
    
    // 检查文件打开是否成功
    if (fp == NULL) {
        perror("Failed to open file");
        return 1;
    }

    printf("ID: ");
    scanf("%i", &id);
    
    // 修正函数调用参数：直接传cli数组，无需文件指针（若cli已初始化）
    position = idPosition(cli, id, 0, 10);
    
    if (position != -1) {
        printf("Position of ID %d is %d\n", id, position);
    } else {
        printf("ID %d not found\n", id);
    }
    
    fclose(fp); // 关闭文件
    return 0;
}

int idPosition(struct client *arr, int idWanted, int i, int top) {
    // 终止条件：超出数组范围，返回-1
    if (i >= top) {
        return -1;
    }
    
    if (arr[i].id == idWanted) {
        return i;
    }
    
    // 递归遍历下一个元素
    return idPosition(arr, idWanted, i+1, top);
}

额外说明 #

如果确实需要从文件读取结构体数据而非使用已初始化的cli数组，可修改idPosition函数，避免递归重复读取（文件指针会自动偏移），调整逻辑如下：

int idPosition(FILE *fp, int idWanted, int top) {
    struct client temp;
    int i = 0;
    rewind(fp); // 重置文件指针到开头
    
    while (i < top && fread(&temp, sizeof(struct client), 1, fp) == 1) {
        if (temp.id == idWanted) {
            return i;
        }
        i++;
    }
    return -1;
}

此时main中调用改为position = idPosition(fp, id, 10);即可。

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