Остановка рекурсии поиска файлов [РЕШЕНО]

[daimon]

QStringList listFiles = QStringList();   // какой в этом скрытый смысл ?

все переменные нужно иниционализировать)), хотя можно и убрать

[mutineer]

QStringList listFiles = QStringList();   // какой в этом скрытый смысл ?

все переменные нужно иниционализировать)), хотя можно и убрать

Лист сам инициализируется в своем конструкторе. А так у тебя создается и удаляется лишний объект

[daimon]

QStringList listFiles = QStringList();   // какой в этом скрытый смысл ?

все переменные нужно иниционализировать)), хотя можно и убрать

Лист сам инициализируется в своем конструкторе. А так у тебя создается и удаляется лишний объект

согласен

[daimon]

вопрос по архитектуре использования потока такого.

что лучше:

1. создать поток один раз и просто чередовать функции start, terminate (своя функция, меняет флаг для останова рекурсии)
     плюс - один раз выделил память и работать с одним потоком,
     минус - если поток не используется, висит в памяти, только одна ветка поиска файлов
2. создавать каждый раз извне (где он используется) и после отработки рекурсии делать deleteLater() и также в terminate (моя функция)  вызывать удаление потока deleteLater(),
    плюс - в памяти не висит просто так, можно запустить независимые ветки поиска файлов,
    минус - время на выделение памяти при создании ветки

[Igors]

вопрос по архитектуре использования потока такого.

что лучше:

В таких случаях лучше спросить (самого себя) "а их может быть 2 или больше?". Напр где-то началось (одно) сканирование, а в др месте потребовалось еще одно (напр с др фолдером). Если такая ситуевина планируется, то второй вариант 

[V1KT0P]

подробнее пожалуйста..

Да чего подробнее то, вот на скорую руку переписал, но не проверял правильность.

Код:

QStringList getListFilesFind(const QString &pathr, QStringList filters)
{
    QStack<QString> dirs;
    QString path;
    QStringList listFiles;
    if(pathr.isEmpty())
        path = QDir::currentPath();
    else
        path = pathr;
    dirs.push(path);
    while (!dirs.isEmpty()) {
        path = dirs.pop();
        QDir dir(path);
        if(!dir.exists())
            continue;

        foreach (QString file, dir.entryList(filters))
            listFiles << QFileInfo(dir, file).absoluteFilePath();

        foreach (QString subDir, dir.entryList(QDir::Dirs | QDir::NoDotAndDotDot)){
            dirs.push(path + QDir::separator() + subDir);
        }
    }
    return listFiles;
}

[daimon]

подробнее пожалуйста..

Да чего подробнее то, вот на скорую руку переписал, но не проверял правильность.

Код:

QStringList getListFilesFind(const QString &pathr, QStringList filters)
{
    QStack<QString> dirs;
    QString path;
    QStringList listFiles;
    if(pathr.isEmpty())
        path = QDir::currentPath();
    else
        path = pathr;
    dirs.push(path);
    while (!dirs.isEmpty()) {
        path = dirs.pop();
        QDir dir(path);
        if(!dir.exists())
            continue;

        foreach (QString file, dir.entryList(filters))
            listFiles << QFileInfo(dir, file).absoluteFilePath();

        foreach (QString subDir, dir.entryList(QDir::Dirs | QDir::NoDotAndDotDot)){
            dirs.push(path + QDir::separator() + subDir);
        }
    }
    return listFiles;
}

работать работает, но вот непонятно чего так

Код

C++ (Qt)
	m_countFiles+= dir.entryList(QDir::Dirs | QDir::NoDotAndDotDot).count();
 
		foreach (QString file, dir.entryList(filters))
		{
			m_currentNumberFile ++;
			emit progress(m_currentNumberFile *100 / m_countFiles);
			listFiles << QFileInfo(dir, file).absoluteFilePath();
		}

доходит до 100% за 10 сек, хотя поиску ещё идёт как минимум 3 минуты, при моём рекурсивном варианте поиска прогресс менялся и бывало 96 97 98 и аж в конце 100
что не так?

[V1KT0P]

подробнее пожалуйста..

Да чего подробнее то, вот на скорую руку переписал, но не проверял правильность.

Код:

QStringList getListFilesFind(const QString &pathr, QStringList filters)
{
    QStack<QString> dirs;
    QString path;
    QStringList listFiles;
    if(pathr.isEmpty())
        path = QDir::currentPath();
    else
        path = pathr;
    dirs.push(path);
    while (!dirs.isEmpty()) {
        path = dirs.pop();
        QDir dir(path);
        if(!dir.exists())
            continue;

        foreach (QString file, dir.entryList(filters))
            listFiles << QFileInfo(dir, file).absoluteFilePath();

        foreach (QString subDir, dir.entryList(QDir::Dirs | QDir::NoDotAndDotDot)){
            dirs.push(path + QDir::separator() + subDir);
        }
    }
    return listFiles;
}

работать работает, но вот непонятно чего так

Код

C++ (Qt)
	m_countFiles+= dir.entryList(QDir::Dirs | QDir::NoDotAndDotDot).count();
 
		foreach (QString file, dir.entryList(filters))
		{
			m_currentNumberFile ++;
			emit progress(m_currentNumberFile *100 / m_countFiles);
			listFiles << QFileInfo(dir, file).absoluteFilePath();
		}

