C语言用函数填充文件内容到数组时的编译警告问题

警告原因 #

你遇到的assignment to ‘char’ from ‘char *’警告，核心是类型不匹配：

• strtok()返回的是char*类型指针，指向分割后的字符串首地址
• 但你定义的dictionary是char[]类型，每个元素仅能存储单个char字符，把指针直接赋值给单个字符，既触发类型转换警告，也完全不符合你要存储字符串的逻辑。

代码中的其他问题 #

1. 数据结构错误：main里的char dict[250]只能存250个单个字符，无法容纳250个字符串
2. 数组越界风险：循环for(j=0; j<=250; j++)会访问dict[250]，超出数组合法下标范围（数组下标从0到249）
3. 字符串指针失效：直接赋值strtok的返回指针不可靠——line会被反复清空复用，后续操作会覆盖之前的字符串内容

修正方案 #

1. 调整数据结构 #

将字典改为字符串数组，用char* dict[250]预分配250个字符串指针，用于存储每个分割后的字符串。

2. 修改readDict函数 #

• 函数参数改为char** dictionary，同时传入最大容量参数防止越界
• 用strdup()（或malloc()+strcpy()）复制strtok返回的字符串到字典元素中，避免指针失效

3. 修正主函数逻辑 #

• 遍历字符串数组时判断元素是否为NULL，避免输出无效内容
• 修正循环条件，杜绝越界访问

修正后的完整代码 #

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

// 修改参数为字符串数组指针，同时传入最大容量
void readDict(char** dictionary, int max_size);

void readDict(char** dictionary, int max_size) {
    int i = 0;
    FILE *fp = fopen("dictionary/top250.txt", "r");
    if(fp == NULL) {
        printf("Error! opening file");
        exit(1);
    }
    char chunk[128];
    size_t len = sizeof(chunk);
    char *line = malloc(len);
    if(line == NULL){
        perror("Unable to allocate memory to buffer");
        exit(1);
    }
    line[0] = '\0';
    while(fgets(chunk, sizeof(chunk), fp) != NULL && i < max_size) {
        // 检查剩余空间是否足够，不够则扩容
        if(len - strlen(line) < sizeof(chunk)) {
            len *= 2;
            char* temp = realloc(line, len);
            if(temp == NULL) {
                perror("Unable to reallocate memory to buffer.");
                free(line);
                exit(1);
            }
            line = temp;
        }
        strcat(line, chunk);
        // 检查是否读取到完整一行
        if(line[strlen(line)-1] == '\n') {
            // 去掉换行符，避免影响分割结果
            line[strlen(line)-1] = '\0';
            
            char* one = strtok(line, "\t");
            char* two = strtok(NULL, "\t");
            char* three = strtok(NULL, "\t");
            char* fourth = strtok(NULL, "\t");
            // 确保第四个分割结果存在，再复制
            if(fourth != NULL) {
                // 用strdup复制字符串到字典，strdup内部会自动malloc内存
                dictionary[i] = strdup(fourth);
                i++;
            }
            line[0] = '\0';
        }
    } 
    fclose(fp);
    free(line);
}

int main(int arg_count, char *arg[]) {
    // 定义字符串数组，最多存250个字符串，初始化为NULL
    char* dict[250] = {NULL};
    readDict(dict, 250);
    
    int j = 0;
    // 遍历到第一个NULL元素为止，或者遍历完250个
    while(j < 250 && dict[j] != NULL) {
        fputs(dict[j], stdout);
        putchar('\n'); // 换行，方便查看
        // 释放每个字符串的内存，避免泄漏
        free(dict[j]);
        j++;
    }
    
    // Rest of the code
    return 0;
}

关键修改说明 #

• char* dict[250]：每个元素是字符串指针，用于存储分割后的第四个字符串
• strdup(fourth)：复制strtok返回的字符串内容，分配独立内存，避免原line被覆盖后指针失效
• 增加max_size参数：限制字典的最大元素数量，防止数组越界
• 循环条件修改：while(j < 250 && dict[j] != NULL)，避免访问无效元素
• 内存释放：主函数中释放每个字符串的内存，避免内存泄漏

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