VPF::Освобождение памяти TThread - Форум программистов

Pori

Дата 4.8.2009, 15:09 (ссылка)

(нет голосов)

Загрузка ...

Шустрый

Профиль
Группа: Участник
Сообщений: 50
Регистрация: 10.9.2007

Репутация: 1
Всего: 1

Всем привет.

В общем сразу код:

Код


TMyThread *Thr;

void __fastcall TForm1::Button1Click(TObject *Sender)
{
for(int i=0;i<=100000;++i)
{
    Thr = new TMyThread(true);
    Thr->Resume();
    Thr->Terminate();
        Thr=NULL;
}
}

//модуль потока
int i;
void __fastcall TMyThread::Execute()
{
    FreeOnTerminate=true;
    for(i=0;i<=100000;++i)
    {
        if (Terminated)
            break;
        Synchronize(&UpdateCaption);
    }
}
//---------------------------------------------------------------------------
void __fastcall TMyThread::UpdateCaption()
{
    Form1->Label1->Caption = i;
    Application->ProcessMessages();
}

Программа валиться после 5-10 секунд работы - недостаточно памяти.

PS: похожая тема, но ответа там не нашел: http://forum.vingrad.ru/topic-251625.html#st_0_view_0

Romikgy

Дата 4.8.2009, 15:26 (ссылка)

(нет голосов)

Загрузка ...

Любитель-программер

Профиль
Группа: Участник Клуба
Сообщений: 7326
Регистрация: 11.5.2005
Где: Porto Franco Odes sa

Репутация: 11
Всего: 146

Цитата(Pori @ 4.8.2009, 14:09

)

Thr->Terminate();
Thr=NULL;

нехорошо не дожидатся окончания работы потока
между ними вставь
Thr->WaitFor()
должно помочь ,
да и правильнее вместо Thr=NULL;
использовать delete (ведь new же вызвано было )

--------------------

Владение русской орфографией это как владение кунг-фу — истинные мастера не применяют его без надобности.

Alca

Дата 4.8.2009, 15:28 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Завсегдатай
Сообщений: 3993
Регистрация: 14.6.2006

Репутация: 25
Всего: 50

Цитата

Код


for(int i=0;i<=100000;++i)

А не много ли потоков ты создаешь?

--------------------

C++ cross-platform library

Pori

Дата 4.8.2009, 15:44 (ссылка)

(нет голосов)

Загрузка ...

Шустрый

Профиль
Группа: Участник
Сообщений: 50
Регистрация: 10.9.2007

Репутация: 1
Всего: 1

Цитата(Romikgy @ 4.8.2009, 15:26)

Огромное спасибо, работает как часы ;)

по поводу delete: так вроде как деструктор освобождает память, разве нет? Стоит FreeOnTerminate=true, это как раз из той темы, что я указывал выше

Цитата(Alca @ 4.8.2009, 15:28)

А не много ли потоков ты создаешь?

Это для теста, так сказать, краш-тест ;)

Alca

Дата 4.8.2009, 15:52 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Завсегдатай
Сообщений: 3993
Регистрация: 14.6.2006

Репутация: 25
Всего: 50

Если

Код


FreeOnTerminate=true;

то delete не надо.

--------------------

C++ cross-platform library

Pori

Дата 4.8.2009, 16:03 (ссылка)

(нет голосов)

Загрузка ...

Шустрый

Профиль
Группа: Участник
Сообщений: 50
Регистрация: 10.9.2007

Репутация: 1
Всего: 1

Вот я тоже так думаю, но вставил delete и о чудо, нет ошибок. А я думал, что будет что-нить типа access violation

Romikgy

Дата 4.8.2009, 16:13 (ссылка)

(нет голосов)

Загрузка ...

Любитель-программер

Профиль
Группа: Участник Клуба
Сообщений: 7326
Регистрация: 11.5.2005
Где: Porto Franco Odes sa

Репутация: 11
Всего: 146

Цитата(Pori @ 4.8.2009, 14:44

)

Стоит FreeOnTerminate=true

да проглядел , все ок

--------------------

Владение русской орфографией это как владение кунг-фу — истинные мастера не применяют его без надобности.

Alca

Дата 4.8.2009, 16:19 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Завсегдатай
Сообщений: 3993
Регистрация: 14.6.2006

Репутация: 25
Всего: 50

Цитата

Код


FreeOnTerminate=true

Память освобождается, но указатель не нулиться, т.к. в деструкторе:

Код


Free()

--------------------

C++ cross-platform library

0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема \| C++ Builder \| Следующая тема »