доходит до 100% за 10 сек, хотя поиску ещё идёт как минимум 3 минуты, при моём рекурсивном варианте поиска прогресс менялся и бывало 96 97 98 и аж в конце 100
что не так?

Ну дык ты же подсчитываешь количество директорий. В твоем рекурсивном после нахождения каждой директории в ней происходил поиск и поэтому время между нахождением следующей директории был долгий. В моем же варианте сразу все директории верхнего уровня находятся и если вложенных директорий нет то и будет 100%. Щас набросаю как я это вижу...

[daimon]

я поправил

Код

C++ (Qt)
m_countFiles += dir.entryList(filters).count();

чтобы плюсовать количество файлов по фильтру именно текущей директории

но всё равно 99-100 100-99

[V1KT0P]

я поправил

Код

C++ (Qt)
m_countFiles += dir.entryList(filters).count();

чтобы плюсовать количество файлов по фильтру именно текущей директории

но всё равно 99-100 100-99

Вот это попробуй, вроде должно работать, но прогресс будет иногда уменьшатся когда будет происходить нахождение новых файлов:

Код:

QStringList getListFilesFind(const QString &pathr, QStringList filters)
{
    QStack<QString> dirs;
    QString path;
    QStringList listFiles;
    qint32 countFiles = 1;
    qint32 currentNumberFile = 0;
    qint32 progress = 0;
    qint32 progressOld = 0;
    if(pathr.isEmpty())
        path = QDir::currentPath();
    else
        path = pathr;
    dirs.push(path);
    while (!dirs.isEmpty()) {
        path = dirs.pop();
        QDir dir(path);
        countFiles += dir.entryList(QDir::NoDotAndDotDot).count();
        ++currentNumberFile;
        if(!dir.exists())
            continue;

        foreach (QString file, dir.entryList(filters)) {
            ++currentNumberFile;
            listFiles << QFileInfo(dir, file).absoluteFilePath();
            progress = currentNumberFile * 100 / countFiles;
            if (progressOld != progress) {
                progressOld = progress;
                emit progress(progress);
            }
        }

        foreach (QString subDir, dir.entryList(QDir::Dirs | QDir::NoDotAndDotDot)){
            dirs.push(path + QDir::separator() + subDir);
        }
    }
    return listFiles;
}

[daimon]

по нулям всё, emit не идёт

[V1KT0P]

по нулям всё, emit не идёт

Да ступил, вместо

Код:

countFiles += dir.entryList(QDir::NoDotAndDotDot).count();

надо было

Код:

countFiles += dir.entryList(QDir::Dirs | QDir::Files | QDir::NoDotAndDotDot).count();

[daimon]

по нулям всё, emit не идёт

Да ступил, вместо

Код:

countFiles += dir.entryList(QDir::NoDotAndDotDot).count();

надо было

Код:

countFiles += dir.entryList(QDir::Dirs | QDir::Files | QDir::NoDotAndDotDot).count();

всё равно не так, попытаюсь объяснить, так мы считаем количество всех файлов в папке, а текущий номер файла идёт по списку количества файлов в директории по фильтру, итог один - прогресс всегда показывает 30-40%

думаю нужно написать countFiles += dir.entryList(filters).count();

и всё равно всегда 100%
мда сурово

видимо прийдёться поставить прогресбар на анимацию))))

[V1KT0P]

по нулям всё, emit не идёт

Да ступил, вместо

Код:

countFiles += dir.entryList(QDir::NoDotAndDotDot).count();

надо было

Код:

countFiles += dir.entryList(QDir::Dirs | QDir::Files | QDir::NoDotAndDotDot).count();

всё равно не так, попытаюсь объяснить, так мы считаем количество всех файлов в папке, а текущий номер файла идёт по списку количества файлов в директории по фильтру, итог один - прогресс всегда показывает 30-40%

думаю нужно написать countFiles += dir.entryList(filters).count();

и всё равно всегда 100%
мда сурово

Дело в том что по мере поиска количество файлов увеличивается и изначально мы не знаем сколько их. Вот последний вариант, больше я не хочу пробовать:

Код:

QStringList getListFilesFind(const QString &pathr, QStringList filters)
{
    QStack<QString> dirs;
    QString path;
    QStringList listFiles;
    qint32 countFiles = 1;
    qint32 currentNumberFile = 0;
    qint32 progress = 0;
    qint32 progressOld = 0;
    if(pathr.isEmpty())
        path = QDir::currentPath();
    else
        path = pathr;
    dirs.push(path);
    while (!dirs.isEmpty()) {
        path = dirs.pop();
        QDir dir(path);
        countFiles += dir.entryList(QDir::Dirs | QDir::NoDotAndDotDot).count();
        ++currentNumberFile;

        progress = currentNumberFile * 100 / countFiles;
        if (progressOld != progress) {
            progressOld = progress;
            emit progress(progress);
        }

        if(!dir.exists())
            continue;

        foreach (QString file, dir.entryList(filters)) {
            listFiles << QFileInfo(dir, file).absoluteFilePath();
        }

        foreach (QString subDir, dir.entryList(QDir::Dirs | QDir::NoDotAndDotDot)){
            dirs.push(path + QDir::separator() + subDir);
        }
    }
    return listFiles;
}

[Igors]

entryList явно не дешевый вызов и делать его дважды-трижды не следует