eclipse插件FindBugs使用笔记
2015-10-14 17:51
615 查看
FindBugs是一个静态分析工具，它检查类或者JAR 文件，将字节码与一组缺陷模式进行对比以发现可能的问题。Findbugs自带检测器，其中有60余种Bad practice，80余种Correctness，1种 Internationalization，12种Malicious code vulnerability，27种Multithreaded correctness，23种Performance，43种Dodgy。
Bad practice 坏的实践
一些不好的实践，下面列举几个：
HE： 类定义了equals()，却没有hashCode()；或类定义了equals()，却使用
Object.hashCode()；或类定义了hashCode()，却没有equals()；或类定义了hashCode()，却使用Object.equals()；类继承了equals()，却使用Object.hashCode()。
SQL：Statement 的execute方法调用了非常量的字符串；或Prepared Statement是由一个非常量的字符串产生。
DE： 方法终止或不处理异常，一般情况下，异常应该被处理或报告，或被方法抛出。
Correctness 一般的正确性问题
可能导致错误的代码，下面列举几个：
NP： 空指针被引用；在方法的异常路径里，空指针被引用；方法没有检查参数是否null；null值产生并被引用；null值产生并在方法的异常路径被引用；传给方法一个声明为@NonNull的null参数；方法的返回值声明为@NonNull实际是null。
Nm： 类定义了hashcode()方法，但实际上并未覆盖父类Object的hashCode()；类定义了tostring()方法，但实际上并未覆盖父类Object的toString()；很明显的方法和构造器混淆；方法名容易混淆。
SQL：方法尝试访问一个Prepared Statement的0索引；方法尝试访问一个ResultSet的0索引。
UwF：所有的write都把属性置成null，这样所有的读取都是null，这样这个属性是否有必要存在；或属性从没有被write。
Internationalization 国际化
当对字符串使用upper或lowercase方法，如果是国际的字符串，可能会不恰当的转换。
Malicious code vulnerability 可能受到的恶意攻击
如果代码公开，可能受到恶意攻击的代码，下面列举几个：
FI： 一个类的finalize()应该是protected，而不是public的。
MS：属性是可变的数组；属性是可变的Hashtable；属性应该是package protected的。
Multithreaded correctness 多线程的正确性
多线程编程时，可能导致错误的代码，下面列举几个：
ESync：空的同步块，很难被正确使用。
MWN：错误使用notify()，可能导致IllegalMonitorStateException异常；或错误的
使用wait()。
No： 使用notify()而不是notifyAll()，只是唤醒一个线程而不是所有等待的线程。
SC： 构造器调用了Thread.start()，当该类被继承可能会导致错误。
Performance 性能问题
可能导致性能不佳的代码，下面列举几个：
DM：方法调用了低效的Boolean的构造器，而应该用Boolean.valueOf(…)；用类似
Integer.toString(1) 代替new Integer(1).toString()；方法调用了低效的float的构造器，应该用静态的valueOf方法。
SIC：如果一个内部类想在更广泛的地方被引用，它应该声明为static。
SS： 如果一个实例属性不被读取，考虑声明为static。
UrF：如果一个属性从没有被read，考虑从类中去掉。
UuF：如果一个属性从没有被使用，考虑从类中去掉。
Dodgy 危险的
具有潜在危险的代码，可能运行期产生错误，下面列举几个：
CI： 类声明为final但声明了protected的属性。
DLS：对一个本地变量赋值，但却没有读取该本地变量；本地变量赋值成null，却没有读取该本地变量。
ICAST： 整型数字相乘结果转化为长整型数字，应该将整型先转化为长整型数字再相乘。
INT：没必要的整型数字比较，如X <= Integer.MAX_VALUE。
NP： 对readline()的直接引用，而没有判断是否null；对方法调用的直接引用，而方法可能返回null。
REC：直接捕获Exception，而实际上可能是RuntimeException。
ST： 从实例方法里直接修改类变量，即static属性。
以下是我使用FindBugs插件对我们现有项目进行的代码审查和优化：
1、文件路径：com/iamzken/test/DepartmentInfo.java
行数：1245行
修正之前代码片段：
public synchronized boolean equals(java.lang.Object obj) {
if (!(obj instanceof DepartmentInfo)) returnfalse;
DepartmentInfo other = (DepartmentInfo)obj;
if (obj == null) returnfalse;
if (this == obj) returntrue;
if (__equalsCalc != null) {
return (__equalsCalc == obj);
}
修正之后代码片段：
public synchronized booleanequals(java.lang.Object obj) {
if (!(obj instanceof DepartmentInfo)) returnfalse;
DepartmentInfo other = (DepartmentInfo)obj;
//if (obj == null) returnfalse;
if (this == obj) returntrue;
if (__equalsCalc != null) {
return (__equalsCalc == obj);
}
改动原因：把if (obj == null)returnfalse;去掉，因为如果代码走到该处的话，obj肯定不会为空！
2、 文件路径：com/iamzken/test/JobInfo.java
行数：861行
修正之前代码片段：
public synchronized booleanequals(java.lang.Object obj) {
if (!(obj instanceof JobInfo)) returnfalse;
JobInfo other = (JobInfo) obj;
if (obj == null) returnfalse;
if (this == obj) returntrue;
if (__equalsCalc != null) {
return (__equalsCalc == obj);
}
修正之后代码片段：
public synchronized booleanequals(java.lang.Object obj) {
if (!(obj instanceof JobInfo)) returnfalse;
JobInfo other = (JobInfo) obj;
//if (obj == null) returnfalse;
if (this == obj) returntrue;
if (__equalsCalc != null) {
return (__equalsCalc == obj);
}
改动原因：把if (obj == null)returnfalse;这句话去掉，因为代码执行到此处，obj肯定不会为null；
3、 文件路径：com/iamzken/test/PersonInfo.java
行数:957
修正之前代码片段：
public synchronized booleanequals(java.lang.Object obj) {
if (!(obj instanceof PersonInfo)) return false;
PersonInfo other = (PersonInfo) obj;
if (obj == null) return false;
if (this == obj) return true;
if (__equalsCalc != null) {
return (__equalsCalc == obj);
}
修正之后代码片段：
public synchronized boolean equals(java.lang.Object obj) {
//if (!(obj instanceof PersonInfo)) return false;
PersonInfo other = (PersonInfo) obj;
if (obj == null) return false;
if (this == obj) return true;
if (__equalsCalc != null) {
return (__equalsCalc == obj);
}
改动原因：把if (!(obj instanceof PersonInfo)) return false;这句话去掉，因为代码执行到此处，obj肯定不会为null；
4、 文件路径：
org/iamzken/test/AccidentScrapModel.java
行数：117
修正之前代码片段：
public synchronized boolean equals(java.lang.Object obj) {
if (!(obj instanceof AccidentScrapModel)) return false;
AccidentScrapModel other =(AccidentScrapModel) obj;
if (obj == null) return false;
if (this == obj) return true;
if (__equalsCalc != null) {
return (__equalsCalc == obj);
}
修正之后代码片段：
public synchronized boolean equals(java.lang.Object obj) {
if (!(obj instanceof AccidentScrapModel)) return false;
AccidentScrapModel other =(AccidentScrapModel) obj;
// if (obj == null) return false;
if (this == obj) return true;
if (__equalsCalc != null) {
return (__equalsCalc == obj);
}
改动原因：把if (obj == null)return false;这句代码去掉，因为代码执行到此处，obj肯定不会为空。
5、 文件路径：
org/iamzken/test/ApprovalRecordModel.java
行数：165
修正之前代码片段：
public synchronized booleanequals(java.lang.Object obj) {
if (!(obj instanceof ApprovalRecordModel)) return false;
ApprovalRecordModel other =(ApprovalRecordModel) obj;
if (obj == null) return false;
if (this == obj) return true;
if (__equalsCalc != null) {
return (__equalsCalc == obj);
}
修正之后代码片段：
public synchronized booleanequals(java.lang.Object obj) {
if (!(obj instanceof ApprovalRecordModel)) return false;
ApprovalRecordModel other =(ApprovalRecordModel) obj;
//if (obj == null) return false;
if (this == obj) return true;
if (__equalsCalc != null) {
return (__equalsCalc == obj);
}
改动原因：
把if (obj == null)return false;这句话去掉，因为代码执行到此处，obj肯定不会为空
6、 文件路径：
org/iamzken/test/CompositeType.java
行数：69
修正之前代码片段：
public synchronized boolean equals(java.lang.Object obj) {
if (!(obj instanceof CompositeType)) return false;
CompositeType other = (CompositeType)obj;
if (obj == null) return false;
if (this == obj) return true;
if (__equalsCalc != null) {
return (__equalsCalc == obj);
}
修正之后代码片段：
public synchronized boolean equals(java.lang.Object obj) {
if (!(obj instanceof CompositeType)) return false;
CompositeType other = (CompositeType)obj;
//if (obj == null) return false;
if (this == obj) return true;
if (__equalsCalc != null) {
return (__equalsCalc == obj);
}
改动原因：
把if (obj == null)return false;这句话去掉，因为代码执行到此处，obj肯定不会为空。
7、 文件路径：
org/iamzken/test/WFProcessStatusModel.java
行数：117
修正之前代码：
public synchronized boolean equals(java.lang.Object obj) {
if (!(obj instanceof WFProcessStatusModel)) return false;
WFProcessStatusModel other =(WFProcessStatusModel) obj;
if (obj == null) return false;
if (this == obj) return true;
if (__equalsCalc != null) {
return (__equalsCalc == obj);
}
修正之后代码：
public synchronized boolean equals(java.lang.Object obj) {
if (!(obj instanceof WFProcessStatusModel)) return false;
WFProcessStatusModel other =(WFProcessStatusModel) obj;
//if (obj == null) return false;
if (this == obj) return true;
if (__equalsCalc != null) {
return (__equalsCalc == obj);
}
改动原因：
把if (obj == null)return false;这句代码去掉，因为代码执行到此处，obj肯定不会为空。
8、 文件路径：
com/iamzken/google/TradeCalcController.java
行数：216
修正之前代码片段：
if(! "4".equals(recountLog.getTypeId())){
processService.processData();
} }
修正之后代码片段：
if(! (4==recountLog.getTypeId())){
processService.processData();
}
改动原因：
recountLog.getTypeId()返回的值是Integer，可以直接通过==进行比较，没必要把4包装成字符串进行equals比较，因为需要对字符串进行解析，降低效率
9、 文件路径：
com/iamzken/google/PaymentSetController.java
行数：307
改动之前代码片段：
if("".equals(tranCycle)||tranCycle==null||tranCycle==""){
flag="1";
改动之后代码片段：
if("".equals(tranCycle)||tranCycle==null){
flag="1";
改动原因：
去掉||tranCycle==""
10、 文件路径：
com/iamzken/google/PartnersWebServiceImpl.java
行数：353
改动之前代码片段：
if(partnersmodel.length<0){
errMsg ="账单信息为空！";
}
改动之后代码片段：
if(partnersmodel.length<=0){
errMsg ="账单信息为空！";
}
改动原因：<改为<=
11、 文件路径：
com/iamzken/google/PartnersWebServiceImpl.java
行数：226
改动之前代码片段：
if(partnersmodel.length<0){
errMsg =errMsg+"账单信息为空！";
}
改动之后代码片段：
if(partnersmodel.length<=0){
errMsg =errMsg+"账单信息为空！";
}
改动原因：<改为<=
12、文件路径：
com/iamzken/google/PaymentSetController.java
行数：163
修改之前代码片段：
if(!"".equals(id)&& id != null && id != ""){
this.paymentSetService.insertPaymentSetItem(paymentset);
jm.setStatus(JsonModel.STATUS_OK);
jm.setMsg("修改成功！");
修改之后代码片段：
if(!"".equals(id) && id != null){
this.paymentSetService.insertPaymentSetItem(paymentset);
jm.setStatus(JsonModel.STATUS_OK);
jm.setMsg("修改成功！");
改动原因：去掉&& id != ""
13、文件路径：
com/iamzken/google/PaymentSetController.java
行数：176
修改之前代码片段：
if(!"".equals(id)&& id != null && id != ""){
jm.setMsg("提交失败！");
修改之后代码片段：
if(!"".equals(id) && id != null){
jm.setMsg("提交失败！");
改动原因：去掉&& id != ""
14、文件路径：com/iamzken/MenuDataService.java
行数：362
修改之前代码片段：
}finally{
try {
fw.close();
} catch (IOExceptione) {
e.printStackTrace();
}
}
修改之后代码片段：
}finally{
try {
if(fw != null){
fw.close();
}
} catch (IOException e) {
e.printStackTrace();
}
改动原因：fw关闭之前加上判空代码，否则有可能出现空指针异常。
15、文件路径：com/iamzken/test/LoginController.java
行数：101-102
修改之前代码片段：
mv.addObject("dfUrl",cmd.getDefaultUrl());
mv.addObject("userName",cmd.getUserName());
修改之后代码片段：
if(cmd != null){
mv.addObject("dfUrl", cmd.getDefaultUrl());
mv.addObject("userName", cmd.getUserName());
}
改动原因：使用cmd对象之前先要进行非空判断
16、 文件路径：
com/iamzken/google/AgentInfo.java
行数：110-114
修改之前代码片段：
public boolean equals(AgentInfo agentInfo){
if(agentName.equals(agentInfo.getAgentName()) )
{
return true;
}
修改之后代码片段：
public boolean equals(AgentInfo obj){
if(obj instanceof AgentInfo){
AgentInfo agentInfo= (AgentInfo)obj;
if(agentName.equals(agentInfo.getAgentName()))
{
returntrue;
}
}
并且要重写hashCode方法。
@Override
public int hashCode(){
return this.getAgentName().hashCode();
}
改动原因：这是equals方法的标准写法。
17、文件路径：
com/iamzken/google/MerchantsTransfer.java
行数：88-98
修改之前代码片段：
public boolean equals(MerchantsTransfer merchantsTransfer){
if(this.merchantName.equals(merchantsTransfer.getMerchantName())
&&this.formerAgentName.equals(merchantsTransfer.getFormerAgentName())
&& this.idcontent.equals(merchantsTransfer.getIdcontent())
&& this.acceptAgentName.equals(merchantsTransfer.getAcceptAgentName())
)
{
return true;
}
return false;
}
修改之后代码片段：
public booleanequals(Object obj){
if(obj instanceofMerchantsTransfer){
MerchantsTransfermerchantsTransfer = (MerchantsTransfer)obj;
if(this.merchantName.equals(merchantsTransfer.getMerchantName())
&&this.formerAgentName.equals(merchantsTransfer.getFormerAgentName())
&&this.idcontent.equals(merchantsTransfer.getIdcontent())
&&this.acceptAgentName.equals(merchantsTransfer.getAcceptAgentName())
)
{
returntrue;
}
}
return false;
}
@Override
public int hashCode() {
returnthis.getMerchantName().hashCode()+
this.getFormerAgentName().hashCode()+
this.getIdcontent().hashCode()+
this.getAcceptAgentName().hashCode();
}
改动原因：equals方法写的不是标准写法，并且只有equals方法没有hashCode方法
18、文件路径：
com/iamzken/google/SpecialMerchantInfo.java
行数：55-64
修改之前代码片段：
public boolean equals(SpecialMerchantInfo merchantInfo){
if(specialMerchantName.equals(merchantInfo.getSpecialMerchantName())
&& membercode.equals(merchantInfo.getMembercode())
&& agentName.equals(merchantInfo.getAgentName())
)
{
return true;
}
return false;
}
修改之后代码片段：
public boolean equals(Object obj){
if(obj instanceof SpecialMerchantInfo){
SpecialMerchantInfo merchantInfo =(SpecialMerchantInfo)obj;
if(specialMerchantName.equals(merchantInfo.getSpecialMerchantName())
&& membercode.equals(merchantInfo.getMembercode())
&& agentName.equals(merchantInfo.getAgentName())
)
{
return true;
}
}
return false;
}
@Override
public int hashCode() {
return this.getSpecialMerchantName().hashCode()+
this.getMembercode().hashCode()+
this.getAgentName().hashCode();
}
改动原因：
equals方法不是标准写法，并且没有hashCode方法。
19、文件路径：
com/iamzken/google/SpringContextsUtil.java
行数：整个文件
修改之前的代码：
package com.iamzken.google;
import org.springframework.beans.BeansException;
import org.springframework.context.ApplicationContext;
import org.springframework.context.ApplicationContextAware;
public class SpringContextsUtilimplements ApplicationContextAware {
private static ApplicationContext applicationContext;
public static Object getBean(StringbeanName) {
return applicationContext.getBean(beanName);
}
public static <T> T getBean(StringbeanName, Class<T> clazs) {
return clazs.cast(getBean(beanName));
}
public void setApplicationContext(ApplicationContextapplicationContext)
throws BeansException {
SpringContextsUtil.applicationContext = applicationContext;
}
}
修改之后的代码：
package com.iamzken.google;
import org.springframework.beans.BeansException;
import org.springframework.context.ApplicationContext;
import org.springframework.context.ApplicationContextAware;
public class SpringContextsUtil implements ApplicationContextAware {
private ApplicationContext applicationContext;
public Object getBean(String beanName) {
return applicationContext.getBean(beanName);
}
public <T> T getBean(String beanName,Class<T> clazs) {
returnclazs.cast(getBean(beanName));
}
public voidsetApplicationContext(ApplicationContext applicationContext)
throws BeansException {
this.applicationContext =applicationContext;
}
}
改动原因：使用非静态方法给静态成员变量直接赋值容易出现多线程安全问题，去掉static，成为实例成员方法和成员变量，或者改为单例模式！
20、 文件路径：
com/iamzken/google/DateUtil.java
行数：179
修改之前代码片段：
public static boolean compareDate(String date1, String date2) {
SimpleDateFormat df = new SimpleDateFormat("yyyy/MM/dd");
boolean flag = false;
try {
date1 = date1.replace(".", "/");
date2 = date2.replace(".", "/");
date1 = date1.replace("-", "/");
date2= date2.replace("-", "/");
修改之后代码片段：
public static boolean compareDate(String date1, String date2) {
boolean flag = false;
try {
date1 = date1.replace(".", "/");
date2 = date2.replace(".", "/");
date1 = date1.replace("-", "/");
date2 = date2.replace("-", "/");
改动原因：去掉无用的SimpleDateFormatdf = new SimpleDateFormat("yyyy/MM/dd");这句话
21、文件路径：
行数：281
修改之前代码片段：
public ModelAndView viewDetail(HttpServletRequest request,
HttpServletResponse response){
ModelAndView m = new ModelAndView(this.detailView);
Map<String, String> map = new HashMap<String,String>();
Long logid = Long.valueOf(request.getParameter("logid" ));
修改之后代码片段：
public ModelAndView viewDetail(HttpServletRequest request,
HttpServletResponse response){
ModelAndView m = new ModelAndView(this.detailView);
Long logid = Long.valueOf(request.getParameter("logid" ));
改动原因：
去掉无用的 Map<String, String>map= new HashMap<String, String>();
这句话
22、文件路径：
com/iamzken/google/TradeZtcDetailService.java
行数：38
修改之前代码片段：
public Map addBatchTradeZtcDetail(List<String[]>list,String updateName) {
// 提示信息Map
Map tips = new HashMap();
// 验证错误信息的集合
List<String> errorList = new ArrayList<String>();
Map<String,SpecialMerchantInfo> map = new HashMap<String,SpecialMerchantInfo>();
// 代理商信息
List<AgentInfo>tradeZtcList = new ArrayList<AgentInfo>();
修改之后代码片段：
public Map addBatchTradeZtcDetail(List<String[]>list,String updateName) {
// 提示信息Map
Map tips = new HashMap();
// 验证错误信息的集合
List<String> errorList = new ArrayList<String>();
// 代理商信息
List<AgentInfo>tradeZtcList = new ArrayList<AgentInfo>();
改动原因：去掉无用的Map<String,SpecialMerchantInfo> map = newHashMap<String,SpecialMerchantInfo>();这句话
23、文件路径：
com/iamzken/google/ExportAgentInfo.java
行数：47
修改之前代码片段：
public void exportExcel(String title, String[] headers,List<T> dataList,
HttpServletResponse response, String pattern) throws IOException {
SimpleDateFormat sdf = new SimpleDateFormat(pattern);
OutputStream out = response.getOutputStream();
// 声明一个工作薄
SXSSFWorkbook workbook = newSXSSFWorkbook(5000);
修改之后代码片段：
public void exportExcel(String title, String[] headers,List<T> dataList,
HttpServletResponse response, String pattern) throws IOException {
OutputStream out = response.getOutputStream();
// 声明一个工作薄
SXSSFWorkbook workbook = newSXSSFWorkbook(5000);
改动原因：
去掉无用的SimpleDateFormat sdf = new SimpleDateFormat(pattern);这句话
24、文件路径：
com/iamzken/google/PartnersWebServiceImpl.java
行数：337
修改之前代码片段：
public List<Map<String,String>>invoiceSolution(String requestNo, String billNo,
String status,PartnersModel[] partnersmodel) {
String errMsg ="";
String [] result=new String[8];
Map<String, Object> map = new HashMap<String, Object>();
修改之后代码片段：
public List<Map<String,String>>invoiceSolution(String requestNo, String billNo,
String status,PartnersModel[] partnersmodel) {
String errMsg ="";
Map<String,Object> map = new HashMap<String,Object>();
改动原因：去掉无用的String [] result=new String[8];这句话
25、文件路径：
com/iamzken/google/ExportCommission.java
行数：49
修改之前代码片段：
public void exportExcel(String title, String[] headers,List<T> dataList,
HttpServletResponse response, String pattern) throws IOException {
SimpleDateFormat sdf = new SimpleDateFormat(pattern);
OutputStream out =response.getOutputStream();
修改之后代码片段：
public void exportExcel(String title, String[] headers,List<T> dataList,
HttpServletResponse response, String pattern) throws IOException {
OutputStream out =response.getOutputStream();
改动原因：去掉无用的SimpleDateFormat sdf = new SimpleDateFormat(pattern);这句话
26、文件路径：
com/iamzken/google/ExportMerchantInfo.java
行数：48
修改之前代码片段：
public void exportExcel(String title, String[] headers,List<SpecialMerchantInfo> dataList,
HttpServletResponse response, String pattern) throws IOException {
SimpleDateFormat sdf = new SimpleDateFormat(pattern);
OutputStream out = response.getOutputStream();
修改之后代码片段：
public void exportExcel(String title, String[] headers,List<SpecialMerchantInfo> dataList,
HttpServletResponse response, String pattern) throws IOException {
OutputStream out =response.getOutputStream();
改动原因：去掉无用的SimpleDateFormat sdf = new SimpleDateFormat(pattern);这句话
27、文件路径：
com/iamzken/google/ExportMerchantRelate.java
行数：48
修改之前代码片段：
public void exportExcel(String title, String[] headers, List dataList,
HttpServletResponse response, String pattern) throws IOException {
SimpleDateFormat sdf = new SimpleDateFormat(pattern);
OutputStream out =response.getOutputStream();
修改之后代码片段：
public void exportExcel(String title, String[] headers, List dataList,
HttpServletResponse response, String pattern) throws IOException {
OutputStream out =response.getOutputStream();
改动原因：去掉无用的SimpleDateFormat sdf = new SimpleDateFormat(pattern);这句话
28、文件路径：
com/iamzken/google/ExportPublicity.java
行数：51
修改之前代码片段：
public void exportExcel(String title, String[] headers,
List<PreAuditItem> dataList, HttpServletResponseresponse,
String pattern) throws IOException {
SimpleDateFormat sdf = new SimpleDateFormat(pattern);
OutputStream out = response.getOutputStream();
修改之后代码片段：
public void exportExcel(String title, String[] headers,
List<PreAuditItem> dataList, HttpServletResponseresponse,
String pattern) throws IOException {
OutputStream out =response.getOutputStream();
改动原因：去掉无用的SimpleDateFormat sdf = new SimpleDateFormat(pattern);这句话
29、文件路径：
com/iamzken/google/ExportTransDetail.java
行数：48
修改之前代码片段：
public void exportExcel(String title, String[] headers, List dataList,
HttpServletResponse response, String pattern) throws IOException {
SimpleDateFormat sdf = new SimpleDateFormat(pattern);
OutputStream out =response.getOutputStream();
修改之后代码片段：
public void exportExcel(String title, String[] headers, List dataList,
HttpServletResponse response, String pattern) throws IOException {
OutputStream out =response.getOutputStream();
改动原因：去掉无用的SimpleDateFormat sdf = new SimpleDateFormat(pattern);这句话
30、文件路径：
com/iamzken/google/ExportVposDetail.java
行数：48
修改之前代码片段：
public void exportExcel(String title, String[] headers, List dataList,
HttpServletResponse response, String pattern) throws IOException {
SimpleDateFormat sdf = new SimpleDateFormat(pattern);
OutputStream out =response.getOutputStream();
修改之后代码片段：
public void exportExcel(String title, String[] headers, List dataList,
HttpServletResponse response, String pattern) throws IOException {
OutputStream out =response.getOutputStream();
改动原因：去掉无用的SimpleDateFormat sdf = new SimpleDateFormat(pattern);这句话
31、文件路径：
com/iamzken/google/ExcelReader.java
行数：157
修改之前代码片段：
public String[]getRowData(int sheetIndex,int rowIndex){
String[] dataArray = null;
if(rowIndex>this.getColumnNum(sheetIndex)){
return dataArray;
}else{
dataArray = new String[this.getColumnNum(sheetIndex)];
return this.dataList.get(rowIndex);
}
修改之后代码片段：
public String[]getRowData(int sheetIndex,int rowIndex){
String[] dataArray = null;
if(rowIndex>this.getColumnNum(sheetIndex)){
return dataArray;
}else{
return this.dataList.get(rowIndex);
}
改动原因：去掉无用的dataArray = newString[this.getColumnNum(sheetIndex)]; 这句话
Bad practice 坏的实践
一些不好的实践，下面列举几个：
HE： 类定义了equals()，却没有hashCode()；或类定义了equals()，却使用
Object.hashCode()；或类定义了hashCode()，却没有equals()；或类定义了hashCode()，却使用Object.equals()；类继承了equals()，却使用Object.hashCode()。
SQL：Statement 的execute方法调用了非常量的字符串；或Prepared Statement是由一个非常量的字符串产生。
DE： 方法终止或不处理异常，一般情况下，异常应该被处理或报告，或被方法抛出。
Correctness 一般的正确性问题
可能导致错误的代码，下面列举几个：
NP： 空指针被引用；在方法的异常路径里，空指针被引用；方法没有检查参数是否null；null值产生并被引用；null值产生并在方法的异常路径被引用；传给方法一个声明为@NonNull的null参数；方法的返回值声明为@NonNull实际是null。
Nm： 类定义了hashcode()方法，但实际上并未覆盖父类Object的hashCode()；类定义了tostring()方法，但实际上并未覆盖父类Object的toString()；很明显的方法和构造器混淆；方法名容易混淆。
SQL：方法尝试访问一个Prepared Statement的0索引；方法尝试访问一个ResultSet的0索引。
UwF：所有的write都把属性置成null，这样所有的读取都是null，这样这个属性是否有必要存在；或属性从没有被write。
Internationalization 国际化
当对字符串使用upper或lowercase方法，如果是国际的字符串，可能会不恰当的转换。
Malicious code vulnerability 可能受到的恶意攻击
如果代码公开，可能受到恶意攻击的代码，下面列举几个：
FI： 一个类的finalize()应该是protected，而不是public的。
MS：属性是可变的数组；属性是可变的Hashtable；属性应该是package protected的。
Multithreaded correctness 多线程的正确性
多线程编程时，可能导致错误的代码，下面列举几个：
ESync：空的同步块，很难被正确使用。
MWN：错误使用notify()，可能导致IllegalMonitorStateException异常；或错误的
使用wait()。
No： 使用notify()而不是notifyAll()，只是唤醒一个线程而不是所有等待的线程。
SC： 构造器调用了Thread.start()，当该类被继承可能会导致错误。
Performance 性能问题
可能导致性能不佳的代码，下面列举几个：
DM：方法调用了低效的Boolean的构造器，而应该用Boolean.valueOf(…)；用类似
Integer.toString(1) 代替new Integer(1).toString()；方法调用了低效的float的构造器，应该用静态的valueOf方法。
SIC：如果一个内部类想在更广泛的地方被引用，它应该声明为static。
SS： 如果一个实例属性不被读取，考虑声明为static。
UrF：如果一个属性从没有被read，考虑从类中去掉。
UuF：如果一个属性从没有被使用，考虑从类中去掉。
Dodgy 危险的
具有潜在危险的代码，可能运行期产生错误，下面列举几个：
CI： 类声明为final但声明了protected的属性。
DLS：对一个本地变量赋值，但却没有读取该本地变量；本地变量赋值成null，却没有读取该本地变量。
ICAST： 整型数字相乘结果转化为长整型数字，应该将整型先转化为长整型数字再相乘。
INT：没必要的整型数字比较，如X <= Integer.MAX_VALUE。
NP： 对readline()的直接引用，而没有判断是否null；对方法调用的直接引用，而方法可能返回null。
REC：直接捕获Exception，而实际上可能是RuntimeException。
ST： 从实例方法里直接修改类变量，即static属性。
以下是我使用FindBugs插件对我们现有项目进行的代码审查和优化：
1、文件路径：com/iamzken/test/DepartmentInfo.java
行数：1245行
修正之前代码片段：
public synchronized boolean equals(java.lang.Object obj) {
if (!(obj instanceof DepartmentInfo)) returnfalse;
DepartmentInfo other = (DepartmentInfo)obj;
if (obj == null) returnfalse;
if (this == obj) returntrue;
if (__equalsCalc != null) {
return (__equalsCalc == obj);
}
修正之后代码片段：
public synchronized booleanequals(java.lang.Object obj) {
if (!(obj instanceof DepartmentInfo)) returnfalse;
DepartmentInfo other = (DepartmentInfo)obj;
//if (obj == null) returnfalse;
if (this == obj) returntrue;
if (__equalsCalc != null) {
return (__equalsCalc == obj);
}
改动原因：把if (obj == null)returnfalse;去掉，因为如果代码走到该处的话，obj肯定不会为空！
2、 文件路径：com/iamzken/test/JobInfo.java
行数：861行
修正之前代码片段：
public synchronized booleanequals(java.lang.Object obj) {
if (!(obj instanceof JobInfo)) returnfalse;
JobInfo other = (JobInfo) obj;
if (obj == null) returnfalse;
if (this == obj) returntrue;
if (__equalsCalc != null) {
return (__equalsCalc == obj);
}
修正之后代码片段：
public synchronized booleanequals(java.lang.Object obj) {
if (!(obj instanceof JobInfo)) returnfalse;
JobInfo other = (JobInfo) obj;
//if (obj == null) returnfalse;
if (this == obj) returntrue;
if (__equalsCalc != null) {
return (__equalsCalc == obj);
}
改动原因：把if (obj == null)returnfalse;这句话去掉，因为代码执行到此处，obj肯定不会为null；
3、 文件路径：com/iamzken/test/PersonInfo.java
行数:957
修正之前代码片段：
public synchronized booleanequals(java.lang.Object obj) {
if (!(obj instanceof PersonInfo)) return false;
PersonInfo other = (PersonInfo) obj;
if (obj == null) return false;
if (this == obj) return true;
if (__equalsCalc != null) {
return (__equalsCalc == obj);
}
修正之后代码片段：
public synchronized boolean equals(java.lang.Object obj) {
//if (!(obj instanceof PersonInfo)) return false;
PersonInfo other = (PersonInfo) obj;
if (obj == null) return false;
if (this == obj) return true;
if (__equalsCalc != null) {
return (__equalsCalc == obj);
}
改动原因：把if (!(obj instanceof PersonInfo)) return false;这句话去掉，因为代码执行到此处，obj肯定不会为null；
4、 文件路径：
org/iamzken/test/AccidentScrapModel.java
行数：117
修正之前代码片段：
public synchronized boolean equals(java.lang.Object obj) {
if (!(obj instanceof AccidentScrapModel)) return false;
AccidentScrapModel other =(AccidentScrapModel) obj;
if (obj == null) return false;
if (this == obj) return true;
if (__equalsCalc != null) {
return (__equalsCalc == obj);
}
修正之后代码片段：
public synchronized boolean equals(java.lang.Object obj) {
if (!(obj instanceof AccidentScrapModel)) return false;
AccidentScrapModel other =(AccidentScrapModel) obj;
// if (obj == null) return false;
if (this == obj) return true;
if (__equalsCalc != null) {
return (__equalsCalc == obj);
}
改动原因：把if (obj == null)return false;这句代码去掉，因为代码执行到此处，obj肯定不会为空。
5、 文件路径：
org/iamzken/test/ApprovalRecordModel.java
行数：165
修正之前代码片段：
public synchronized booleanequals(java.lang.Object obj) {
if (!(obj instanceof ApprovalRecordModel)) return false;
ApprovalRecordModel other =(ApprovalRecordModel) obj;
if (obj == null) return false;
if (this == obj) return true;
if (__equalsCalc != null) {
return (__equalsCalc == obj);
}
修正之后代码片段：
public synchronized booleanequals(java.lang.Object obj) {
if (!(obj instanceof ApprovalRecordModel)) return false;
ApprovalRecordModel other =(ApprovalRecordModel) obj;
//if (obj == null) return false;
if (this == obj) return true;
if (__equalsCalc != null) {
return (__equalsCalc == obj);
}
改动原因：
把if (obj == null)return false;这句话去掉，因为代码执行到此处，obj肯定不会为空
6、 文件路径：
org/iamzken/test/CompositeType.java
行数：69
修正之前代码片段：
public synchronized boolean equals(java.lang.Object obj) {
if (!(obj instanceof CompositeType)) return false;
CompositeType other = (CompositeType)obj;
if (obj == null) return false;
if (this == obj) return true;
if (__equalsCalc != null) {
return (__equalsCalc == obj);
}
修正之后代码片段：
public synchronized boolean equals(java.lang.Object obj) {
if (!(obj instanceof CompositeType)) return false;
CompositeType other = (CompositeType)obj;
//if (obj == null) return false;
if (this == obj) return true;
if (__equalsCalc != null) {
return (__equalsCalc == obj);
}
改动原因：
把if (obj == null)return false;这句话去掉，因为代码执行到此处，obj肯定不会为空。
7、 文件路径：
org/iamzken/test/WFProcessStatusModel.java
行数：117
修正之前代码：
public synchronized boolean equals(java.lang.Object obj) {
if (!(obj instanceof WFProcessStatusModel)) return false;
WFProcessStatusModel other =(WFProcessStatusModel) obj;
if (obj == null) return false;
if (this == obj) return true;
if (__equalsCalc != null) {
return (__equalsCalc == obj);
}
修正之后代码：
public synchronized boolean equals(java.lang.Object obj) {
if (!(obj instanceof WFProcessStatusModel)) return false;
WFProcessStatusModel other =(WFProcessStatusModel) obj;
//if (obj == null) return false;
if (this == obj) return true;
if (__equalsCalc != null) {
return (__equalsCalc == obj);
}
改动原因：
把if (obj == null)return false;这句代码去掉，因为代码执行到此处，obj肯定不会为空。
8、 文件路径：
com/iamzken/google/TradeCalcController.java
行数：216
修正之前代码片段：
if(! "4".equals(recountLog.getTypeId())){
processService.processData();
} }
修正之后代码片段：
if(! (4==recountLog.getTypeId())){
processService.processData();
}
改动原因：
recountLog.getTypeId()返回的值是Integer，可以直接通过==进行比较，没必要把4包装成字符串进行equals比较，因为需要对字符串进行解析，降低效率
9、 文件路径：
com/iamzken/google/PaymentSetController.java
行数：307
改动之前代码片段：
if("".equals(tranCycle)||tranCycle==null||tranCycle==""){
flag="1";
改动之后代码片段：
if("".equals(tranCycle)||tranCycle==null){
flag="1";
改动原因：
去掉||tranCycle==""
10、 文件路径：
com/iamzken/google/PartnersWebServiceImpl.java
行数：353
改动之前代码片段：
if(partnersmodel.length<0){
errMsg ="账单信息为空！";
}
改动之后代码片段：
if(partnersmodel.length<=0){
errMsg ="账单信息为空！";
}
改动原因：<改为<=
11、 文件路径：
com/iamzken/google/PartnersWebServiceImpl.java
行数：226
改动之前代码片段：
if(partnersmodel.length<0){
errMsg =errMsg+"账单信息为空！";
}
改动之后代码片段：
if(partnersmodel.length<=0){
errMsg =errMsg+"账单信息为空！";
}
改动原因：<改为<=
12、文件路径：
com/iamzken/google/PaymentSetController.java
行数：163
修改之前代码片段：
if(!"".equals(id)&& id != null && id != ""){
this.paymentSetService.insertPaymentSetItem(paymentset);
jm.setStatus(JsonModel.STATUS_OK);
jm.setMsg("修改成功！");
修改之后代码片段：
if(!"".equals(id) && id != null){
this.paymentSetService.insertPaymentSetItem(paymentset);
jm.setStatus(JsonModel.STATUS_OK);
jm.setMsg("修改成功！");
改动原因：去掉&& id != ""
13、文件路径：
com/iamzken/google/PaymentSetController.java
行数：176
修改之前代码片段：
if(!"".equals(id)&& id != null && id != ""){
jm.setMsg("提交失败！");
修改之后代码片段：
if(!"".equals(id) && id != null){
jm.setMsg("提交失败！");
改动原因：去掉&& id != ""
14、文件路径：com/iamzken/MenuDataService.java
行数：362
修改之前代码片段：
}finally{
try {
fw.close();
} catch (IOExceptione) {
e.printStackTrace();
}
}
修改之后代码片段：
}finally{
try {
if(fw != null){
fw.close();
}
} catch (IOException e) {
e.printStackTrace();
}
改动原因：fw关闭之前加上判空代码，否则有可能出现空指针异常。
15、文件路径：com/iamzken/test/LoginController.java
行数：101-102
修改之前代码片段：
mv.addObject("dfUrl",cmd.getDefaultUrl());
mv.addObject("userName",cmd.getUserName());
修改之后代码片段：
if(cmd != null){
mv.addObject("dfUrl", cmd.getDefaultUrl());
mv.addObject("userName", cmd.getUserName());
}
改动原因：使用cmd对象之前先要进行非空判断
16、 文件路径：
com/iamzken/google/AgentInfo.java
行数：110-114
修改之前代码片段：
public boolean equals(AgentInfo agentInfo){
if(agentName.equals(agentInfo.getAgentName()) )
{
return true;
}
修改之后代码片段：
public boolean equals(AgentInfo obj){
if(obj instanceof AgentInfo){
AgentInfo agentInfo= (AgentInfo)obj;
if(agentName.equals(agentInfo.getAgentName()))
{
returntrue;
}
}
并且要重写hashCode方法。
@Override
public int hashCode(){
return this.getAgentName().hashCode();
}
改动原因：这是equals方法的标准写法。
17、文件路径：
com/iamzken/google/MerchantsTransfer.java
行数：88-98
修改之前代码片段：
public boolean equals(MerchantsTransfer merchantsTransfer){
if(this.merchantName.equals(merchantsTransfer.getMerchantName())
&&this.formerAgentName.equals(merchantsTransfer.getFormerAgentName())
&& this.idcontent.equals(merchantsTransfer.getIdcontent())
&& this.acceptAgentName.equals(merchantsTransfer.getAcceptAgentName())
)
{
return true;
}
return false;
}
修改之后代码片段：
public booleanequals(Object obj){
if(obj instanceofMerchantsTransfer){
MerchantsTransfermerchantsTransfer = (MerchantsTransfer)obj;
if(this.merchantName.equals(merchantsTransfer.getMerchantName())
&&this.formerAgentName.equals(merchantsTransfer.getFormerAgentName())
&&this.idcontent.equals(merchantsTransfer.getIdcontent())
&&this.acceptAgentName.equals(merchantsTransfer.getAcceptAgentName())
)
{
returntrue;
}
}
return false;
}
@Override
public int hashCode() {
returnthis.getMerchantName().hashCode()+
this.getFormerAgentName().hashCode()+
this.getIdcontent().hashCode()+
this.getAcceptAgentName().hashCode();
}
改动原因：equals方法写的不是标准写法，并且只有equals方法没有hashCode方法
18、文件路径：
com/iamzken/google/SpecialMerchantInfo.java
行数：55-64
修改之前代码片段：
public boolean equals(SpecialMerchantInfo merchantInfo){
if(specialMerchantName.equals(merchantInfo.getSpecialMerchantName())
&& membercode.equals(merchantInfo.getMembercode())
&& agentName.equals(merchantInfo.getAgentName())
)
{
return true;
}
return false;
}
修改之后代码片段：
public boolean equals(Object obj){
if(obj instanceof SpecialMerchantInfo){
SpecialMerchantInfo merchantInfo =(SpecialMerchantInfo)obj;
if(specialMerchantName.equals(merchantInfo.getSpecialMerchantName())
&& membercode.equals(merchantInfo.getMembercode())
&& agentName.equals(merchantInfo.getAgentName())
)
{
return true;
}
}
return false;
}
@Override
public int hashCode() {
return this.getSpecialMerchantName().hashCode()+
this.getMembercode().hashCode()+
this.getAgentName().hashCode();
}
改动原因：
equals方法不是标准写法，并且没有hashCode方法。
19、文件路径：
com/iamzken/google/SpringContextsUtil.java
行数：整个文件
修改之前的代码：
package com.iamzken.google;
import org.springframework.beans.BeansException;
import org.springframework.context.ApplicationContext;
import org.springframework.context.ApplicationContextAware;
public class SpringContextsUtilimplements ApplicationContextAware {
private static ApplicationContext applicationContext;
public static Object getBean(StringbeanName) {
return applicationContext.getBean(beanName);
}
public static <T> T getBean(StringbeanName, Class<T> clazs) {
return clazs.cast(getBean(beanName));
}
public void setApplicationContext(ApplicationContextapplicationContext)
throws BeansException {
SpringContextsUtil.applicationContext = applicationContext;
}
}
修改之后的代码：
package com.iamzken.google;
import org.springframework.beans.BeansException;
import org.springframework.context.ApplicationContext;
import org.springframework.context.ApplicationContextAware;
public class SpringContextsUtil implements ApplicationContextAware {
private ApplicationContext applicationContext;
public Object getBean(String beanName) {
return applicationContext.getBean(beanName);
}
public <T> T getBean(String beanName,Class<T> clazs) {
returnclazs.cast(getBean(beanName));
}
public voidsetApplicationContext(ApplicationContext applicationContext)
throws BeansException {
this.applicationContext =applicationContext;
}
}
改动原因：使用非静态方法给静态成员变量直接赋值容易出现多线程安全问题，去掉static，成为实例成员方法和成员变量，或者改为单例模式！
20、 文件路径：
com/iamzken/google/DateUtil.java
行数：179
修改之前代码片段：
public static boolean compareDate(String date1, String date2) {
SimpleDateFormat df = new SimpleDateFormat("yyyy/MM/dd");
boolean flag = false;
try {
date1 = date1.replace(".", "/");
date2 = date2.replace(".", "/");
date1 = date1.replace("-", "/");
date2= date2.replace("-", "/");
修改之后代码片段：
public static boolean compareDate(String date1, String date2) {
boolean flag = false;
try {
date1 = date1.replace(".", "/");
date2 = date2.replace(".", "/");
date1 = date1.replace("-", "/");
date2 = date2.replace("-", "/");
改动原因：去掉无用的SimpleDateFormatdf = new SimpleDateFormat("yyyy/MM/dd");这句话
21、文件路径：
行数：281
修改之前代码片段：
public ModelAndView viewDetail(HttpServletRequest request,
HttpServletResponse response){
ModelAndView m = new ModelAndView(this.detailView);
Map<String, String> map = new HashMap<String,String>();
Long logid = Long.valueOf(request.getParameter("logid" ));
修改之后代码片段：
public ModelAndView viewDetail(HttpServletRequest request,
HttpServletResponse response){
ModelAndView m = new ModelAndView(this.detailView);
Long logid = Long.valueOf(request.getParameter("logid" ));
改动原因：
去掉无用的 Map<String, String>map= new HashMap<String, String>();
这句话
22、文件路径：
com/iamzken/google/TradeZtcDetailService.java
行数：38
修改之前代码片段：
public Map addBatchTradeZtcDetail(List<String[]>list,String updateName) {
// 提示信息Map
Map tips = new HashMap();
// 验证错误信息的集合
List<String> errorList = new ArrayList<String>();
Map<String,SpecialMerchantInfo> map = new HashMap<String,SpecialMerchantInfo>();
// 代理商信息
List<AgentInfo>tradeZtcList = new ArrayList<AgentInfo>();
修改之后代码片段：
public Map addBatchTradeZtcDetail(List<String[]>list,String updateName) {
// 提示信息Map
Map tips = new HashMap();
// 验证错误信息的集合
List<String> errorList = new ArrayList<String>();
// 代理商信息
List<AgentInfo>tradeZtcList = new ArrayList<AgentInfo>();
改动原因：去掉无用的Map<String,SpecialMerchantInfo> map = newHashMap<String,SpecialMerchantInfo>();这句话
23、文件路径：
com/iamzken/google/ExportAgentInfo.java
行数：47
修改之前代码片段：
public void exportExcel(String title, String[] headers,List<T> dataList,
HttpServletResponse response, String pattern) throws IOException {
SimpleDateFormat sdf = new SimpleDateFormat(pattern);
OutputStream out = response.getOutputStream();
// 声明一个工作薄
SXSSFWorkbook workbook = newSXSSFWorkbook(5000);
修改之后代码片段：
public void exportExcel(String title, String[] headers,List<T> dataList,
HttpServletResponse response, String pattern) throws IOException {
OutputStream out = response.getOutputStream();
// 声明一个工作薄
SXSSFWorkbook workbook = newSXSSFWorkbook(5000);
改动原因：
去掉无用的SimpleDateFormat sdf = new SimpleDateFormat(pattern);这句话
24、文件路径：
com/iamzken/google/PartnersWebServiceImpl.java
行数：337
修改之前代码片段：
public List<Map<String,String>>invoiceSolution(String requestNo, String billNo,
String status,PartnersModel[] partnersmodel) {
String errMsg ="";
String [] result=new String[8];
Map<String, Object> map = new HashMap<String, Object>();
修改之后代码片段：
public List<Map<String,String>>invoiceSolution(String requestNo, String billNo,
String status,PartnersModel[] partnersmodel) {
String errMsg ="";
Map<String,Object> map = new HashMap<String,Object>();
改动原因：去掉无用的String [] result=new String[8];这句话
25、文件路径：
com/iamzken/google/ExportCommission.java
行数：49
修改之前代码片段：
public void exportExcel(String title, String[] headers,List<T> dataList,
HttpServletResponse response, String pattern) throws IOException {
SimpleDateFormat sdf = new SimpleDateFormat(pattern);
OutputStream out =response.getOutputStream();
修改之后代码片段：
public void exportExcel(String title, String[] headers,List<T> dataList,
HttpServletResponse response, String pattern) throws IOException {
OutputStream out =response.getOutputStream();
改动原因：去掉无用的SimpleDateFormat sdf = new SimpleDateFormat(pattern);这句话
26、文件路径：
com/iamzken/google/ExportMerchantInfo.java
行数：48
修改之前代码片段：
public void exportExcel(String title, String[] headers,List<SpecialMerchantInfo> dataList,
HttpServletResponse response, String pattern) throws IOException {
SimpleDateFormat sdf = new SimpleDateFormat(pattern);
OutputStream out = response.getOutputStream();
修改之后代码片段：
public void exportExcel(String title, String[] headers,List<SpecialMerchantInfo> dataList,
HttpServletResponse response, String pattern) throws IOException {
OutputStream out =response.getOutputStream();
改动原因：去掉无用的SimpleDateFormat sdf = new SimpleDateFormat(pattern);这句话
27、文件路径：
com/iamzken/google/ExportMerchantRelate.java
行数：48
修改之前代码片段：
public void exportExcel(String title, String[] headers, List dataList,
HttpServletResponse response, String pattern) throws IOException {
SimpleDateFormat sdf = new SimpleDateFormat(pattern);
OutputStream out =response.getOutputStream();
修改之后代码片段：
public void exportExcel(String title, String[] headers, List dataList,
HttpServletResponse response, String pattern) throws IOException {
OutputStream out =response.getOutputStream();
改动原因：去掉无用的SimpleDateFormat sdf = new SimpleDateFormat(pattern);这句话
28、文件路径：
com/iamzken/google/ExportPublicity.java
行数：51
修改之前代码片段：
public void exportExcel(String title, String[] headers,
List<PreAuditItem> dataList, HttpServletResponseresponse,
String pattern) throws IOException {
SimpleDateFormat sdf = new SimpleDateFormat(pattern);
OutputStream out = response.getOutputStream();
修改之后代码片段：
public void exportExcel(String title, String[] headers,
List<PreAuditItem> dataList, HttpServletResponseresponse,
String pattern) throws IOException {
OutputStream out =response.getOutputStream();
改动原因：去掉无用的SimpleDateFormat sdf = new SimpleDateFormat(pattern);这句话
29、文件路径：
com/iamzken/google/ExportTransDetail.java
行数：48
修改之前代码片段：
public void exportExcel(String title, String[] headers, List dataList,
HttpServletResponse response, String pattern) throws IOException {
SimpleDateFormat sdf = new SimpleDateFormat(pattern);
OutputStream out =response.getOutputStream();
修改之后代码片段：
public void exportExcel(String title, String[] headers, List dataList,
HttpServletResponse response, String pattern) throws IOException {
OutputStream out =response.getOutputStream();
改动原因：去掉无用的SimpleDateFormat sdf = new SimpleDateFormat(pattern);这句话
30、文件路径：
com/iamzken/google/ExportVposDetail.java
行数：48
修改之前代码片段：
public void exportExcel(String title, String[] headers, List dataList,
HttpServletResponse response, String pattern) throws IOException {
SimpleDateFormat sdf = new SimpleDateFormat(pattern);
OutputStream out =response.getOutputStream();
修改之后代码片段：
public void exportExcel(String title, String[] headers, List dataList,
HttpServletResponse response, String pattern) throws IOException {
OutputStream out =response.getOutputStream();
改动原因：去掉无用的SimpleDateFormat sdf = new SimpleDateFormat(pattern);这句话
31、文件路径：
com/iamzken/google/ExcelReader.java
行数：157
修改之前代码片段：
public String[]getRowData(int sheetIndex,int rowIndex){
String[] dataArray = null;
if(rowIndex>this.getColumnNum(sheetIndex)){
return dataArray;
}else{
dataArray = new String[this.getColumnNum(sheetIndex)];
return this.dataList.get(rowIndex);
}
修改之后代码片段：
public String[]getRowData(int sheetIndex,int rowIndex){
String[] dataArray = null;
if(rowIndex>this.getColumnNum(sheetIndex)){
return dataArray;
}else{
return this.dataList.get(rowIndex);
}
改动原因：去掉无用的dataArray = newString[this.getColumnNum(sheetIndex)]; 这句话
内容来自用户分享和网络整理，不保证内容的准确性，如有侵权内容，可联系管理员处理 
相关文章推荐
- java中的显示转换和隐式转换
- Struts1和Struts2核心控制器的执行原理
- Java集合框架（Set与Map，HashSet与HashMap，TreeSet与TreeMap）
- NET-JAVA 四 java代码注释规范
- SpringMVC深入理解
- 韩顺平 java io编程
- 保存Eclipse控制台输出的内容
- Java swing 实现图片闪烁
- JAVA反射机制获取类和方法
- Java的类名与文件名必须一致
- 用 Javassist 进行类转换
- java泛型小结
- java synchronized关键字详解
- spring一些注解的意思
- Spring in Action 第二部分学习笔记
- 问个spring自身框架中的log.info()怎么才能不往catalina.out写的问题
- Java集合collection编程题及答案
- java.lang.IndexOutOfBoundsException: Invalid index 0, size is 0
- eclipse安装maven插件
- java输出excel数据的方法示例（一种是到excel，一种是结构化数据到文本文件，可以用editplus之类的看）