下面java代码存在哪些问题？

Question

Package com.jxlife;

import java.util.*;
import java.math*;
import com.jxlife.pa.policy;
import com.jxlife.pa.policyproduct;
impost com.jxlife.pa.policyDAO;

public class ICalculatepremium {
public ICalculatepremium( ) {
}
/**
*获取某个保单的保费
*
*@param policyid
*@return
*/
Public bigDecimal calculatepremium(Long policyId,string calculateType) {
BigDecimal a = nu11;
Policy policy = PolicyDAO.load.longvalue( ) );
for (int I =θ; i <= policy.getproducts( ).size( ); i++ ) {
Policyproduct p =policy.getProducts( ).get(i);
a=a.add(p.getPremium( ) );
}
Return a;
}
}

Answer 1

你这是要大家来找茬么~~~
1、Package 第一个字母p为小写；
2、import java.math星; 星前边少了一个"."；
3、Public bigDecimal 第一个字母p为小写，bigDecimal应为 BigDecimal；
4、string calculateType 第一个字母S为大写；
5、PolicyDAO.load.longvalue( ) ); longValue()的V为大写，并且这个load是方法么，是的话括弧在哪，还有最后多了半个括弧啥情况；
6、int I =θ; 这里 i 为小写，还有这个 θ 根本不是数字 0；
7、i <= policy.getproducts( ).size( ); 这边为了性能，计算最好不要放在循环里，可以定义int len = policy.getproducts( ).size( ); 放在外边,
这边用i <= len；
8、Policyproduct p =policy.getProducts( ).get(i); 同7，为了性能考虑，变量p的定义放在循环外，还有policy.getProducts( ) 也放在循环外；
9、Return a; 应为return a；
10、还有一点 ICalculatepremium 类命名规范一般将单字母”I“开头的类定义为接口，虽然不如此也不会报错，但是还是按规范来的好。
11、可能还有其它问题、、、