如鹏网 大学生计算机学习社区

CowNew开源团队

http://www.cownew.com 邮件请联系 about521 at 163.com

  BlogJava :: 首页 :: 新随笔 :: 联系 :: 聚合  :: 管理 ::
  363 随笔 :: 2 文章 :: 808 评论 :: 0 Trackbacks

上午我试了下JDBMonitor(http://www.cownew.com),并阅读了部分代码
我觉得想法挺好的,不过我也发现了一个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增加一个主动关闭的方法

posted on 2006-05-22 01:34 CowNew开源团队 阅读(840) 评论(0)  编辑  收藏

只有注册用户登录后才能发表评论。


网站导航: