优抚优待代码自查

| 浏览次数:

 远程调用问题:

 逻辑问题:

 1. com.ufgov.mz.mz.ppms.treatmapplication.ace.handler.AceHeadTailAfterEditHandler

 作匹配 标准和待遇计算 方法比较长 2. 抚恤优待的资金发放存储过程比较混乱,同时采用了游标和直接插入的方法,并且游标使用的还是循环插入,这样会导致效率低下。应整理后统一用批量插入的方法来执行存储过程。

 3. If()else()分支中,有共同的代码,应当抽出相同的部分放到 if 外面来执行,优化代码。

 4. 嵌套的 if(),存在可以直接“与”判断的情况,不用嵌套也能完成功能。

 Pub 问题:

 1.pub 两个常量类也加注释。

 2.pub 里面不用了的接口删除。

 3.pub 不用了的话删掉,带确定和取消的弹出框到底是继承那个类。

 4.发现不少的有 getByorgs 的方法把,这些不用了的删掉,目前使用动态人员库的找上下级组织的方法。

 Ppms 问题 1.接口注释写上, 2.动态人员库的差异 VO 合并有问题 3.类名规范化。

 4.AceHeadTailAfterEditHandler 类硬编码问题。

 5.代理加上注释。

 6.不要为了一个流程图的名字而各自写一个类。

 7.方法按钮 setName 和 setCode 要学会使用注入的形式, 只是提议。

 8.检查有没不必要的组织切换事件 删掉。

【优抚优待代码自查】相关推荐

工作总结最新推荐

NEW