在单独的控制器功能中使用 Multer 文件上传

Question

我正在编写一个 Express 应用程序，为了避免我的routes.js文件混乱，我创建了一个单独的，UploadController如下所示：

// UploadController.js
const multer = require('multer')
const storage = multer.diskStorage({
  destination: function (req, file, cb) {
    cb(null, 'public/' + file.fieldname + '/')
  },
  filename: function (req, file, cb) {
    cb(null, Date.now() + '-' + file.originalname)
  }
})

const upload = multer({
  storage: storage
})

module.exports = {
  upload
}

我routes像这样使用，具体来说这只是我的注册路线作为示例：

app.post('/register',
    RegistrationPolicy.validate,
    UploadController.upload.single('avatar'),
    UserController.register)

这工作得很好。理想情况下，我需要multipart/form-data在上传运行之前检查请求类型是否为，然后在上传完成后运行：

req.body.avatarUri = req.file.destination + req.file.filename

以便在UserController.register运行时上传文件的URI将存储在我的数据库中。

我能想到的最简洁的upload方法是在我的控制器中创建一个自定义方法，该方法执行检查、上传操作，然后是 URI 分配。问题是，将 multer 放在这样的函数中间：

async upload (req, res, next, field) {
    // ...check request type...
    multerUpload.single(field)
    // ...assign URI...
    next()
}

给我 Error: Route.post() requires a callback function but got a [object Promise]

是否可以按照我描述的方式使用multer？谁能指出我可能有什么误解？我是 Node+Express 的新手，所以如果我错过了一些需要的信息，请告诉我。

Answer 1

你提出的函数不符合Express中间件的签名规范。签名需要是：

function(requestObject, responseObject, nextCallback){}

函数的返回值并不重要（但Promise自从你声明它以来你就返回 a async）。您正在传递一个错误中间件，因为您要返回一个带有 4 个参数的函数。

我认为您应该分解逻辑以将请求类型检查到它自己的中间件中，单独链接到 multer，然后也在它自己的中间件中进行请求转换，除非我遗漏了一些东西：

function checkMultipart(req, res, next) {
    const contentType = req.headers["content-type"];
    // Make sure it's multipart/form
    if (!contentType || !contentType.includes("multipart/form-data")) {
        // Stop middleware chain and send a status
        return res.sendStatus(500);
    }
    next();
}

function rewriter(req, res, next) {
    // Set the request fields that you want
    req.body.avatarUri = req.file.destination + req.file.filename;
    next();
}

router.post("/", checkMultipart, upload.single("avatar"), rewriter, (req, res, next) => {});