Поиск слов в файле в Qt

[stasJun]

Сообщение от Alex11223

где они совпадают-то? вы ж к completed пытаетесь подключить fileThreadProgressChanged.

так правильно. сигнал- completed
слот fileThreadProgressChanged
а что нужно подключать?
progressChanged тоже сигнал

Писи. Вы извиняйте. я уже зпутался окончалтельно и ничего уже не пойму, что куда нужно, что нужно делать

[Alex11223]

Так в моем примере с 2 сигналами у них разные параметры, одинаковые были в начальном коде, где был только сигнал для количества байт (то есть в моем примере начальный completed переименован progressChanged и добавлен новый сигнал для результата поиска, его я назвал completed, но он не имеет отношения к тому что было раньше).

В предыдущем сообщении все ж описано. http://programmersforum.ru/showthrea...=5#post1637969

[stasJun]

Сообщение от Alex11223

Так в моем примере с 2 сигналами у них разные параметры, одинаковые были в начальном коде, где был только сигнал для количества байт (то есть в моем примере начальный completed переименован progressChanged и добавлен новый сигнал для результата поиска, его я назвал completed, но он не имеет отношения к тому что было раньше).

В предыдущем сообщении все ж описано. http://programmersforum.ru/showthrea...=5#post1637969

переделал:

 for (size_t index = 0; index < mThreads.size(); ++index)
            connect(mThreads[index], SIGNAL(progressChanged(qint64)), this, SLOT(fileThreadProgressChanged(qint64)));

я просто в упор не мог понять, и не подумал что там дествительно переменовали переменную и переменная с completed другие аргументы принимает.
Теперь без все ок. но подскажите как сделать чтобы не 1 поток- 1 слово
а 1 поток- много слов работало. а то не прикольно что кансел в конце только срабатывает и прогрес бар через wait

[Alex11223]

"1 поток- много" никак не связано с "не прикольно что кансел в конце только срабатывает и прогрес бар через wait".

Чтобы работала отмена (и чтобы вообще не замирал весь UI во время поиска) надо

Сообщение от Alex11223

не делать ничего после запуска потоков (убрать wait), всю обработку результата и т.п. делать только по сигналу (completed, ...)

........

в completed наверно надо передавать имя файла и результат (в данном случае bool найдено/не найдено).
При его получении выводить результат и считать количество завершенных потоков для определения завершения всего поиска.

А чтобы работало 1 поток=много файлов надо разобраться в коде/примере и доделать его под это.

тут http://programmersforum.ru/showthrea...=4#post1637756

Сообщение от Alex11223

Как минимум found сейчас неправильно устанавливается, будет всегда true после первого файла в котором нашли нужную строку.

[stasJun]

Сообщение от Alex11223

"1 поток- много" никак не связано с "не прикольно что кансел в конце только срабатывает и прогрес бар через wait".

Чтобы работала отмена (и чтобы вообще не замирал весь UI во время поиска) надо

если я убираю wait сразу креш при запуске програмы.

в completed наверно надо передавать имя файла и результат (в данном случае bool найдено/не найдено).
При его получении выводить результат и считать количество завершенных потоков для определения завершения всего поиска.

- я ж это делаю, вывожу в емите

[Alex11223]

Сообщение от stasJun

если я убираю wait сразу креш при запуске програмы.

Что именно в коде сейчас? Естественно надо убрать не только wait, а все что происходит в обработчике клика кнопки после запуска потоков (wait, вывод файлов в QListWidget, удаление потоков, ...)

[stasJun]

Сообщение от Alex11223

Что именно в коде сейчас? Естественно надо убрать не только wait, а все что происходит в обработчике клика кнопки после запуска потоков (wait, вывод файлов в QListWidget, удаление потоков, ...)

я закоментил что удалил

void FindFilesWidget::startClicked()
{
    if(mActionState == ActionState::STOPPED)
    {
        mActionState = ActionState::RUNNING;
        mStartButton->setText("&Cancel");
        const int idealThreadCount = QThread::idealThreadCount();
        mThreads.reserve(idealThreadCount);

        QDirIterator dirIt(mSearchDirectoryLine->text(), QDirIterator::Subdirectories);
        std::vector<QString> paths;
        while (dirIt.hasNext())
        {
            dirIt.next();
            if (QFileInfo(dirIt.filePath()).isFile())
            {
                if (QFileInfo(dirIt.filePath()).suffix() == "txt")
                    paths.push_back(dirIt.filePath());
            }
        }
        for (int  index = 0; index < paths.size(); ++index)
        {
            QFileInfo fileInfo(paths[index]);
            mFileSize += fileInfo.size();
        }
        const std::vector<Range> ranges = GenerateRanges(idealThreadCount, paths.size());
        std::vector<QString> foundFiles;
        QMutex mutex;

        for (size_t index = 0; index < idealThreadCount; ++index)
            mThreads.push_back(new FindWordInFileThread(paths, mSearchWordLine->text(), ranges[index].mStart, ranges[index].mLength, foundFiles, &mutex));

        for (size_t index = 0; index < mThreads.size(); ++index)
            connect(mThreads[index], SIGNAL(progressChanged(qint64)), this, SLOT(fileThreadProgressChanged(qint64)));


        for (size_t index = 0; index < idealThreadCount; ++index)
            mThreads[index]->start();
        //for (size_t index = 0; index < idealThreadCount; ++index)
            //mThreads[index]->wait();

        std::sort(foundFiles.begin(), foundFiles.end());

        //for (size_t index = 0; index < idealThreadCount; ++index)
            //delete mThreads[index];

        mFoundFilesList->clear();
        for (size_t index = 0; index < foundFiles.size(); ++index)
            mFoundFilesList->addItem(foundFiles[index]);
    }
    else
    {
        mActionState = ActionState::STOPPED;
        mStartButton->setText("&Start");
        for (size_t index = 0; index < mThreads.size(); ++index)
                    mThreads[index]->cancelClicked();
    }
}

[Alex11223]

крешится по идее не должно от этого, но незакомментированные строки после start не имеют никакого смысла. Этот код же выполнится сразу после запуска, когда еще нет ничего в foundFiles.

Надо убрать foundFiles вообще и делать все что происходит тут в слоте для completed.

[stasJun]

Сообщение от Alex11223

крешится по идее не должно от этого, но незакомментированные строки после start не имеют никакого смысла. Этот код же выполнится сразу после запуска, когда еще нет ничего в foundFiles.

Надо убрать foundFiles вообще и делать все что происходит тут в слоте для completed.

а foundFiles где искать?
у меня сигнал completed . а слота такого нету

[Alex11223]

Не нужен foundFiles, я ж описал уже 2 раза.

Сделать сигнал в который передается имя файла и результат поиска (найден/не найден), в слоте для него добавлять в UI этот файл если найден.

И там же можно считать либо количество завершенных потоков (если 1 поток=1 файл), либо количество завершенных файлов (если 1 поток=много файлов, и количество файлов известно до запуска) для определения завершения всего поиска.
То есть например до запуска сохранить их число, при каждым срабатывании слота отнимать 1, когда стало 0 — значит поиск завершился (включить обратно кнопку Старт и т.п.)

Сообщение от stasJun

у меня сигнал completed . а слота такого нету

А зачем нужен сигнал если нигде нет слота для него?