java - 服务层和控制器中的代码分离
问题描述
我已经为重置密码和应用程序的其他部分编写了代码,我想将服务层与 dao 和控制器分开;我的控制器代码是:
@RequestMapping(value = "", method = RequestMethod.PUT)
public ResponseModel resetPassword(@Valid @RequestBody AuthenticationRequestModel authenticationRequestModel, HttpServletRequest request) {
String ip = WebUtils.getClientIp(request);
Optional<SecuritySMS> securitySMS = securitySMSService.getLastValidSMS(authenticationRequestModel.getMobile());
if (!securitySMS.isPresent()) {
return new ResponseModel(messages.get("sms.security.expired"), ResponseModel.ResponseStatus.ERROR);
}
SecuritySMS sms = securitySMS.get();
if (!sms.isConfirmed()) {
return new ResponseModel(messages.get("sms.security.expired"), ResponseModel.ResponseStatus.ERROR);
} else if (!sms.getIp().equalsIgnoreCase(ip)) {
return new ResponseModel(messages.get("sms.security.ip.changed"), ResponseModel.ResponseStatus.ERROR);
}
Optional<User> user = userService.findByMobile(sms.getMobile());
if (!user.isPresent()) {
return new ResponseModel(messages.get("sms.reset.user.nonexistent"), ResponseModel.ResponseStatus.ERROR);
}
userService.updatePassword(user.get(), authenticationRequestModel.getPassword());
return authHelper.loginWithHttpResponse(authenticationRequestModel);
}
我对将代码移动到服务层感到困惑;上述方法是正确的还是必须将某些代码(例如下面的代码)移至服务层?如果我将此代码移动到服务层,我怎样才能得到响应?布尔值(不可接受,因为我想向用户返回正确的消息),字符串还是异常?
String ip = WebUtils.getClientIp(request);
Optional<SecuritySMS> securitySMS = securitySMSService.getLastValidSMS(authenticationRequestModel.getMobile());
if (!securitySMS.isPresent()) {
return new ResponseModel(messages.get("sms.security.expired"), ResponseModel.ResponseStatus.ERROR);
}
SecuritySMS sms = securitySMS.get();
if (!sms.isConfirmed()) {
return new ResponseModel(messages.get("sms.security.expired"), ResponseModel.ResponseStatus.ERROR);
} else if (!sms.getIp().equalsIgnoreCase(ip)) {
return new ResponseModel(messages.get("sms.security.ip.changed"), ResponseModel.ResponseStatus.ERROR);
}
Optional<User> user = userService.findByMobile(sms.getMobile());
if (!user.isPresent()) {
return new ResponseModel(messages.get("sms.reset.user.nonexistent"), ResponseModel.ResponseStatus.ERROR);
}
解决方案
MVS 只是一种简化,它不是一个完美的方法。这就是为什么它提出了许多类似的问题。
- 控制器应该输入多少逻辑?
- 模型应该包含任何逻辑吗?
- 视图应该包含逻辑吗?
像这样的问题很难回答,但控制器需要验证输入并在需要时重定向到视图。您的代码确实看起来很像域逻辑,稍后应该在服务中使用,但它显然更容易在您的控制器中管理。对于领域逻辑(也称为业务逻辑、业务规则和领域知识),我们可以将其解释为做出关键业务决策的逻辑。
因此,确实,您是在 Controller 中做出决定,但在服务中管理这些决定会使解决方案复杂化。所以,我会把它留在那里,反正最后,除了你和你的团队之外没有人会为这个代码做出贡献,所以如果你对它感到满意,那很好。
但是,如果您要移动代码,我建议您使用异常。在我看来,服务应该只返回所需的模型或资源。最好使用异常处理错误。
推荐阅读
- angular - 如何使用 ionic 3 中的 Range 更改所有页面的字体大小?
- c# - 如何使用Linq 像在c# 中查询数组一样使用Linq 查询元组?
- windows - Windows:在特定处理器组上启动进程
- arrays - 在 Julia 中创建一个联合类型的空多维数组
- php - 获取每个子站点的首页 ID 有不同的结果
- c# - 通过两阶段反序列化反序列化循环引用
- java - 如何在 Android Studio 的所有页面中保存一个功能(即 SFX 或音乐等设置)?
- python - 如何用许多单独的函数制作密度图?
- java - Android 中快速、低功耗/cpu 字符串匹配
- c++ - 程序在 shell 排序后输出全 0