一、 编码

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

应该改为

for(int i=0,len=list.size();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

String title = “标题”;

}

改为

String title = null;

for(int i=0;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. 使用统一的,保证不同浏览器下的页面兼容

建议使用:

html PUBLIC “-//W3C//DTD XHTML 1.0 Transitional//EN” “http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd”>

23. HTML结构要完整、正确

标准的HTML文档结构:

html PUBLIC “-//W3C//DTD XHTML 1.0 Transitional//EN” “http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd”>

Insert title here

……

……

其它:

…………

……

24. HTML标签要完整

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

………

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

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

26. HTML代码使用标准缩进

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

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

标题

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

菜单

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

.menu{

color:black;

font-size:13px;

}

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

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

function checkForm(){

……

}

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

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

……


版权声明:本文为weixin_29284201原创文章,遵循 CC 4.0 BY-SA 版权协议,转载请附上原文出处链接和本声明。
原文链接:https://blog.csdn.net/weixin_29284201/article/details/114136255