Ошибка при работе с динамическими компонентами

@snoopi · Регистрация: 04.01.2010

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

Делаю менюшку для игры.
Пункты меню - Label и Image. При нажатии на какой либо пункт меню - очищаю форму от всего и создаю новые пункты меню.
И все казалось бы легко и просто пока движешься по меню вперед, а вот когда возвращаешься назат то после прорисовки возникает ошибка "access violation at address 40005905 in module 'rtl60.bpl'"
Выкладываю код как есть:

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
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
//---------------------------------------------------------------------------
 
#include <vcl.h>
#pragma hdrstop
 
#include "Unit1.h"
#include "Unit3.h"
#include "Unit4.h"
#include "Unit2.h"
#include "Game.h"
//---------------------------------------------------------------------------
#pragma package(smart_init)
#pragma resource "*.dfm"
TForm1 *Form1;
//void __fastcall *(char* p);
//void __fastcall (*pf)(char*);
//---------------------------------------------------------------------------
 
void del_comp(int c=0)
{
 int buf = Form1->ComponentCount;
 for(int i=c; i<buf; i++)
  Form1->Components[c]->Free();
 Form1->Refresh();
}
 
TADOTable *TForm1::Create_table(AnsiString TableName, TForm *form)
{
        TADOTable *tab1 = new TADOTable(form);
        tab1->ConnectionString = "Provider=Microsoft.Jet.OLEDB.4.0;Data Source="
                 +ExtractFilePath(Application->ExeName)
                 +"\Game.mdb;Persist Security Info=False";
        tab1->TableName = TableName;
        tab1->Active = true;
        tab1->First();
        return tab1;
}
 
void TForm1::print_menu()
{
          del_comp();
          //ShowMessage(Form1->ComponentCount);
          TADOTable *tab = Create_table("Setting",Form1);
          int cout=0;
          while(!tab->Eof)
          {
           cout++;
           tab->Next();
          }
          tab->First();
          Form1->Height = (1+cout)*(40)+100;
 
          int MenyTop = Form1->Height/2;
          MenyTop += ((1+cout)/2*(35+5))-35;
 
          TLabel *lab =  new TLabel(Form1);
          lab->Parent = Form1;
          lab->Font->Charset = RUSSIAN_CHARSET;
          lab->Font->Name = "Times New Roman";
          lab->Transparent = true;
          lab->BringToFront();
          lab->Font->Size = 10;
          lab->Caption = "Ñîçäàòü ïðîôèëü";
          lab->Left = Form1->Width/2 - lab->Width/2;
          lab->Top = MenyTop;
          lab->OnClick = Form1->Label1Click;
           //-----
          TImage *img = new TImage(Form1);
          img->Picture->LoadFromFile("C:\\1.bmp");
          img->Width = 130;
          img->Height = 35;
          img->Transparent = true;
          img->Stretch = true;
          img->Parent = Form1;
          img->Top = lab->Top - lab->Height/2;
          img->Left = Form1->Width/2 - img->Width/2;
          lab->BringToFront();
          int i=1;
          while(!tab->Eof)
          {
           //-----
           TLabel *lab =  new TLabel(Form1);
           lab->Parent = Form1;
           lab->Font->Charset = RUSSIAN_CHARSET;
           lab->Font->Name = "Times New Roman";
           lab->Transparent = true;
           lab->BringToFront();
           lab->Font->Size = 10;
           lab->Caption = tab->Fields->Fields[1]->AsString;
           lab->Left = Form1->Width/2 - lab->Width/2;
           lab->Top = MenyTop - i*(lab->Height+25);
           lab->OnClick = Form1->LabClick;
           //-----
           TImage *img = new TImage(Form1);
           img->Picture->LoadFromFile("C:\\1.bmp");
           img->Width = 130;
           img->Height = 35;
           img->Transparent = true;
           img->Stretch = true;
           img->Parent = Form1;
           img->Top = lab->Top - lab->Height/2;
           img->Left = Form1->Width/2 - img->Width/2;
           lab->BringToFront();
           tab->Next();
           i++;
          };
          tab->Free();
}
 
