巧妙地解决Log4j文件重命名失败的问题
2014年07月06日


一、问题描述

1、大家在用log4j时,可能会遇到下面的一个报错信息:

Failed to rename [/opt/proj.log] to [/opt/proj.log.2014-03-20].

这个算是log4j.DailyRollingFileAppender的一个BUG,网上很多修改方案,不过我能搜到的,都不理想


2、一般性问题:文件重命名失败

详细的内容可以参见论坛的帖子:http://bbs.csdn.net/topics/330195396

关键内容如下:

我也遇到了和LZ相同的问题,也是无法重命名,但是可以先重命名,然后再执行其他操作!如果先执行其他的操作,比如读这个文件,然后再修改这个文件名字,就无法重命名了,我先把代码贴出来了,大家帮忙了啊!!

File file = new File(fileName);
File newFile = new File(fileName.replace("2008", "2011"));

BufferedReader bReader = new BufferedReader(new FileReader(file));
PrintWriter pWriter = new PrintWriter(newFile.getPath());
String linesString = "";
if (!file.exists()) {
	throw new RuntimeException("文件不存在");
}else{
	while((linesString = bReader.readLine())!=null){
	System.out.println(linesString.replace("2008-", "2009-"));
	}
	bReader.close();
	pWriter.close();			
}
if(file.renameTo(newFile)){
	System.out.println("Rename file succeed");
}else {
	System.out.println("Rename file failed");
}

你这个只是单纯的为文件重命名,没进行其他的操作,现在的问题是,进行了其他的读写流之类的操作,然后重命名操作失败了!只是单纯的重命名,大家都能成功的!!



二、解决方案

在我写这篇文章之前,还没有真正地有效解决这个问题。

我的解决方案是这样的:

第一步:

先把原文件copy一份,copy的那份就命名为要重命名的文件。

比如proj.log,我拷贝一份,命名成proj.2014-03-20.log


第二步:

本来我的思路是这样的:删除原proj.log,然后再新建一个proj.log

但是我发现,原proj.log文件不但无法重命名,而且无法删除。


所以,我换了种思路:直接打开proj.log的输入流,然后把它的文件内容清空


总结一下,变相的文件重命名思路:直接上代码,参见注释。

    File target = new File(newFilename);
    if (target.exists()) { //新文件若存在,则删掉
      target.delete();
    }
    File file = new File(fileName); // 旧文件
    boolean result = file.renameTo(target); //将旧文件更名
    if(result) {
      LogLog.debug(fileName +" -> "+ newFilename);
    } else { //更名失败,则采取变相的更名方法
        try{
            FileUtils.cloneFile(file, target); // 将旧文件拷贝给新文件
            LogLog.debug(fileName +" -> "+ newFilename);
            //清空旧文件
            emptyFileContent(file);
        } catch (IOException e) {
            LogLog.error("Failed to rename ["+fileName+"] to ["+newFilename+"].");
        }
    }
	
    /** 清空文件内容 */
    public static void emptyFileContent(File file) {
        FileOutputStream out = null;
        try {
            out = new FileOutputStream(file);
            out.write(new byte[0]);
        } catch (Exception e) {
            LogLog.error("Can't not empty " + file.getName());
        } finally {
            IOUtils.close(out);
        }
    }