Express + Multer：请求体验证失败时如何阻止图片保存

你的问题根源在于：upload.single('image')作为路由中间件，会在后续的验证、业务逻辑之前就完成图片的磁盘写入。所以哪怕后续验证失败，图片已经被存储到磁盘中了。下面提供两种可行的解决方案：

方案一：验证失败后删除已上传文件 #

这是对现有代码改动最小的方案，在验证出错或业务逻辑不通过时，手动删除已保存的图片文件。

修改后的路由代码 #

const fs = require('fs'); // 引入Node.js内置文件系统模块

router.post('/', [Auth, Admin], upload.single('image'), async (req, res) => {
    if (!req.file)
        return res.status(400).send("you didn't upload image (jpeg OR png)");
    
    try {
        const { error } = validate(Object.assign({}, req.body, { image: req.file.path }));
        if (error) {
            // 验证失败，删除已上传的图片
            fs.unlinkSync(req.file.path);
            return res.status(400).send(error.details[0].message);
        }

        let singer= await Singer.findOne({ singername: req.body.singername});
        if (singer) {
            // 歌手已存在，删除图片
            fs.unlinkSync(req.file.path);
            return res.status(400).send('Singer already added');
        }

        singer= new Singer({
            singername: req.body.singername,
            tag: req.body.tag,
            image: (process.env.BASE_URL || config.get('BASE_URL')) + req.file.path,
        });
        await singer.save();
        return res.status(200).send(singer);
    } catch (err) {
        // 捕获数据库等其他异常，同时删除已上传图片
        if (req.file) fs.unlinkSync(req.file.path);
        res.status(500).send('服务器内部错误');
    }
});

说明：

• fs.unlinkSync是同步删除文件的方法，写法直观，适合这个场景；如果担心阻塞事件循环，也可以改用异步的fs.unlink配合回调。
• 所有可能导致请求失败的分支都要加上删除逻辑，避免残留无效文件。

方案二：先验证再保存文件（内存存储暂存） #

改用Multer的内存存储，先把图片暂存到内存中，等验证和业务逻辑全部通过后，再手动写入磁盘。这种方式从根源上避免了无效文件的生成。

第一步：修改上传中间件 #

const multer=require('multer')

// 使用内存存储，文件会暂存在req.file.buffer中
const storage= multer.memoryStorage()

const filefilter=(req,file,cb)=>{
    if (file.mimetype === 'image/jpeg' || file.mimetype === 'image/png') {
        cb(null, true);
    } else {
        cb(null, false);
    }
}

const upload=multer({
    storage:storage,
    limits:{
        fileSize:1024*1024*5
    },
    fileFilter:filefilter
});

module.exports=upload;

第二步：修改路由代码 #

const fs = require('fs');
const path = require('path');

router.post('/', [Auth, Admin], upload.single('image'), async (req, res) => {
    if (!req.file)
        return res.status(400).send("you didn't upload image (jpeg OR png)");
    
    try {
        // 先验证请求体（此时还未保存图片，无需传入image路径）
        const { error } = validate(req.body);
        if (error) {
            return res.status(400).send(error.details[0].message);
        }

        let singer= await Singer.findOne({ singername: req.body.singername});
        if (singer) {
            return res.status(400).send('Singer already added');
        }

        // 验证通过，生成文件名和存储路径
        const fileName = new Date().toISOString().replace(/:/g,'-') + req.file.originalname;
        const filePath = path.join('./images/', fileName);
        
        // 将内存中的图片写入磁盘
        fs.writeFileSync(filePath, req.file.buffer);

        // 创建并保存歌手数据
        singer= new Singer({
            singername: req.body.singername,
            tag: req.body.tag,
            image: (process.env.BASE_URL || config.get('BASE_URL')) + filePath,
        })
        await singer.save();
        return res.status(200).send(singer);

    } catch (err) {
        res.status(500).send('服务器内部错误');
    }
});

说明：

• 内存存储适合文件大小限制较小的场景（你的限制是5MB，完全没问题），不会产生磁盘IO浪费。
• 验证通过后才执行磁盘写入，彻底避免无效文件残留。

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