# 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（含同类合并描述） |

**说明**：模块以**契约层**为主，异常/日志问题多在实现侧；当前以**命名、魔法值、格式**为主。