javascript - 在 Node.js 中将业务逻辑从控制器重构为服务
问题描述
我正在学习 Node.js,但我在代码重构方面遇到了问题。我阅读了 Node.js 中的代码架构和良好的编码实践,我想重构我的代码。
我当前的代码:
用户控制器.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.'
})
});
});
}
我想将所有业务逻辑放入服务中。我试过这样的事情:
用户控制器.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 });
}
}
用户服务.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 的新手。如何解决这个问题?
解决方案
谢谢你的回答。我修改了我的代码,现在看起来像这样:
用户控制器.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 });
}
}
用户服务.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;
}
我的代码正确吗?我说的是编码架构。工作正常,但我不确定我的代码是否是“良好实践”编程。我问是因为我想开始编写整个后端并且我不确定我的代码。
我认为这部分代码看起来不太好:
} 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 语句。这是正确的解决方案吗?我正在寻求建议,可以在这里更改哪些内容以使代码更专业。
推荐阅读
- npm - 如何跳过通过“npm ci”安装可选依赖项?
- android - 如何使用折线的 setTag 在谷歌地图中保存自定义数据
- git - 为什么 git 有时会显示,有时不会显示消息“您的分支在 n 次提交之前领先于 'whatever/upstream'?”
- html - 重叠 div 的动态/响应式解决方案
- swift - 从 WKWebView 生成 PDF 时不需要的右边距
- python - 如何解决 Python 中未找到 Html 模块的错误?
- android - 如何使用房间更改日期格式?
- javascript - 使用 React Typescript CRA 设置 Electron
- angular - 我怎样才能去实现角度源代码?
- haskell - 如何传递未用于调用函数的参数?