服务层和控制器中的代码分离

Question

我已经为重置密码和应用程序的其他部分编写了代码，我想将服务层与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);
}

Answer 1

MVS只是一种简化，它不是一种完美的方法。这就是为什么它提出了如此多类似于这个问题的原因。

• 控制器应该包含多少逻辑
• 一个模型应该包含任何逻辑吗
• 视图是否应该包含逻辑

像这样的问题很难回答，但控制器需要验证输入并在需要时重定向到视图。您的代码看起来确实很像域逻辑，稍后应该在服务中使用，但它显然更容易在控制器中进行管理。对于领域逻辑(也称为业务逻辑、业务规则和领域知识(，我们可以将其解释为做出业务关键决策的逻辑。

因此，的确，您在Controller中进行决策，但在服务中管理这些决策会使解决方案复杂化。所以，我只想把它留在那里，不管怎样，最终，除了你和你的团队之外，没有人会为这个代码做出贡献，所以如果你对此感到满意，那就好了。

但是，如果要移动代码，我建议使用异常。在我看来，服务应该只返回所需的模式或资源。最好使用异常处理错误。