一、 编码

1. 约束自己,规范编码习惯

充足的代码注释、标准缩进的格式、注意命名规范。参考《开发管理规范》

"看上去"专业能促进代码质量。越是难看的代码,在它的演化过程中会越来越差。因为当你看到你要bugfix的代码很乱,那么在bugfix的时候一般也会草草了事。一个方法有400行,在修改这个方法的时候大家不会在意再加上几十行代码。因为它看起来很差,它就是很差,我没必要美化它。反过来说,如果待改的代码很整洁很规范,那么修改者也会比较小心。

2. 避免冗长的方法和类

应将方法设计成简要的、功能性单元,用它描述和实现一个不连续的类接口部分。理想情况下,方法应简明扼要。若长度很大,可考虑通过某种方式将其分割成较短的几个方法。这样做也便于类内代码的重复使用(有些时候,方法必须非常大,但它们仍应只做同样的一件事情)。

3. 不要向标准输出写无用数据

System.out.println(); 此语句在程序中随处可见,都是在调试时使用的,在程序正式运行时也没有将其去掉,结果就是日志中有大量的无用数据,不仅使得日志不易分析,也增加了系统的开销。

如要调试可用LogWriter,将WEB-INF/ini/merpserver.ini中loglevel设为4

4. 调用方法前注意参数校验,判断参数是否为空或无意义的值

5. 使用对象前,检查对象是否为空

if(names!=null&&names.length>0){
String[] nameArray = names.split(‘,’);
}
if(userEn!=null){
String name = userEn.getName();
}

6. 避免过多过常的创建java对象

尽量避免在经常调用的方法、循环中new对象,由于系统不仅要花费时间来创建对象,而且还要花时间对这些对象进行垃圾回收和处理,在我们可以控制的范围内,最大限度的重用对象,最好能用基本的数据类型或数组来替代对象。

7. 尽量避免随意使用类成员变量

当某个对象被定义为stataic变量所引用,那么gc通常是不会回收这个对象所占有的内存的。此时类成员变量的生命周期与类同步,如果类不卸载,那么该对象会常驻内存,直到程序终止

8. 减少对变量的重复计算

for(int i=0;i<list.size();i++)

应该改为

for(int i=0,len=list.size();i<len;i++)

并且在循环中应该避免使用复杂的表达式,在循环中,循环条件会被反复计算,如果不使用复杂表达式,而使循环条件值不变的话,程序将会运行的更快

9. 避免不必要的创建对象

A a = new A();
if(i==1){
list.add(a);
}

应该改为

if(i==1){
A a = new A();
list.add(a);
}

10. 原则上循环里面不要声明对象,一律在循环外面声明

for(int i=0;i<size;i++){
String title = “标题”;
}

改为

String title = null;
for(int i=0;i<size;i++){
title = “标题”;
}

11. 尽量在finally块中释放资源

程序中使用到的资源应当被释放,以避免资源泄漏。这最好在finally块中去做。不管程序执行的结果如何,finally块总是会执行的,以确保资源的正确关闭。

12. 使用StringBuilder和StringBuffer进行字符串连接

StringBuffer提供了同步机制,所以并发线程访问是线程安全的,适合多线程。

StringBuilder没有提同步机制,所以线程不安全,适合单线程,但如果是单线程的话,要比StringBuffer快。

13. 遍历HashMap使用entrySet

当需要遍历HashMap的时候,请尽量使用entrySet,而不要用keySet,entrySet的效率要比keySet高,实际上使用entrySet是只需要遍历一次hash,即将key和value的映射关系放入到entry中,再取之;而keySet需要两次遍历hash,第一次取所有的key,第二次用key去取出对应的value。

Iterator iter = hashMap.entrySet().iterator();
while (iter.hasNext()) {
Map.Entry entry = (Map.Entry) iter.next();
String key = String.valueOf(entry.getKey());
String val = String.valueOf(entry.getValue());
}

14. 尽量缓存经常使用的对象

尽可能将经常使用的对象进行缓存,可以使用数组,或HashMap的容器来进行缓存,但这种方式可能导致系统占用过多的缓存,性能下降。

推荐参考《开发指南》,使用EHCache

15. 使用统一的工具类

