Файлы Java-файлов не закрываются

Question

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

Касается:Freeing Java File Handles , Java keeps file locks no matter what

package me.test; 

import java.io.File; 
import java.util.logging.FileHandler; 
import java.util.logging.Formatter; 
import java.util.logging.Level; 
import java.util.logging.LogRecord; 
import java.util.logging.Logger; 

public class Test { 
    Logger log = Logger.getAnonymousLogger(); 
    FileHandler handle; 

    final static String newline = System.lineSeparator(); 
    /** 
    * @param args 
    */ 
    public static void main(String[] args) { 
     Test t = new Test(); 
     t.run(); 
    } 
    public void run() 
    { 
     for (int i = 0; i < 6; i++) { 
      testLogs(); 
      change(); 
     } 
     testLogs(); 
     if (handle != null) 
     { 
      handle.close(); 
      log.removeHandler(handle); 
     } 
    } 
    public static FileHandler craftFileHandler(File file, boolean append) 
    { 
     if (file == null) 
      return null; 
     FileHandler fh = null; 
     try { 
      fh = new FileHandler(file.getPath(), append); 
      fh.setFormatter(new Formatter() { 

       @Override 
       public String format(LogRecord record) { 
        return "[test] " + "[" + record.getLevel().toString() + "]" + String.format(record.getMessage(), record.getParameters()) + newline; 
       } 
      }); 
      return new FileHandler(file.getPath(), append); 
     } catch (Exception e) { 
      if (fh != null) 
       fh.close(); 
      return null; 
     } 
    } 

    public void change() 
    { 
     if (handle != null) 
     { 
      handle.flush(); 
      handle.close(); 
      log.removeHandler(handle); 
     } 
     handle = null; 
     File f = new File("log.log"); 
     handle = craftFileHandler(f, true); 
     System.out.println(f.getAbsolutePath()); 
     if (handle != null) 
      log.addHandler(handle); 
    } 
    public void testLogs() 
    { 
     if (log == null) 
     { 
      log = Logger.getLogger("test"); 
      log.setLevel(Level.ALL); 
     } 
     log.info("This is info #1"); 
     log.warning("Warning 1"); 
     log.info("meh info again."); 
     log.severe("SEVERE HELL YA NICE TEST"); 
     log.info("You sure its good here?"); 
     log.info("Handler count " + log.getHandlers().length); 
    } 
} 

Этот код предназначен быть тестовый код. Я сделал этот тестовый файл, чтобы понять, как исправить эту проблему в проекте, который у меня есть.

Причина, по которой я столкнулся, состоит в том, что проблема возникает слишком быстро, чтобы объяснить. Таким образом, цикл был лучшим способом имитировать его. В моем проекте есть конфиг для файла журнала, чтобы выбрать, куда его поместить. Но если файл не изменен в config при его перезагрузке. Он имеет тенденцию оставлять файл заблокированным и создавать дополнительные файлы. EVERY reload

Я хотел бы получить эту работу. Если это нормально работает. Тогда я могу правильно реализовать его в своем проекте.

Answer 1

Вы получаете несколько файлов, созданных, потому что вы создаете FileHandler и никогда закрывая его.

fh = new FileHandler(file.getPath(), append); 
... 
return new FileHandler(file.getPath(), append); 

Исправить?

return fh; 

В заключение или нет абсолютно никакой разницы. В этом случае вы действительно хотите закрыться в блоке catch, поскольку ничего не сможете закрыть, если вы этого не сделаете.

Answer 2

Ну, один вопрос здесь:

try { 
      fh = new FileHandler(file.getPath(), append); 
      fh.setFormatter(new Formatter() { 

       @Override 
       public String format(LogRecord record) { 
        return "[test] " + "[" + record.getLevel().toString() + "]" + String.format(record.getMessage(), record.getParameters()) + newline; 
       } 
      }); 
      return new FileHandler(file.getPath(), append); 
     } catch (Exception e) { 
      if (fh != null) 
       fh.close(); 
      return null; 

Вы никогда не закрываем файл в заявлении попробовать, только закрыв его, если есть ошибка. Вы должны закрыть файл, как только вы сделали с ним:

try { 
      fh = new FileHandler(file.getPath(), append); 
      fh.setFormatter(new Formatter() { 

       @Override 
       public String format(LogRecord record) { 
        return "[test] " + "[" + record.getLevel().toString() + "]" + String.format(record.getMessage(), record.getParameters()) + newline; 
       } 
      }); 
      //close fh 
      fh.close(); 
      return new FileHandler(file.getPath(), append); 
     } catch (Exception e) { 
      if (fh != null) 
       fh.close(); 
      return null; 

Answer 3

Всегда близкие ресурсы внутри наконец блок-

try { 
    fh = new FileHandler(file.getPath(), append); 
    fh.setFormatter(new Formatter() { 

    @Override 
    public String format(LogRecord record) { 
     return "[test] " + "[" + record.getLevel().toString() + "]" + String.format(record.getMessage(), record.getParameters()) + newline; 
    } 
    }); 
    return new FileHandler(file.getPath(), append); 
} catch (Exception e) { 

    return null; 
    // never close in catch 

} finally { 
    // lastly close anything that may be open 
    if (fh != null){ 
     try { 
      fh.close(); 
     } catch (Exception ex){ 
      // error closing 
     } 
    } 
} 

Answer 4

После использования метод журнала закрывает все обработчики.

this.logger.log(Level.SEVERE, (exception.getClass().getName() + ": " + exception.getMessage()) + "\r\n" + exception.getCause() + "\r\n" + "\r\n"); 

    for (Handler handler : this.logger.getHandlers()) 
    { 
     handler.close(); 
    }