2016-10-04 635 views
2

此代码:FileAlreadyExistsException抛出,并且重写文件

File tmpFile = File.createTempFile(PREFIX_TMP, null, new File(reportPath)); 
     logger.debug("The report will be saved in the temporary file '{}'", tmpFile.getName()); 

     reportWriter.write(tmpFile, report); 

     Calendar calendar = Calendar.getInstance(); 

     boolean isFileMoved = false; 
     while (!isFileMoved) { 
      String reportName = String.format(report.getName(), calendar.getTime()); 
      File reportFile = new File(reportPath, reportName); 

      try { 
       Files.move(tmpFile.toPath(), reportFile.toPath()); 
       isFileMoved = true; 

       logger.info("The temporary file '{}' is renamed to '{}'", tmpFile.getName(), reportFile.getName()); 
      } catch (FileAlreadyExistsException e) { 
       logger.warn("The file '{}' already exists in the folder '{}': adding a second to generation time", 
         reportName, reportPath); 
      } 

      calendar.add(Calendar.SECOND, 1); 
     } 

生成以下,是不可能的,登录的语句:

2016年10月4日10:35:11674 [WARN] [ _Executor-1] abcMyGenerator - 的 文件 'myFile_01092016103511.csv' 已经存在的文件夹 '/ MYDIR' 中:添加第二代时间

2016年10月4日10:35:11677 [WARN ] [_Executor-3] abcMyGenerator - 的 文件 'myFile_01092016103511.csv' 已经存在的文件夹 '/ MYDIR' 中:添加第二代时间

2016年10月4日10:35:11677 [INFO ] [_Executor-1] abcMyGenerator - 的 临时文件 'tmp2103892505851032137.tmp' 被重命名为 'myFile_01092016103512.csv'

2016年10月4日10:35:11680 [INFO] [_Executor-3] ABC MyGenerator - 将 临时文件'tmp6843688962754506611.tmp'更名为 'myFile_01092016103512.csv'

Executor 3重写该文件,即使它应该抛出FileAlreadyExistsException。

没有抛出异常,并且文件的数据已被覆盖。

回答

1

Executor 3覆盖文件,即使它应该抛出FileAlreadyExistsException。

如果您希望Files.move(...)以原子方式操作,则需要使用ATOMIC_MOVE option。默认情况下,它首先测试目标是否存在,然后执行易受thread race conditions影响的移动。以下是javadocs for Files.move(...)

ATOMIC_MOVE - 将移动执行为原子文件操作。如果文件系统不支持原子移动,则会引发异常。使用ATOMIC_MOVE,您可以将文件移动到目录中,并确保任何正在监视目录的进程访问完整的文件。

如果你必须有FileAlreadyExistsException支持,那么你将需要保护的动作与​​块只允许一个线程在同一时间运行它。鉴于你在这个块中正在做IO,​​应该没有什么性能损失。

0

Files.move使用Files.exists内部,状态可以使用多线程时之间发生了变化英寸

相关问题