为什么在 Producers 中使用 @Singleton 而不是 @ApplicationScoped？

Question

LoggerProducer.java 是一个用于生成要注入到 CDI bean 中的 Logger 的类：

@Inject 
Logger LOG;

完整代码：

import javax.ejb.Singleton;

/**
 * @author rveldpau
 */
@Singleton
public class LoggerProducer {

    private Map<String, Logger> loggers = new HashMap<>();

    @Produces
    public Logger getProducer(InjectionPoint ip) {
        String key = getKeyFromIp(ip);
        if (!loggers.containsKey(key)) {
            loggers.put(key, Logger.getLogger(key));
        }
        return loggers.get(key);
    }

    private String getKeyFromIp(InjectionPoint ip) {
        return ip.getMember().getDeclaringClass().getCanonicalName();
    }
}

问题：@Singleton 可以安全地变成@ApplicationScoped 吗？

我的意思是，为什么有人要在这里使用 EJB？是否有技术原因，因为不涉及任何事务，并且（AFAIK）无论如何它都是线程安全的？

_{我显然指的是javax.enterprise.context.ApplicationScoped，而不是javax.faces.bean.ApplicationScoped。}

Answer 1

@Singleton 注释不仅提供事务，而且默认提供线程安全。因此，如果您将其替换为@ApplicationScoped，您将失去同步。因此，为了使其正确，您需要这样做：

@ApplicationScoped
public class LoggerProducer {

   private final ConcurrentMap<String, Logger> loggers = new ConcurrentHashMap<>();

   @Produces
   public Logger getProducer(InjectionPoint ip) {
      String key = getKeyFromIp(ip);
      loggers.putIfAbsent(key, Logger.getLogger(key));
      return loggers.get(key);
   }

   private String getKeyFromIp(InjectionPoint ip) {
     return ip.getMember().getDeclaringClass().getCanonicalName();
  }
}

如果你将地图设为静态，你也可以完全没有任何范围