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后返回async。您正在传递Error Middleware，因为您正在返回一个包含4个参数的函数。

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

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) => {});

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

1 个答案: