JDBMonitor程序的几个待改进之处(mistapostle)

上午我试了下JDBMonitor(http:// [url]www.cownew.com[/url]),并阅读了部分代码
我觉得想法挺好的,不过我也发现了一个BUG
我试了下DEMO里面TEST,结果发现如果使用SocketDBListener则整个程序不能正常结束
我查了下代码发现是SocketDBListener里面的THREAD不是后台的,而且也没有手工关闭的原因
我改成后台的之后就没问题了

另外还发现一个问题,
在SocketDBListener的代码里有这么一段:
protected void finalize() throws Throwable
{
super.finalize();
for(int i=0,n=clientList.size();i<n;i++)
{
Socket socket = (Socket) clientList.get(i);
socket.close();
}
}
这样有2个问题:
1)super.finalize() 最好在最后调用,而且最好在FIANLLY里用,如:
finally{
super.finalize();
}
2) 关闭socket之类的工作在fianlize里面做不太好,在我的测试里,当MAIN方法进行完了后
finalize并没有执行,而且因为SocketDBListener里面的THREAD还在,所以shoutdownHook也没有执行;
而且即使该为后台线程,能正常关闭了,finalize也不一定进行的
所以建议为IDBListener增加一个主动关闭的方法

本文出自 “CowNew开源团队” 博客,转载请与作者联系!

你可能感兴趣的:(职场,程序,休闲,JDBMonitor,mistapostle)