菜鸟写的代码如何优化性能太慢。

0 0

菜鸟写的代码如何优化性能太慢。5

public Paginate getBookInfoByPublishDateAndPublisher(Paginate paginate,
			String unionSiteID,String publisher, Date fromDate, Date toDate, String bookName,
			Double price1, Double price2, String vendor, String bookCategory) {
		// TODO Auto-generated method stub
		StringBuffer whereCause =new StringBuffer( "not exists(select l from UnionSiteProductList l where l.unionSiteID=? and b.bookID =l.productID)");
		Vector paras = new Vector();
		paras.add(unionSiteID);
		if (publisher != null && publisher.trim().length() > 0) {
			whereCause.append( (whereCause.length() > 0 ? " and " : "")
					+ "b.publisherName like ?");
			paras.add("%" + publisher + "%");
		}
		if (fromDate != null) {
			whereCause.append( (whereCause.length() > 0 ? " and " : "")
					+ "b.publishDate >= ?");
			paras.add(fromDate);
		}
		if (toDate != null) {
			whereCause.append( (whereCause.length() > 0 ? " and " : "")
					+ "b.publishDate <= ?");
			paras.add(toDate);
		}
		if (bookName != null && bookName.trim().length() > 0) {
			whereCause.append((whereCause.length() > 0 ? " and " : "")
					+ "b.bookName like ?");
			paras.add("%" + bookName + "%");
		}
		if (price1 != null) {
			whereCause.append( (whereCause.length() > 0 ? " and " : "")
					+ "pp.productPrice >= ?");
			paras.add(price1);
		}
		if (price2 != null) {
			whereCause.append( (whereCause.length() > 0 ? " and " : "")
					+ "pp.productPrice <= ?");
			paras.add(price2);
		}
		if (vendor != null && vendor.trim().length() > 0) {
			whereCause.append((whereCause.length() > 0 ? " and " : "")
					+ "v.vendorName like ?");
			paras.add("%" + vendor + "%");
		}
		if (bookCategory != null && bookCategory.trim().length() > 0) {
			whereCause.append( (whereCause.length() > 0 ? " and " : "")
					+ "bcv.bookCategoryName like ?");
			paras.add("%" + bookCategory + "%");
		}
		String hql = "select b,pp,v,bcv from Book b,ProductItem i,Product p, "
				+ "ProductPrice pp,Vendor v,BookCategoryView bcv "
				+ " where b.bookID=i.entityID and b.deleteFlag=0 and b.flag=4 "
				+ " and i.entityType=1 and i.deleteFlag=0"
				+ " and i.productID=p.productId and p.productType=1 and p.deleteFlag=0 and p.isValid=1"
				+ " and p.productId=pp.productId and pp.deleteFlag=0"
				+ " and b.vendorID = v.vendorID "
				+ " and b.bookCategoryCode=bcv.bookCategoryCode ";		


		if (whereCause.length() > 0) {
			hql = hql + " and " + whereCause;
		}
		paginate = this.pagedQuery(hql, paginate, paras.toArray());
		return paginate;
	}

半年javaweb开发，接受拍砖
性能很慢，请各位帮忙看下如何优化。小弟在此谢过了

职场话题

2011年5月03日 11:09

hekuilove
1
0 0 3

6个答案按时间排序按投票排序

0 0

最好把StringBuffer改成StringBuilder
其他的貌似没有影响性能的问题，再看一下这段代码放的位置吧

2011年5月08日 12:16

jackzlz
5
0 0 2

添加评论

0 0

stringbuffer 使用的时候应该先开辟空间否则直接使用string去累加好了,
我以前总结的一点点代码可读性的东西：
http://evn.iteye.com/blog/866171
这样的代码性能上不会有太大问题的, 性能问题要从大方面开始考虑然后再考虑这些细节
借助代码性能工作我也有总结过,可以去看看.
一定是从大的开始解决然后解决小的代码问题

2011年5月07日 01:50

chenxiang105
2039
0 0 7

添加评论

0 0

另外，whereCause.length() 会小于等于0么……

2011年5月03日 22:08

jianpc
45
0 0 1

添加评论

0 0

whereCase的时候用StringBuffer了，为什么String hql = xxxx的时候不用呢？另外，如果几行字符串的拼装就能让你明显感觉性能下降了，那么这个系统问题真就大了（虽然字符串的操作开销很大，但我不认为楼主这个系统存在这个问题），所以还是从别的地方分析下影响性能的原因，例如sql语句本身。

2011年5月03日 22:05

jianpc
45
0 0 1

添加评论

0 0

这怎么设计OO的，hibernate给初学者，真是害死人了，配置好映射关系就不需要写关联查询了，会自动生成

# String hql = "select b,pp,v,bcv from Book b,ProductItem i,Product p, "
#                 + "ProductPrice pp,Vendor v,BookCategoryView bcv "
#                 + " where b.bookID=i.entityID and b.deleteFlag=0 and b.flag=4 "
#                 + " and i.entityType=1 and i.deleteFlag=0"
#                 + " and i.productID=p.productId and p.productType=1 and p.deleteFlag=0 and p.isValid=1"
#                 + " and p.productId=pp.productId and pp.deleteFlag=0"
#                 + " and b.vendorID = v.vendorID "
#                 + " and b.bookCategoryCode=bcv.bookCategoryCode ";

这个hql改成sql吧

2011年5月03日 16:58