调用FindFiles遍历C:\失败但遍历C:\Windows成功的原因及建议

问题原因分析 #

这个问题我之前也碰到过，主要是对Windows API的FindFirstFile行为理解不到位，再加上路径处理的小细节问题导致的，咱们一步步拆解：

1. FindFirstFile的参数逻辑误解 #

当你传入L"c:\\"给FindFiles时，FindFirstFile拿到的是没有通配符的根目录路径。按照Windows API的规则，这种情况下函数只会检查该目录是否存在，而不会枚举目录下的所有内容。虽然它会返回一个有效的句柄，但紧接着调用FindNextFile会立刻返回FALSE（因为没有更多匹配项），导致你看不到任何文件输出，看起来像是“执行失败”。

而FindFiles(L"c:\\Windows")能“正常执行”其实是巧合：FindFirstFile找到了Windows这个目录本身，打印完路径后循环就结束了——它同样没有枚举Windows目录内的内容，只是刚好有一条输出让你误以为正常。

2. CombinePaths的路径处理隐患 #

当处理根目录路径时，PathRemoveFileSpec不会修改路径（因为根目录没有父目录），这部分本身没问题，但如果FindFirstFile返回的是.或..（根目录下默认存在的特殊目录），组合出来的路径会是C:\.或C:\..，这虽然合法，但并不是你想要的实际文件/目录路径。

解决建议 #

针对这两个问题，咱们可以一步步修正代码：

1. 让FindFiles自动处理目录枚举逻辑 #

修改FindFiles函数，先判断传入的路径是文件还是目录，如果是目录，自动添加通配符*来触发枚举行为，同时跳过.和..这两个特殊目录：

void FindFiles(std::wstring const &pattern) {
    std::wstring searchPath = pattern;
    // 检查路径是否为目录
    DWORD attr = GetFileAttributes(searchPath.c_str());
    if (attr != INVALID_FILE_ATTRIBUTES && (attr & FILE_ATTRIBUTE_DIRECTORY)) {
        // 确保目录路径末尾有反斜杠（如果没有的话）
        if (!searchPath.empty() && searchPath.back() != L'\\') {
            searchPath.push_back(L'\\');
        }
        // 添加通配符，触发目录内容枚举
        searchPath += L"*";
    }

    WIN32_FIND_DATA fd;
    HANDLE h = FindFirstFile(searchPath.c_str(), &fd);
    if (h == INVALID_HANDLE_VALUE) {
        wprintf(L"FindFirstFile. Err=%d\n", GetLastError());
        return;
    }

    do {
        // 跳过.和..特殊目录
        if (wcscmp(fd.cFileName, L".") == 0 || wcscmp(fd.cFileName, L"..") == 0) {
            continue;
        }
        std::wstring fullPath = CombinePaths(pattern, fd.cFileName);
        wprintf(L"FullPath=%s\n", fullPath.c_str());
    } while (FindNextFile(h, &fd));

    // 区分正常结束和错误结束
    DWORD err = GetLastError();
    if (err != ERROR_NO_MORE_FILES) {
        wprintf(L"FindNextFile. Err=%d\n", err);
    }

    FindClose(h);
}

2. 优化CombinePaths函数（可选但推荐） #

如果你的编译器支持C++17及以上，强烈建议改用std::filesystem来处理路径，它比Windows API更简洁可靠，还能跨平台：

#include <filesystem>
namespace fs = std::filesystem;

std::wstring CombinePaths(std::wstring const &pattern, LPCWSTR filename) {
    fs::path basePath(pattern);
    // 如果传入的是文件路径，取它的父目录；如果是目录，直接使用
    if (fs::is_regular_file(basePath)) {
        basePath = basePath.parent_path();
    }
    // 路径拼接
    return (basePath / filename).wstring();
}

如果不能使用C++17，也可以修正原来的CombinePaths，去掉冗余的操作，让它更健壮：

std::wstring CombinePaths(std::wstring const &pattern, LPCWSTR filename) {
    std::wstring retVal(MAX_PATH, L'\0');
    // PathCombine会自动处理路径拼接，不管base路径是否有反斜杠
    PathCombine(retVal.data(), pattern.c_str(), filename);
    // 去掉字符串末尾多余的空字符
    retVal.resize(wcslen(retVal.c_str()));
    return retVal;
}

3. 调用方式无需修改 #

现在不管你调用FindFiles(L"c:\\Windows")还是FindFiles(L"c:\\")，都会正确枚举目标目录下的所有文件和子目录，不会再出现“执行失败”的情况。

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