JAVA-synchronized锁的使用误区

JAVA-synchronized锁的使用误区

最近在处理手头项目时，接到leader通知说，某个加了锁的方法总是拿到空值，然后导致程序报错。看了日志发现是，某个线程private的concurrentHashmap对象变为空导致的。查看该类的函数，发现：
1 有一个函数负责清空该map，并重新从数据库读取值并装填。
2 有一个函数负责返回该map内的所有values。

synchronized(cars){
				cars.clear();
			/*
			some code here to refill the map
			*/
				}
			}

另一个读取map的方法

synchronized(cars){
			return cars.values();
		}

乍看之下似乎没有问题，但是线上确实有问题，那么这段代码就肯定有问题。
查了查synchronized关键字的使用方法，发现有以下的说明：

原来return会直接释放锁，因此压根锁不住这个对象，导致多线程可以同时操作该对象：
A线程访问读值方法，B线程访问重填方法。看起来似乎在遇到synchronized关键字的时候只有一个可以获得访问权，但是读值方法拿到锁之后立马释放了，在调用cars.values()之前，B线程可以获取cars对象，并成功清空。由于此时A调用已经没有锁检测了，A在此时获取values自然会拿到空。

简单的解决方法

1 直接对读map的函数用synchronized关键字声明：这个函数本身内部就只是对cars加锁并返回，对函数加锁的效果是一样的，不过性能可能有差异。
2 在函数内声明对象a，在锁内部读取map值并赋值给a，最后返回a。