java - 插入 Map 的值不会在循环的第二个循环中保留在那里

Question

下面的函数将具有给定扩展名的所有文件从 rootDirectory 复制到给定目标。当文件名称不同时它工作得很好，但是当有两个文件具有相同名称时(请参阅递归调用 - 它可以位于子目录中)，它不会执行应有的操作。如果有更多同名文件，则应复制这两个文件并重命名第二个文件(在其名称中添加 _1、_2、...)。

我发现我正在使用的 map 可能有问题 - 每次复制文件时，我想保存它的名称并添加计数器来计算它被复制的次数(因此可以添加适当的数量以其名)。你能帮我解决这个问题吗？

void copy(File rootDirectory, String destination, String fileExtension) {
    File destFile = new File(destination);
    HashMap<String, Integer> counter = new HashMap<>();
    for (File file : rootDirectory.listFiles()) {
        try {
            if (file.isDirectory()) { copy(file, destination, fileExtension);
            } else if (getExtension(file.getPath().toLowerCase()).equals(fileExtension.toLowerCase())) { 
                if (!destFile.exists()) { destFile.mkdirs();}
                String fileName = file.getName();
                if(counter.containsKey(fileName)){  // <<-- IS NEVER TRUE 
                    int count = counter.get(fileName);
                    count++;
                    counter.put(fileName, count);
                    int i = fileName.contains(".") ? fileName.lastIndexOf('.') : fileName.length();
                    fileName = fileName.substring(0, i) + "_" + count + fileName.substring(i);
                } else{ counter.put(fileName, 0);
                }
                Files.copy(file.toPath(), Paths.get(destination + "\\" + fileName), StandardCopyOption.REPLACE_EXISTING);
            }
        } catch (IOException e) {
            //...
        }
    }
} 

Answer 1

您正在使用递归。换句话说，你总是从一个新的空 map 开始。将 map 放在您的方法之外，这将解决您的问题。