Browse Source

add code review new filelanzhouhailiang_one-master_merge-Liuxy-1733972680766-lwhT.md

master
Administrator 4 months ago
parent
commit
d9b962807e
  1. 40
      2024-12-12/lanzhouhailiang_one-master_merge-Liuxy-1733972680766-lwhT.md

40
2024-12-12/lanzhouhailiang_one-master_merge-Liuxy-1733972680766-lwhT.md

@ -0,0 +1,40 @@
### 审核报告
#### ProductOutTwoController.java
- **行号 68**:在 `confirmPass` 方法中,使用 `productOutTwoService.confirmPass(whereJson)` 直接调用服务层方法,没有问题。确保 `whereJson` 参数不为空,则无需额外空指针检查。
- **行号 73**:新增 `outPointPass` 方法,该方法同样直接调用服务层方法 `productOutTwoService.outPointPass(whereJson)`,合理。没有发现不必要的空指针判断。
#### ProductOutTwoService.java
- **行号 65**:`confirmPass` 方法无问题,直接调用服务实现层。
- **行号 70**:新增 `outPointPass` 方法,直接调用服务实现层,合理。
#### ProductOutTwoServiceImpl.java
- **行号 150-166**:`confirmPass` 方法实现无问题,逻辑合理。
- **行号 170-190**:`outPointPass` 方法实现正确,使用了 WmsToAcsService 进行操作,并返回结果。
#### InBillQueryController.java
- **行号 39**:`query2` 方法接收参数 `product_area` 为数组,确保 `product_area` 不为 `null` 或空数组。
- **行号 77**:`download2` 方法接收参数 `product_area` 为数组,确保 `product_area` 不为 `null` 或空数组。
#### InBillQueryService.java
- **行号 25**:`queryAll` 方法无问题,直接调用服务实现层。
- **行号 30**:`queryAll2` 方法新增参数 `stor_id`,确保 `stor_id` 不为 `null` 或空数组。
- **行号 58**:`download` 方法无问题,直接调用服务实现层。
- **行号 63**:`download2` 方法新增参数 `stor_id`,确保 `stor_id` 不为 `null` 或空数组。
#### InBillQueryServiceImpl.java
- **行号 105-125**:`queryAll` 方法实现正确,逻辑合理。
- **行号 130-165**:`queryAll2` 方法实现正确,逻辑合理。
- **行号 355-434**:`download` 方法实现正确,逻辑合理。
- **行号 440-470**:`download2` 方法实现正确,逻辑合理。
#### ST_IVT_INBILLQUERY.wql
- **行号 26**:`areas` 和 `stor_id` 参数处理正确,无问题。
- **行号 767-772**:`queryAll` 方法 WQL 逻辑正确。
- **行号 905-910**:`download` 方法 WQL 逻辑正确。
#### index2.vue
- **行号 15**:`el-select` 组件使用正确,`stor_id` 选项处理合理。
### 总结
总体来说,代码改动合理,逻辑清晰,无多余操作。建议在调用服务层方法时,确保入参不为空,以避免潜在的空指针异常。
Loading…
Cancel
Save