文件迁移程序仅生成空文件问题：指针无值的排查与修复求助

嘿，我来帮你捋捋这个文件移动程序的问题！从你描述的现象（新文件为空、指针s无值）和给出的代码片段来看，核心问题大概率出在文件大小获取后的指针重置和缺失的文件读取逻辑上，咱们一步步拆解：

核心问题排查 #

你提到的两个异常其实是连锁反应：

• 指针s无值 → 要么没读到源文件内容，要么内存分配的就是空空间
• 新文件为空 → 因为没有把s里的有效内容写进去，根源还是s没拿到数据

从你截断的sizeofFile(f...)代码来看，90%的概率是这个函数没正确重置文件指针——当你用fseek跳到文件末尾取大小后，指针停在末尾，后续读文件自然读不到任何内容，s就成了空的。

完整修复方案 #

1. 先把sizeofFile函数改对 #

这是最关键的一步，必须在获取文件大小后把指针重置回文件开头：

long sizeofFile(FILE* fp) {
    if (fp == NULL) return 0;
    // 跳到文件末尾获取大小
    fseek(fp, 0, SEEK_END);
    long fileSize = ftell(fp);
    // 必须重置指针到开头！不然后续读不到内容
    fseek(fp, 0, SEEK_SET);
    return fileSize;
}

2. 补全changeFileLoc的完整逻辑 #

你的原代码只打开了源文件，缺了读取内容、写入新文件的核心步骤，而且没有错误检查，很容易踩坑。下面是补全后的安全版本：

#include "Header.h"
#include <stdio.h>
#include <stdlib.h>
#include <string.h>

// 正确实现的文件大小获取函数
long sizeofFile(FILE* fp) {
    if (fp == NULL) return 0;
    fseek(fp, 0, SEEK_END);
    long fileSize = ftell(fp);
    fseek(fp, 0, SEEK_SET); // 关键：重置指针到文件开头
    return fileSize;
}

void changeFileLoc(char filename[], char location[]) {
    char newname[255];
    // 用snprintf替代sprintf，避免缓冲区溢出（更安全）
    snprintf(newname, sizeof(newname), "%s\\%s", location, filename);

    // 打开源文件（二进制模式）
    FILE* fpSrc = fopen(filename, "rb");
    if (fpSrc == NULL) {
        perror("打开源文件失败");
        return;
    }

    // 获取文件大小
    long fileSize = sizeofFile(fpSrc);
    if (fileSize <= 0) {
        fclose(fpSrc);
        perror("源文件为空或获取大小失败");
        return;
    }

    // 分配内存存储文件内容，同时检查分配是否成功
    char* s = (char*)malloc(fileSize);
    if (s == NULL) {
        fclose(fpSrc);
        perror("内存分配失败");
        return;
    }

    // 读取源文件内容到s
    size_t readBytes = fread(s, 1, fileSize, fpSrc);
    if (readBytes != fileSize) {
        free(s);
        fclose(fpSrc);
        perror("读取源文件不完整");
        return;
    }

    // 打开目标文件（二进制模式写入）
    FILE* fpDest = fopen(newname, "wb");
    if (fpDest == NULL) {
        free(s);
        fclose(fpSrc);
        perror("打开目标文件失败");
        return;
    }

    // 把内容写入新文件
    size_t writeBytes = fwrite(s, 1, fileSize, fpDest);
    if (writeBytes != fileSize) {
        perror("写入目标文件不完整");
    }

    // 必须清理资源：关闭文件、释放内存
    free(s);
    fclose(fpSrc);
    fclose(fpDest);
}

3. 关键修复点说明 #

• 指针重置：sizeofFile里的fseek(fp,0,SEEK_SET)是救命的一步，没它的话读完文件大小后指针在末尾，fread啥也读不到，s自然是空的
• 安全路径拼接：用snprintf代替sprintf，避免newname缓冲区溢出（比如路径太长的情况）
• 错误检查：给每个IO操作、内存分配加了错误提示，方便你后续排查问题
• 资源清理：所有打开的文件都关闭，分配的内存都释放，避免内存泄漏和文件占用

额外小提示 #

如果目标路径不存在，程序会创建文件失败，你可以加个路径检查的逻辑（比如Windows下用mkdir，需要包含<direct.h>）；另外如果要处理长文件名，newname的255长度可能不够，可以改成_MAX_PATH（Windows系统定义的最大路径长度）。

内容的提问来源于stack exchange，提问作者F.Wu