Динамическое выделение памяти под массив структур (new/delete)

@lordimid · Регистрация: 02.10.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
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
prsAnnouncments *wifunctions::prsFindAnnouncments(TCHAR *prsPageSource)
{
    TCHAR *lpPage;   // Указатель на текущее положение в странице
    lpPage=prsPageSource;
    prsStrings *lppsi,psi;
    lppsi=&psi; 
    prsStringsInit(lppsi);
    prsAnnouncments *PanTemp,*pan;
 
    int i=0;
    do
    {
        if(i>0)
        {
            pan = new prsAnnouncments[i];
        //  pan = prsCopyAnnounceStructs(pan,PanTemp,i);
            delete  PanTemp;
 
        }
        else
            pan = new prsAnnouncments;
 
        lpPage=prsFindString(lppsi->TextAdress,lpPage);                              
    //  lpPage=prsCopyStringToAnn(L'"',lpPage,(pan+i)->AnAddr);                      
        wcscpy(pan->AnAddr,L"DFfdsfdsfdsfdsf3443543лдодл");
 
        if(i>0)
        {
            PanTemp = new prsAnnouncments[i];
        //  PanTemp=prsCopyAnnounceStructs(PanTemp,pan,i+1);
 
            delete  pan;
        }else
        {
            PanTemp = new prsAnnouncments;
        //  *PanTemp=*pan;
        //  wcscpy_s(PanTemp->AnAddr,pan->AnAddr);
            delete pan;
        }
        
        i++;
    }while(true);
    return pan;
}

Дело в том, что когда я вызываю функцию

C++
1
        wcscpy(pan->AnAddr,L"DFfdsfdsfdsfdsf3443543лдодл");

то компилятор, когда доходит до точки delete pan;

C++
1
2
3
4
5
6
7
        if(i>0)
        {
            PanTemp = new prsAnnouncments[i];
        //  PanTemp=prsCopyAnnounceStructs(PanTemp,pan,i+1);
 
            [B]delete  pan;[/B]
        }else

выкидывает меня на кусок кода:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
void __cdecl _free_base (void * pBlock)
{
 
        int retval = 0;
 
 
        if (pBlock == NULL)
            return;
 
        RTCCALLBACK(_RTC_Free_hook, (pBlock, 0));
 
        retval = HeapFree(_crtheap, 0, pBlock);
        if (retval == 0)
        {
            errno = _get_errno_from_oserr(GetLastError());
        }
}

(файл free.c) и замирает.
В чем дело, подскажите пожалуйста? Что я делаю не так? Вся суть в том, что если элемент массива структур больше нуля, то при присвоении ему строки, память почему-то освобождается с ошибками. Если функцию

C++
1
        wcscpy(pan->AnAddr,L"DFfdsfdsfdsfdsf3443543лдодл");

убрать из кода, то ошибок не возникает.

@AlexVRud · 24.03.2016, 17:44

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

то компилятор, когда доходит до точки delete pan;

Тут и ошибка, для массива должно быть

C++
1
            delete [] pan;

@TheCalligrapher · 24.03.2016, 18:48

Бессмысленно гадать, где у вас в коде ошибка, пока вы не приведете мало-мальски осмысленный кусок кода. Что такое prsAnnouncments? Почему в вашем посте не видно полного детального объявления этого типа?

Разумеется, угадать "на кофейной гуще", что именно у вас там происходит труда не составит - ваша wcscpy переполняет целевой буфер, рушит динамическую память и все потом падает. Но тем не менее - потрудитесь постить код целиком.

И, как уже было сказано, удаление массивов в С++ делается через delete [].

Croessmah · 24.03.2016, 19:01

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

то компилятор, когда доходит до точки delete pan;

lordimid, как минимум, несоответствие new и delete

@lordimid · 25.03.2016, 09:55 **[ТС]**

Извиняюсь. Вот более развернутая версия моего примера.

файл maincode.cpp

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
#include "resource.h"
wifunctions wifunc,*wfi;
 
