NodeJS更新产品时无法删除images文件夹旧图片该如何解决？

核心错误点 #

• 你给fs.unlinkSync传入的是Express静态资源的访问路由路径/images/xxx，但Node.js的fs文件操作模块需要的是服务器本地磁盘的实际文件路径，系统找不到对应路径的文件，所以删除失败
• fs.unlinkSync是同步API，不需要加await关键字，只有异步的fs.promises.unlink或者回调式的fs.unlink才需要处理异步逻辑

修复步骤 #

1. 首先构造正确的本地文件路径，推荐用path模块拼接绝对路径，避免相对路径的工作目录偏差问题
2. 先判断旧图片文件是否存在，避免删除不存在的文件抛出异常
3. 移除同步API前多余的await

修复后代码 #

首先在文件顶部引入path模块：

import path from 'path'
// 如果是CommonJS规范用：const path = require('path')

修改updateProduct里的删除逻辑：

export const updateProduct = async (req, res)=>
{
    try{
        let image 
        const oldProduct = await Product.findOne({_id:req.params.id})
        const {name,price,quantity,category} = req.body
        
        if(req.file)
        {
         image = req.file.filename
         // 构造本地实际文件路径，这里假设images文件夹在项目根目录
         const oldImagePath = path.resolve('./images', oldProduct.image)
         // 先判断文件是否存在再执行删除
         if(fs.existsSync(oldImagePath)){
             fs.unlinkSync(oldImagePath)
         }

        }else{
            image = oldProduct.image
        }
        const productToUpdate = {name,category,quantity,price,image}
        // 这里不需要new Product，直接传更新对象即可，避免生成多余的_id字段
        await Product.updateOne({_id:req.params.id}, productToUpdate)
        
         res.status(200).json('product Updated')
     }catch(error)
     {
         res.status(404).json({message:error.message})
     }
}

额外优化提示 #

• 原代码里new Product()再传入updateOne属于冗余操作，直接传普通对象即可，避免生成不必要的新_id导致异常
• 如果是高并发场景推荐用异步的fs.promises.unlink替代同步API，避免阻塞事件循环

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