在for循环前只生成一次对象可以吗？

本人目前遇到的一个问题，在公司眼前跑的系统中，有一个项目中设了两个定时器，每天到定时的时候严重影响服务器性能，致使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们给予建议，希望大家能畅所欲言，各抒已见。

评论

4 楼 geeksun 2009-04-27  

另外，检查下是不是pageTexts太大了，如果太大了，则需要开一个单独的线程来进行处理，没处理1000（假设值）就sleep一次
这句作何解释？

3 楼 geeksun 2009-04-27  

多谢上面两位，
# //------------------这段代码可以考虑用BeanUtil代替------------------- 
#     SeoPageTextsBean pageTextBean = new SeoPageTextsBean(); 
#     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())); 
#     //---------------------------------------------------------------
这段程序我现在改为用 BeanUtils来操作啦，现在对程序明白了许多。

2 楼 vlinux 2009-04-24  

看到这样的代码我每次都有想按照自己的风格重新排版的冲动，同时修正了会造成所有Map都引用同一个对象的BUG
可以很明确的说，这段代码如果真的有效率问题，也不会是因为循环语句导致的开销太大
1.检查AnalyseUrl.analyse(pageUrl);
2.检查SEOPageTexts.SEO_Page_Texts_Map.put(url,pageTextBean );
＊3.检查是否是由于所有Map都引用同一个对象造成的

for(TSeoPageTexts seoPageText : pageTexts) {
	String pageUrl = seoPageText.getPageUrl();
	if( null == pageUrl || "".equals(pageUrl) ) {
		continue;
	}//if
	String url = AnalyseUrl.analyse(pageUrl);
	if( null == url ) {
		continue;
	}//if
	url = url.toLowerCase().trim();
	//------------------这段代码可以考虑用BeanUtil代替-------------------
	SeoPageTextsBean pageTextBean = new SeoPageTextsBean();
	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()));
	//---------------------------------------------------------------
	if ("".equals(url)){
		logger.info("AnalyseUrl分析后结果为空字符，原url串为: "+pageUrl);
	} else {
		SEOPageTexts.SEO_Page_Texts_Map.put(url,pageTextBean );
	}//if
}//for

另外，检查下是不是pageTexts太大了，如果太大了，则需要开一个单独的线程来进行处理，没处理1000（假设值）就sleep一次

1 楼 luckaway 2009-04-24  

放在外面还是每次都会创建SeoPageTextsBean！



而且是你这样写，程序是有错的！

put到Map里是同一个对象！Map里的所有元素都引用同一个对象！