【CI/CD构建】关于不小心将springMVC注解写在service层

背景

之前写一个接口的时候没有察觉到将@RequestBody这个注解带到service层了。

今天提交代码的时候,插件没有检测到这个低级错误,导致试飞构建连maven编译都过不了,maven找不到程序包org.springframework.web.bind.annotation这个包

结果

试飞构建不成功

背后原因

如果将@RequestBody写在 Service层会出现的问题

  1. 编译错误:
    Service 层通常是业务逻辑层,它不直接处理 HTTP 请求。@RequestBody是 Spring MVC 用于处理 HTTP 请求的注解,它依赖于 Spring MVC 的请求处理机制。如果在 Service 层使用@RequestBody,会导致编译错误,因为 Service 层没有相应的请求处理上下文来解析这个注解。
  2. 逻辑混乱:
    从设计模式和分层架构的角度来看,这违背了职责分离原则。控制器层应该负责接收和解析 HTTP 请求,将请求数据转换为业务逻辑层(Service 层)可以理解的对象形式,然后将这些对象传递给 Service 层进行业务处理。如果在 Service 层处理请求体的绑定,会使 Service 层与 HTTP 请求细节耦合,导致代码难以维护和测试。例如,在单元测试 Service 层时,不应该涉及 HTTP 请求相关的内容,而@RequestBody的存在会引入这些不相关的因素。
  3. 无法正常工作:
    即使通过一些不合理的方式(如在不适当的上下文中强制加载 Spring MVC 相关的组件)让代码能够编译通过,@RequestBody在 Service 层也无法正确地获取和解析 HTTP 请求体中的数据,因为它不在请求处理的流程中。它缺少 Spring MVC 的DispatcherServlet等组件对请求的分发和处理环境。

结语

希望uu们不会和我一样犯这种低级错误(●´ω`●)ゞ

相关推荐
_运维那些事儿3 小时前
GitLabCI-CD入门
运维·ci/cd·容器·云计算·k8s·运维开发
henujolly5 小时前
How do you troubleshoot a CI failure?
java·开发语言·ci/cd
切糕师学AI19 小时前
什么是灰度发布(Gray Release)?
devops·持续部署·持续集成·灰度发布·release·gray release
慧都小项1 天前
解析CI/CD工具TeamCity的可视化失败追踪功能
ci/cd
游戏开发爱好者81 天前
在 Windows、Linux 与 CI 环境下命令行上传 IPA 到 App Store
linux·windows·ios·ci/cd·小程序·uni-app·iphone
freejackman1 天前
持续集成-Jenkins 基础教程
java·python·ci/cd·自动化·jenkins·持续部署·持续集成
Xxtaoaooo1 天前
React Native 跨平台鸿蒙开发实战:构建 CI/CD 与自动化发布流程
react native·ci/cd·harmonyos
卓码软件测评2 天前
第三方CNAS/CMA软件测试测评机构【LoadRunner的JSON和XML响应数据的关联和处理技巧】
测试工具·ci/cd·性能优化·单元测试·测试用例
jason.zeng@15022072 天前
Task进行CI/CD简单又适用
ci/cd
康康的AI博客2 天前
用Claude Code构建企业级GitHub CI/CD安全自动化流程
安全·ci/cd·github