使用同步而登錄

Question

在我的應用程序已經寫了使用的java.util.logging

import java.io.IOException; 
import java.io.InputStream; 
import java.util.Properties; 
import java.util.logging.FileHandler; 
import java.util.logging.Level; 
import java.util.logging.SimpleFormatter; 



public class Logger { 

    public final static String PROPERTIES_FILE = "Logger.properties"; 
    private static java.util.logging.Logger logger = null; 

    private static void initialize() { 
     try { 
      logger = java.util.logging.Logger.getLogger(Logger.class.getName()); 
      FileHandler fh = new FileHandler("D:\\MyLogFile.log", true); 
      logger.addHandler(fh); 
      logger.setLevel(Level.ALL); 
      SimpleFormatter formatter = new SimpleFormatter(); 
      fh.setFormatter(formatter); 
      logger.log(Level.INFO, "Test Message Logged"); 

     } 
     catch (IOException e) { 
      System.out.println("Warning: Unable to read properties file " + 
          PROPERTIES_FILE); 
     } 
     } 

    public static synchronized java.util.logging.Logger getLogger(String name) 
    { 
     if(logger==null) 
     { 
     Logger.initialize(); 
     } 
     logger.getLogger(name); 
     return logger; 
    } 


} 

我自己的日誌記錄工具是否需要使用同步進行getLogger方法？請提出您的意見。 （此代碼在多線程環境中運行）

Answer 1

我同意懶惰初始化在這裏似乎不必要的其他評論者。初始化變量記錄最簡單的方法是在靜態初始化，這是保證僅在類加載時執行一次：

public class Logger { 

    public final static String PROPERTIES_FILE = "Logger.properties"; 
    private static java.util.logging.Logger logger = null; 

    private static void initialize() { 
     try { 
      logger = java.util.logging.Logger.getLogger(Logger.class.getName()); 
      FileHandler fh = new FileHandler("D:\\MyLogFile.log", true); 
      logger.addHandler(fh); 
      logger.setLevel(Level.ALL); 
      SimpleFormatter formatter = new SimpleFormatter(); 
      fh.setFormatter(formatter); 
      logger.log(Level.INFO, "Test Message Logged"); 

     } 
     catch (IOException e) { 
      System.out.println("Warning: Unable to read properties file " + 
          PROPERTIES_FILE); 
     } 
    } 

    static { 
     initialize(); 
    } 

    public static java.util.logging.Logger getLogger(String name) 
    { 
     logger.getLogger(name); 
     return logger; 
    } 


} 

然而，您可以避免與double-checked locking大多數同步。

public class Logger { 

    // note: volatile is required 
    private volatile static java.util.logging.Logger logger = null; 

    //... 

    public static java.util.logging.Logger getLogger(String name) 
    { 
     if(logger==null) 
     { 
      synchronized(Logger.class) 
      { 
       if(logger == null) 
       Logger.initialize(); 
       } 
      } 
     } 
     logger.getLogger(name); 
     return logger; 
    } 
} 

事實上，在你的情況，我認爲你能避免同步完全如果你重寫你的初始化函數，以便它之前完全將其分配到（揮發性）類變量配置中的局部變量的記錄：

private volatile static java.util.logging.Logger logger = null; 
private static void initialize() { 
    try { 
     Logger logger = java.util.logging.Logger.getLogger(Logger.class.getName()); 
     FileHandler fh = new FileHandler("D:\\MyLogFile.log", true); 
     logger.addHandler(fh); 
     logger.setLevel(Level.ALL); 
     SimpleFormatter formatter = new SimpleFormatter(); 
     fh.setFormatter(formatter); 
     logger.log(Level.INFO, "Test Message Logged"); 

     Logger.logger = logger; 
    } 
    catch (IOException e) { 
     System.out.println("Warning: Unable to read properties file " + 
         PROPERTIES_FILE); 
    } 

    public static java.util.logging.Logger getLogger(String name) 
    { 
     if(logger==null) 
     { 
     Logger.initialize(); 
     } 
     logger.getLogger(name); 
     return logger; 
    } 
} 

這有一個潛在的有初始化（）執行幾次，但我不認爲你不在乎，只要每一個getLogger調用都會有一個有效的記錄器實例，即使該實例變化。

Answer 2

是的，您需要​​那裏。首先要避免多次撥打initialize()，其次要讓logger更改對其他線程可見。

這提出了一個問題：爲什麼你不能熱切地註冊initialize()並避免同步？

Answer 3

我不會推薦它，因爲你會招致同步的開銷在每次調用getLogger()時，你只需要初始化一次。我應該這樣做，而不是：

private static boolean initialized = false; 

private static synchronized void initialize() { 
    try { 
     if(!initialized) 
     {    
     // Do initialization 
     initialized = true; 
     } 

    } 
    catch (IOException e) { 
     System.out.println("Warning: Unable to read properties file " + 
         PROPERTIES_FILE); 
    } 
    } 

public static java.util.logging.Logger getLogger(String name) 
{ 
    if(logger==null) 
    { 
    Logger.initialize(); 
    } 
    logger.getLogger(name); 
    return logger; 
} 

這樣，如果第二個線程進來，而第一線仍處於初始化（），第二個線程將阻止，因爲在初始化同步的（）。一旦第一個線程完成初始化，第二個線程繼續，看到初始化爲真，不會重新初始化，然後繼續。