本规范规定了使用Java语言编程时排版、注释、命名、编码和JTEST的规则和建议。
本规范适用于使用Java语言编程的部门和产品。
下列文件中的条款通过本规范的引用而成为本规范的条款。凡是注日期的引用文件,其随后所有的修改单(不包括勘误的内容)或修订版均不适用于本规范,然而,鼓励根据本规范达成协议的各方研究是否可使用这些文件的最新版本。凡是不注日期的引用文件,其最新版本适用于本规范。
序号 |
编号 |
名称 |
1 |
公司-0000 |
《软件编程规范总则》 同总则的部分使用*号 |
2 |
公司-DKBA200106-003 |
《Java语言编程规范》 |
|
|
|
规则:编程时强制必须遵守的原则。
建议:编程时必须加以考虑的原则。
格式:对此规范格式的说明。
说明:对此规范或建议进行必要的解释。
示例:对此规范或建议从正、反两个方面给出例子。
说明:对于由开发工具自动生成的代码可以有不一致。
示例:如下例子不符合规范。
for (...) {
... // program code
}
if (...)
{
... // program code
}
void example_fun( void )
{
... // program code
}
应如下书写。
for (...)
{
... // program code
}
if (...)
{
... // program code
}
void example_fun( void )
{
... // program code
}
示例:
if (filename != null
&& new File(logPath + filename).length() < LogConfig.getFileSize())
{
... // program code
}
public static LogIterator read(String logType, Date startTime, Date endTime,
int logLevel, String userName, int bufferNum)
示例:如下例子不符合规范。
LogFilename now = null; LogFilename that = null;
应如下书写
LogFilename now = null;
LogFilename that = null;
示例:如下例子不符合规范。
if(writeToFile) writeFileThread.interrupt();
应如下书写:
if(writeToFile)
{
writeFileThread.interrupt();
}
示例:如下例子不符合规范。
if(log.getLevel() < LogConfig.getRecordLevel())
{
return;
}
LogWriter writer;
应如下书写
if(log.getLevel() < LogConfig.getRecordLevel())
{
return;
}
LogWriter writer;
int index;
说明:以免用不同的编辑器阅读程序时,因TAB键所设置的空格数目不同而造成程序布局不整齐。JBuilder、UltraEdit等编辑环境,支持行首TAB替换成空格,应将该选项打开。
说明:采用这种松散方式编写代码的目的是使代码更加清晰。
由于留空格所产生的清晰性是相对的,所以,在已经非常清晰的语句中没有必要再留空格,如果语句已足够清晰则括号内侧(即左括号后面和右括号前面)不需要加空格,多重括号间不必加空格,因为在Java语言中括号已经是最清晰的标志了。
在长语句中,如果需要加的空格非常多,那么应该保持整体清晰,而在局部不加空格。给操作符留空格时不要连续留两个以上空格。
示例:
(1) 逗号、分号只在后面加空格。
int a, b, c;
(2)比较操作符, 赋值操作符"="、 "+=",算术操作符"+"、"%",逻辑操作符"&&"、"&",位域操作符"<<"、"^"等双目操作符的前后加空格。
if (current_time >= MAX_TIME_VALUE)
a = b + c;
a *= 2;
a = b ^ 2;
(3)"!"、"~"、"++"、"--"、"&"(地址运算符)等单目操作符前后不加空格。
flag = !isEmpty; // 非操作"!"与内容之间
i++; // "++","--"与内容之间
(4)"."前后不加空格。
p.id = pid; // "."前后不加空格
(5) if、for、while、switch等与后面的括号间应加空格,使if等关键字更为突出、明显。
if (a >= b && c > d)
格式:
类定义
{
类的公有属性定义
类的保护属性定义
类的私有属性定义
类的公有方法定义
类的保护方法定义
类的私有方法定义
}
说明:注释的原则是有助于对程序的阅读理解,在该加的地方都加了,注释不宜太多也不能太少,注释语言必须准确、易懂、简洁。可以用注释统计工具来统计。
说明:方便JavaDoc收集
示例:
com/huawei/iin/websmap/comm/package.html
说明:在详细描述中应该说明这个包的作用以及在整个项目中的位置。
格式:
一句话简述。
详细描述。
产品模块名称和版本
公司版权信息
示例:
为 WEBSMAP 提供通信类,上层业务使用本包的通信类与 SMP-B 进行通信。
详细描述。。。。。。。。
IIN V100R001 WEBSMAP
(C) 版权所有 2000-2001 华为技术有限公司
说明:注意以 /* 开始避免被 JavaDoc 收集
示例:
/*
* 注释内容
*/
package com.huawei.iin.websmap.comm;
说明:文件名可选。
格式:
/*
* 文件名:[文件名]
* 版权:〈版权〉
* 描述:〈描述〉
* 修改人:〈修改人〉
* 修改时间:YYYY-MM-DD
* 跟踪单号:〈跟踪单号〉
* 修改单号:〈修改单号〉
* 修改内容:〈修改内容〉
*/
说明:每次修改后在文件头部写明修改信息,CheckIn的时候可以直接把蓝色字体信息粘贴到VSS的注释上。在代码受控之前可以免去。
示例:
/*
* 文件名:LogManager.java
* 版权:Copyright 2000-2001 Huawei Tech. Co. Ltd. All Rights Reserved.
* 描述: WIN V200R002 WEBSMAP 通用日志系统
* 修改人: 张三
* 修改时间:2001-02-16
* 修改内容:新增
* 修改人: 李四
* 修改时间:2001-02-26
* 跟踪单号:D20103
* 修改单号:WSS368
* 修改内容:。。。。。。
* 修改人: 王五
* 修改时间:2001-03-25
* 跟踪单号:D27153
* 修改单号:WSS498
* 修改内容:。。。。。。
*/
说明:方便JavaDoc收集
示例:
package com.huawei.iin.websmap.comm;
/**
* 注释内容
*/
public class CommManager
说明:可根据需要列出:版本号、生成日期、作者、内容、功能、与其它类的关系等。 如果一个类存在Bug,请如实说明这些Bug。
格式:
/**
* 〈一句话功能简述〉
* 〈功能详细描述〉
* @author [作者]
* @version [版本号, YYYY-MM-DD]
* @see [相关类/方法]
* @since [产品/模块版本]
* @deprecated
*/
说明:描述部分说明该类或者接口的功能、作用、使用方法和注意事项,每次修改后增加作者和更新版本号和日期,@since 表示从那个版本开始就有这个类或者接口,@deprecated 表示不建议使用该类或者接口。
示例:
/**
* LogManager 类集中控制对日志读写的操作。
* 全部为静态变量和静态方法,对外提供统一接口。分配对应日志类型的读写器,
* 读取或写入符合条件的日志纪录。
* @author 张三,李四,王五
* @version 1.2, 2001-03-25
* @see LogIteraotor
* @see BasicLog
* @since CommonLog1.0
*/
示例:
/**
* 注释内容
*/
private String logType;
/**
* 注释内容
*/
public void write()
格式:
/**
* 〈一句话功能简述〉
* 〈功能详细描述〉
* @param [参数1] [参数1说明]
* @param [参数2] [参数2说明]
* @return [返回类型说明]
* @exception/throws [违例类型] [违例说明]
* @see [类、类#方法、类#成员]
* @deprecated
*/
说明:@since 表示从那个版本开始就有这个方法;@exception或throws 列出可能仍出的异常;@deprecated 表示不建议使用该方法。
示例:
/**
* 根据日志类型和时间读取日志。
* 分配对应日志类型的LogReader,指定类型、查询时间段、条件和反复器缓冲数,
* 读取日志记录。查询条件为null或0的表示没有限制,反复器缓冲数为0读不到日志。
* 查询时间为左包含原则,即 [startTime, endTime) 。
* @param logTypeName 日志类型名(在配置文件中定义的)
* @param startTime 查询日志的开始时间
* @param endTime 查询日志的结束时间
* @param logLevel 查询日志的级别
* @param userName 查询该用户的日志
* @param bufferNum 日志反复器缓冲记录数
* @return 结果集,日志反复器
* @since CommonLog1.0
*/
public static LogIterator read(String logType, Date startTime, Date endTime,
int logLevel, String userName, int bufferNum)
说明:异常注释用@exception或@throws表示,在JavaDoc中两者等价,但推荐用@exception标注Runtime异常,@throws标注非Runtime异常。异常的注释必须说明该异常的含义及什么条件下抛出该异常。
说明:可使程序排版整齐,并方便注释的阅读与理解。
示例:如下例子,排版不整齐,阅读稍感不方便。
public void example( )
{
// 注释
CodeBlock One
// 注释
CodeBlock Two
}
应改为如下布局。
public void example( )
{
// 注释
CodeBlock One
// 注释
CodeBlock Two
}
示例:如下例子,显得代码过于紧凑。
//注释
program code one
//注释
program code two
应如下书写:
//注释
program code one
//注释
program code two
说明:这些语句往往是程序实现某一特定功能的关键,对于维护人员来说,良好的注释帮助更好的理解程序,有时甚至优于看设计文档。
说明:这样比较清楚程序编写者的意图,有效防止无故遗漏break语句。
说明:错误的注释不但无益反而有害。
说明:在使用缩写时或之前,应对缩写进行必要的说明。
说明:除非必要,不应在代码或表达中间插入注释,否则容易使代码可理解性变差。
说明:清晰准确的函数、变量等的命名,可增加代码可读性,并减少不必要的注释。
说明:注释的目的是解释代码的目的、功能和采用的方法,提供代码以外的信息,帮助读者理解代码,防止没必要的重复注释信息。
示例:如下注释意义不大。
// 如果 receiveFlag 为真
if (receiveFlag)
而如下的注释则给出了额外有用的信息。
// 如果从连结收到消息
if (receiveFlag)
说明:当代码段较长,特别是多重嵌套时,这样做可以使代码更清晰,更便于阅读。
示例:参见如下例子。
if (...)
{
program code1
while (index < MAX_INDEX)
{
program code2
} // end of while (index < MAX_INDEX) // 指明该条while语句结束
} // end of if (...) // 指明是哪条if语句结束
说明:注释语言不统一,影响程序易读性和外观排版,出于对维护人员的考虑,建议使用中文。
说明:调试程序的时候可以方便的使用 /* 。。。*/ 注释掉一长段程序。
说明:JavaDoc工具收集简介的时候使用选取第一句话。
示例:如下是对设置属性的流程注释
//1、 判断输入参数是否有效。
。。。。。
// 2、设置本地变量。
。。。。。。
示例:这里主要是对闰年算法的说明。
//1. 如果能被4整除,是闰年;
//2. 如果能被100整除,不是闰年.;
//3. 如果能被400整除,是闰年.。
格式:
com.huawei.产品名.模块名称
com.huawei.部门名称. 项目名称
示例:
融合WEBSMAP包名 com.huawei.iin.websmap
通用消息转发包名 com.huawei.insa2.msgtrans
示例:OrderInformation, CustomerList, LogManager, LogConfig, SmpTransaction
示例:
private void calculateRate();
public void addNewOrder();
格式:
get + 非布尔属性名()
is + 布尔属性名()
set + 属性名()
动词()
动词 + 宾语()
示例:
public String getType();
public boolean isFinished();
public void setVisible(boolean);
public void show();
public void addKeyListener(Listener);
示例:
private customerName;
private orderNumber;
private smpSession;
示例:
public final static int MAX_VALUE = 1000;
public final static String DEFAULT_START_DATE = "2001-12-08";
示例:
public class Person
{
private String name;
private static List properties;
public void setName (String name)
{
this.name = name;
}
public void setProperties (List properties)
{
Person.properties = properties;
}
示例:
Application 类型的,命名以App 结尾——MainApp
Frame 类型的,命名以Frame 结尾——TopoFrame
Panel 类型的,建议命名以Panel 结尾——CreateCircuitPanel
Bean 类型的,建议命名以Bean 结尾——DataAccessBean
EJB 类型的,建议命名以EJB 结尾——DBProxyEJB
Applet 类型的,建议命名以Applet 结尾——PictureShowApplet
示例:getCustomerInformation() 改为 getCustomerInfo()
示例: protected void setUserName(), private void calculateRate()
示例:customers, orderItems
说明:虽然为仅用一两行就可完成的功能去编方法好象没有必要,但用方法可使功能明确化,增加程序可读性,亦可方便维护、测试。
说明:对于模块间接口方法的参数的合法性检查这一问题,往往有两个极端现象,即:要么是调用者和被调用者对参数均不作合法性检查,结果就遗漏了合法性检查这一必要的处理过程,造成问题隐患;要么就是调用者和被调用者均对参数进行合法性检查,这种情况虽不会造成问题,但产生了冗余代码,降低了效率。
说明:划分类的时候,应该尽量把逻辑处理、数据和显示分离,实现类功能的单一性。
示例:
数据类不能包含数据处理的逻辑。
通信类不能包含显示处理的逻辑。
说明: 父类如果实现了比较合理的toString() ,子类可以继承不必再重写。
示例:
public TopoNode
{
private String nodeName;
public String toString()
{
return "NodeName : " + nodeName;
}
}
示例:
try
{
// ... ...
}
catch(IOException ioe)
{
//... ...
}
finally
{
try
{
out.close();
}
catch (IOException ioe)
{
//... ...
}
}
说明:若有特殊原因必须用注释加以说明。
示例:
try
{
//.... ...
}
catch (IOException ioe)
{
ioe.printStackTrace ();
}
说明:便于问题定位。
示例:
throw new IOException("Writing data error! Data: " + data.toString());
说明:
非运行期异常是由外界运行环境决定异常抛出条件的异常,例如文件操作,可能受权限、磁盘空间大小的影响而失败,这种异常是程序本身无法避免的,需要调用者明确考虑该异常出现时该如何处理方法,因此非运行期异常必须有throws子句标出,不标出或者调用者不捕获该类型异常都会导致编译失败,从而防止程序员本身疏忽。
运行期异常是程序在运行过程中本身考虑不周导致的异常,例如传入错误的参数等。抛出运行期异常的目的是防止异常扩散,导致定位困难。因此在做异常体系设计时要根据错误的性质合理选择自定义异常的继承关系。
还有一种异常是Error 继承而来的,这种异常由虚拟机自己维护,表示发生了致命错误,程序无法继续运行例如内存不足。我们自己的程序不应该捕获这种异常,并且也不应该创建该种类型的异常。
说明:
一个系统或者模块应该统一规划异常类型和返回码的含义。
但是不能用异常来做一般流程处理的方式,不要过多地使用异常,异常的处理效率比条件分支低,而且异常的跳转流程难以预测。
说明:防止阅读程序时产生误解,防止因默认的优先级与设计思想不符而导致程序出错。
示例:
下列语句中的表达式
word = (high << 8) | low (1)
if ((a | b) && (a & c)) (2)
if ((a | b) < (c & d)) (3)
如果书写为
high << 8 | low
a | b && a & c
a | b < c & d
(1)(2)虽然不会出错,但语句不易理解;(3)造成了判断条件出错。
示例:如下的程序可读性差。
if (state == 0)
{
state = 1;
... // program code
}
应改为如下形式:
private final static int TRUNK_IDLE = 0;
private final static int TRUNK_BUSY = 1;
private final static int TRUNK_UNKNOWN = -1;
if (state == TRUNK_IDLE)
{
state = TRUNK_BUSY;
... // program code
}
说明:使用int index[] 格式使程序的可读性较差
示例:
如下程序可读性差:
public int getIndex()[]
{
....
}
如下程序可读性好:
public int[] getIndex()
{
....
}
说明:代码发布的时候可以统一关闭调试代码,定位问题的时候又可以打开开关。
示例:NullPointException抛出时常常描述为空,这样往往看不出是出了什么错。
说明: 如果程序中需要分类处理,则将异常根据分类组织成继承关系。如果确实有很多异常类型首先考虑用异常描述来区别,throws/exception子句标明的异常最好不要超过三个。
说明:若此段代码各语句之间有实质性关联并且是完成同一件功能的,那么可考虑把此段代码构造成一个新的方法。
说明:主要类包括:
1、能完成独立功能的类,如通讯。
2、具有完整界面的类,如一个对话框、一个窗口、一个帧等。
3、JavaBean 类。
示例:
public static void main(String[] arguments)
{
CreateCircuitDialog circuitDialog1 = new CreateCircuitDialog (null, "Ciruit", false);
circuitDialog1.setVisible(true);
}
说明:由于集合保存了对象的句柄,虚拟机的垃圾收集器就不会回收。
说明:便于程序阅读和查找。
示例:矩形的长与宽关系较密切,放在一起。
rect.length = 10;
rect.width = 5;
说明:高技巧语句不等于高效率的程序,实际上程序的效率关键在于算法。
继承层次 5层
类的行数 1000行(包含{})
类的属性 10个
类的方法 20个
类友好方法 10个
类私有方法 15个
类保护方法 10个
类公有方法 10个
类调用方法 20个
方法参数 5个
return语句 1个
方法行数 30行
方法代码 20行
注释比率 30--50%
为了规范大家的编程行为,借鉴前人的编程经验,避免低级错误的不断发生,特发布MDSP产品Java十大低级错误,供所有Java开发人员学习:
1、方法和变量命名随意而不规范,没有类注释、方法注释或注释不规范,代码修改后,不同步修改注释,导致注释与代码不符。
2、数据类不重载toString()方法(说明:编程规范要求“所有的数据类必须重载toString() 方法,返回该类有意义的内容”)。
3、对方法的调用不进行空指针判断而造成空指针异常。
4、数据库操作、IO操作的资源没有及时释放,数据库结果集和连接释放顺序不正确,或者使用没有必要的预处理。
5、循环体内包含了大量没有必要在循环中处理的语句,循环体内循环获取数据库连接,循环体内进行不必要的try-catch操作。(说明:编程规范中建议“不要在循环体内调用同步方法和使用 try-catch 块”)
6、嵌套使用try-catch,或者try-catch后面没有必要的finally操作(说明:数据库操作、IO操作等需要使用结束close()的对象必须在try -catch-finally 的finally中close())。
7、不对数组下标作范围校验。
8、equals操作时没有将常量放在equals操作符的左边(说明:字符串变量与常量比较时,先写常量,这样可以避免空指针异常)。
9、字符串转化为数字时没有做异常处理。
10、没有在异常分支记录日志导致问题定位困难。
代码案例(说明:所有样例都是错误的代码):
1、方法和变量命名随意而不规范,没有类注释、方法注释或注释不规范,代码修改后,不同步修改注释,导致注释与代码不符。
public class MethodName
{
//错误:没有类的注释
//错误:方法的注释不规范、方法名不规范
public void a( int i )
{
//错误:变量名称太短
String b = "";
//错误:变量名称太长,超过40个字符
String longVariableLongVariableLongVariableLongVariable= "";
}
/**
* <一句话功能简述>方法命名约定:应以小写字母开头、同时命名不应含有下划线字符。
* <功能详细描述>
* @param i
* @see [类、类#方法、类#成员]
*/
public void Check_Method( int i )
{
// 错误:方法名称不规范
//错误:变量名称不规范
String b = "";
}
/**
* <一句话功能简述>变量、方法名、类名、接口名避免出现美元字符$。
* <功能详细描述>
* @param i
* @see [类、类#方法、类#成员]
*/
public void Fo$o ()
{
// 错误:方法名称不规范
}
/**
* <一句话功能简述>注意hashCode方法大小写。此规则用于推测作者可能是要实现hashCode方法,但大小写错了。
* <功能详细描述>
* @return
* @see [类、类#方法、类#成员]
*/
public int hashcode()
{
// 错误:方法名称不规范
int temp = 0;
// oops,this probably was supposed to be hashCode
return temp;
}
/**
*
* @param o 只有参数类型和返回值类型不一致,方法名equals完全相同(包括大小写也相同),推测可能是想实现equals方法。
* @return
*/
public int equals(Object o)
{
// 错误:方法名称不规范
// oops, thisprobably was supposed to be boolean equals
int temp = 0;
return temp;
}
/**
* <一句话功能简述>只有参数类型和返回值类型不一致,方法名equals完全相同(包括大小写也相同),推测可能是想实现equals方法。
* <功能详细描述>
* @param s
* @return
* @see [类、类#方法、类#成员]
*/
public booleanequals(String s)
{
// 错误:方法名称不规范
// oops, thisprobably was supposed to be equals(Object)
return true;
}
/**
* <一句话功能简述>'getX()'方法同时返回值又是 'boolean' ,则建议用'isX()'命名。checkParameterizedMethods属性用于决定是否对带参数的方法检测。
* <功能详细描述>
* @return
* @see [类、类#方法、类#成员]
*/
public boolean getFoo()
{
// 错误:方法名称不规范
// bad
return true;
}
}
2、数据类不重载toString()方法(说明:编程规范要求“所有的数据类必须重载toString() 方法,返回该类有意义的内容”)。
public classContent extendsBaseMetaData
{
private String oldCID;
private String cId;
private String spId;
private inttype;
private int serviceType;
.........................
/**
* 设置修改标识
* @param update_flag修改标识
*/
public voidsetUpdate_flag(String update_flag)
{
this.updateFlg= update_flag;
}
public voidsetVersion(String version)
{
this.version= version;
}
public StringtoString()
{
//没有重载toString方法或者重载无意义的toString方法
return null;
}
}
3、对方法的调用不进行空指针判断而造成空指针异常。
/**
* <一句话功能简述>演示不进行空指针判断造成空指针异常
* <功能详细描述>
*
* @see [相关类/方法]
* @since [产品/模块版本]
*/
public class NullPoint
{
/**
* <一句话功能简述>
* <功能详细描述>
* @return
* @see [类、类#方法、类#成员]
*/
public String returnNull()
{
Stringtemp = null;
return temp;
}
public void checkNull()
{
String str1 = "aa";
//错误:没有对returnNull()的值进行为空判断,会造成空指针
if (str1.equals(returnNull()))
{
str1 = returnNull();
}
print(str1);
}
public void print(String str)
{
}
}
4、数据库操作、IO操作的资源没有及时释放,数据库结果集和连接释放顺序不正确,或者使用没有必要的预处理。
/**
* <一句话功能简述>该类用来演示没有释放资源的的错误
* <功能详细描述>
* @see [相关类/方法]
* @since [产品/模块版本]
*/
public class ReleaseRes
{
public void releaseDB ()
{
Connectionconn = null;
APLResultSet rs = null;
Stringsql= "";
try
{
conn =DB.getConnection();
rs =DB.query(sql);
if (rs.next())
{
return rs.getString("currentdate");
}
else
{
returnDateTool.getCurrentDate(pattern);
}
Conn.close();
//错误:连接没有在finally正确释放,会出现异常时导致在运行一段时间后,系统由于连接池满载而崩溃。
}
catch (DBException e)
{
...................
}
//错误:资源APLResultSet没有正确释放
}
/**
* <一句话功能简述>
* <功能详细描述>
* @param file
* @see [类、类#方法、类#成员]
*/
public void releaseIO (Filefile)
{
StringBuffer sb = null;
BufferedReader in = null;
try
{
in = new BufferedReader(new FileReader(file));
sb = new StringBuffer();
for (String line; (line = in.readLine()) != null;)
{
sb.append(line + "\r\n");
}
}
catch (IOException ex)
{
Throwex;
}
//错误:IO资源没有正确释放,需要在finally中释放资源
}
5、循环体内包含了大量没有必要在循环中处理的语句,循环体内循环获取数据库连接,循环体内进行不必要的try-catch操作。(说明:编程规范中建议“不要在循环体内调用同步方法和使用 try-catch 块”)
/**
* <一句话功能简述>该类用来演示循环中的错误
* <功能详细描述>
* @see [相关类/方法]
* @since [产品/模块版本]
*/
public class CheckCycle
{
public void cycleInfo()
{
String[] Str = null;
//获取各个部件的通知地址
Str =outNotify.getResult();
StringBuffer sb = null;
BufferedReader in = null;
Connectionconn = null;
APLResultSet rs = null;
Stringsql= "";
File file= null;
for (int i = 0; i { //错误:判断完全可以在for外部进行而不是在内内部。 if( null != Str) { System.out.print(Str[i]); } //错误:不要在循环中获取数据库连接。 conn =DB.getConnection(); //错误:循环体内进行无必要的try-catch操作,完全可以放在循环外边。 try { //错误:不要在循环中获取IO连接。 in= new BufferedReader(new FileReader(file)); sb= new StringBuffer(); } catch (IOException ex) { Throw ex; } } } } 6、嵌套使用try-catch,或者try-catch后面没有必要的finally操作(说明:数据库操作、IO操作等需要使用结束close()的对象必须在try -catch-finally 的finally中close())。 /** * <一句话功能简述>演示嵌套循环的错误 * <功能详细描述> * @see [相关类/方法] * @since [产品/模块版本] */ public class NestingTry { /** * <一句话功能简述> * <功能详细描述> * @see [类、类#方法、类#成员] */ public void checkTry() { StringBuffer sb = null; BufferedReader in = null; File file = null; try { initFtpClient(this.ftpClient,getFtpServConfig()); // 把绝对路径加入到文件中 String[] absoluteFiles null; //错误:完全没有必要在内容进行异常处理,而应该将异常处理放在try块后面进行处理。 try { absoluteFiles =addAbsolutePath(this.ftpClient, files); } Catch(IOException ex) { Throw ex; } File file = null; in = new BufferedReader(newFileReader(file)); sb = new StringBuffer(); } catch (Exception e) { throw newFtpOperationsException(FtpConst.Ftp_Operations_Del_Error, "IOException wheninvoking deleteFiles method", e); } //错误:IO资源没有正确释放,必须有finally进行释放资源 } } 7、不对数组下标作范围校验。 /** * <一句话功能简述>该类用来演示数据下标越界 * <功能详细描述> * @see [相关类/方法] * @since [产品/模块版本] */ public class ArrayOver { public void checkArray() { //获取一个数组对象 String[] cIds =ContentService.queryByName(name); if(null != cIds) { //错误:只是考虑到cids有可能为null的情况,但是cids完全有可能是个0长度的数组,因此cIds[0]有可能数组下标越界。 Stringcid=cIds[0]; } } } 8、equals操作时没有将常量放在equals操作符的左边(说明:字符串变量与常量比较时,先写常量,这样可以避免空指针异常)。 /** * 定义常量 */ public static final String SP_NAME = "SPNAME"; /** * <一句话功能简述>常量比较 * <功能详细描述> * @see [类、类#方法、类#成员] */ public void equalsConst() { Stringtemp = null; //错误;//在temp为null的情况下,此时会导致意想不到的异常抛出,如果将常量放在左边,就不会有这种问题。 if(temp.equals(SP_NAME)) { temp = SP_NAME + "AA"; } } 9、字符串转化为数字时没有做异常处理。 /** * <一句话功能简述>字符传传唤成数字 * <功能详细描述> * @see [类、类#方法、类#成员] */ public void switchStr() { String temp = "123a"; //错误:对字符串转换为数字没有进行任何异常处理。可能抛出未知异常。 if(Integer.parseInt(temp)== 123) { //...... } } 10、没有在异常分支记录日志导致问题定位困难。 /** * 根据objectId删除业务信息 * * @param id 业务objectId * @throws IllegalServiceStatusException 非法业务状态异常 * @see [类、类#方法、类#成员] */ public void deleteService(String id) throwsIllegalServiceStatusException { log.enterMethodDebugLog(new Object[][] {{"id", id}}); MdspTServiceinfo serviceInfo =getMdspServiceByObjectId(id); if (null == serviceInfo) { return; //错误:异常分支没有记录日志,导致问题定位困难。 } ........ }