Генерация лабиринта. Ошибка с памятью. Двумерным динамический массив

@Artawower · Регистрация: 03.04.2016

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

Добрый день, двумерный динамический массив до этого не юзал, понадобился для реализации генерации лабиринта в глубину по графу. Без использования динамического массива программа работает стабильно, кроме того заметил что при первом запуске программа всегда работает нормально(есть предположение что какая-то переменная засирается, но я не нашел какая..) Собственно кривенький код прилагаю (та часть что в комментах вроде как причина еррора)

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
//---------------------------------------------------------------------------
 
#include <vcl.h>
#pragma hdrstop
 
#include "Unit2.h"
//---------------------------------------------------------------------------
#pragma package(smart_init)
#pragma resource "*.dfm"
TForm2 *Form2;
//---------------------------------------------------------------------------
__fastcall TForm2::TForm2(TComponent* Owner)
    : TForm(Owner)
{
}
//---------------------------------------------------------------------------
void __fastcall TForm2::Button1Click(TObject *Sender)
{
    PaintBox1->Canvas->Brush->Color = clWhite;
    PaintBox1->Canvas->Rectangle(0, 0, PaintBox1->Height, PaintBox1->Width);
    int N = StrToInt(Edit1->Text);
    int **mas = new int*[N];
    for (int i = 0; i < N; i++)
    {
        mas[i] = new int[N];
    }
 
    for (int i = 0; i < N; i++)
    {
        for (int j = 0; j < N; j++)
        {
            mas[j][i] = -1;
        }
    }
 
    for (int i = 1; i < N - 1; i += 2 )
    {
        for (int j = 1; j < N - 1; j += 2)
        {
             mas[j][i] = -2;
        }
    }
 
int forward = random(N - 2);
 
int side = 0, ni = 0, nj = 0;
    while (mas[nj][ni] != -2)
    {
        side = random(4);
        forward = random(N);
        if (side == 0)
        {
            ni = 1;
            nj = forward;
            if (mas[nj][ni] == -2)
            {
                mas[forward][0] = 0;
            }
        }
        if (side == 1)
        {
            ni = N - 3;
            nj = forward;
            if (mas[nj][ni] == -2)
            {
                mas[forward][N - 2] = 0;
            }
        }
        if (side == 2)
        {
            ni = forward;
            nj = 1;
            if (mas[nj][ni] == -2)
            {
                mas[0][forward] = 0;
            }
        }
        if (side == 3)
        {
            ni = forward;
            nj = N - 3;
            if (mas[nj][ni] == -2)
            {
                mas[N - 2][forward] = 0;
            }
        }
    }
int enj = N - 1, eni = N - 1;
    while (mas[enj][eni] != -2)
    {
        side = random(4);
        forward = random(N);
        if (side == 0)
        {
            eni = 1;
            enj = forward;
            if (mas[enj][eni] == -2)
            {
                mas[forward][0] = 0;
            }
        }
        if (side == 1)
        {
            eni = N - 3;
            enj = forward;
            if (mas[enj][eni] == -2)
            {
                mas[forward][N - 2] = 0;
            }
        }
        if (side == 2)
        {
            eni = forward;
            enj = 1;
            if (mas[enj][eni] == -2)
            {
                mas[0][forward] = 0;
            }
        }
        if (side == 3)
        {
            eni = forward;
            enj = N - 3;
            if (mas[enj][eni] == -2)
            {
                mas[N - 2][forward] = 0;
            }
        }
    }
 
mas[nj][ni] = 0;
 
boolean checked_end = true;
 
while (checked_end == true)
    {
        forward = random(4);
    if ((mas[nj + 2][ni] == -2) | (mas[nj - 2][ni] == -2) | (mas[nj][ni + 2] == -2)
        | (mas[nj][ni - 2] == -2))
        {
            if ((forward == 0) && (mas[nj + 2][ni] == -2))
            {
                mas[nj + 2][ni] = 0;
                mas[nj + 1][ni] = 0;
                nj += 2;
            }
            else
 
            if ((forward == 1) && (mas[nj - 2][ni] == -2))
            {
                mas[nj - 2][ni] = 0;
                mas[nj - 1][ni] = 0;
                nj -= 2;
            }
 
            if ((forward == 2) && (mas[nj][ni + 2] == -2))
            {
                mas[nj][ni + 2] = 0;
                mas[nj][ni + 1] = 0;
                ni += 2;
            }
            else
 
            if ((forward == 2) && (mas[nj][ni - 2] == -2))
            {
                mas[nj][ni - 2] = 0;
                mas[nj][ni - 1] = 0;
                ni -= 2;
            }
 
        }
        else
        {
        checked_end = false;
        }
    }
 
 
/*
boolean CHECKED_0 = true;
    while (CHECKED_0 == true)
    {
        for (int i = 0; i < N; i++)
        {
            for (int j = 0; j < N; j++)
            {
                if (mas[j][i] == 0)
                {
                    if ((mas[j + 2][i] == -2) | (mas[j - 2][i] == -2) |
                     (mas[j][i + 2] == -2) | (mas[j][i - 2] == -2))
                    {
                        forward = random(4);
                        if ((forward == 0) && (mas[j + 2][i] == -2))
                        {
                            mas[j + 2][i] = 0;
                            mas[j + 1][i] = 0;
                        }
                        else if ((forward == 1) && (mas[j - 2][i] == -2))
                        {
                            mas[j - 2][i] = 0;
                            mas[j - 1][i] = 0;
                        }
                        if ((forward == 2) && (mas[j][i + 2] == -2))
                        {
                            mas[j][i + 2] = 0;
                            mas[j][i + 1] = 0;
                        }
                        if ((forward == 3) && (mas[j][i - 2] == -2))
                        {
                            mas[j][i - 2] = 0;
                            mas[j][i - 1] = 0;
                        }
                    }
                }
            }
        }
 
        CHECKED_0 = false;
        for (int i = 0; i < N; i++)
        {
           for (int j = 0; j < N; j++)
           {
                if (mas[j][i] == -2)
                {
                    CHECKED_0 = true;
                    break;
                }
           }
        }
 
    }      */
 
 
 
Label1->Caption = IntToStr(forward);
    int scale = (PaintBox1->Height / N);
    for (int i = 0; i < N; i++)
    {
        for (int j = 0; j < N; j++)
        {
            if (mas[j][i] == -1)
            {
                PaintBox1->Canvas->Brush->Color = clBlack;
                PaintBox1->Canvas->Rectangle(i*scale, j*scale, i*scale+scale, j*scale+scale);
            }
            if (mas[j][i] == 0)
            {
                PaintBox1->Canvas->Brush->Color = clPurple;
                PaintBox1->Canvas->Rectangle(i*scale, j*scale, i*scale+scale, j*scale+scale);
            }
        }
    }
delete mas;
//mas = nullptr;
}
 
 
//---------------------

