java nio 파일관련해서 리팩토링 하는 요령을 알고싶습니다

Question

// 파일 최종 수정 시간 @Override public List searchFileDate(String src, String keyword) { Path dirPath = Paths.get(src); List fileDate = new ArrayList(); SimpleDateFormat sdf = new SimpleDateFormat("yyyy-MM-dd aa hh:mm"); Stream walk; try { walk = Files.walk(dirPath); List result = walk.filter(Files::isRegularFile).collect(Collectors.toList()); for (Path path : result) { BasicFileAttributes attr = Files.readAttributes(path, BasicFileAttributes.class); Date lastModifiedTime = new Date(attr.lastModifiedTime().toMillis()); String strPath = path.toString(); String dateList = " 최종수정 " + sdf.format(lastModifiedTime); if (strPath.contains(keyword)) { fileDate.add(dateList); } } } catch (FileNotFoundException e) { e.printStackTrace(); } catch (Exception e) { e.printStackTrace(); } return fileDate; } // 폴더 최종 수정 시간 @Override public List searchDirDate(String src, String keyword) { Path dirPath = Paths.get(src); List dirDate = new ArrayList(); SimpleDateFormat sdf = new SimpleDateFormat("yyyy-MM-dd aa hh:mm"); Stream walk; try { walk = Files.walk(dirPath); List result = walk.filter(Files::isDirectory).collect(Collectors.toList()); for (Path path : result) { BasicFileAttributes attr = Files.readAttributes(path, BasicFileAttributes.class); Date lastModifiedTime = new Date(attr.lastModifiedTime().toMillis()); String strPath = path.toString(); String dateList = " 최종수정 " + sdf.format(lastModifiedTime); if (strPath.contains(keyword)) { dirDate.add(dateList); } } } catch (FileNotFoundException e) { e.printStackTrace(); } catch (Exception e) { e.printStackTrace(); } return dirDate; }제가 파일과 폴더를 관리하는 프로그램을 만들고 있습니다. 완성은 거의 다 됐는데 폴더와 파일을 항상 구분해 주다 보니 거의 같은 내용인 메소드를 계속해서 만들게 되더라고요. 리팩토링을 해서 깔끔하게 줄이고 싶은데 감을 못 잡고 있습니다. 어느 부분을 어떻게 바꾸는지 요령을 알려주시면 감사하겠습니다. 중복되는 메소드내용을 빼내서 하나로 합쳐라 이런 건 알겠는데 막상 하려면 막막해서요. 거의 작성한 모든 코드가 저렇게 작은 부분 제외하고는 항상 동일합니다

HJOW · Accepted Answer

이런 경우라면, 메소드를 하나로 합치는 걸 고려해볼 수 있습니다. 파일 경로를 알고 있다면, File 객체를 만들어서 그게 파일인지 폴더인지 구분할 수 있습니다. 두 기능을 모두 수행할 수 있는 메소드를 새로 만듭니다. 기존 매개변수 그대로 가져오고, 매개변수 하나 추가하여 디렉토리만 작업할지 파일만 작업할 지 입력을 받습니다.(boolean 이나 int 타입이 적절할 것 같네요.) 합친 메소드 안에서 기존 동작 그대로 가져오되, 파일/폴더에 따라 달라지는 부분만 if문으로 분기하여 처리합니다.(아예 메소드 시작지점에서 파일인지 폴더인지 검사해서 boolean 타입으로 가지고 있으면 됩니다.) 기존 메소드들은 내용 싹 지우고, 대신 그 안에서 새로 만든 메소드를 호출하고 그 결과를 단순 리턴합니다.이렇게 되면 공통부분은 분기할 필요가 없어 중복코드가 줄어들 수 있고 기존 코드와의 호환성도 유지됩니다.

대전개발자커뮤니티 · Answer

단순히 파일이나 폴더 생성, 변경, 삭제를 관리하고 싶으시면 간단히 WatchService 클래스 이용하시면 됩니다.

처음배워요 · Answer

대전개발자커뮤니티 HJOW 두분 다 답변 감사드립니다.

yeori · Answer

두 메소드를 하나로 합쳐도 될듯한데 그냥 합치더라도 메소드가 너무 많은 일을 합니다.기능1. tree 구조를 순회하는 기능(walk)기능2. 조건에 맞는 파일을 찾는 기능(return fileData;)기능3. 그러한 파일들에 대해서 하고 싶은 일들아마 원래 원하는건 이게 아닐까 싶습니다.기능1. 특정 디렉토리 내에 기능2. 주어진 이름을 포함하는 파일(디렉토리)을 찾아서 기능3. 언제 수정했는지 알고 싶다. 기능1에 대해서 아래와 같이 재귀호출하는 함수를 준비합니다. static void walkInternal(File cur, Predicate fileCallback, List holder) { boolean isDir = cur.isDirectory(); if (fileCallback.test(cur)) { holder.add(cur); } if (isDir) { File [] children = cur.listFiles(); for (File child : children) { walkInternal(child, fileCallback, holder); } } } static List walk(File cur, Predicate fileCallback) { List files = new ArrayList<>(); walkInternal(cur, fileCallback, files); return files; }메소드 walk 는 도우미 역할을 하고(조건에 해당하는 파일을 반환), 실제로 디렉토리를 타고 돌아다니는 일은 walkInternal이 다합니다(이 부분을 Files.walk 로 대체하신듯. 암거나 써도 됨)walkInternal은 디렉토리 구조를 돌아다니며 인자로 받은 callback 함수를(fileCallback) 실행해서 호출한 쪽에게 물어보는 역할을 합니다(기능2를 메소드 밖으로 빼낸 지점) test 메소드가 true를 반환하면 리스트에 담아줍니다. # 기능2기능2처럼 특정 조건에 해당하는 파일을 찾는 기능은 그때그때 달리질 수 있습니다.이런 부분은 walk 를 호출하는 쪽에서 메소드 호출시 callback 구현체를 넘겨줍니다. public static void main(String[] args) { File tmp = new File("d:/tmp"); Predicate jsFilter = (file) -> { String fname = file.getName().toLowerCase(); return fname.endsWith(".js"); }; List jsFiles = walk(tmp, jsFilter); jsFiles.stream().forEach(file -> { System.out.println(file.getAbsolutePath() + ": " + new Date(file.lastModified())); }); }* jsFilter - 여기서는 js 파일만 찾고 싶음. 파일이름이 ".js"로 끝나면 true 반환(기능 2에 해당)* d:/tmp 에 대해서 traversing 해봤음 기능3은 반환받은 리스트 jsFiles를 통해서 예를 들어서 언제 마지막으로 편집했는지..? 뭐 이런걸 알고 싶은듯 합니다. walk 메소드 아래와 같이 mapper 구현체를 추가해서 사용할 수도 있습니다. static List walk( File cur, Predicate fileCallback, Function mapper) { List files = new ArrayList<>(); walkInternal(cur, fileCallback, files); // 여기서 원하는 값으로 변환해줌 List result = new ArrayList<>(); for (File in : files) { result.add(mapper.apply(in)); } return result; }* 제네릭 변수 T와 mapper(Function) 을 써서 파일을 변환함근데 java stream api가 있으니까 귀찮게 만들지 않고 걍 stream().map(...) 으로 쓰는듯 합니다.

java nio 파일관련해서 리팩토링 하는 요령을 알고싶습니다

4개의 답변

Q&A 목록