需要同步吗

由于这几张表读取的非常频繁。于是便相对第一次查询的结果进行缓存。如果缓存中没有相应的数据再从数据库中读取。
现的问题，这个方法上要不要考滤同步的问题。
我是这样想的。当同一时刻有多个用户同时查询相同的数据时。此时要查询的数据刚好在缓存中没有。那么这时要从数据库中读取。
假设A查询时执行的到的代码为：List conInfoList = new ArrayList();
而B查询执行到代码为:if(pageConInfoMap.get(riskCode + comCode) != null)
那么现在的问题是A将结果查询出放入缓存。而B又一次将结果查询出放入缓存中。缓存用的是map不会产生重复记录的情况。
但是本来要查询一次数据库，其它的都要从缓存中读取。可是假设的情况却读了两次，在实际运行中，有可不只是两次。所以说我是否应当要取读数据进行缓存的方法设为同步呢。

代码如下:

已经将其设为同步的

public class PageAutoConfigServer {

	private static final Map pageConInfoMap = new HashMap();
	
	public synchronized List getPageConfigInfo(String riskCode, String comCode)
			throws Exception {
		if(pageConInfoMap.get(riskCode + comCode) != null){
			return (List)pageConInfoMap.get(riskCode + comCode);
		}
		
		DbPool dbPool = new DbPool();
		List conInfoList = new ArrayList();
		try {
			dbPool.open(SysConst.getProperty("DDCCDATASOURCE"));
			conInfoList = new DBPageFiledAutoConfig().getPageElementsConfigInfo(riskCode, comCode, dbPool);
			pageConInfoMap.put(riskCode + comCode, conInfoList);
		} catch (Exception e) {
			throw e;
		} finally{
			dbPool.close();
		}

		return conInfoList;
	}

}

 说明此方法是在jsp页面中直接调用的

 

评论

7 楼 xuzhfa123 2009-06-15  

不需要同步

6 楼 taowen 2009-06-15  

加锁有两个效果，一个是同步一个是内存可见性。不知道普通的HashMap能不能保证Thread A写入的值，会被Thread B看到。也许不行，也许行，也许部分行部分不行。至少需要加一个ReadWriteLock吧？

5 楼 taupo 2009-06-15  

但是如果查询出来的数据又被改了，跟新数据库的同时还得更新缓存。。。

所以不是只读或者很少改的数据还是不要缓存了

4 楼 taupo 2009-06-15  

我认为
不需要同步。。。。

只是在把数据加入缓存的时候判断是否在缓存中已经存在，如果存在就不加入

3 楼 ywlqi 2009-06-15  

同步会产生效率问题，我觉得不加同步也可以，毕竟LZ所担心的情况还是少数，即使两次访问数据库也不会产生什么问题。

另外LZ的例子在JSP中是怎么调用的？new PageAutoConfigServer()吗？如果这样并不能真正做到同步，要PageAutoConfigServer 类是单例的才可以，要不就把方法作为static的

2 楼 shadowlin 2009-06-15  

可以用concurrent里面的那个ConcurrentHashMap嘛.

1 楼 taowen 2009-06-15  

private static final Map pageConInfoMap = new HashMap(); 
全局共享的map怎么不需要同步呢？