控制器应调用单个还是多个服务层函数？兼谈登录注册业务逻辑界定

一、明确业务逻辑的定义 #

直白来说，业务逻辑是和你的业务规则强绑定的核心流程——它解决的是“业务应该按什么步骤走”的问题，而非单纯的技术实现细节。

结合你的登录、注册示例：

• 登录的业务逻辑：验证请求合法性（字段齐全、用户存在等）→ 校验密码正确性 → 生成并返回令牌
• 注册的业务逻辑：验证注册信息合法性 → 加密密码 → 创建用户 → 发送验证OTP → 返回提示信息

这些串联起来的流程，都是业务规则要求必须遵循的步骤，属于典型的业务逻辑范畴。

二、控制器中的多服务调用逻辑是否属于业务逻辑？要不要迁移到服务层？ #

结论很明确：这部分串联服务的逻辑完全属于业务逻辑，必须迁移到服务层。

原因分析 #

控制器（Controller）的核心职责应该是做「HTTP门面处理」：接收请求、解析参数、调用服务层、封装HTTP响应（状态码、响应体格式），它不应该参与业务流程的串联。

你当前代码的问题：

1. 登录控制器：把「先校验再生成令牌」的流程写在控制器里，如果后续业务要求登录前新增验证码校验，你不得不修改控制器代码，而非仅调整服务层，违反了单一职责原则。
2. 注册控制器：把「校验→加密密码→创建用户→发OTP」的核心流程串在控制器中，这部分是注册业务的核心规则，属于服务层的职责范围。而且如果其他场景（比如后台系统创建用户）需要复用这套注册流程，控制器里的代码根本无法复用。

优化后的代码示例 #

登录服务层新增业务方法

// 服务层代码
public Map<String, Object> processLogin(String email, String password) {
    Map<String, Object> result = new HashMap<>();
    String validationMsg = validateLogin(email, password);
    if (validationMsg != null) {
        result.put("err", validationMsg);
        return result;
    }
    AbstractMap.SimpleEntry<String, String> tokens = loginUser(email, password);
    if (tokens == null) {
        result.put("err", "Password is incorrect");
        return result;
    }
    result.put("access_token", tokens.getKey());
    result.put("refresh_token", tokens.getValue());
    return result;
}

登录控制器简化

@PostMapping("/login")
public ResponseEntity<Map<String, Object>> login(@RequestBody LoginRequest loginReq){
    Map<String, Object> result = service.processLogin(loginReq.getEmail(), loginReq.getPassword());
    if (result.containsKey("err")) {
        return ResponseEntity.status(409).body(result);
    }
    return ResponseEntity.ok(result);
}

注册服务层新增业务方法

// 服务层代码
public Map<String, Object> processRegister(Person person) {
    Map<String, Object> result = new HashMap<>();
    String validationMsg = validateRegister(person);
    if (validationMsg != null) {
        result.put("err", validationMsg);
        return result;
    }
    person.setPassword(hashPassword(person.getPassword()));
    Person newPerson = createPerson(person);
    emailVerificationSendOTP(newPerson.getEmail());
    result.put("msg", "Enter OTP sent by mail");
    result.put("Person", newPerson);
    return result;
}

注册控制器简化

@PostMapping("/register")
public ResponseEntity<Map<String, Object>> register(@RequestBody Person person) {
    Map<String, Object> result = service.processRegister(person);
    if (result.containsKey("err")) {
        System.out.print(result.get("err"));
        return ResponseEntity.status(409).body(result);
    }
    return ResponseEntity.ok(result);
}

三、分层职责总结 #

• Controller：仅处理HTTP相关操作，比如接收请求参数、调用服务层、根据服务返回结果封装HTTP响应（设置状态码、响应体格式）。
• Service：包揽所有业务逻辑，包括业务流程串联、规则校验、调用DAO操作数据等，是业务规则的核心载体。
• Model：仅负责数据封装，比如LoginRequest、Person等实体类，仅作为数据容器，不包含业务逻辑。

内容的提问来源于stack exchange，提问作者Ganesh Dagadi