void print_menu2()
{
          del_comp();
          //ShowMessage(Form1->ComponentCount);
          AnsiString buf[4]={"Íàçàä","Íàñòðîéêè","Èãðà Ñëîâà","Èãðà Ãîðîäà"};
          Form1->Height = 4*(40)+100;
 
          int MenyTop = Form1->Height/2;
          MenyTop += 45;
 
          for(int i=0; i<4; i++)
          {
           TLabel *lab =  new TLabel(Form1);
           lab->Caption = buf[i];
           lab->Top = MenyTop - i*(lab->Height+25);
           lab->Left = Form1->Width/2 - lab->Width/2;
           lab->Font->Charset = RUSSIAN_CHARSET;
           lab->Font->Name = "Times New Roman";
           lab->Transparent = true;
           lab->Font->Size = 10;
           lab->Parent = Form1;
           switch (i)
           {
            case 0: lab->OnClick = Form1->LabBack; break;
            case 1: lab->OnClick = Form1->LabBack; break;
            case 2: lab->OnClick = Form1->LabMenu3; break;
            case 3: lab->OnClick = Form1->LabMenu3; break;
           }
           //---
           TImage *img = new TImage(Form1);
           img->Picture->LoadFromFile("C:\\1.bmp");
           img->Width = 130;
           img->Height = 35;
           img->Transparent = true;
           img->Stretch = true;
           img->Parent = Form1;
           img->Top = lab->Top - lab->Height/2;
           img->Left = Form1->Width/2 - img->Width/2;
           lab->BringToFront();
          }
}
 
void print_menu3()
{
          del_comp();
          AnsiString buf[3]={"Íàçàä","Ñåòåâàÿ èãðà","Êîìïàíèÿ"};
          Form1->Height = 3*(40)+100;
 
          int MenyTop = Form1->Height/2;
          MenyTop += (3/2*(35+5))-35;
 
          for(int i=0; i<3; i++)
          {
           TLabel *lab =  new TLabel(Form1);
           lab->Caption = buf[i];
           lab->Top = MenyTop - i*(lab->Height+25);
           lab->Left = Form1->Width/2 - lab->Width/2;
           lab->Font->Charset = RUSSIAN_CHARSET;
           lab->Font->Name = "Times New Roman";
           lab->Transparent = true;
           lab->Font->Size = 10;
           lab->Parent = Form1;
           switch (i)
           {
            case 0: lab->OnClick = Form1->Lab2Back; break;
            case 1: lab->OnClick = Form1->LabSeti; break;
            case 2: lab->OnClick = Form1->LabKompania; break;
            //case 3: lab->OnClick = Form1->LabBack; break;
           }
           //---
           TImage *img = new TImage(Form1);
           img->Picture->LoadFromFile("C:\\1.bmp");
           img->Width = 130;
           img->Height = 35;
           img->Transparent = true;
           img->Stretch = true;
           img->Parent = Form1;
           img->Top = lab->Top - lab->Height/2;
           img->Left = Form1->Width/2 - img->Width/2;
           lab->BringToFront();
          }
}
 
 
__fastcall TForm1::TForm1(TComponent* Owner)
        : TForm(Owner)
{
}
 
void back_image()
{
        //for(int )
}
 
void __fastcall TForm1::LabKompania(TObject *Sender)
{
        Form1->Hide();
        //Form2->player = ((TLabel*)Sender)->Caption;
        Form4->start_game( ((TLabel*)Sender)->Caption );
        Form4->Show();
}
 
void __fastcall TForm1::LabSeti(TObject *Sender)
{
        Form1->Hide();
        Form2->Show();
}
 
void __fastcall TForm1::LabMenu3(TObject *Sender)
{
        print_menu3();
}
 
void __fastcall TForm1::LabClick(TObject *Sender)
{
        print_menu2();
}
 
void __fastcall TForm1::Lab2Back(TObject *Sender)
{
        print_menu2();
        //ShowMessage("It's OK");
}
 
void __fastcall TForm1::LabBack(TObject *Sender)
{
        Form1->Close();
}
 
//---------------------------------------------------------------------------
 
void TForm1::FormArc(TForm *form)
{
        HRGN Region;
        Region = CreateRoundRectRgn(0,
                               0,
                               form->Width,
                               form->Height,
                               30,
                               30);
        SetWindowRgn(form->Handle, Region, true);
}
 
void __fastcall TForm1::FormCreate(TObject *Sender)
{
        print_menu();
        FormArc(Form1);
}
//---------------------------------------------------------------------------
void __fastcall TForm1::FormActivate(TObject *Sender)
{
        //print_menu();
}
//---------------------------------------------------------------------------
 
void __fastcall TForm1::Label1Click(TObject *Sender)
{
        Form1->Hide();
        Form3->Show();
}
//---------------------------------------------------------------------------
 
