2019/9/29 对云端方法检查,指出需要重构的方法
1、云端方法:GetOuRecipeDtlCountByMzRegId,存在魔法数,空串,未视图化。
2、云端方法:GetCustomTreeDto,存在魔法数需要使用枚举,非单表域不要使用:DataRepository.GetQueryable。
3、云端方法:GetUspOuRecipePrint,存在魔法数需要使用枚举,空串,字符拼接。
4、云端需要拆分到公共域和getview。
5、云端方法:GetRptWorkSheetList,空串判断。
6、云端方法:GetUspLoadWorkSheet,空串判断,if else 带花括号。
7、云端方法:OuRecipeDtlPrint.GetSetting,这个类写的不规范,为什么自定义封装类也在这个类文件下?ReturnList,系统参数直接使用。
8、云端方法:GetOuExecuteDtl,入参lstuspOuRecipeForOuexecute现在没有用到了,不需要new bsUnitRatioService,ouExecuteDtlService,直接调用公共服务域的就行了,不需要声明LimitTotalMz,Count == 1。
9、云端方法:GetPatOuExecuteList,魔法数使用枚举。
10、云端方法:DoSaveBalance,魔法数。
11、云端方法:GetReCallPat,入参,参数,命名不规范,不需要new服务,使用公共服务域的,字符串拼接,循环查询。
12、云端方法:入参命名不规范,在循环内查询或者新增数据。
13、云端方法:不要使用db来执行数据语句。
13、云服务方法:AddMergeFeeInfo 使用了New()实现类。
没有使用基础域的方法:
14、云端方法:循环查询 。
14、云端方法:AddSaveMergePartFeeList 硬编码。
15、云端方法:GetPatInInvoiceDtlList 过滤条件。
16、云端方法:CheckInInvoiceDtlDeleteAble所入的值有可能是“ ”、“”、null。
17、云端方法:GetSettlementInfo。
18、云服务:SetInInvoiceDtlYbType 循环访问数据库,和条件判断写错了。