QFtp多文件下载异常求助：仅最后一个文件有数据或崩溃

嘿，我来帮你拆解这个QFtp下载的坑！你遇到的两个问题——仅最后一个文件有数据、关闭QFile时崩溃，本质上都是QFile指针被重复覆盖+文件生命周期管理混乱导致的，咱们一步步来解决：

核心问题根源 #

1. 仅最后一个文件有数据：每次触发listInfo信号时，你都用reportFile = new QFile(...)覆盖了之前的指针，前面创建的QFile对象完全没和对应的FTP下载操作绑定，FTP的写入流始终指向最后创建的那个文件，前面的文件自然没数据，大小为0。
2. 关闭QFile时崩溃：指针被多次覆盖后，之前的QFile对象变成了野指针，当你尝试通过这个指针调用close()或delete时，要么访问了已释放的内存，要么操作的对象根本不是目标文件，直接触发崩溃。

修复方案：为每个文件绑定独立生命周期 #

我们需要为每个要下载的文件创建专属的QFile对象，通过QFtp的命令完成信号，把文件的打开、写入、关闭操作和对应的下载流程绑定起来，确保每个文件都能被正确处理。

修正后的完整代码示例 #

QFtp *ftp = new QFtp(this);

// 连接成功后登录
connect(ftp, &QFtp::connected, [ftp]() {
    ftp->login(FTP_USERNAME, FTP_PASSWORD);
});

// 登录成功后触发文件列表查询
connect(ftp, &QFtp::loggedIn, [ftp]() {
    ftp->list();
});

// 处理每个文件的下载逻辑
connect(ftp, &QFtp::listInfo, [this, ftp](const QUrlInfo &ftpUrlInfo) {
    if (ftpUrlInfo.isFile()) {
        // 为每个文件创建独立的QFile对象
        QFile *reportFile = new QFile("some local path/" + ftpUrlInfo.name());
        if (!reportFile->open(QIODevice::WriteOnly)) {
            qWarning() << "Failed to open local file:" << reportFile->fileName();
            delete reportFile;
            return;
        }

        // 绑定下载完成后的操作：关闭文件并安全释放内存
        connect(ftp, &QFtp::commandFinished, reportFile, [reportFile, ftp](int id, bool error) {
            // 只处理当前文件对应的下载命令
            if (ftp->currentCommand() == QFtp::Get) {
                if (!error) {
                    qInfo() << "Download success:" << reportFile->fileName();
                } else {
                    qWarning() << "Download failed:" << reportFile->fileName();
                }
                reportFile->close();
                // 用deleteLater确保在事件循环中安全释放对象
                reportFile->deleteLater();
                // 断开临时连接，避免干扰后续命令
                disconnect(ftp, &QFtp::commandFinished, reportFile, nullptr);
            }
        });

        // 开始下载当前文件
        ftp->get(ftpUrlInfo.name(), reportFile);
    }
});

// 启动FTP连接
ftp->connectToHost(FTP_HOST, FTP_PORT);

关键优化点 #

• 独立文件对象：不再复用同一个指针，每个下载操作对应唯一的本地文件，彻底解决0字节问题。
• 命令完成绑定关闭：通过commandFinished信号，等FTP下载命令结束后再关闭文件，避免提前关闭导致写入失败。
• 安全内存管理：用deleteLater()替代直接delete，确保文件对象在事件循环中被安全释放，杜绝野指针崩溃。
• 命令过滤：只处理QFtp::Get类型的命令，避免其他FTP操作（比如list）触发错误的文件关闭逻辑。

额外建议：用智能指针更省心 #

如果你的项目支持C++11及以上，推荐用std::unique_ptr管理QFile对象，彻底避免野指针和内存泄漏：

auto reportFile = std::make_unique<QFile>("some local path/" + ftpUrlInfo.name());
if (!reportFile->open(QIODevice::WriteOnly)) {
    qWarning() << "Failed to open local file:" << reportFile->fileName();
    return;
}

// 转移智能指针所有权到lambda中
connect(ftp, &QFtp::commandFinished, this, [ftp, file = std::move(reportFile)](int id, bool error) {
    if (ftp->currentCommand() == QFtp::Get && file) {
        file->close();
        qInfo() << "Downloaded:" << file->fileName();
    }
});

ftp->get(ftpUrlInfo.name(), reportFile.get());

这样就能完美解决你遇到的两个问题啦！

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