feat: yidao temp idcard protocol

[Sean-LL]

No description provided.

[szl97]

@claude 检查提交的代码，使用中文回复

[szl97]

Claude finished @szl97's task —— View job

---

代码审查完成

已完成的审查任务

• 读取并分析新增的Java文件
• 检查代码结构和最佳实践
• 提供详细的代码审查反馈

---

📋 代码审查报告

本次提交为易道OCR身份证识别功能添加了3个新的协议文件，整体代码质量良好，符合项目规范。以下是详细的审查结果：

✅ 代码优点

1. 架构设计合理

   • 遵循了项目现有的协议适配器模式
   • YidaoOcrProperty 正确继承了 OcrProperty 基类
   • 实现了必要的接口：IMemoryClearable, ITransfer

2. 代码规范性好

   • 使用了Lombok注解减少样板代码
   • JSON序列化配置正确，使用了 @JsonProperty 和 @JsonIgnore
   • 包结构清晰：com.ke.bella.openapi.protocol.ocr.tmp_idcard

3. 内存管理优秀

   • YidaoOcrTmpIdcardRequest:31-36 实现了大数据清理机制
   • 使用 volatile boolean cleared 确保线程安全
   • 及时清理 imageBase64 避免内存泄漏

🔍 发现的问题和建议

1. 敏感信息安全 (api/sdk/src/main/java/com/ke/bella/openapi/protocol/ocr/tmp_idcard/YidaoOcrTmpIdcardRequest.java:18-19)

@JsonProperty("app_secret")
private String appSecret;

• ⚠️ 安全风险: appSecret 敏感信息未在 clearLargeData() 中清理
• 💡 建议: 在内存清理时也清除敏感信息：

@Override
public void clearLargeData() {
    if(!cleared) {
        this.imageBase64 = null;
        this.appSecret = null;  // 建议添加
        this.cleared = true;
    }
}

2. 代码风格细节 (api/sdk/src/main/java/com/ke/bella/openapi/protocol/ocr/tmp_idcard/YidaoOcrTmpIdcardRequest.java:32)

if(!cleared) {

• 💅 风格建议: 运算符前后添加空格提高可读性：if (!cleared) {

3. 字段验证 (api/sdk/src/main/java/com/ke/bella/openapi/protocol/ocr/tmp_idcard/YidaoOcrTmpIdcardRequest.java:22-25)

• 📝 功能建议: 考虑添加字段验证，确保 imageBase64 和 imageUrl 不能同时为空
• 可以添加 @Valid 注解或自定义验证逻辑

4. 访问修饰符一致性 (api/sdk/src/main/java/com/ke/bella/openapi/protocol/ocr/YidaoOcrProperty.java:16)

AuthorizationProperty auth;

• 🔧 建议: 添加访问修饰符保持一致性：private AuthorizationProperty auth;

📊 代码质量评分

评估项目	评分	说明
架构设计	⭐⭐⭐⭐⭐	完全符合项目架构模式
代码规范	⭐⭐⭐⭐☆	整体规范，个别细节可改善
安全性	⭐⭐⭐☆☆	需要加强敏感信息处理
可维护性	⭐⭐⭐⭐⭐	结构清晰，易于维护
性能考虑	⭐⭐⭐⭐⭐	良好的内存管理机制

总体评价: 这是一个高质量的代码提交，正确实现了易道OCR身份证识别协议。主要需要关注敏感信息的安全处理，其他都符合项目标准。建议在合并前修复敏感信息清理的安全问题。

---