2019/9/29 对云端方法检查,指出需要重构的方法

1、云端方法:GetOuRecipeDtlCountByMzRegId,存在魔法数,空串,未视图化。

2019/9/29 对云端方法检查,指出需要重构的方法

2、云端方法:GetCustomTreeDto,存在魔法数需要使用枚举,非单表域不要使用:DataRepository.GetQueryable。

2019/9/29 对云端方法检查,指出需要重构的方法

3、云端方法:GetUspOuRecipePrint,存在魔法数需要使用枚举,空串,字符拼接。

2019/9/29 对云端方法检查,指出需要重构的方法

4、云端需要拆分到公共域和getview。

2019/9/29 对云端方法检查,指出需要重构的方法

5、云端方法:GetRptWorkSheetList,空串判断。

2019/9/29 对云端方法检查,指出需要重构的方法

6、云端方法:GetUspLoadWorkSheet,空串判断,if else 带花括号。

2019/9/29 对云端方法检查,指出需要重构的方法

7、云端方法:OuRecipeDtlPrint.GetSetting,这个类写的不规范,为什么自定义封装类也在这个类文件下?ReturnList,系统参数直接使用。

2019/9/29 对云端方法检查,指出需要重构的方法

8、云端方法:GetOuExecuteDtl,入参lstuspOuRecipeForOuexecute现在没有用到了,不需要new bsUnitRatioService,ouExecuteDtlService,直接调用公共服务域的就行了,不需要声明LimitTotalMz,Count == 1。 2019/9/29 对云端方法检查,指出需要重构的方法

9、云端方法:GetPatOuExecuteList,魔法数使用枚举。

2019/9/29 对云端方法检查,指出需要重构的方法

10、云端方法:DoSaveBalance,魔法数。

2019/9/29 对云端方法检查,指出需要重构的方法

2019/9/29 对云端方法检查,指出需要重构的方法

11、云端方法:GetReCallPat,入参,参数,命名不规范,不需要new服务,使用公共服务域的,字符串拼接,循环查询。

2019/9/29 对云端方法检查,指出需要重构的方法

12、云端方法:入参命名不规范,在循环内查询或者新增数据。

2019/9/29 对云端方法检查,指出需要重构的方法

13、云端方法:不要使用db来执行数据语句。

2019/9/29 对云端方法检查,指出需要重构的方法

13、云服务方法:AddMergeFeeInfo 使用了New()实现类。

2019/9/29 对云端方法检查,指出需要重构的方法

2019/9/29 对云端方法检查,指出需要重构的方法

没有使用基础域的方法:

2019/9/29 对云端方法检查,指出需要重构的方法

2019/9/29 对云端方法检查,指出需要重构的方法

14、云端方法:循环查询 。

2019/9/29 对云端方法检查,指出需要重构的方法

14、云端方法:AddSaveMergePartFeeList 硬编码。

2019/9/29 对云端方法检查,指出需要重构的方法

2019/9/29 对云端方法检查,指出需要重构的方法

 

15、云端方法:GetPatInInvoiceDtlList 过滤条件。

2019/9/29 对云端方法检查,指出需要重构的方法

2019/9/29 对云端方法检查,指出需要重构的方法

16、云端方法:CheckInInvoiceDtlDeleteAble所入的值有可能是“ ”、“”、null。

2019/9/29 对云端方法检查,指出需要重构的方法

2019/9/29 对云端方法检查,指出需要重构的方法

17、云端方法:GetSettlementInfo。

2019/9/29 对云端方法检查,指出需要重构的方法

2019/9/29 对云端方法检查,指出需要重构的方法

2019/9/29 对云端方法检查,指出需要重构的方法

18、云服务:SetInInvoiceDtlYbType 循环访问数据库,和条件判断写错了。

2019/9/29 对云端方法检查,指出需要重构的方法

2019/9/29 对云端方法检查,指出需要重构的方法