Работа с Thread

@Vovik_0_1 · Регистрация: 28.07.2011

Студворк — интернет-сервис помощи студентам

Привет всем. Не могу понять в чем дело. Создаю второй поток но программа всё равно зависает. В чём дело?

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
//---------------------------------------------------------------------------
 
#include <vcl.h>
#pragma hdrstop
 
#include "Unit1.h"
#include "Unit2.h"
//---------------------------------------------------------------------------
#pragma package(smart_init)
#pragma resource "*.dfm"
TForm1 *Form1;
Analiz *Thread;
//---------------------------------------------------------------------------
 
//---------------------------------------------------------------------------
__fastcall TForm1::TForm1(TComponent* Owner)
        : TForm(Owner)
{
}
//---------------------------------------------------------------------------
void __fastcall TForm1::Button3Click(TObject *Sender)
{
  Thread = new Analiz(false);
   Thread->FreeOnTerminate = true;
}

его h

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
#ifndef Unit1H
#define Unit1H
//---------------------------------------------------------------------------
#include <Classes.hpp>
#include <Controls.hpp>
#include <StdCtrls.hpp>
#include <Forms.hpp>
#include <ComCtrls.hpp>
//---------------------------------------------------------------------------
class TForm1 : public TForm
{
__published:    // IDE-managed Components
        TLabel *Label1;
        TButton *Button3;
        TRichEdit *RichEdit1;
        TProgressBar *ProgressBar1;
        TLabel *Label2;
        TProgressBar *ProgressBar2;
        TLabel *Label3;
        void __fastcall Button3Click(TObject *Sender);
private:    // User declarations
public:     // User declarations
        __fastcall TForm1(TComponent* Owner);
};
//---------------------------------------------------------------------------
extern PACKAGE TForm1 *Form1;
//---------------------------------------------------------------------------
#endif

Вот Unit потока:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
#include <vcl.h>
#pragma hdrstop
 
#include "Unit2.h"
#include "Unit1.h"
#pragma package(smart_init)
//---------------------------------------------------------------------------
 
//   Important: Methods and properties of objects in VCL can only be
//   used in a method called using Synchronize, for example:
//
//      Synchronize(UpdateCaption);
//
//   where UpdateCaption could look like:
//
//      void __fastcall Analiz::UpdateCaption()
//      {
//        Form1->Caption = "Updated in a thread";
//      }
//---------------------------------------------------------------------------
void __fastcall Analiz::Analizing()
{
  TStringList*s = new TStringList;
    TStringList*s2 = new TStringList;
    s->LoadFromFile("log.ant");
    s2->LoadFromFile("log2.ant");
    Form1->ProgressBar1->Max = s2->Count;
    Form1->ProgressBar2->Max = s->Count;
    for (int i = 0; i < s2->Count; i++) {
        for (int q = 0; q < s->Count; q++) {
            if (s2->Strings[i] == s->Strings[q]) {s->Delete(q);  break;  }
            else if (q == s->Count - 1)
      {
      Form1->RichEdit1->SelAttributes->Color = clBlue;
  Form1->RichEdit1->SelText = "Добавлено- ";
  Form1->RichEdit1->SelAttributes->Color = clBlack;
  Form1->RichEdit1->SelText = s2->Strings[i]+"\n";
        }
      Form1->ProgressBar2->StepIt();
    }
   Form1->ProgressBar1->StepIt();
   }
   //Узнаем какие файлы были удалены
   Form1->ProgressBar1->Max = s->Count;
   for (int i=0;i<s->Count;i++)
   {
    Form1->RichEdit1->SelAttributes->Color = clRed;
  Form1->RichEdit1->SelText = "Удалено- ";
  Form1->RichEdit1->SelAttributes->Color = clBlack;
  Form1->RichEdit1->SelText = s->Strings[i]+"\n";
   Form1->ProgressBar1->StepIt();
   }
   delete s;
   delete s2;
   Form1->ProgressBar2->Position = 0;
   Form1->ProgressBar1->Position = 0;
}
__fastcall Analiz::Analiz(bool CreateSuspended)
        : TThread(CreateSuspended)
{
}
//---------------------------------------------------------------------------
void __fastcall Analiz::Execute()
{
  Synchronize(Analizing);
}

вот его h

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
#ifndef Unit2H
#define Unit2H
//---------------------------------------------------------------------------
#include <Classes.hpp>
//---------------------------------------------------------------------------
class Analiz : public TThread
{            
private:
void __fastcall Analizing();
protected:
        void __fastcall Execute();
public:
        __fastcall Analiz(bool CreateSuspended);
};
//---------------------------------------------------------------------------
#endif

