文件名验证

Question

我正在每分钟在一个线程中验证目录中的许多文件。 目前我正在使用下面的功能，工作正常。 我的问题是我是否可以重写这个方法来加快运行速度（java8？），希望能够缩短。

public boolean validateFile(String fileName) { 
    fileName = fileName.substring(0, fileName.lastIndexOf('.')); 
    String[] values = fileName.split("_"); 
    if (values.length == 4) { 
     if(!values[1].matches("0*")) { 
      if(!fileName.contains(" ")) { 
       if(values[3].equals("PASS") || values[3].equals("FAIL")) { 
        return true; 
       } 
      } 
     } 
    } 
    return false; 
} 

规则验证：

1. 文件名不能包含任何空白字符。
2. 文件名拆分器始终为_字符。
3. 值[1]不能只包含零（）。它必须有12位数字。
4. 值[3]必须包含“PASS”或“FAIL”。
5. 数组值[]长度必须总是4

可以运行和叉上http://ideone.com/dh8j9M

Answer 1

像这样此代码与文件名的例子 - 只是分支的一个普通的变换以Java的8。可能更具可读性，但不会更快。

return Optional.ofNullable("test") // your fileName 
      .filter(x -> !x.contains(" ")) 
      .map(x -> x.substring(0, x.lastIndexOf("."))) 
      .map(x -> x.split("_")) 
      .filter(arr -> arr.length == 4) 
      .filter(arr -> !arr[1].matches("0*")) 
      .filter(arr -> arr[3].equals("PASS") || arr[3].equals("FAIL")) 
      .isPresent(); 

Answer 2

即使@Eugene的回答是更表现，但我认为你也可以打破整个regex成片，使之清楚。例如：

private static final String PREFIX = "\\w+"; 
private static final String DIGIT12 = "(?:[1-9]\\d{11}|0\\d{11})"; 
private static final String TIMESTAMP = "[1-9]\\d{11}"; 
private static final String STATUS = "(?:PASS|FAIL)"; 
private static final String EXTENSION = "\\p{Alpha}+"; 

private static final String FILENAME_REGEX = 
     format("%s_%s_%s_%s\\.%s", PREFIX, DIGIT12, TIMESTAMP, STATUS, EXTENSION); 

private boolean validateFile(String fileName) { 
    return fileName.matches(FILENAME_REGEX); 
}