Spring Boot中HTTP接口场景下业务逻辑的分层设计疑问

你提的这个问题其实是很多Spring Boot刚入门的开发者都会踩的坑——既混淆了@HttpExchange这类注解的用途，又对分层架构的职责边界有点迷茫，咱们一步步捋清楚。

首先先纠正一个关键误解：你定义的TransactionClient接口用了@HttpExchange，但这个注解是Spring 6+用来定义声明式HTTP客户端的，不是用来写自己服务的端接口的！简单说，这个接口是给你的服务用来调用外部其他HTTP服务用的，比如你的服务需要调用第三方的交易接口，就可以通过这个接口生成代理客户端，而不是用来替代Controller定义自己的API。你现在的Controller用@RestController+@GetMapping这些才是正确定义服务端HTTP接口的方式。

然后回到你最关心的问题：业务逻辑到底该放Controller还是Service层？

先说结论：生产环境下绝对不要把业务逻辑放在Controller里，视频里这么做只是为了快速演示、简化代码而已，完全不是最佳实践。

咱们来明确各层的职责：

• Controller层：只做和HTTP请求响应相关的事——接收请求参数、做参数合法性校验、调用Service层的方法、封装并返回响应。它的职责是“桥梁”，把HTTP请求转成业务调用，再把业务结果转成HTTP响应，不该包含任何业务规则。
• Service层：这才是封装业务逻辑的地方，处理所有和业务规则相关的逻辑，比如你代码里的“保存交易时自动设置当前时间”、“更新交易时保留原交易日期”这些，都属于业务逻辑，必须放在Service层。另外，事务注解@Transactional也更适合放在Service层，因为事务是和业务操作绑定的。
• Repository层：只负责和数据库交互，单表的CRUD操作都在这里，不要在这里加业务逻辑。

结合你的代码，给你一个优化后的分层示例：

1. 新增Service层 #

@Service
public class TransactionService {
    private final TransactionRepository transactionRepository;

    public TransactionService(TransactionRepository transactionRepository) {
        this.transactionRepository = transactionRepository;
    }

    public List<Transaction> findAllTransactions() {
        return transactionRepository.findAll();
    }

    public Transaction findTransactionById(Integer id) {
        return transactionRepository.findById(id)
                .orElseThrow(() -> new RuntimeException("交易记录不存在")); // 这里建议抛自定义异常，而非返回null
    }

    @Transactional
    public ResponseEntity<HttpStatus> saveTransaction(Transaction transaction) {
        // 业务逻辑：自动设置交易时间
        transaction.setDateOfTransaction(new Timestamp(System.currentTimeMillis()));
        transactionRepository.save(transaction);
        return ResponseEntity.ok(HttpStatus.OK);
    }

    @Transactional
    public void updateTransaction(Transaction transaction, Integer id) {
        Transaction existingTransaction = transactionRepository.findById(id)
                .orElseThrow(() -> new RuntimeException("交易记录不存在"));
        // 业务逻辑：保留原交易日期
        transaction.setId(id);
        transaction.setDateOfTransaction(existingTransaction.getDateOfTransaction());
        transactionRepository.save(transaction);
    }

    @Transactional
    public void deleteTransaction(Integer id) {
        transactionRepository.deleteById(id);
    }
}

2. 简化Controller层 #

@RestController
@RequestMapping("/transactions")
public class TransactionController {
    private final TransactionService transactionService;

    public TransactionController(TransactionService transactionService) {
        this.transactionService = transactionService;
    }

    @GetMapping
    public List<Transaction> findAll() {
        return transactionService.findAllTransactions();
    }

    @GetMapping("/{id}")
    public Transaction findOne(@PathVariable("id") Integer id) {
        return transactionService.findTransactionById(id);
    }

    @PostMapping
    public ResponseEntity<HttpStatus> saveTransaction(@RequestBody Transaction transaction) {
        return transactionService.saveTransaction(transaction);
    }

    @PatchMapping("/{id}")
    public void updateTransaction(@RequestBody Transaction transaction, @PathVariable("id") Integer id) {
        transactionService.updateTransaction(transaction, id);
    }

    @DeleteMapping("/{id}")
    public void deleteTransaction(@PathVariable("id") Integer id) {
        transactionService.deleteTransaction(id);
    }
}

3. 正确使用@HttpExchange的例子 #

如果你的服务需要调用外部的交易HTTP接口，比如某个第三方支付平台的交易查询接口，这时候@HttpExchange才派上用场：

// 声明式HTTP客户端，用来调用外部服务
@HttpExchange("https://third-party-pay.com/api/transactions")
public interface ThirdPartyTransactionClient {
    @GetExchange
    List<Transaction> getAllTransactions();

    @GetExchange("/{id}")
    Transaction getTransactionById(@PathVariable("id") Integer id);
}

// 在Service里注入这个客户端，调用外部服务
@Service
public class PaymentService {
    private final ThirdPartyTransactionClient thirdPartyTransactionClient;

    public PaymentService(ThirdPartyTransactionClient thirdPartyTransactionClient) {
        this.thirdPartyTransactionClient = thirdPartyTransactionClient;
    }

    public List<Transaction> getExternalTransactions() {
        return thirdPartyTransactionClient.getAllTransactions();
    }
}

最后再强调一下：分层架构的核心是单一职责原则，这样代码才会易于维护、复用和测试。视频里的简化写法只能用来快速入门，实际项目中一定要严格分层，不然随着业务复杂度提升，Controller会变得臃肿不堪，代码耦合度极高，后期维护起来会非常痛苦。

备注：内容来源于stack exchange，提问作者Jerry598