是否可以直接使用上次计算的结果？ Java 同步块

Question

为什么Synchronized block在连续两次请求分解同一个值时可以直接使用之前的计算结果？

Java Concurrency in Practice 2-8 的代码

    public class two_8_CachedFactorizer  implements Servlet {
    @GuardedBy("this") private BigInteger lastNumber;
    @GuardedBy("this") private BigInteger[] lastFactors;
    @GuardedBy("this") private long hits;
    @GuardedBy("this") private long cacheHits;

    public synchronized long getHits(){return hits;}
    public synchronized double getCacheHitRatio(){
        return (double) cacheHits/(double)hits;
    }
    @Override
    public  void service(ServletRequest req, ServletResponse res) throws ServletException, IOException {
        BigInteger i=extractFromRequest(req);
        BigInteger[] factors=null;
        synchronized (this) {
            ++hits;
            if (i.equals(lastNumber)) {
                ++cacheHits;
                factors = lastFactors.clone();
            }
        }
            if (factors==null)
            {
                factors=factor(i);//question in here ：Suppose two identical requests arrive here at the same time
                synchronized (this)
                {
                    lastNumber=i;
                    lastFactors=factors.clone();
                }
            }
            encodeIntoResponse(res,factors);
        }

}

Answer 1

我相信您手头上至少有一个比赛条件。可能是较早发布的factor(i) 在稍后发布的factor(i) 之后完成；覆盖后来发布的factor(i) 的值。所以 lastFactor 和 lastNumber 可以更新到较早的版本。为了解决这个问题，我会在最后一个同步块中添加一些检查。

在当前代码中，您可以让多个线程执行相同的计算。我不确定这是否可取。