int TmpInt;
extern ParseConfigs apc,*lpapc;
 
 
 
int WINAPI WinMain(HINSTANCE hInst,HINSTANCE hPrevInstance,LPSTR lpszCmdLine,int CmdShow)
{
    MSG msg;
    wfi=&wifunc;
 
.....................................
 
    while(GetMessage(&msg,NULL,0,0))
    {
        TranslateMessage(&msg);
        DispatchMessage(&msg);
    }
    return msg.wParam;
}
 
 
LRESULT CALLBACK MainWindowFunction(HWND hwnd,UINT msg,WPARAM wParam,LPARAM lParam)
{
    wfi=&wifunc;
    switch(msg) 
    {
 
......................................................................
 
 
    case WM_COMMAND:
        {
            switch(LOWORD(wParam))
            {
 
                case ID_SEND:
                    {
                        wfi=&wifunc;
                        CreateThread(NULL,0,ParseThread,(LPVOID)NULL,0,&(wfi->dwIDThread));
                        return 0;
                    }
            }
 
            return 0;
        }
 
......................................................................
 
    case WM_DESTROY:
        {
            PostQuitMessage(0);
            return 0;
        }
    }
    return DefWindowProc(hwnd,msg,wParam,lParam);
}
 
 
DWORD WINAPI ParseThread(LPVOID cnt)
{
    wfi=&wifunc;
......................................................................
 
wfi->prsFindAnnouncments(PageUTF);
 
......................................................................
 
    return 0;
}

файл resource.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
29
30
31
32
33
34
35
36
37
38
39
#include <Tchar.h>
//#include <Shlwapi.h>
#include <windows.h>
#include <windowsx.h>
#include <Winbase.h>
#include <iostream>
#include <fstream>
#include <commctrl.h>
#include <WinInet.h>
#pragma comment(lib,"wininet")
 
................................................................................................ 
 
extern HWND wihw,wiEdit,hwHost,hwSend,hwDlgOptions,hwDlgEdit;
extern HINSTANCE hInstance;
 
................................................................................................ 
 
DWORD WINAPI ParseThread(LPVOID);
 
LRESULT CALLBACK MainWindowFunction(HWND,UINT,WPARAM,LPARAM);
 
struct prsAnnouncments 
{
    int AnMin,prsAnHr,prsAnY,prsAnMon,prsAnDay;          //
    TCHAR AnAddr[100];
    TCHAR AnTittle[100];
    TCHAR AnnDiscr[1000];
    int AnPrice;
};
 
................................................................................................ 
 
class wifunctions{
public:
................................................................................................ 
prsAnnouncments *prsFindAnnouncments(TCHAR *prsPageSource);   
................................................................................................ 
};

файл resourceopen.cpp

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
#include "resource.h"
................................................................................................ 
 
prsAnnouncments *wifunctions::prsFindAnnouncments(TCHAR *prsPageSource)
{
    TCHAR *lpPage;   // Указатель на текущее положение в странице
    lpPage=prsPageSource;
    prsAnnouncments *PanTemp,*pan;
 
    int i=0;
    do
    {
        if(i>0)
        {
            pan = new prsAnnouncments[i];
        //  pan = prsCopyAnnounceStructs(pan,PanTemp,i);
            if(i==1)
                delete  PanTemp;
            else
                delete[] PanTemp;
 
        }
        else
            pan = new prsAnnouncments;
 
        lpPage=prsFindString(lppsi->TextAdress,lpPage);                                  // Сначала ищем адрес обьявления
    //  lpPage=prsCopyStringToAnn(L'"',lpPage,(pan+i)->AnAddr);                         // Копируем его в структуру prsAnnouncments
        wcscpy((pan+i)->AnAddr,L"DFfdsfdsfdsfdsf3443543лдодл");
 
        if(i>0)
        {
            PanTemp = new prsAnnouncments[i];
        //  PanTemp=prsCopyAnnounceStructs(PanTemp,pan,i+1);
 
            delete[]  pan;
        }else
        {
            PanTemp = new prsAnnouncments;
        //  *PanTemp=*pan;
            wcscpy_s(PanTemp->AnAddr,pan->AnAddr);
            delete pan;
        }
        
        i++;
    }while(true);
    return pan;
}
................................................................................................

