C代码出现Incompatible pointer type错误，请求排查指针与内存问题

问题描述 #

我编写了一段读取逗号分隔格式物品文件的C代码，出现了不兼容指针类型错误，不确定是否缺失内存分配或存在其他指针问题，恳请帮忙排查解决。

原代码 #

#include <stdio.h>
#include <string.h>
#define MAX_ITEM_NAME_STRING    25
#define MAX_ITEM_DESCRIPTION_STRING     80

typedef struct 
{
    char name[MAX_ITEM_NAME_STRING]; //25字符长度
    char description[MAX_ITEM_DESCRIPTION_STRING]; //80字符长度
}Item;

int ItemReadItems(Item *items[], int maxItems, char *filename){
    FILE * itemFile = fopen(filename, "r");
    // 文件内容是6行格式："comb","a small comb"
    char ch;// 注释：从逗号分隔的txt文件加载物品列表
    char line[MAX_ITEM_DESCRIPTION_STRING + MAX_ITEM_NAME_STRING + 10];
    int i =0;
    if(itemFile == NULL)
    { return -1;}// 无法打开文件则返回-1
    for(i<maxItems; i++)
    {// 将items.txt内容填充到item数组中
        
        while(fgets(line, sizeof(line), itemFile) != NULL){
          strcpy(items[i]->name, strtok(line, ","));
          strcpy(items[i]->description, strtok(line, ","));
        }
    }
    fclose(itemFile);
    return i;// 返回从文件加载的物品数量   
}

错误信息（翻译后） #

/mnt/c/Users/liloc/Documents/CSCV352/Exercise 6 Files/Exercise6_1/Item.c: 在函数‘ItemReadItems’中:
/mnt/c/Users/liloc/Documents/CSCV352/Exercise 6 Files/Exercise6_1/Item.c:15:15: 警告：传递‘fgets’的参数1时类型不兼容 [-Wincompatible-pointer-types]
   15 |         fgets(line, sizeof(line), itemFile);
      |               ^~~~
      |               |
      |               char **
从/mnt/c/Users/liloc/Documents/CSCV352/Exercise 6 Files/Exercise6_1/Item.h:3包含的文件中，
                 从/mnt/c/Users/liloc/Documents/CSCV352/Exercise 6 Files/Exercise6_1/Item.c:1包含的文件中：
/usr/include/stdio.h:564:14: 附注：期望‘char * restrict’但实参的类型为‘char **’
  564 | extern char *fgets (char *__restrict __s, int __n, FILE *__restrict __stream)
      |              ^~~~~
/mnt/c/Users/liloc/Documents/CSCV352/Exercise 6 Files/Exercise6_1/Item.c:16:39: 警告：传递‘strtok’的参数1时类型不兼容 [-Wincompatible-pointer-types]
   16 |         strcpy(items[i]->name, strtok(line, ","));
      |                                       ^~~~
      |                                       |
      |                                       char **
从/mnt/c/Users/liloc/Documents/CSCV352/Exercise 6 Files/Exercise6_1/Item.h:4包含的文件中，
                 从/mnt/c/Users/liloc/Documents/CSCV352/Exercise 6 Files/Exercise6_1/Item.c:1包含的文件中：
/usr/include/string.h:336:14: 附注：期望‘char * restrict’但实参的类型为‘char **’
  336 | extern char *strtok (char *__restrict __s, const char *__restrict __delim)
      |              ^~~~~~
/mnt/c/Users/liloc/Documents/CSCV352/Exercise 6 Files/Exercise6_1/Item.c:17:46: 警告：传递‘strtok’的参数1时类型不兼容 [-Wincompatible-pointer-types]
   17 |         strcpy(items[i]->description, strtok(line, ","));
      |                                              ^~~~
      |                                              |
      |                                              char **
从/mnt/c/Users/liloc/Documents/CSCV352/Exercise 6 Files/Exercise6_1/Item.h:4包含的文件中，
                 从/mnt/c/Users/liloc/Documents/CSCV352/Exercise 6 Files/Exercise6_1/Item.c:1包含的文件中：
/usr/include/string.h:336:14: 附注：期望‘char * restrict’但实参的类型为‘char **’
  336 | extern char *strtok (char *__restrict __s, const char *__restrict __delim)
      |              ^~~~~~

