本人目前遇到的一个问题,在公司眼前跑的系统中,有一个项目中设了两个定时器,每天到定时的时候严重影响服务器性能,致使CPU的占有率达到了100%,有时会宕机,我修改了一下跑定时的程序,如下:
SeoPageTextsBean pageTextBean = new SeoPageTextsBean();
int len = pageTexts.size();
for (int i=0;i<len;i++)
{
TSeoPageTexts seoPageText = (TSeoPageTexts) pageTexts.get(i);
String pageUrl = seoPageText.getPageUrl();
if (pageUrl!=null&&!"".equals(pageUrl))
{
pageTextBean.setPageUrl(pageUrl);
pageTextBean.setTitle(seoPageText.getTitle());
pageTextBean.setDescription(seoPageText.getDescription());
pageTextBean.setKeywords(seoPageText.getKeywords());
pageTextBean.setH1Text(seoPageText.getH1Text());
pageTextBean.setTopText(seoPageText.getTopText());
pageTextBean.setH2Text(seoPageText.getH2Text());
pageTextBean.setLeftText(seoPageText.getLeftText());
pageTextBean.setBottomText((seoPageText.getBottomText()));
String url =AnalyseUrl.analyse(pageTextBean.getPageUrl());
if (url!=null)
{
url =url.toLowerCase().trim();
if ("".equals(url)){
if(log.isInfoEnabled()){
log("======分析后结果为空字符!===== 原url串为: "+pageTextBean.getPageUrl());
}
}
else{ SEOPageTexts.SEO_Page_Texts_Map.put(url,pageTextBean );
}
}
}
}
请注意这句 SeoPageTextsBean pageTextBean = new SeoPageTextsBean(); ,我把SeoPageTextsBean对象的生成写在for循环的前边,因为从程序优化的角度考虑,这样可以在程序中只生成一次SeoPageTextsBean的实例,比在for循环中不断地生成新实例的性能上会减少开销,在本机上测试,pageTextBean 在for循环中也有值,就是不知道这样的写法在程序run的时候会不会有什么bug存在,这个请有这方面经验的Jer们给予建议,,希望大家能畅所欲言,各抒已见。
这样是不行的,最后你的map中有不同的url但他们都指向同一个对象(最后的那个对象)
放在外面还是每次都会创建SeoPageTextsBean!
而且是你这样写,程序是有错的!
put到Map里是同一个对象!Map里的所有元素都引用同一个对象!
这是宕机的原因?感觉不应该啊!
[quote] SeoPageTextsBean pageTextBean = new SeoPageTextsBean(); [/quote]
这个是多余的,你不用这个中间对象的,
你是搞维护的吗?????