差不多一两个月前吧,出于性能的考量给HSXB Project 做了优化,效果非常的理想,当时就想把优化HSXB的方法写出来与大家分享,可总由于这或那的,而一拖再拖,其实要想写出好的、详细的东西,是需要静下心来写,可最近我不可能有这样的条件,如是乎,就简要说几条,也算个一个过场吧。
1、程序的结构不能乱,架构要好,写代码的时候不需要去刻意的强调你所谓的性能,但你写的代码要知道(预判)他的可能瓶颈在那里。
2、对程序做后天优化,效果会非常棒,有时候一个小小的调整对程序的性能来说,可能是量级的提高。
3、好的架构,优化会很轻松,说实话HSXB的结构很不错,但代码我不满意,主要责任在我,一、HSXB是我写的第一个完全Java多线程程序,其次那个时候我身上有三四个项目,所以精力不够,三、接收我项目的那人还是个Java 新手。
4、技术人员嘛,能做到知其然知其所以然,这是棒中棒啊,这样做优化的空间就更大了,这也是我一直在苦苦追寻中,不容易 ...
5、软件业还没到工厂时代,软件还是门艺术 ...
下面给段代码,看你能否帮助优化一下,这段代码有好几处值得优化,但只要性能还可以的时候,我是不会去优化她,因为那样增加了一点负责度,有时候显得有些不值得 ...
/**
* Copyright (C) 2006 the original author or authors.
*
* This software is published under the terms of the GNU Public License (GPL),
* a copy of which is included in this distribution.
*/
package com.boar;
import java.util.Iterator;
import java.util.concurrent.Executors;
import java.util.concurrent.ScheduledExecutorService;
import java.util.concurrent.TimeUnit;
import com.boar.domain.Workgroup;
import com.boar.domain.WorkgroupManager;
import com.boar.domain.impl.DBWorkgroupManager;
/**
* Boar Robot Starter
*
* @author <a href="zhuaming@gmail.com">Ben </a>
*/
public class BoarStarter {
private static BoarManager bManager = BoarManager.getInstance();
private static WorkgroupManager workgroupManager = new DBWorkgroupManager();
public static void main(String args[]) {
Iterator<Workgroup> it = workgroupManager.getWorkgroups().iterator();
while(it.hasNext()){
Workgroup workgroup = it.next();
Boar boar = new Boar(workgroup.getWorkgroupID(),workgroup.getUsername(),workgroup.getPassword());
boar.openConnection();
bManager.addBoar(workgroup.getWorkgroupID(),boar);
}
ScheduledExecutorService scheduledExecute = Executors.newSingleThreadScheduledExecutor();
scheduledExecute.scheduleAtFixedRate(new BoarWorkerThread(),1,60, TimeUnit.SECONDS);
}
private static class BoarWorkerThread implements Runnable {
public void run() {
Iterator it = bManager.getBoars();
while(it.hasNext()){
Boar boar = (Boar)it.next();
if (boar.isConnection()) {
if (boar.getAgentSessionCount() > 1) {
boar.closeConnection(); //
}
} else {
boar.openConnection();
if (boar.getAgentSessionCount() > 1){
boar.closeConnection(); //
}
}
}
}
}
}