java - 跳出同步块(synchronized block)？

Question

我有这个:

synchronized(c){
   if (!items.hasNext() && (c.getFinishedCount() == c.getStartedCount())) {
      f.done(null, results);
      return;
    }

    if(c.isBelowCapacity()){
      RunMap(items,m,results,c,s,f);
    }
}

但最终当我调用 RunMap() 或 f.done() 时我不需要在 c 上同步，我只需要在 c 上同步当我执行条件检查/if 语句时。

我是否应该让 c 上的方法同步，然后摆脱同步块(synchronized block)？

Answer 1

Should I just make the methods on c synchronized, and then get rid of the synchronized block?

不，更好的方法是在同步块(synchronized block)中进行检查，然后再做剩下的工作:

boolean bFDone = false;
boolean bRunMap = false;

synchronized (c) {
    bFDone = !items.hasNext() && (c.getFinishedCount() == c.getStartedCount());
    bRunMap = !bFDone && c.isBelowCapacity();
}

if (bFDone) {
    f.done(null, results);
} else if (bRunMap) {
    RunMap(items,m,results,c,s,f);
}

这个版本的一大优势是，所有可变代码实际上是同步的，其余(可能很慢)的东西稍后发生。

正如对此答案的评论中正确提到的那样，如果您的初始假设是正确的，即整个 block 不需要同步，这只会正常工作。如果您需要立即对条件使用react，那么该 block 实际上需要(完全)同步。