使用Java原子变量执行方法调用

问题描述

假设我有一个这样的代码块

if (counter < 100) {
    synchronized (counter)
        if (counter < 100) {
            doSomething();
            counter.incrementAndGet();
        }
    }
}

其中counter是AtomicLong。我将如何将该块转换为不再使用synchronized关键字，并且仍然保持其正确性？还是不可能？

解决方法

这完全取决于您未显示的doSomething。它可能依赖于在同步块中被调用，甚至可能无法并行化。

我认为可能会接受的一种替代方法是

if (counter.getAndIncrement() < 100) {
    doSomething();
}

但是假设doSomething总是抛出异常。在您的代码中，计数器永远不能递增，因此条件将始终解析为true。在上面的示例中，它将被称为前100次，即使它们失败，也将被计为迭代。

简而言之，为了让我们说是否保持正确性，那么您需要非常明确地定义正确性意味着什么。

如果您不知道正确的含义，那么最好还是保留它。