前两个版本的代码 都或多或少存在一定的问题,虽然可能微乎其微,但是程序需要严谨再严谨,
第一个版本问题: 局限于单机版,依赖于 Jvm的锁
第二个版本问题: 极端情况下,解锁逻辑的问题,线程B的锁,可能会被线程A解掉,这种情况实际上是不合理的。
1. 由于是客户端自己生成过期时间,所以需要强制要求分布式下每个客户端的时间必须同步。
2. 当锁过期的时候,如果多个客户端同时执行jedis.getSet()方法,那么虽然最终只有一个客户端可以加锁,
但是这个客户端的锁的过期时间可能被其他客户端覆盖。
3. 锁不具备拥有者标识,即任何客户端都可以解锁。
版本一: http://www.cnblogs.com/xifenglou/p/8807323.html
版本二: http://www.cnblogs.com/xifenglou/p/8883717.html
所以基于以上问题,第三个版本出来了,
Talk is cheap, show me the code!
import org.springframework.util.StopWatch;
import redis.clients.jedis.Jedis;
import java.util.concurrent.CountDownLatch;
import java.util.concurrent.CyclicBarrier;
import java.util.concurrent.ExecutorService;
import java.util.concurrent.Executors;
/**
* 使用RedisTool.tryGetDistributedLock
* 实现 分布式锁
* 终极版本
*/
public class TicketRunnable3 implements Runnable {
private CountDownLatch count;
private CyclicBarrier barrier;
private static final Integer Lock_Timeout = 10000;
private static final String lockKey = "LockKey";
private volatile static boolean working = true;
public TicketRunnable3(CountDownLatch count, CyclicBarrier barrier) {
this.count = count;
this.barrier = barrier;
}
private int num = 20; // 总票数 此处可随意 写一个数,保证线程能运行起来,真正的共享变量不应该写死在程序中, 应该从redis中获取,这样模拟多进程多线程的并发访问
public void sellTicket(Jedis jedis) {
String name = Thread.currentThread().getName();
try{
boolean getLock = RedisTool.tryGetDistributedLock(jedis,lockKey, name,Lock_Timeout);
if( getLock){
if(!working)
return;
// Do your job
num = Integer.parseInt(jedis.get("ticket"));
if (num > 0) {
num--;
jedis.set("ticket",num+"");
if(num!=0)
System.out.println("================"+Thread.currentThread().getName()+"================= 售出票号" + (num+1)+",还剩" + num + "张票--" );
else {
System.out.println("================"+Thread.currentThread().getName()+"================= 售出票号" + (num+1)+",票已经票完!--");
working = false;
}
}
}else{
//System.out.println();
i