void __fastcall TForm1::FormKeyPress(TObject *Sender, char &Key)
{
        if(Key == VK_ESCAPE) Form1->Close();
}
//---------------------------------------------------------------------------

Помогите пожалуйста, у меня уже мозги плавятся

. почти всё перепробовал

@Maluda · 19.06.2011, 21:40

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

C++
1
2
3
4
5
6
7
void del_comp(int c=0)
{
 int buf = Form1->ComponentCount;
 for(int i=c; i<buf; i++)
  Form1->Components[c]->Free();
 Form1->Refresh();
}

Весь твой жуткий код разбирать лень, но вот здесь сразу ошибка!!!

Я думаю вот здесь у тебя должно было быть вот так

C++
1
Components[i]

@snoopi · 19.06.2011, 21:46 **[ТС]**

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

Весь твой жуткий код разбирать лень, но вот здесь сразу ошибка!!!

Я думаю вот здесь у тебя должно было быть вот так

C++
1
Components[i]

в этой процедуре ошибке нету, она все подчищает под 0.
а если удалять по Components[i] то выдаст ошибку, поскольку при удалении 1 компонента происходит сдвиг вниз всех остальных. хотя можно делать и так, но тогда надо начинать с конца а не с начала.

Вообще я заметил что эту ошибке билдер выдает через раз, и в непредсказуемом месте. Хоть и код там даже и не трогал(

@Maluda · 19.06.2011, 21:56

Да, согласен, ты своим кодом и меня заморочил,
я не подумал о том, что элементы удаляются и изменяется значение ComponentCount

C++
1
2
3
while (Form1->ComponentCount) {
        Form1->Components[0]->Free();
}

Добавлено через 5 минут
Пару советов для тебя: используй комментарии кода, объясняющие,
что у тебя происходит и чего ты хотел добиться используя ту или иную функцию,
иначе пройдёт полгода и ты сам не разберёшься в своей программе,
не говоря уже о постороннем человеке

@snoopi · 19.06.2011, 21:57 **[ТС]**

Он ошибку выдает на 232 строке т.е. если пытаешься вернутся в первое меню.
И самое интересное что он это меню нормально прорисовывает а потом ругается.
мб попробовать просто отследить эту ошибку через catch

@Maluda · 19.06.2011, 22:05

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

просто отследить эту ошибку через catch

try catch тебе не поможет! Так как у тебя явно ошибка доступа к памяти.
То есть пытаешься обратиться к объекту через указатель, а объекта не существует,
или объект другого класса

@snoopi · 19.06.2011, 22:09 **[ТС]**

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

try catch тебе не поможет! Так как у тебя явно ошибка доступа к памяти.
То есть пытаешься обратиться к объекту через указатель, а объекта не существует,
или объект другого класса

fuck(((

что же делать с этой ошибкой?

@Maluda · 19.06.2011, 22:16

Ставь на 112 строке (где у тебя начинается) функция print_menu2 точку останова,
а затем построчно трассируй программу, хотя я вроде как ошибок в коде этой функции не вижу

Добавлено через 4 минуты

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

C++
1
2
3
4
5
6
7
8
9
10
void __fastcall TForm1::LabClick(TObject *Sender)
{
        print_menu2();
}
 
void __fastcall TForm1::Lab2Back(TObject *Sender)
{
        print_menu2();
        //ShowMessage("It's OK");
}

2 вопроса: 1) Зачем две функции, выполняющие один и тот же код
2) зачем нужно делать функцию, выполняющую другую функцию?

@snoopi · 19.06.2011, 22:25 **[ТС]**

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

Ставь на 112 строке (где у тебя начинается) функция print_menu2 точку останова,
а затем построчно трассируй программу, хотя я вроде как ошибок в коде этой функции не вижу

Добавлено через 4 минуты

2 вопроса: 1) Зачем две функции, выполняющие один и тот же код
2) зачем нужно делать функцию, выполняющую другую функцию?

согласен) я просто пишу на скорость поэтому и не замечаю такие мелочи. а вообще ошибка вылетает 1 раз в самом начале и потом все нормально идет. может быть как нибудь память увеличить или что то кроме этого

