Java文件句柄不会关闭

时间:2013-08-22 23:15:43

标签: java file filehandle java.util.logging

之前已经问过这个问题,但问题看起来有点复杂。我已经使用了之前与此相关的所有解决方案。

相关: Freeing Java File HandlesJava 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);
    }
}

此代码旨在成为测试代码。我制作了这个测试文件,这样我就可以弄清楚如何在我拥有的项目中解决这个问题。

我有这个循环的原因是因为问题发生得太快而无法解释。因此循环是模拟它的最佳方式。在我的项目中,有一个日志文件的配置,可以选择放置它的位置。但是如果在重新加载时配置中没有更改文件。它往往会使文件锁定并创建额外的文件每次重新加载

我想让这个工作。如果这开始正常工作。然后我可以在我的项目中正确实现它。

4 个答案:

答案 0 :(得分: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) {

    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   
        }
    }
}

答案 1 :(得分:1)

您正在创建多个文件,因为您正在创建FileHandler并且从不关闭它。

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

修复?

return fh;

最后与否绝对没有区别。在这种情况下,你实际上确实希望在catch块中关闭,因为如果不这样做,任何东西都无法关闭它。

答案 2 :(得分:0)

嗯,有一个问题在这里:

 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语句中关闭文件,只有在出现错误时才关闭它。您应该在完成后立即关闭该文件:

 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;

答案 3 :(得分:0)

使用后,log方法关闭所有处理程序。

    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();
    }