По новому стандарту С++ на сколько я знаю указывать [] после delete необязательно. Пробовал со скобками и без них - результат тот же.

@TheCalligrapher · 25.03.2016, 11:24

Ну так значит в первой версии кода вы нас просто обманывали? Там доступ в wcscpy ведется через pan, а здесь вдруг стало pan+i. Что это за неожиданные различия? В чем дело?

В последнем варианте все ясно: вы выделяете масив размера i, а потом доступаетесь к элементу pan+i - это очевидный выход за пределы массива. Потому все и падает.

P.S. В каком это новом стандарте С++ вы усмотрели разрешение не указывать [] в delete - я не знаю. Ничего подобного там нет.

Croessmah · 25.03.2016, 11:46

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

В каком это новом стандарте С++ вы усмотрели разрешение не указывать [] в delete - я не знаю.

Наверное, мы просто все отстали от жизни

@lordimid · 25.03.2016, 17:05 **[ТС]**

TheCalligrapher
Да вы правы, я перепутал. Это размер массива не нужно указывать в скобках после delete [].
Спасибо за подсказку, что-то у меня глаз замылился. Действительно проблема была в том, что я выходил за пределы массива. Блин надо же было на таком пустяке два дня убить. Еще раз благодарю.
PS
А по поводу того, что я немного функцию изменил, то это не обмана ради. Просто у меня одинаковая ошибка появлялась, поэтому я и выложил последний вариант этой функции. Так что не серчайте, обманывать я вас не хотел

.

Новые блоги и статьи Все статьи Все блоги /
Камера 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 и по. . .

@TheCalligrapher Вездепух 12937 / 6804 / 1821 Регистрация: 18.10.2014 Сообщений: 17,218
	24.03.2016, 18:48
	Бессмысленно гадать, где у вас в коде ошибка, пока вы не приведете мало-мальски осмысленный кусок кода. Что такое `prsAnnouncments`? Почему в вашем посте не видно полного детального объявления этого типа? Разумеется, угадать "на кофейной гуще", что именно у вас там происходит труда не составит - ваша `wcscpy` переполняет целевой буфер, рушит динамическую память и все потом падает. Но тем не менее - потрудитесь постить код целиком. И, как уже было сказано, удаление массивов в С++ делается через `delete []`. 0

@TheCalligrapher Вездепух 12937 / 6804 / 1821 Регистрация: 18.10.2014 Сообщений: 17,218
	25.03.2016, 11:24
	Сообщение было отмечено lordimid как решение Решение Ну так значит в первой версии кода вы нас просто обманывали? Там доступ в `wcscpy` ведется через `pan`, а здесь вдруг стало `pan+i`. Что это за неожиданные различия? В чем дело? В последнем варианте все ясно: вы выделяете масив размера `i`, а потом доступаетесь к элементу `pan+i` - это очевидный выход за пределы массива. Потому все и падает. P.S. В каком это новом стандарте С++ вы усмотрели разрешение не указывать [] в `delete` - я не знаю. Ничего подобного там нет. 1

@lordimid 6 / 5 / 1 Регистрация: 02.10.2011 Сообщений: 111
	25.03.2016, 17:05 [ТС]
	TheCalligrapher Да вы правы, я перепутал. Это размер массива не нужно указывать в скобках после delete []. Спасибо за подсказку, что-то у меня глаз замылился. Действительно проблема была в том, что я выходил за пределы массива. Блин надо же было на таком пустяке два дня убить. Еще раз благодарю. PS А по поводу того, что я немного функцию изменил, то это не обмана ради. Просто у меня одинаковая ошибка появлялась, поэтому я и выложил последний вариант этой функции. Так что не серчайте, обманывать я вас не хотел . 0

Динамическое выделение памяти под массив структур (new/delete)

Решение