如何在C语言中将二进制文件转为变量？求助代码问题排查

存在的问题： #

• 返回值类型不匹配：函数声明返回char*，但文件打开失败时返回1（int类型），这会导致编译警告或运行时未定义行为，应该返回NULL。
• EOF判断逻辑错误：fgetc()返回int类型（范围是0-255或EOF即-1），用char类型变量接收后，若char是无符号类型，EOF会被转换为255，永远不等于EOF，导致循环无限执行，最终越界访问内存。
• 未检查内存分配结果：malloc()可能返回NULL（内存不足时），后续直接操作string会触发崩溃。
• 文件大小获取不可靠：fseek(file, 0, SEEK_END)和ftell()对部分特殊文件（如管道、字符设备文件）无效，ftell()会返回-1，导致size错误。
• 读取效率低下：逐个字符调用fgetc()读取文件，远不如一次性用fread()读取高效。

修复后的代码： #

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

char* fileToVariable(const char* filename)
{
    FILE* file = fopen(filename, "rb");
    if (file == NULL)
    {
        printf("Error opening file");
        return NULL;
    }

    // 获取文件大小，处理ftell失败的情况
    if (fseek(file, 0, SEEK_END) != 0)
    {
        printf("Failed to seek file");
        fclose(file);
        return NULL;
    }
    long size = ftell(file);
    if (size == -1)
    {
        printf("Failed to get file size");
        fclose(file);
        return NULL;
    }
    printf("%ld\n", size);

    if (fseek(file, 0, SEEK_SET) != 0)
    {
        printf("Failed to seek file to start");
        fclose(file);
        return NULL;
    }

    // 分配内存并检查结果
    char* string = malloc(size + 1);
    if (string == NULL)
    {
        printf("Memory allocation failed");
        fclose(file);
        return NULL;
    }

    // 一次性读取文件内容
    size_t read_count = fread(string, 1, size, file);
    if (read_count != size)
    {
        printf("Failed to read entire file");
        free(string);
        fclose(file);
        return NULL;
    }
    string[size] = '\0';

    fclose(file);
    return string;
}

额外说明： #

• 函数参数filename改为const char*，符合只读参数的惯例。
• 所有文件操作后都检查返回值，确保错误被及时处理。
• 使用long类型存储文件大小，避免int的范围限制（大文件会溢出）。
• 不再使用fgetc()循环，改用fread()提升读取效率。
• 任何错误分支都确保关闭文件并释放已分配的内存，避免资源泄漏。

内容的提问来源于stack exchange，提问作者חננאל חמי