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

@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();
}

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

Новые блоги и статьи Все статьи Все блоги /
Модель ЗдрввоСохранения 7: больше работников, больше ресурсов. anaschu 08.04.2026 работников и заданий может быть сколько угодно, но настроено всё так, что используется пока что только 20%	Дальние перспективы сервера - слоя сети с космологическим дизайном интефейса карты и логики. Hrethgir 07.04.2026 Дальнейшее ближайшее планирование вывело к размышлениям над дальними перспективами. И вот тут может быть даже будут нужны оценки специалистов, так как в дальних перспективах всё может очень сильно. . .	Горе от ума kumehtar 07.04.2026 Эта мне ментальная установка, что вот прямо сейчас, мол, мне для полного счастья не хватает (нужное вписать), и когда я этого достигну - тогда и полный кайф. Одна из самых сильных ловушек на пути. . . .	Использование значений реквизитов справочника в документе, с определенными условиями и правами Maks 07.04.2026 1. Контроль срока действия договора Алгоритм из решения ниже реализован на примере нетипового документа "ЗаявкаНаРаботу", разработанного в конфигурации КА2. Задача: уведомлять пользователя, если. . .
Доступность команды формы по условию Maks 07.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "СписаниеМатериалов", разработанного в конфигурации КА2. Задача: сделать доступной кнопку (команда формы "ЗавершитьСписание") при. . .	Уведомление о неверно выбранном значении справочника Maks 06.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "НарядПутевка", разработанного в конфигурации КА2. Задача: уведомлять пользователя, если в документе выбран неверный склад. . .	Установка Qt Creator для C и C++: ставим среду, CMake и MinGW без фреймворка Qt 8Observer8 05.04.2026 Среду разработки Qt Creator можно установить без фреймворка Qt. Есть отдельный репозиторий для этой среды: https:/ / github. com/ qt-creator/ qt-creator, где можно скачать установщик, на вкладке Releases:. . .	AkelPad-скрипты, структуры, и немного лирики.. testuser2 05.04.2026 Такая программа, как AkelPad существует уже давно, и также давно существуют скрипты под нее. Тем не менее, прога живет, периодически что-то не спеша дополняется, улучшается. Что меня в первую очередь. . .

@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