Добавлено через 5 минут
Да уж эта ошибка вылетает совершенно в разных моментах( а может и вообще не вылетает
может давай скину весь проект на мыло?

@Maluda · 19.06.2011, 22:25

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

а вообще ошибка вылетает 1 раз в самом начале

Короче, всё понятно. У тебя явно ошибка в твоей функции del_comp!!! Первый раз она вылетает, потому что или удаляет с ошибкой что-то.

А следующие разы у тебя всё нормально, так как у тебя ComponentCount равно 0 и условие просто не выполняется.

@snoopi · 19.06.2011, 22:58 **[ТС]**

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

Короче, всё понятно. У тебя явно ошибка в твоей функции del_comp!!! Первый раз она вылетает, потому что или удаляет с ошибкой что-то.

А следующие разы у тебя всё нормально, так как у тебя ComponentCount равно 0 и условие просто не выполняется.

Да ты прав. закоментиравал вызов этой функции и ошибки больше нету, но что делать с ненужными элементами?

@Maluda · 19.06.2011, 23:03

Можно всё создавать динамически, но лучше разобраться какой же именно компонент такое даёт.

@snoopi · 19.06.2011, 23:54 **[ТС]**

Всетаки дело не в той функции. По моему все это из-за того что мы удаляем компонент по которому вызываем событие(LabClick и тому подобное) ведь событие ссылается на какойто элемент которого потом несуществует.
понимаешь меня?

Добавлено через 29 минут
Все вопрос решен!
Функция удаления всех компонентов кроме того которого мы вызываем =)

C++
1
2
3
4
5
6
7
void del_comp(TObject *s)
{
 int buf = Form1->ComponentCount-1;
 for(int i=buf; i>=0; i--)
  if(s == Form1->Components[i]) ((TLabel*)s)->Visible = false;
   else Form1->Components[i]->Free();
}

А вот собственно вызов:

C++
1
2
3
4
5
void __fastcall TForm1::LabMenu3(TObject *Sender)
{
        del_comp(Sender);
        print_menu3();
}

что называется: сам спросил - сам ответил

Новые блоги и статьи Все статьи Все блоги /
http://iceja.net/ математические сервисы iceja 20.01.2026 Обновила свой сайт http:/ / iceja. net/ , приделала Fast Fourier Transform экстраполяцию сигналов. Однако предсказывает далеко не каждый сигнал (см ограничения http:/ / iceja. net/ fourier/ docs ). Также. . .	http://iceja.net/ сервер решения полиномов iceja 18.01.2026 Выкатила http:/ / iceja. net/ сервер решения полиномов (находит действительные корни полиномов методом Штурма). На сайте документация по API, но скажу прямо VPS слабенький и 200 000 полиномов. . .	Расчёт переходных процессов в цепи постоянного тока igorrr37 16.01.2026 / * Дана цепь постоянного тока с R, L, C, k(ключ), U, E, J. Программа составляет систему уравнений по 1 и 2 законам Кирхгофа, решает её и находит переходные токи и напряжения на элементах схемы. . . .	Восстановить юзерскрипты Greasemonkey из бэкапа браузера damix 15.01.2026 Если восстановить из бэкапа профиль Firefox после переустановки винды, то список юзерскриптов в Greasemonkey будет пустым. Но восстановить их можно так. Для этого понадобится консольная утилита. . .
Сукцессия микоризы: основная теория в виде двух уравнений. anaschu 11.01.2026 https:/ / rutube. ru/ video/ 7a537f578d808e67a3c6fd818a44a5c4/	WordPad для Windows 11 Jel 10.01.2026 WordPad для Windows 11 — это приложение, которое восстанавливает классический текстовый редактор WordPad в операционной системе Windows 11. После того как Microsoft исключила WordPad из. . .	Classic Notepad for Windows 11 Jel 10.01.2026 Old Classic Notepad for Windows 11 Приложение для Windows 11, позволяющее пользователям вернуть классическую версию текстового редактора «Блокнот» из Windows 10. Программа предоставляет более. . .	Почему дизайн решает? Neotwalker 09.01.2026 В современном мире, где конкуренция за внимание потребителя достигла пика, дизайн становится мощным инструментом для успеха бренда. Это не просто красивый внешний вид продукта или сайта — это. . .

@snoopi 0 / 0 / 0 Регистрация: 04.01.2010 Сообщений: 31
	19.06.2011, 21:57 [ТС]
	Он ошибку выдает на 232 строке т.е. если пытаешься вернутся в первое меню. И самое интересное что он это меню нормально прорисовывает а потом ругается. мб попробовать просто отследить эту ошибку через catch 0

@Maluda 1280 / 598 / 116 Регистрация: 18.08.2009 Сообщений: 832
	19.06.2011, 23:03
	Можно всё создавать динамически, но лучше разобраться какой же именно компонент такое даёт. 0