码迷,mamicode.com
首页 > 移动开发 > 详细

[android]同步块代码中wait和sleep方法持有锁状态的区别

时间:2015-03-31 12:54:59      阅读:2539      评论:0      收藏:0      [点我收藏+]

标签:android   sleep   wait   

需求

最近项目中这么一个功能,希望app监听android file system,当有文件create/modify/detele时,app收到通知,把对应的变化信息告诉服务器


需求的解决方案

当我们用手机拍照时,会把拍照的文件保存在sdcard的DCIM文件夹下。当文件生成时,我们会接收到create/modify/write_close消息,我们要做的是在write_close之后,再发送消息给服务器,这样一来,当我们收到create消息时,就需要把当前线程阻塞住,直到收到write_close消息之后再调用notifyAll来开启阻塞的线程。


问题

问题出在:我们在接收到write_close消息之后,虽然调用了setState(true),但是isChange的值并没有马上改变,而是过了好几秒之后才被改变


来看看核心代码

while (count > 0) {  //阻塞线程的代码,一下代码简称:循环
				
				synchronized (this) {
					if (!isChange) { //当isChange不变为true时,一直等待,直到接收到write_close消息,调用setState(true)
						try {
							Log.d(LOG_TAG, "sleeping ......");
							this.wait(1000);  //这里原先调用的是TimeUnit.MILLISECONDS.sleep(1000);
						} catch (InterruptedException e) {
							e.printStackTrace();
						}
						continue;
					}
				}

				//...省略代码sendMsg2Server();结束后break
			}

public synchronized void setState(boolean isChange){
			this.isChange = true;
			this.notifyAll();//原来是没有加这句,在改成wait(1000)之后,调用notifyAll()
		}


问题解决方案

把TimeUnit.MILLISECONDS.sleep(1000)改成wait(1000),并且在setState(true)里notifyAll()。


问题的根源

在if(!isChange)和setState()里都用了同步块,同步锁是this。当我们调用sleep(1000)时,循环并没有交出同步锁,而是持有锁直到1秒sleep完成,这时setState()和循环会竞争这个this锁,setState()没有竞争到这个锁是很正常的,只要setState()没有竞争到锁,循环就会sleep 1秒,这样就会造成打印了好几句Log.d(LOG_TAG, "sleeping ......");之后才改变isChange的值的情况。


当我们把代码改成wait(1000)之后,循环马上把锁交了出去,setState()拿到锁,马上改变isChange的值,并且通知循环,这样就能进入后面sendMsg2Server()的代码了

[android]同步块代码中wait和sleep方法持有锁状态的区别

标签:android   sleep   wait   

原文地址:http://blog.csdn.net/vic_fang/article/details/44775179

(0)
(0)
   
举报
评论 一句话评论(0
登录后才能评论!
© 2014 mamicode.com 版权所有  联系我们:gaon5@hotmail.com
迷上了代码!