为什么在vertx的Handlebars库中调用Synchronize?

时间:2018-09-19 00:15:30

标签: java asynchronous thread-safety vert.x handlebars.java

我试图从vertx车把库io.vertx.ext.web.templ.handlebars.impl.HandlebarsTemplateEngineImpl class中了解为什么在此方法中使用synchronized块:

@Override
public void render(Map<String, Object> context, String templateFile, Handler<AsyncResult<Buffer>> handler) {
    try {
        int idx = templateFile.lastIndexOf('/');
        String prefix = "";
        String basename = templateFile;
        if (idx != -1) {
            prefix = templateFile.substring(0, idx);
            basename = templateFile.substring(idx + 1);
        }
        Template template = isCachingEnabled() ? cache.get(templateFile) : null;
        if (template == null) {
            synchronized (this) {
                loader.setPrefix(prefix);
                // Strip leading slash from Utils##normalizePath
                template = handlebars.compile(basename);
                if (isCachingEnabled()) {
                    cache.put(templateFile, template);
                }
            }
        }
        Context engineContext = Context.newBuilder(context).resolver(getResolvers()).build();
        handler.handle(Future.succeededFuture(Buffer.buffer(template.apply(engineContext))));
    } catch (Exception ex) {
        handler.handle(Future.failedFuture(ex));
    }
}

请像个白痴一样向我解释!

1 个答案:

答案 0 :(得分:1)

首先,同步问题绝不是“白痴”问题。
我也花了一些时间看这段代码,但仍然不是100%确信它完全正确。

此处synchronized处于阻塞状态的主要原因是为了防止以下两种方法乱序执行:

loader.setPrefix(prefix);
...
template = handlebars.compile(basename);

您会看到,把手为装载机提供了参考:

loader = new Loader(vertx);
...
handlebars = new Handlebars(loader);

没有同步阻止的可能情况是

T1 sets prefix to A and switches
T2 sets prefix to B and switches
T1 compiles template with prefix set to B, while thinking it's still A