Почему при обходе слишком глубокой директории выводит исключение?

Question

[Rienopely]

FileSearch

import java.io.File;
import java.util.HashSet;

public class FileSearcher {

    //Директория для поиска
    private final File dir;
    //Список папок в директории
    private final HashSet<File> listDirectories;
    
    private boolean findSwitch = true;
    
    //Конструктор принимающий путь к директории
    public FileSearcher(String dirPath) {
        dir = new File(dirPath);
        listDirectories = new HashSet<>();
        //Получить первые папки для списка
        for(File i : dir.listFiles()){
            if(i.isDirectory()){
                listDirectories.add(i);
            }
        } 
    }
    
    public void dirFileWalk(){
        for(File dirs : listDirectories){
            findSwitch = false;
            for(File i : dirs.listFiles()){
                if(i.isDirectory() && !listDirectories.contains(i)){
                    listDirectories.add(i);
                    findSwitch = true;
                }
            }
        }
    }
    
    public HashSet<File> start(){
        while(findSwitch)
            dirFileWalk();
        return listDirectories;
    }

}

Main

public class Main {
    
    public static void main(String[] args){
        FileSearcher a = new FileSearcher
                 
                       
        ("c:/Users/Denis/Desktop/pages");
        HashSet<File> dirs = a.start();   
    }
}

Output
Exception in thread "main" java.util.ConcurrentModificationException
at java.util.HashMap$HashIterator.nextNode(HashMap.java:1429)
at java.util.HashMap$KeyIterator.next(HashMap.java:1453)
at com.phaeton.core.FileSearcher.dirFileWalk(FileSearcher.java:28)
at com.phaeton.core.FileSearcher.start(FileSearcher.java:41)
at com.phaeton.starter.Main.main(Main.java:14)
Java Result: 1

Answer 1

[Данил Антошкин]

Ошибку может напишите ?

Comments

[Rienopely]

Добавил

[Данил Антошкин]

Rienopely: Ты изменяешь коллекцию одновременно когда проходишь по ней итератором

[Rienopely]

Данил Антошкин: Спасибо

Answer 2

[X-NoNAME]

Может из-за этого?

for(File dirs : listDirectories){
       listDirectories.add(i);
 }

На правильность не претендую, но я бы сделал так:

Set<File> copyListDirectories = new HashSet<>(listDirectories);
 for(File dirs : copyListDirectories ){
       listDirectories.add(i);
 }

Comments

[Rienopely]

Да, спасибо. Я уже разобрался)

[Rienopely]

Не знаете как кодировку Windows-1251 в UTF-8 конвертировать, чтобы потом крякозябры не было?)

[X-NoNAME]

Блин, не заметил комментарии к предыдущему ответу.

[X-NoNAME]

Rienopely: на каком этапе и где меняете кодировку?

[Rienopely]

XNoNAME: Мне нужно прочитать файлы , они могут быть в кодировке windows-1251 или UTF-8. Как попадется, я не знаю. И мне нужно перезаписать их все в UTF-8. Русские символы в этих файлах будут отображаться не правильно. Нельзя это никак исправить?

[X-NoNAME]

Rienopely: Тут нужна библиотечка, которая бы автоматически определяла кодировку, нагуглил эту: jchardet.sourceforge.net

[Rienopely]

XNoNAME: Большое спасибо)