有没有办法优化下面的代码:
public BigDecimal calculate(Policy policy) {
BigDecimal total = BigDecimal.ZERO.setScale(2, RoundingMode.HALF_EVEN);
for (PolicyObject policyObject : policy.getPolicyObjects()) {
for (PolicySubObject policySubObject : policyObject.getPolicySubObjects()) {
for (Risk risk : policySubObject.getRisks()) {
for (Rate rate : risk.getRates()) {
// CompareTo returns -1 if BigDecimal is smaller then to compared Big decimal
// 0 if equals and 1 if greater.
if (policySubObject.getSumInsured().compareTo(rate.getRangeStart()) >= 0
&& policySubObject.getSumInsured().compareTo(rate.getRangeEnd()) < 0) {
total = total.add(policySubObject.getSumInsured().multiply(rate.getPremiumRate()));
}
}
}
}
}
return total;
}
4 个嵌套循环连同一个条件看起来非常令人不安。最终需要检查每个费率来计算溢价,我找不到更好的方法来做到这一点。
这里似乎没有明显的解决方案,如果这里需要循环,则无法摆脱它们。
本文收集自互联网,转载请注明来源。
如有侵权,请联系 [email protected] 删除。
我来说两句