使用hanwebcommon.jar中的通用方法,具体参考《开发指南》

使用项目中已经存在的工具类,不要重复创造功能近似的类和方法,如果必要可进行扩展

如:接收参数使用Convert.getParameter(request, 参数名);

16. 减少不必要的空格和空行,参考《开发管理规范》

17. java代码中不要出现黄色警告。注释或删除未使用的变量;保存时去掉多余的import;…

18. 前台接收Stirng类型参数,要进行跨站脚本和sql注入过滤

Convert.getParameter(request,"keyword","",true,true);

19. 不要在jsp中实现业务逻辑,放到类中去完成

分层设计实现了软件之间的解耦;便于进行分工;便于维护;提高软件组件的重用;便于替换某种产品,比如持久层用的是hibernate,需要更换产品用toplink,就不用改其他业务代码,直接把配置一改;便于产品功能的扩展;便于适用用户需求的不断变化。

20. 避免在循环体中使用try-catch 块,最好在循环体外使用try--catch 块以提高系统性

21. oracle大字段操作

先插入一个空的clob类型 empty_clob(),然后再单独更新clob字段

InsertSql insql = new InsertSql( strTableName ); 
    insql.addString("vc_name", name);
    if (("oracle").equals(SysInit.getM_strDB_Type())){
     insql.addClob("vc_adress");
     insql.addClob("vc_path");
    }else{
        insql.addString("vc_adress", address);
        insql.addString("vc_path", path);
}
boolean bl = Manager.doExcute(strAppID , insql.getSql());
if(bl){
if (("oracle").equals(SysInit.getM_strDB_Type())){
String[] strFieldValue = {address, path};
String[] strFieldName ={"vc_adress","vc_path"};
Manager.doClob(strAppID, strFieldName, strFieldValue, strTableName, " WHERE i_id = " + getMaxId());
}
}

22. 使用统一的<!DOCTYPE>,保证不同浏览器下的页面兼容

建议使用:

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">

23. HTML结构要完整、正确

标准的HTML文档结构:

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8" />
<title>Insert title here</title>
……
</head>
<body>
……
</body>
</html>

其它:

<ul>
<li>……</li>
<li>……</li>
</ul>
<table>
<tr>
<td>……</td>
</tr>
</table>

24. HTML标签要完整

标签名称及属性统一使用小写,标签要成对出现,如:

<div id=”search”>………</div>

不用于包含内容的标签,可在标签结尾使用“/”标记结束,如:

<input type=”text” name=”username” value=”tony” />
<br/>

25. 标签属性值必须用双引号包住

26. HTML代码使用标准缩进

27. 脚本每一条语句都要以分号结尾

28. 具有独特性、不需要重复使用的样式,使用内嵌样式:

<div style=”title”>标题<div>

能够重复使用的样式,在样式表中定义:

<li class=”menu”>菜单</li>

页面内使用的样式,使用内嵌样式表:

<style type="text/css">
.menu{
color:black;
font-size:13px;
}
</style>

多个页面公用的样式使用链入外部样式表:

<link href="../global.css" rel="stylesheet" type="text/css" />

29. 页面内使用的脚本函数,在head中定义:

<script type=”text/javascript”>
function checkForm(){
……
}
</script>

重复使用的脚本函数或代码较多的脚本,写到外本脚本文件中,在head中链接:

<script type=”text/javascript” src=”../checkform.js” />

不影响页面本身展示的脚本可考虑放在body结束的位置调用,如广告代码:

……
<script type=”text/javascript” src=”../adv.js” />
</body>

30. 风格统一,保存时要对代码进行格式化,Ctrl+Shift+F

31. 养成程序优化的意识

现在我们经常遇到的一个问题是,程序在开发阶段,执行的完全正常,找测试人员测试也没问题,但是一到上线后,性能马上出问题了,运行速度像蜗牛,客 户不堪忍受,为什么?简单的说,开发人员自测,估计也就几条数据,测试人员测试,估计也就几十上百条数据,一般的程序代码,在这个数量级,性能瓶颈根本就 表现不出来。但是上线后,客户的数据一般都上升到10000级别的,如果程序代码中没有做到细微之处都很严谨的话,问题就马上暴露出来了。