Работа с 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 данные могут оставаться не тронутыми но уже как мусор

Новые блоги и статьи Все статьи Все блоги /
Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Programma_Boinc 28.12.2025 Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Налог на собак: https:/ / ********/ gallery/ V06K53e Финансовый отчет в Excel: https:/ / ********/ gallery/ bKBkQFf Пост отсюда. . .	Кто-нибудь знает, где можно бесплатно получить настольный компьютер или ноутбук? США. Programma_Boinc 26.12.2025 Нашел на реддите интересную статью под названием Anyone know where to get a free Desktop or Laptop? Ниже её машинный перевод. После долгих разбирательств я наконец-то вернула себе. . .	Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка. Programma_Boinc 23.12.2025 Рецензия / Мнение/ Перевод Нашел на реддите интересную статью под названием The Thinkpad X220 Tablet is the best budget school laptop period . Ниже её машинный перевод. Thinkpad X220 Tablet —. . .	PhpStorm 2025.3: WSL Terminal всегда стартует в ~ and_y87 14.12.2025 PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .	Как объединить две одинаковые БД Access с разными данными VikBal 11.12.2025 Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.
Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro	Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .	От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .	Поиск "дружественных имён" СОМ портов Argus19 22.11.2025 Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .	Сколько Государство потратило денег на меня, обеспечивая инсулином. Programma_Boinc 20.11.2025 Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .

@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 8484 / 6151 / 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 8484 / 6151 / 615 Регистрация: 10.12.2010 Сообщений: 28,683 Записей в блоге: 30
	29.06.2012, 22:54
	После delete данные могут оставаться не тронутыми но уже как мусор 0