分析现在公司AtomicLong应用代码

最近刚换工作，项目组人员很忙，没人理我，无事闲看代码，看的第一段代码，发现很有意思，特记录下来；

/**
	 * 获取流水号
	 * 
	 * @return
	 * @throws SerialGenneratorException 
	 */
	protected String getSerial() throws SerialGenneratorException {

		synchronized (serial) {//serial是AtomicLong类型
			long sn = serial.getAndIncrement();
			if (sn + 1 > this.getLimit()) {//如果达到上限，则更新上限
				serial.set(this.allocate());//更新上限
				sn = serial.getAndIncrement();
			}
			// System.out.println(Thread.currentThread().getId()+"------"+sn);
			return seiralFormat(sn, length);
		}
	}

自我分析后，很明显，改成以下代码，效率会成倍提高

	protected String getSerial() throws SerialGenneratorException {
		long sn = serial.getAndIncrement();
		if (sn + 1 > this.getLimit()) {// 如果达到上限，则更新上限
			synchronized (serial) {// serial是AtomicLong类型
				if (sn + 1 > this.getLimit()) {// 如果仍然达到上限，则更新上限
					serial.set(this.allocate());// 更新上限
					sn = serial.getAndIncrement();
				}
			}
		}
		return seiralFormat(sn, length);
	}

评论

11 楼 hnist0603 2018-05-10  

锋之弥漫 写道

itao 写道

AtomicLong等类在实现同步时,没有用synchronized关键字,而是直接使用了最低层(本地c语言实现代码)来完成的。所以他本身是同步的，个人认为，不需要使用synchronized关键字就可以。

赞同2楼的说法，因为既然AtomicLong已经是线程安全类，那么对于serial来说就是线程安全的，这是一个复合操作，其他对象在访问serial时是线程安全的，原子性能保证getSerial方法的返回值也是线程安全的，因为该方法内的操作都是对serial对象的操作，既然它是线程安全的因此这个方法内的操作要么都做，要么都不做，原子性能保证它的线程安全，所以没有必要使用synchronized。假设需要的话，那就没有必要使用AtomicLong了，用Long就可以了啊，因为可以用synchronized包起来保证线程安全

光保证AtomicLong的状态正确还不行，这里还用了this.allocate()， 如果两个线程同时去allocate()的话会造成浪费和不必要，甚至发生业务上的错误，所以这里还是需要synchronized来同步检查的，博主这样改是正确的

10 楼 angole 2015-01-02  

代码2之所以性能比代码1高，是因为避免了很多不必要的同步等待。
代码2不满足if条件的可以直接跳转了，而代码1实现等待，再判断。
另外虽然AtomicLong类是线程安全的，但多个原子操作的复合操作就未必是线程安全的。
所以代码中的synchronized还是必要的。

9 楼 MrLee23 2014-07-18  

建议直接在方法名上直接使用synchronized，这样保证方法内所有的操作都是安全的，还能给调用者一个提示，为此方法是安全的。

8 楼 wzx8118 2014-01-22  

我看到原子量要和lock一块用，更安全，不知道对不对

7 楼 锋之弥漫 2013-05-02  

itao 写道

AtomicLong等类在实现同步时,没有用synchronized关键字,而是直接使用了最低层(本地c语言实现代码)来完成的。所以他本身是同步的，个人认为，不需要使用synchronized关键字就可以。

赞同2楼的说法，因为既然AtomicLong已经是线程安全类，那么对于serial来说就是线程安全的，这是一个复合操作，其他对象在访问serial时是线程安全的，原子性能保证getSerial方法的返回值也是线程安全的，因为该方法内的操作都是对serial对象的操作，既然它是线程安全的因此这个方法内的操作要么都做，要么都不做，原子性能保证它的线程安全，所以没有必要使用synchronized。假设需要的话，那就没有必要使用AtomicLong了，用Long就可以了啊，因为可以用synchronized包起来保证线程安全

6 楼 wml199039 2012-06-15  

itao 写道

AtomicLong等类在实现同步时,没有用synchronized关键字,而是直接使用了最低层(本地c语言实现代码)来完成的。所以他本身是同步的，个人认为，不需要使用synchronized关键字就可以。

原子方法正是从指令层（并不是二楼所说的C语言）来解决同步问题的，
if (sn + 1 > this.getLimit()) 这句必须放在synchronized块中
因为可能会有两个线程“同时”执行该语句，这样就会导致业务错误

5 楼 wml199039 2012-06-15  

原子方法正是从指令层（并不是二楼所说的C语言）来解决同步问题的，
if (sn + 1 > this.getLimit()) 这句必须放在synchronized块中
因为可能会有两个线程“同时”执行该语句，这样就会导致业务错误

4 楼 XINRUIBAOBAO 2012-06-08  

不赞同，2楼的意见，这种先检查再运行的代码最好还是要synchronized一下

3 楼 jack547155187 2012-03-29  

赞成2楼的意见

2 楼 itao 2012-03-14  

AtomicLong等类在实现同步时,没有用synchronized关键字,而是直接使用了最低层(本地c语言实现代码)来完成的。所以他本身是同步的，个人认为，不需要使用synchronized关键字就可以。

1 楼 zhou2008gang 2012-01-11  

如果是多线程来访问你的程序，会不会有问题？long sn = serial.getAndIncrement();  
这个有没有同步？