Исходник

mimicria · 12.04.2016, 08:57

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

delete mas;

Основное правило - как память выделялась, так она и освобождаться должна.
В цикле сначала delete [] mas[i]; потом delete [] mas;

@Artawower · 12.04.2016, 17:27 **[ТС]**

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

и опять же при первом запуске все работает.

Добавлено через 24 минуты
Нашел ошибку, ошибка в логике была небольшая, в массиве присутствовало 2 нуля (начало и конец лабиринта) не вписывающихся в общий алгоритм.

Новые блоги и статьи Все статьи Все блоги /
Камера 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. Пошагово создадим проект для загрузки изображения. . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL3_image 8Observer8 10.02.2026 Содержание блога Библиотека SDL3_image содержит инструменты для расширенной работы с изображениями. Пошагово создадим проект для загрузки изображения формата PNG с альфа-каналом (с прозрачным. . .	Установка Qt-версии Lazarus IDE в Debian Trixie Xfce volvo 10.02.2026 В общем, достали меня глюки IDE Лазаруса, собранной с использованием набора виджетов Gtk2 (конкретно: если набирать текст в редакторе и вызвать подсказку через Ctrl+Space, то после закрытия окошка. . .	SDL3 для Web (WebAssembly): Работа со звуком через SDL3_mixer 8Observer8 08.02.2026 Содержание блога Пошагово создадим проект для загрузки звукового файла и воспроизведения звука с помощью библиотеки SDL3_mixer. Звук будет воспроизводиться по клику мышки по холсту на Desktop и по. . .

@Artawower 1 / 1 / 0 Регистрация: 03.04.2016 Сообщений: 16
	12.04.2016, 17:27 [ТС]
	Толком ничего не изменилось, по сути при нажатии кнопки под массив должна выделяться память заного, не уверен что проблема в этом. Трабл где-то в закоменченном коде, но я его не вижу и опять же при первом запуске все работает. Добавлено через 24 минуты Нашел ошибку, ошибка в логике была небольшая, в массиве присутствовало 2 нуля (начало и конец лабиринта) не вписывающихся в общий алгоритм. 0