[代码评审1]代码评审

来源:p2p系统repayCalc.cs,本代码可能参考自线下系统

一 函数语言使用问题

1)Backdate日期类型改为DateTime,则可以不用字符串拼接

2)还款日计算错误(1个月 !=  30天)

3)面向对象编程,而不是都通过函数计算

 

4)构造函数 替代 字符串拼接,字符串拼接和类型转换耗费CPU

5)冗余代码删除,错误情况:第一个还款日晚于借款结束日期,也不可能是按月还款了,应该在UI里面、或者函数开头处理掉,不应该在算法内部

5)类型重构,减少类型转换,删除无用中间变量

同时enddate变量属于重新定义,去掉

 

二 对象定义的简单方式

三 计算错误

对“每期归还本金”进行四舍五入之后,剩余本金发生变化,通过“0.004”这种方式是错误的。

 

移入循环,算法改成:

1)本次归还本金=剩余本金/剩余期数

2)本次归还本金四舍五入

3)修改剩余本金、剩余期数

四 重复代码消除

1.分析

代码主体分成2块,

第一块是实际计息日等于还贷日(5日计息,5日还贷)

第二块是实际计息日不等于还贷日(5日计息,10日还贷,中间不免息。。5日计息,3日还贷,中间不免息)

这两块通过if else控制,内里充斥无数重复代码

比如以下,导致代码数量激增

 

2. 数据错误在开头就应该处理好,这样能尽早给出提示, 减少函数内部执行时间

重复代码提取之后,与外面类似代码合并

3.处理算头也算尾需求,原先代码两个问题:

a if else代码块分别都有计算,重复

b 只处理了实际借款天数,没有处理还款日,算头的情况,还款日需要提前一天

修改计息日为实际计息日即可]

原先代码:

 

修改后代码:

五 计息日!=还贷日时,第一期利息计算错误

六 业务场景和思路问题

1)首先还贷日应该根据放款日自动生成

2)即使是如下两种情况

a)第一块是实际计息日等于还贷日(5日计息,5日还贷)

b)第二块是实际计息日不等于还贷日(5日计息,10日还贷,中间不免息。。5日计息,3日还贷,中间不免息)

在实际实现的时候, if else也应该调换,因为第一块是第二块的简化情况

解决完第一期,修改初始情况,中间期数算法等于第一块,然后剩下最后一期,最后一期和第一期算法相同。

 3)如果遵守第一个情况,那么第二种的复杂情况不会产生。

 删除if语句块

七 借款周期计算问题

1) 天数/每月天数(实际天数,和固定为30天)

2) 根据借款月数转换

 八 重构为策略模式

具体模式介绍:http://www.cnblogs.com/promise-7/archive/2012/05/29/2524357.html

策略又称做政策(Policy)模式【GOF95】。下面是一个示意性的策略模式结构图:

 

这个模式涉及到三个角色:

  • 环境(Context)角色:持有一个Strategy类的引用。
  • 抽象策略(Strategy)角色:这是一个抽象角色,通常由一个接口或抽象类实现。此角色给出所有的具体策略类所需的接口。
  • 具体策略(ConcreteStrategy)角色:包装了相关的算法或行为。

1)目前的结构是这样的

一个类有很多不同的算法函数

 

2)我们需要重构成

一个抽象类,各种算法继承这个抽象类,实现类通过实例化抽象类使用具体的算法

a)定义需要被继承的类

b)定义继承上述类的算法

c)如下代码重构到Calc()方法内

重构后如下,为什么注释掉isInt,isInt的现实意义是付款方便,如果是为了付款方便,那么应该本金和利息相加后取整

调用处这么写就可以了

RepaymentPlanCalc rPlanCalc = new AverageCapital(myRateCalc.Borrowsum, totalqishu, myRateCalc.Borrowrate, time1);
backlist = rPlanCalc.Calc();

原文地址:https://www.cnblogs.com/mspeer/p/5756813.html