Files
starRiverProperty/docs/reviews/alibaba-audit-2026-04-24/02-maven-cloudwalk-device-manager.md
hpd840321 7b2bd307f1 Initial commit: reorganized source tree
- backend/: 13 Maven modules (cw-elevator-application, cloudwalk-cloud, intelligent-cwoscomponent, ninca-crk, etc.)
- frontend/: 4 Vue projects (elevator-front, cwos-portal, alarm-front, front_acs) + decompiled + scripts
- scripts/: build, test-env, tools (Docker Compose, service templates, API parity)
- docs/: AGENTS.md, superpowers specs, architecture docs
- .gitignore: standard Java/Maven exclusions

Moved from legacy maven-*/ root layout to backend/ organized structure.
2026-05-09 09:56:45 +08:00

27 lines
1.6 KiB
Markdown
Raw Permalink Blame History

This file contains ambiguous Unicode characters
This file contains Unicode characters that might be confused with other characters. If you think that this is intentional, you can safely ignore this warning. Use the Escape button to reveal them.
# 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(含同类合并描述) |
**说明**:模块以**契约层**为主,异常/日志问题多在实现侧;当前以**命名、魔法值、格式**为主。