@BRcr · 29.06.2012, 16:31

Метод Synchronize выполняет переданную ему функцию в контексте основного потока, а у тебя вся работа вынесена в эту функцию.

@Vovik_0_1 · 29.06.2012, 16:35 **[ТС]**

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

вся работа вынесена в эту функцию

И что теперь делать?

Добавлено через 1 минуту
Делал как в Архангельском там вроде тоже так.

Добавлено через 57 секунд

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

Synchronize

А без него же вроде не льзя.. Конфликты будут.

@BRcr · 29.06.2012, 16:36

Теперь нагрузку распределять по-уму, чего делать...
Все обращения к vcl через Synchronize(), остальное в Execute().

@Vovik_0_1 · 29.06.2012, 17:02 **[ТС]**

Вот так чтоли?

C++
1
Synchronize(Form1->RichEdit1->SelText = "Добавлено- ";);

Правдо не работает(( Тоесть для каждого RichEdit1?? Или нужно писать опять функцию? только там будут уже одни vcl

Добавлено через 21 минуту
Вот так?????

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
#include <vcl.h>
#pragma hdrstop
 
#include "Unit2.h"
#include "Unit1.h"
#pragma package(smart_init)
TStringList*s2 = new TStringList;
TStringList*s = new TStringList;
int i = 0;
int ie=0;
//---------------------------------------------------------------------------
 
//   Important: Methods and properties of objects in VCL can only be
//   used in a method called using Synchronize, for example:
//
//      Synchronize(UpdateCaption);
//
//   where UpdateCaption could look like:
//
//      void __fastcall Analiz::UpdateCaption()
//      {
//        Form1->Caption = "Updated in a thread";
//      }
//---------------------------------------------------------------------------
void __fastcall Analiz::Analizing()
{
  Form1->RichEdit1->SelAttributes->Color = clBlue;
  Form1->RichEdit1->SelText = "Добавлено- ";
  Form1->RichEdit1->SelAttributes->Color = clBlack;
  Form1->RichEdit1->SelText = s2->Strings[i]+"\n";
}
void __fastcall Analiz::Analizing2()
{
   Form1->RichEdit1->SelAttributes->Color = clRed;
  Form1->RichEdit1->SelText = "Удалено- ";
  Form1->RichEdit1->SelAttributes->Color = clBlack;
  Form1->RichEdit1->SelText = s->Strings[ie]+"\n";
   Form1->ProgressBar1->StepIt();
}
void __fastcall Analiz::Analizing3()
{
 Form1->ProgressBar2->StepIt();
}
void __fastcall Analiz::Analizing4()
{
  Form1->ProgressBar1->StepIt();
}
void __fastcall Analiz::Analizing5()
{
    Form1->ProgressBar2->Position = 0;
   Form1->ProgressBar1->Position = 0;
}
void __fastcall Analiz::Analizing6()
{
    Form1->ProgressBar1->Max = s2->Count;
    Form1->ProgressBar2->Max = s->Count;
}
void __fastcall Analiz::Analizing7()
{
    Form1->ProgressBar1->Max = s->Count;
}
__fastcall Analiz::Analiz(bool CreateSuspended)
        : TThread(CreateSuspended)
{
}
//---------------------------------------------------------------------------
void __fastcall Analiz::Execute()
{
    s->LoadFromFile("log.ant");
    s2->LoadFromFile("log2.ant");
      Synchronize(Analizing6);
    for (i; i < s2->Count; i++) {
        for (int q = 0; q < s->Count; q++) {
            if (s2->Strings[i] == s->Strings[q]) {s->Delete(q);  break;  }
            else if (q == s->Count - 1)
      {
        Synchronize(Analizing);
        }
      Synchronize(Analizing3);
    }
      Synchronize(Analizing4);
   }
   //Узнаем какие файлы были удалены
     Synchronize(Analizing7);
   for (ie;ie<s->Count;ie++)
   {
    Synchronize(Analizing2);
   }
   delete s;
   delete s2;
    Synchronize(Analizing5);
  }

@Avazart · 29.06.2012, 17:56

Да...
Только так:

C++
1
2
3
4
#include "Unit1.h"
#pragma package(smart_init)
TStringList*s2 = new TStringList; // Туфта
TStringList*s = new TStringList;  // Туфта

Никто не делает...
Это должно быть в классе потока и в Execute();

@BRcr · 29.06.2012, 18:08

Vovik_0_1, ну, и как? Виснет теперь?

@Vovik_0_1 · 29.06.2012, 21:52 **[ТС]**

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

Виснет теперь?

Теперь нет спасибо!

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

Никто не делает...

Только из за этого это туфта?

@Avazart · 29.06.2012, 21:54

Нет, потому что это не рабочая туфта

@Vovik_0_1 · 29.06.2012, 21:56 **[ТС]**

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

потому что это не рабочая туфта

Почему не рабочая? Всё прекрастно работает! А как ты предлогаеш?

@Avazart · 29.06.2012, 21:57

А при повторном создании потока ошибку разве не выбивает?

А как ты предлогаеш?

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

Это должно быть в классе потока и в Execute();

Т.е s1,s2 должны быть членами класса потока ну либо хотябы создаваться внутри Execute()

@Vovik_0_1 · 29.06.2012, 21:59 **[ТС]**

Да нет вроде!

Не по теме:

Поток останавливать нужно или он сам завершится когда код закрнчится?

@Avazart · 29.06.2012, 22:01

Да нет вроде!

Ну ждите... будет потому как s1 и s2 после окончания "первого" потока уже не существуют

@Vovik_0_1 · 29.06.2012, 22:51 **[ТС]**

Вот так:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
#ifndef Unit2H
#define Unit2H
//---------------------------------------------------------------------------
#include <Classes.hpp>
//---------------------------------------------------------------------------
class Analiz : public TThread
{
private:
void __fastcall Analizing();
void __fastcall Analizing2();
void __fastcall Analizing3();
void __fastcall Analizing4();
void __fastcall Analizing5();
void __fastcall Analizing6();
void __fastcall Analizing7();
protected:
        void __fastcall Execute();
public:
        __fastcall Analiz(bool CreateSuspended);
        TStringList*s2;
        TStringList*s;
};

И там:

C++
1
2
3
4
5
6
7
8
9
10
11
void __fastcall Analiz::Execute()
{
s2 = new TStringList;
s = new TStringList;
    s->LoadFromFile("log.ant");
    s2->LoadFromFile("log2.ant");
 
................
................
..............
............

Добавлено через 4 минуты
Только почему то в s и s1 как появились данные при первом запуске так и остались. В чём дело? Всё же должен был удалять оператор delete s; and delete s2;

Добавлено через 8 минут
Avazart, красава! Спасибо разобрался!

@Avazart · 29.06.2012, 22:54

После delete данные могут оставаться не тронутыми но уже как мусор

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами 8Observer8 20.02.2026 Содержание блога Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .	Конвертировать закладки radiotray-ng в m3u-плейлист damix 19.02.2026 Это можно сделать скриптом для PowerShell. Использование . \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json> Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом. # Check if. . .	Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI Eddy_Em 18.02.2026 Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .
И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.	«Знание-Сила» zbw 12.02.2026 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»	SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 12.02.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 11.02.2026 Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .

@BRcr 4043 / 2333 / 292 Регистрация: 03.02.2011 Сообщений: 5,066 Записей в блоге: 10
	29.06.2012, 16:31
	Метод Synchronize выполняет переданную ему функцию в контексте основного потока, а у тебя вся работа вынесена в эту функцию. 0

@BRcr 4043 / 2333 / 292 Регистрация: 03.02.2011 Сообщений: 5,066 Записей в блоге: 10
	29.06.2012, 16:36
	Теперь нагрузку распределять по-уму, чего делать... Все обращения к vcl через Synchronize(), остальное в Execute(). 1

@BRcr 4043 / 2333 / 292 Регистрация: 03.02.2011 Сообщений: 5,066 Записей в блоге: 10
	29.06.2012, 18:08
	Vovik_0_1, ну, и как? Виснет теперь? 0

@Avazart 8488 / 6155 / 615 Регистрация: 10.12.2010 Сообщений: 28,683 Записей в блоге: 30
	29.06.2012, 21:54
	Нет, потому что это не рабочая туфта 0

@Vovik_0_1 40 / 40 / 6 Регистрация: 28.07.2011 Сообщений: 431
	29.06.2012, 21:59 [ТС]
	Да нет вроде! Не по теме: Поток останавливать нужно или он сам завершится когда код закрнчится? 0

@Avazart 8488 / 6155 / 615 Регистрация: 10.12.2010 Сообщений: 28,683 Записей в блоге: 30
	29.06.2012, 22:54
	После delete данные могут оставаться не тронутыми но уже как мусор 0