java代码审查检查表

发表于:2008-2-26 17:24

字体: | 上一篇 | 下一篇 | 我要投稿

 作者:未知    来源:网络转载

    
可靠性(总则/变量和语句)      
重要   100 是否已经消除了所有警告?
重要 Y 40 常数变量是否声明为final
重要   80 对象使用前是否进行了检查?
重要   80 局部对象变量使用后是否被复位为NULL
重要   70 对数组的访问是否是安全的?(合法的index取值为[0, MAX_SIZE-1])。
重要   20 是否确认没有同名变量局部重复定义问题?
    20 程序中是否只使用了简单的表达式?
重要 Y 20 是否已经用()使操作符优先级明确化?
重要 Y 20 所有判断是否都使用了(常量==变量)的形式?
    80 是否消除了流程悬挂?
重要   80 是否每个if-else if-else语句都有最后一个else以确保处理了全集?
重要   80 是否每个switch-case语句都有最后一个default以确保处理了全集?
    80 for循环是否都使用了包含下限不包含上限的形式?(k=0; k<MAX
重要   40 XML标记书写是否完整,字符串的拼写是否正确?
    40 对于流操作代码的异常捕获是否有finally操作以关闭流对象?
    20 退出代码段时是否对临时对象做了释放处理?
重要   40 对浮点数值的相等判断是否是恰当的?(严禁使用==直接判断)
可靠性(函数)      
重要 Y 60 入口对象是否都被进行了判断不为空?
重要 Y 60 入口数据的合法范围是否都被进行了判断?(尤其是数组)
重要 Y 20 是否对有异常抛出的方法都执行了try...catch保护?
重要 Y 80 是否函数的所有分支都有返回值?
重要   50 int的返回值是否合理?(负值为失败,非负值成功)
    20 对于反复进行了int返回值判断是否定义了函数来处理?
    60 关键代码是否做了捕获异常处理?
重要   60 是否确保函数返回CORBA对象的任何一个属性都不能为null?
重要   60 是否对方法返回值对象做了null检查,该返回值定义时是否被初始化?
重要   60 是否对同步对象的遍历访问做了代码同步?
重要   80 是否确认在对Map对象使用迭代遍历过程中没有做增减元素操作?
重要   60 线程处理函数循环内部是否有异常捕获处理,防止线程抛出异常而退出?
    20 原子操作代码异常中断,使用的相关外部变量是否恢复先前状态?
重要   100 函数对错误的处理是恰当的?
可维护性      
重要   100 实现代码中是否消除了直接常量?(用于计数起点的简单常数例外)
    20 是否消除了导致结构模糊的连续赋值?(如a= (b=d+c )
    20 是否每个return前都要有日志记录?
    20 是否有冗余判断语句?(如:if (b) return true; else return false;
    20 是否把方法中的重复代码抽象成私有函数?
22/2<12
《2023软件测试行业现状调查报告》独家发布~

关注51Testing

联系我们

快捷面板 站点地图 联系我们 广告服务 关于我们 站长统计 发展历程

法律顾问:上海兰迪律师事务所 项棋律师
版权所有 上海博为峰软件技术股份有限公司 Copyright©51testing.com 2003-2024
投诉及意见反馈:webmaster@51testing.com; 业务联系:service@51testing.com 021-64471599-8017

沪ICP备05003035号

沪公网安备 31010102002173号