mirror of
https://github.com/hpd840321/starRiverProperty.git
synced 2026-06-09 08:20:31 +08:00
bf20a13c96
Made-with: Cursor
1.6 KiB
1.6 KiB
02 maven-cloudwalk-device-manager — 阿里巴巴规范走查(整理版)
范围:maven-cloudwalk-device-manager/**/*.java(排除 target/),约 271 个文件,以 RPC 接口、DTO、枚举、常量为主。
问题清单(节选)
| 严重度 | 位置 | 类别 | 说明 | 建议 |
|---|---|---|---|---|
| P1 | .../constant/DeviceConstant.java |
命名 | OCEAN_DEFALUE_APPLICATIONID 拼写错误 |
更正为 DEFAULT 并全局替换 |
| P1 | 多文件 *DeltParam |
命名 | Delt 与 Delete 语义不符 |
新代码用 Delete,对外 API 需评估兼容 |
| P2 | DeviceConstant.java |
规约 | 常量集中在 interface 中(常量接口反模式) |
改为 final 类 + private 构造 |
| P2 | DeviceConstant.java |
格式 | 大括号与缩进不符合主流 K&R / 4 空格 | 统一格式化 |
| P2 | .../DeviceGroupService.java |
命名 | 方法 gets 语义不清 |
listByCondition 等 |
| P2 | 多文件 | 校验 | @Size(max=999) 与数字型 message 散落 |
抽 ValidationConstants + 码表文档 |
| P2 | 多文件 | 依赖注解 | 广泛使用 org.hibernate.validator.constraints.NotBlank |
优先 javax.validation.constraints.NotBlank |
| P2 | .../em/DeviceUpgradeRevertStatusEnum.java |
卫生 | 若仍含反编译尾部注释需清理 | 已建议用 strip_jdcore_java_noise.py 全仓处理 |
小结
| 级别 | 数量(约) |
|---|---|
| P0 | 0 |
| P1 | 2 |
| P2 | 26(含同类合并描述) |
说明:模块以契约层为主,异常/日志问题多在实现侧;当前以命名、魔法值、格式为主。