Отдельный экземпляр переменной для всего приложения в многопотоковой среде

Question

У меня этот код ниже.

private static volatile Properties props = null; 
    private static volatile StanfordCoreNLP pipeline = null; 
    /** 
    * 
    * @return 
    */ 
    static { 
     if (props == null) { 
      props = new Properties(); 
      props.setProperty("annotators", "tokenize, ssplit, parse, sentiment"); 
     } 
     if (pipeline == null) { 
      pipeline = new StanfordCoreNLP(props); 
     } 
    } 

Я хочу иметь один экземпляр переменной props и pipeline по всему моему приложению, которое многопоточные.

Является ли мой код правильным или мне что-то не хватает?

Спасибо.

Answer 1

ваш код кажется правильным, как вы создаете экземпляр во время загрузки класса, поэтому вам нужно только создать добытчик для них

Но я рекомендую использовать одноплодную шаблон в этом случае, как показано ниже

я использовал одноплодный шаблон с двойной блокировкой для многопоточной среды

public class Utility{ 

     private static final Object masterLock = new Object(); 
     private static volatile Properties props = null; 
     private static volatile StanfordCoreNLP pipeline = null; 

     static Properties getPropertiesInstance(){ 

      if(props == null){ 

       synchronized (masterLock) { 
        if(props == null){ 
         props = new Properties(); 
        } 
       } 
      } 

      return props; 
     } 

     static StanfordCoreNLP getStanfordCoreNLPInstance(){ 

      if(pipeline == null){ 

       synchronized (masterLock) { 
        if(pipeline == null){ 
         pipeline = new StanfordCoreNLP(); 
        } 
       } 
      } 

      return pipeline; 
     } 
} 

Answer 2

можно избавиться от volatile с помощью вызова метода и по-прежнему сохраняет безопасность потока через статический initializa Тион.

private static final Properties props = initProperties(); 
private static Properties initProperties() { 
    Properties props = new Properties(); 
    props.setProperty("annotators", "tokenize, ssplit, parse, sentiment"); 
    return props; 
} 

public static Properties getProperties() { 
    return props; 
} 

Edit: Чтобы ответить на ваш вопрос, хотя, да код в OP действительно Потокобезопасная, хотя то, что я дал, прежде чем это способ, которым я хотел бы сделать это лично.

Answer 3

volatile не требуется в вашем случае. Java гарантирует, что статическая инициализация будет правильно синхронизирована. Также нет необходимости в вызове метода. Также см. Are Java static initializers thread safe? и https://www.cs.umd.edu/~pugh/java/memoryModel/jsr-133-faq.html