在Node.js中将业务逻辑从控制器重构为服务

Question

我正在学习Node.js，但我在代码重构方面遇到了问题。我阅读了Node.js中的代码体系结构和良好的编码实践，我想重构我的代码。

我的当前代码:

User.Controller.js

const bcrypt = require('bcryptjs');
const User = require('../models/user');

exports.createUser = (req, res, next) => {
  bcrypt.hash(req.body.password, 10)
    .then(hash => {
      const user = new User({
        email: req.body.email,
        password: hash
      });
      user.save()
        .then(result => {
          res.status(201).json({
            message: 'User created!',
            result: result
          })
        })
        .catch(err => {
          res.status(400).json({
            message: 'An unknown error has occurred.'
          })
        });
    });
}

我想把所有的业务逻辑放到服务中。我试过这样的方法:

User.Controller.js

const UserService = require('../services/user.service');

exports.createUser = async function (req, res, next) {

  try {
    var result = await UserService.createUser(req.body.email, req.body.password);
    return res.status(200).json({ result: result, message: "User created!" });
  } catch (e) {
    return res.status(400).json({ message: e.message });
  }

}

user.service.js

const bcrypt = require('bcryptjs');
const User = require('../models/user.model');

exports.createUser = async function (email, password) {

  bcrypt.hash(password, 10)
    .then(hash => {
      const user = new User({
        email: email,
        password: hash
      });
      user.save()
        .then(result => {
          return result;
        })
        .catch(err => {
          throw new Error(err);
        });
    });

}

但关于诺言，我有很多错误:

(node:3760) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not ha
ndled with .catch(). (rejection id: 1)
(node:3760) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

我是node.js和JavaScript的新手。怎么解决这个问题？

Answer 1

谢谢你的回答。我修改了我的代码，现在看起来是这样的:

User.Controller.js

const UserService = require('../services/user.service');

exports.createUser = async function (req, res, next) {

  try {
    let user = await UserService.createUser(req.body.email, req.body.password);
    return res.status(201).json({ data: user, message: 'User created!' });
  } catch (e) {
    let errorMsg;
    let statusCode;
    if(e.errors['email'].kind === 'unique') {
      statusCode = 422;
      errorMsg = 'E-mail already exists.';
    } else {
      statusCode = 500;
      errorMsg = 'An unknown error has occurred.';
    }
    return res.status(statusCode).json({ message: errorMsg });
  }

}

user.service.js

const bcrypt = require('bcryptjs');
const User = require('../models/user.model');

exports.createUser = async function (email, password) {
  const hash = bcrypt.hashSync(password, 10);
  const user = new User({
    email: email,
    password: hash
  });
  await user.save();
  return result;
}

我的代码正确吗？我说的是编码架构。工作很好，但我不确定我的代码是否是“良好的实践”编程。我这样问是因为我想开始写整个后端，而我不确定我的代码。

null

  } catch (e) {
    let errorMsg;
    let statusCode;
    if(e.errors['email'].kind === 'unique') {
      statusCode = 422;
      errorMsg = 'E-mail already exists.';
    } else {
      statusCode = 500;
      errorMsg = 'An unknown error has occurred.';
    }
    return res.status(statusCode).json({ message: errorMsg });
  }

例如，在其他控制器中，我将有5个If语句。这是正确的解决方案吗？我在这里征求意见，什么可以改变，使代码专业化。

Answer 2

如果你想要tu使用承诺就使用承诺，当编码一致性是黄金的时候。如果您想使用async await（承诺的语法糖），那么在任何地方都要坚持使用async await。

承诺在结尾处使用链式捕获处理错误。

MyPromise().then(() => bar).catch((err) =>  do something with err )

对于async/await，应该将

try { } catch (err) {}

在您的例子中，它只是说在任何包含承诺的异步函数中，您应该用try catch包装它们。

希望有帮助。我最后的建议是，先学一种方法，然后再学另一种方法，不要把它们混在一起。