最近利用tair做分布式锁的时候,在注入NS的时候发现一个bug,具体是这样的:
@Component
public class SyncLock implements DisposableBean, DistributedLock {
private static final Logger logger = LoggerFactory.getLogger(SyncLock.class);
private static final int VERSION = 0xffff;
private static final int EXPIRE = 60*60;
private static final List<String> KEYS = new CopyOnWriteArrayList<>();
@Value("${tair.namespace}")
private static int NS;
@Autowired
private TairManager tairManager;
/**
* @param key 名称
* @param expireTime 秒数
* @return 是否成功获取锁
*/
@Override
public boolean tryLock(String key, long expireTime, TimeUnit timeUnit) {
ResultCode rc = tairManager.put(NS, key, 0, VERSION, (int) timeUnit.toSeconds(expireTime));
if (ResultCode.SUCCESS.equals(rc)) {
//lock获取成功
KEYS.add(key);
return true;
} else if (ResultCode.VERERROR.equals(rc)) {
//lock获取失败,已经被别人获取,可能需要等会儿再获取
logger.info("lock获取失败,已经被其他线程获取.........");
return false;
} else {
//其他错误
logger.info("tryLock 获取失败,resultCode:", rc);
return false;
}
}
测试过程中发现,一直返回获取锁失败,debug发现原来是NS=0,原来是注入失败,怎么会注入失败呢?
为了再次确认,我在类实例化之前打印出了NS的值,发现NS确实没有注入进来。奇怪,其它地方都注入进来了,怎么注入失败呢?
@PostConstruct
public void postConstruct(){
System.out.println("NS={}" + NS);
}
对比发现这里分布式锁使用的NS成员变量是static静态成员变量,相当于一个类变量,类变量在类加载的时候就初始化了。而@value默认是set方法注入,是实例化对象过程中注入的。
所以这里static静态变量没有注入。解决办法也很简单:
1.使用成员变量去掉static
@Value("${tair.namespace}")
private int NS;
2.使用静态变量的话,@value注解加在set方法上面
private static int NS;
@Value("${tair.namespace}")
public void setNS(int NS) {
this.NS = NS;
}