NodeJs MVC 架构，这是通心粉代码吗？

Question

我是一名初级 NodeJS 开发人员，目前在加密货币交易平台上工作。这是项目文件夹结构。

/app
    /controllers
        /user
        /order
        /settings
        ...
        index.js
    /middlewares
    /models
    /routes
        user.js
        order.js
    /services
        user.js
        order.js
        ...
    /views
    index.js
/config
/migrations
/public
/utils
server.js
.env
...

现在，一开始有点不知所措，但后来，我在应用程序中移动变得很舒服。
除了一个特定的文件！ 控制器目录中的 index.js， 这是它的设置方式。

const { readdirSync, lstatSync } = require('fs');
const { validationSchema } = require('../../utils/validator');

module.exports = readdirSync('app/controllers')
  .filter(name => lstatSync(`app/controllers/${name}`).isDirectory())
  .reduce((controllersAccumulator, dir) => Object.assign(
    controllersAccumulator,
    {
      [`${dir}Controller`]: readdirSync(`app/controllers/${dir}`)
        .map(fileName => require(`./${dir}/${fileName}`))
        .reduce((accum, controllerFile) => Object.assign(
          accum,
          Object.keys(controllerFile).reduce(validationSchema.bind(null, dir, controllerFile), {}),
        ), {}),
    },
  ), {});

我不得不承认，这对我来说一直很害怕，只是看看而已！ 所以简单来说就是，它将路由请求映射到控制器目录中的处理程序。

例如，如果一个用户想发出一个帖子请求注册一个新帐户：路由路径将是这样的：

// in the routes dir, user.js 
const { userCOntroller } = require('../controllers/');

router.post('/registration', userController.registration);


// in the /controllers, then /user, there will be a registration.js that includes:
const UserService = require('../../services/user');

exports.registration = async (req, res) => await UserService.create(req.body);


//then in the /services directory, user.js
...
class UserService {
  static create(body) { ... }
  ...
}

module.exports = UserService

所以我仍然无法理解的是，我们是如何在最后一个 sn-p 的用户路由中导入 userController 的？这就是/controllers 中的 index.js 文件所产生的内容！
当我问团队中的高级人员时，他们说，是的，它很难阅读，但它的代码更少。 好吧，好吧：\

那么，为了使这个文件更具可读性，可以做些什么不同的事情， 换句话说，有没有办法重构它？ 提前致谢！

Answer 1

解释第一个代码 sn-p 的最简单方法是重写它并添加适当的 cmets：

//Get all file names inside the controllers directory
const fileList = readdirSync('app/controllers');
//Get only directories from the file list
const onlyDirectories = fileList.filter(name => lstatSync(`app/controllers/${name}`).isDirectory());
//Create an empty object that will be exported from this module. Properties of this object will be assigned in the "for of" loop
const objectToExport = {};
//Import all files from the given directory and assign them to objectToExport
for (let directoryName of onlyDirectories){
    //First get all file names
    const controllerFiles = readdirSync(`app/controllers/${directoryName}`);
    //Then require each of these files
    const controllerModules = controllerFiles.map(fileName => require(`./${directoryName}/${fileName}`));
    //Assign the imported modules to `objectToExport`
    controllerModules.forEach((controllerModule) => {
        //Validate each module and assign it only if it passes validation
        if (validationSchema(directoryName, controllerModule)){
            objectToExport[`${directoryName}Controller`] = controllerModule;
        }
    });
}

module.exports = objectToExport;

下面我将解决您来自 cmets 的后续问题。 生成的对象现在如下所示：

{
    userController: {
        registration: [Function],
        ...
    },
    orderController: {
        ...
    },
    ...
}

注册函数最终出现在 userController 属性中，因为它是由 registration.js导出，然后由我的第一个代码 sn 中的 require 语句导入 -p。为了在其他文件中直接使用这个函数，你必须通过以下方式对其进行解构：

const { userController: {registration} } = require('../controllers/');
//now you can use it like this:
router.post('/registration', registration);

您的最后一个问题是关于req.body。如您所见，该函数有两个参数：req 和 res：

exports.registration = async (req, res) => await UserService.create(req.body);

然后将其作为中间件传递给您的路由器。我假设您使用Express.js 作为您的框架。如果是这样，req 和res 会被路由器自动传递给registration 函数。这就是UserService 在此函数中使用它们的方式。 body 属性由 Express 自动创建，如 here 所述。