Удаление динамически созданных TSpeedButton

@Chelybey · Регистрация: 27.01.2016

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

Имеется такой вот код, оптимизации 0, но сейчас не об этом.
При попытке удалить массив из But, происходит ошибка.

Project Project1.exe raised exception class EAccessViolation with 'Access violation at address 00476550 in module
'Project1.exe'. Read of address 022EC510'. Process stopped. Use Step or Run to continue.

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
//---------------------------------------------------------------------------
 
#include <vcl.h>
#pragma hdrstop
 
#include "Unit2.h"
#include "Unit1.h"
#include <Math.hpp>
//---------------------------------------------------------------------------
#pragma package(smart_init)
#pragma resource "*.dfm"
TForm2 *Form2;
TSpeedButton *But[30][30];
int O,U,W;
//---------------------------------------------------------------------------
__fastcall TForm2::TForm2(TComponent* Owner)
        : TForm(Owner)
{
 
 
}
//---------------------------------------------------------------------------
void __fastcall TForm2::FormClose(TObject *Sender, TCloseAction &Action)
{
Form1->Close();
}
//---------------------------------------------------------------------------
 
int pole(int N, int M, int Bomb)
  {
 
        randomize();
 
        int z[721];
              for(int x=1;x<=(Bomb);x++)
       {
         z[x] = RandomRange(1,N*M);
        for (int i = 1; i <= x; i++)
        {
        for (int j = i + 1; j <= x; j++)
         if (z[i] == z[j])
         { x--; }
        }
       }
    int b=1 ;
        for(int i = 0; i < N; i++)
      for(int j = 0; j < M; j++)
       {
 
        But[j][i] = new TSpeedButton(Form2);
        But[j][i]->Parent = Form2;
        But[j][i]->Top = 55 + (j * 20);
        But[j][i]->Left = 25 + (i * 20);
        But[j][i]->Width = 20;
        But[j][i]->Height = 20;
        But[j][i]->AllowAllUp = true;
        But[j][i]->NumGlyphs = 3;
        But[j][i]->GroupIndex = b;
        But[j][i]->Tag = 0;
        But[j][i]->Name = "But"+IntToStr(b);
         b++;
         for(int x=1;x<=(Bomb);x++)
         {
        if (But[j][i]->GroupIndex == z[x])
        {
        But[j][i]->Glyph->LoadFromFile("mina16x16.bmp");
        But[j][i]->NumGlyphs = 1;
       But[j][i]->Tag = 1;
        }
       }
       }
        
       Form2->Width = 20*N+62;
       Form2->Height = 20*M+105;
 
        Form2->HeaderControl1->Sections->Insert(0)->Width  = 90;
        Form2->HeaderControl1->Sections->Insert(1)->Width  = Form2->Width - 180;
 
        Form2->Image2->Parent = Form2->HeaderControl1;
        Form2->Image2->Left = 7;
        Form2->Image2->Top = 3;
 
        Form2->Image1->Parent = Form2->HeaderControl1;
        Form2->Image1->Left = Form2->StaticText2->Left + 36;
        Form2->Image1->Top = 3;
 
        Form2->Button1->Left = (Form2->Width/2-21);
    Form2->StaticText2->Caption = Bomb;
      O = N;U = M;W = Bomb; return O,U,W;
 
  }
 
void __fastcall TForm2::Timer1Timer(TObject *Sender)
{
Form2->StaticText1->Caption = Timer1->Tag++;
}
//---------------------------------------------------------------------------
 
 
 
 
void __fastcall TForm2::FormPaint(TObject *Sender)
{
for(int i=0;i<O;i++)
for(int j=0;j<U;j++)
{
   if((But[j][i]->NumGlyphs == 1)&&(But[j][i]->Down == true))
   {
   But[j][i]->Down = true;
   But[j][i]->NumGlyphs = 3;
    }
    if(But[j][i]->Down == true)
    {
     But[j][i]->AllowAllUp = false;
    }
    }
        Form2->Timer1->Enabled = true;
 
}
//---------------------------------------------------------------------------
 
 
 
void __fastcall TForm2::FormClick(TObject *Sender)
{
 /* for(int i = 0; i < O; i++)
      for(int j = 0; j < U; j++)
       {
       But[j][i]->Down = true;
       }
      */
 
}
//---------------------------------------------------------------------------
 
 
 
void __fastcall TForm2::Button1Click(TObject *Sender)
{
        for(int i=0;i<O;i++)
                for(int j=0;j<U;j++)
                {
                         if(!But[j][i] == NULL)
                         delete But[j][i];
                }
}
//---------------------------------------------------------------------------

Если будут идеи на счет оптимизации кода, приму к сведению.

Добавлено через 15 минут
С удалением получилось решить проблему своеобразным образом, но с оптимизацией всё по прежнему.

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
void __fastcall TForm2::Button1Click(TObject *Sender)
{
        for(int i=0;i<O;i++)
                for(int j=0;j<U;j++)
                {
                         if(!But[j][i]->Down == true)
                         {
                                But[j][i]->Down = true;
                         }
                         if(But[j][i]->Down == true)
                         {
                            delete But[j][i];
                         } 
                         
                }
}

