mirror of
https://github.com/hpd840321/starRiverProperty.git
synced 2026-06-09 08:20:31 +08:00
反编译工作区
23 lines
1.7 KiB
Markdown
23 lines
1.7 KiB
Markdown
# 06 `maven-cw-elevator-application` — 阿里巴巴规范走查（整理版）
|
||
|
||
**范围**：`maven-cw-elevator-application/**/*.java`（排除 `target/`）；优先 `*Service*`、`*Impl*`、`*Controller*` 及 **>150 行** 大文件。
|
||
|
||
## 问题清单（节选）
|
||
|
||
| 严重度 | 位置 | 类别 | 说明 | 建议 |
|
||
|--------|------|------|------|------|
|
||
| 阻断 | `PersonRuleServiceImpl.java` | 契约 | `edit` / `timeDetail` / `pageByApp` 等直接 **`return null`** | 返回明确 `CloudwalkResult` 失败体或 `UnsupportedOperationException` |
|
||
| 严重 | `AcsElevatorDeviceGetWayController.java` | 安全 | `getKey` 将配置密钥字段下发 HTTP | 鉴权、短期令牌、禁止明文密钥接口化 |
|
||
| 严重 | `AcsElevatorDeviceGetWayController.java` | 性能 | `queryZoneTreeCode` 多层循环内重复远程/DB 调用 | 批量查询或缓存 |
|
||
| 主要 | 多处 `*ServiceImpl` / `*Controller` | 异常 | 大量 `catch (Exception)` | 收窄并保留 cause |
|
||
| 主要 | `AcsElevatorDeviceServiceImpl.java` | 分层 | `@Repository` 误用于 Service；注入实现类而非接口 | `@Service` + 面向接口 |
|
||
| 主要 | 各 Controller | 校验 | `@RequestBody` 缺 `@Valid` | 补 Bean Validation |
|
||
| 主要 | `AbstractCloudwalkController.java` | NPE | `RequestContextHolder.getRequestAttributes()` 可能 null | 判空 |
|
||
| 次要 | 多处 | 命名 | `getByDeciveCode` 等 **Device** 拼写错误 | 统一修正（评估 API 兼容） |
|
||
| 次要 | `FileController.java` | API | `DatatypeConverter.parseBase64Binary` | `Base64.getDecoder()` |
|
||
| 次要 | 全模块 | 依赖 | Fastjson 使用 | 版本治理或迁移 Jackson |
|
||
|
||
## 小结
|
||
|
||
优先处理 **对外返回 null**、**密钥类接口** 与 **N+1 调用**；其次统一 **异常与分层注解**。
|