Java中Iterator迭代器next()的使用

        最近工作中之前写了一个删除超过3个月的历史文件的功能，一直没空验证，这两天看了一下，发现这功能压根没有把文件删除掉，最后排查发现是Iterator迭代器使用的问题，一个很基础的知识，所以工作了几年，这种错误太低级了，记录一下以免以后再犯。

代码片段如下：

Iterator<String> it = fileList.iterator();
while(it.hasNext()) {
    log.logInfo("删除文件: " + it.next()); // 这里调用了迭代器的next()方法，获取到当前文件的路径
    File file = new File(it.next()); // 这里又调用了一次next()方法，此时获取到的是下一个文件路径
    if (file.exists() && !file.isDirectory()) {
        file.delete();
    }
}

很简单的一段代码，就是遍历文件删除文件，但是Iterator迭代器用的比较少，没注意到next()方法每次调用时会指向list的下一个元素，导致log日志调用的next的文件路径都没有被删除到，这里还有可能导致报一个：NoSuchElementException的异常

修改后的代码如下：

 

Iterator<String> it = fileList.iterator();
while(it.hasNext()) {
    String filePath = it.next()；// 获取到当前文件的路径赋值给一个变量
    log.logInfo("删除文件: " + filePath ); 
    File file = new File(filePath);
    if (file.exists() && !file.isDirectory()) {
        file.delete();
    }
}