错误分析与修复方案 #

核心错误原因 #

1. for循环语法错误：原代码中for(i<maxItems; i++)完全不符合C语言for循环的语法（正确格式应为for(初始化; 条件; 增量)），这个语法错误导致编译器对变量line的类型解析混乱，误判为char**类型，进而触发不兼容指针类型警告。
2. 循环结构逻辑错误：外层for循环和内层while循环嵌套错误，会导致一次性读取文件所有行并覆盖到同一个items[i]中，无法正确填充多个Item对象。
3. strtok使用错误：第二次调用strtok时仍传入line，会重新从头分割字符串，正确做法是第二次及之后调用传入NULL，继续分割当前剩余的字符串。
4. 未处理带引号的字段：文件中字段是带双引号的，直接分割后会把引号也存入name和description，需要去除首尾的引号。
5. 潜在野指针风险：函数参数Item *items[]是Item指针数组，若调用者未为每个items[i]分配内存，会导致items[i]->name这类访问触发未定义行为。

修复后的代码 #

#include <stdio.h>
#include <string.h>
#include <stdlib.h> // 用于malloc内存分配

#define MAX_ITEM_NAME_STRING    25
#define MAX_ITEM_DESCRIPTION_STRING     80

typedef struct 
{
    char name[MAX_ITEM_NAME_STRING];
    char description[MAX_ITEM_DESCRIPTION_STRING];
}Item;

// 辅助函数：去除字符串首尾的双引号
void trim_quotes(char *str) {
    int len = strlen(str);
    if (len > 0 && str[0] == '"') {
        memmove(str, str + 1, len);
        len--;
    }
    if (len > 0 && str[len - 1] == '"') {
        str[len - 1] = '\0';
    }
}

int ItemReadItems(Item *items[], int maxItems, char *filename){
    FILE *itemFile = fopen(filename, "r");
    char line[MAX_ITEM_DESCRIPTION_STRING + MAX_ITEM_NAME_STRING + 10];
    int count = 0;

    if (itemFile == NULL) {
        return -1; // 打开失败返回-1
    }

    // 正确的循环逻辑：读取一行处理一个Item，直到文件结束或达到maxItems
    while (count < maxItems && fgets(line, sizeof(line), itemFile) != NULL) {
        // 去除换行符（fgets会读取换行符到line中）
        line[strcspn(line, "\n")] = '\0';

        // 为当前Item分配内存（如果调用者未预先分配）
        items[count] = (Item*)malloc(sizeof(Item));
        if (items[count] == NULL) {
            fclose(itemFile);
            return count; // 返回已成功加载的数量
        }

        // 分割字符串：第一次传line，后续传NULL
        char *token = strtok(line, ",");
        if (token != NULL) {
            trim_quotes(token);
            // 确保字符串长度不超过数组容量，避免溢出
            strncpy(items[count]->name, token, MAX_ITEM_NAME_STRING - 1);
            items[count]->name[MAX_ITEM_NAME_STRING - 1] = '\0'; // 手动添加终止符
        }

        token = strtok(NULL, ",");
        if (token != NULL) {
            trim_quotes(token);
            strncpy(items[count]->description, token, MAX_ITEM_DESCRIPTION_STRING - 1);
            items[count]->description[MAX_ITEM_DESCRIPTION_STRING - 1] = '\0';
        }

        count++;
    }

    fclose(itemFile);
    return count; // 返回成功加载的物品数量
}

关键修复点说明 #

• 修正for循环语法：移除错误的嵌套循环，改用单while循环控制读取行数和最大物品数。
• 正确使用strtok：第二次调用传入NULL，实现对同一行字符串的连续分割。
• 处理带引号字段：新增trim_quotes函数，去除字符串首尾的双引号。
• 内存分配：在函数内为每个Item分配内存（若调用者未预先分配），避免野指针访问；同时添加内存分配失败的处理逻辑。
• 字符串安全处理：改用strncpy替代strcpy，并手动添加字符串终止符，避免数组溢出。
• 去除换行符：使用strcspn去除fgets读取到的换行符，避免字符串中包含多余的换行字符。

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