check返回一个中间件，但您没有使用它，也没有向它传递所需的参数。

const { check, validationResult } = require("express-validator");

module.exports = {
    validateUsers(req, res, next) {
        if (req.method === "POST") {
            console.log(req.body.email);
            const middleware = check(["email", "Must consist only of letters."]).isEmail();
            middleware(req, res, next)
           
        }

        const errors = validationResult(req);

        if (!errors.isEmpty()) {
            return res.json(errors);
        }  else {
            return next();
        }
    }
}

我的建议是，这有点不符合人体工程学。它甚至可能不工作，因为从check返回的中间件可能调用next，然后您的代码就没有意义了。它的设计是为了不同的使用。它可能会更干净，如：

const { body, validationResult } = require("express-validator");

module.exports = {
    validateUsers: [
            body(["email", "Must consist only of letters."]).isEmail(),
            (req, res, next) => {
                const errors = validationResult(req);

                if (!errors.isEmpty()) {
                    return res.json(errors);
                }  else {
                    return next();
                }
            }
        ]
}

然后像这样使用：

app.post('/thing', validateUsers, (req, res, next) => {
   // you'd get here only if there were no issues
})