@ndbn · 12.03.2016, 00:29

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

if(!But[j][i]->Down == true)
* * * * * * * * * * * * *{
* * * * * * * * * * * * * * * * But[j][i]->Down = true;
* * * * * * * * * * * * *}
* * * * * * * * * * * * *if(But[j][i]->Down == true)
* * * * * * * * * * * * *{
* * * * * * * * * * * * * * delete But[j][i];
* * * * * * * * * * * * *}

Если не true, то true и если true, то удалить.
Можно сократить до "удалить".

Я не уверен, но кажется, вы неверно применяете оператор delete, его нужно применять для динамического массива, а у вас он статический, в вашем случае его удаление нужно производить наподобии

C++
1
2
But[j][i]->Free();
But[j][i] = null;

@nick42 · 12.03.2016, 11:17

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

Если будут идеи на счет оптимизации кода, приму к сведению.

У меня немного другой подход к построению поля. Во вложении набросок на тему вашего приложения. Массив ХХ состоит из 2 (чистое поле) и 4 (поле с миной). При создании поля используется клонирование SpeedButton посредством memorystream. Глифы для нее грузятся единожды из ресурсов (загружать их в цикле из файла для каждой кнопки - это перебор, по моему).

@Chelybey · 12.03.2016, 13:16 **[ТС]**

C++
1
2
But[j][i]->Free();
But[j][i] = null;

Ещё вчера такой способ вызывал ошибку, а сейчас прокнул. Ну и муть я заварил(.

Добавлено через 38 минут
Запустил программу без компиляции прямо из архива, сработала шустро).
Но при компиляции на моём c++ Builder 6, с добавлением

C++
1
2
#include <memory>
#include <algorithm>

по скорости заполнения = изначально моему.
Похоже всё из-за разных версий Builder.

Добавлено через 8 минут
Извиняюсь, похоже это тормоза из-за запуска из под Builder. происходят тормоза.
При закрытом Builder, всё прокает.

@nick42 · 12.03.2016, 13:41

Ну, создание 900 кнопок требует времени. На моем компе сейчас - это около 4-х секунд. Можно не создавать кнопки динамически, а просто изменять их видимость (если требуются разноразмерные матрицы), а сами кнопки расставить в дизайн-режиме.

@Chelybey · 12.03.2016, 21:28 **[ТС]**

nick42, И последний вопрос.
Каким образом уже в программе можно будет изменить размер массива, например на 64, 256, 480, или на N*M, где
"M" и "N" в пределах 8 - 30.
Ведь если оставить 900, тогда при размере поля 8*8, мины буду генерироваться за пределом поля.

Добавлено через 1 час 39 минут
Хотя можно не отвечать, нормально разобраться у меня получается только в половине кода, а время не резиновое.
Проще свою багаюзную бодягу добить, нежели разбирать в том о чём вообще представления нет.
По предмету в колледже дают самые азы, а требуют вагон, да ещё и времени 1 месяц.
А тут ещё и экзамен по 6-ти предметам одновременно скоро, а к проге ещё и документацию пилить надо.
Короче ПОЛНЫЙ ******.
OFF TOP.
P.S. Спс всем кто пытался помочь.
R.I.P.

@nick42 · 12.03.2016, 23:13

.. Вот со студентами всегда так: пытаешься помочь решить задачку, идеи какие-то вытаскиваешь... - а им только хуже от этого.

Новые блоги и статьи Все статьи Все блоги /
Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. 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 Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .

@nick42 Практикантроп 4841 / 2726 / 534 Регистрация: 23.09.2011 Сообщений: 5,798
	12.03.2016, 13:41
	Ну, создание 900 кнопок требует времени. На моем компе сейчас - это около 4-х секунд. Можно не создавать кнопки динамически, а просто изменять их видимость (если требуются разноразмерные матрицы), а сами кнопки расставить в дизайн-режиме. 0

@Chelybey 1 / 1 / 0 Регистрация: 27.01.2016 Сообщений: 45
	12.03.2016, 21:28 [ТС]
	nick42, И последний вопрос. Каким образом уже в программе можно будет изменить размер массива, например на 64, 256, 480, или на NM, где "M" и "N" в пределах 8 - 30. Ведь если оставить 900, тогда при размере поля 88, мины буду генерироваться за пределом поля. Добавлено через 1 час 39 минут Хотя можно не отвечать, нормально разобраться у меня получается только в половине кода, а время не резиновое. Проще свою багаюзную бодягу добить, нежели разбирать в том о чём вообще представления нет. По предмету в колледже дают самые азы, а требуют вагон, да ещё и времени 1 месяц. А тут ещё и экзамен по 6-ти предметам одновременно скоро, а к проге ещё и документацию пилить надо. Короче ПОЛНЫЙ ******. OFF TOP. P.S. Спс всем кто пытался помочь. R.I.P. 0

@nick42 Практикантроп 4841 / 2726 / 534 Регистрация: 23.09.2011 Сообщений: 5,798
	12.03.2016, 23:13
	.. Вот со студентами всегда так: пытаешься помочь решить задачку, идеи какие-то вытаскиваешь... - а им только хуже от этого. 0