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

[Alex11223]

И куда смотреть?

У потока есть такие варианты завершения:

1. После поиска во всех файлах ("happy path")
2. При ошибке (не удалось открыть файл, ...)
3. Отмена (вызов cancel)

В коде выше обрабатывается только 3 вариант (вызов cancel у всех потоков).
И даже для этого случая не видно уничтожения потоков/очистки mThreads.

[stasJun]

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

И куда смотреть?

У потока есть такие варианты завершения:

1. После поиска во всех файлах ("happy path")
2. При ошибке (не удалось открыть файл, ...)
3. Отмена (вызов cancel)

В коде выше обрабатывается только 3 вариант (вызов cancel у всех потоков).
И даже для этого случая не видно уничтожения потоков/очистки mThreads.

Для проверки файлов у меня есть проверка,

void FindWordInFileThread::run()
{
    mIsCancelled = false;
    for (size_t index = mStartIndex; index < mStartIndex + mNumElements; ++index)
    {

        QFile file(mFilePath[index]);
        if (!file.open(QIODevice::ReadOnly | QIODevice::Text))
           return;
        QTextStream textStream(&file);
        while (!textStream.atEnd())
        {
            if (mIsCancelled)
            {
                file.close();
                return;
            }
            QString line = textStream.readLine();
            if (line.contains(mWord, Qt::CaseInsensitive))
            {
                emit fileFound(file.fileName());
                break;
            }
        }
        if (!mIsCancelled)

        emit progressChanged(file.size());
        qDebug() << file.size();
        file.close();
    }
}

если удалять потоки:

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

то крешится

[Alex11223]

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

Для проверки файлов у меня есть проверка,

В потоке-то есть, но никто извне не узнает об этом (ну или хотя бы просто о том, что поток завершился)

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

если удалять потоки: то крешится

Ну так удалять надо не когда вздумалось, а после их завершения.

[stasJun]

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

В потоке-то есть, но никто извне не узнает об этом (ну или хотя бы просто о том, что поток завершился)

Ну так удалять надо не когда вздумалось, а после их завершения.

удаляю

FindFilesWidget::~FindFilesWidget()
{
    for (size_t index = 0; index < mIdealThreadCount; ++index)
        delete mThreads[index];
}

[stasJun]

а как сделать сигнал на закрытия програмы хрестиком?

QObject::connect(тут как  хрестик обозначить?, SIGNAL(clicked()), &app, SLOT(quit()));

и как мне проверять mThreads пустой или нет?
isEmpty подойдет?

[Alex11223]

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

удаляю

Так деструктор виджета вызовется только при его закрытии.

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

а как сделать сигнал на закрытия програмы хрестиком?

http://stackoverflow.com/questions/1...x-close-button

....
protected:
    void closeEvent(QCloseEvent *event); override;

[stasJun]

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

Так деструктор виджета вызовется только при его закрытии.

так мне и нужно)
если я не закрываю виджет но заново жму кнопку, то мне нужно сделать иф, если mThreads не пусчтой то я все удаляю и делаю mThreads.clear();
mThreads.reserve(mIdealThreadCount) ;
правильно я думаю?

if (mActionState == ActionState::STOPPED)
    {
        mFoundFilesList->clear();
        mProgressBar->setValue(0);
        mBytesProcessed = 0;
        mActionState = ActionState::RUNNING;
        mStartButton->setText("&Cancel");
        //тут надо иф  этот  делать
        mThreads.clear();
        mThreads.reserve(mIdealThreadCount);

[Alex11223]

да, можно так (только еще удалить все объекты если не пустой), но обработка всех вариантов завершения описанных выше все равно нужна же, так что можно и там очищать.

[stasJun]

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

да, можно так (только еще удалить все объекты если не пустой), но обработка всех вариантов завершения описанных выше все равно нужна же, так что можно и там очищать.

я сделаю как я думаю, если препод скажет что не везде удаляю переделаю)
я правильно конект для хрестика сделал? я в самом конце делаю

void FindFilesWidget::closeEvent(QCloseEvent* event)
{
    QWidget::closeEvent(event);
}

  ................
else
    {
        mActionState = ActionState::STOPPED;
        mStartButton->setText("&Start");
        for (size_t index = 0; index < mThreads.size(); ++index)
            mThreads[index]->cancel();
    }
    QObject::connect(this, SIGNAL(clicked()), SLOT(closeEvent()));
}

[Alex11223]

Да удаление-то ладно, но там ж сейчас всегда будет висеть кнопка отмены (пока ее не нажмут) когда на самом деле потоки давно остановились.

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

я правильно конект для хрестика сделал?

это не сигнал, просто переопределяемая функция родительского класса.