Форум программистов, компьютерный форум CyberForum.ru

С++ для начинающих

Войти
Регистрация
Восстановить пароль
 
casper007
67 / 67 / 22
Регистрация: 12.12.2013
Сообщений: 400
#1

Переполнение стека во время освобождения памяти - C++

18.02.2014, 21:47. Просмотров 330. Ответов 5
Метки нет (Все метки)

Здравствуйте! СТолкнулся с такой проблемой, что при работе деструктора программа аварийно завершается, а в дебаггере появляется сообщение о том, что стек заполнен. Оказалось, что дело в освобождении памяти оператором delete. Я закомментировал, и все работает, правда с утечкой памяти. MVS 2010 C++ .
Детруктор:
C++
1
2
3
4
5
6
7
TUchenik::~TUchenik()
{
    if (HF!=NULL)       {DeleteObject(HF); HF=NULL;}
    if (DC!=NULL)       {DeleteObject(DC); DC=NULL;}
    if (Uchenik!=NULL)
    {delete []Uchenik; Uchenik=NULL;};
}
Место, где память выделяется:
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
void TUchenik::Jornal(void)
{
    if(Uchenik != NULL) {delete []Uchenik; Uchenik = NULL;};
    wifstream File;
    locale loc("rus_rus.1251");
    File.imbue(loc);
    File.open(NameOfFile);
    while(!File.eof())
    {
        
        File>>name; File>>surname; File>>klass; File>>bukva;
        if ((wcslen(name))==0) break;
        NNN++;
    }
    File.close();
    Uchenik =new TUchenik[NNN];
    File.open(NameOfFile);
    for (int i=0; i<NNN;i++)
    {
        File>>Uchenik[i].name;
        File>>Uchenik[i].surname;
        File>>Uchenik[i].klass;
        File>>Uchenik[i].bukva;
        if (wcslen(Uchenik[i].name)==0) 
            break;
    }
    File.close();
}


Функция главного окна (на всякий случай)
Кликните здесь для просмотра всего текста
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
LRESULT CALLBACK Glaw_Win(HWND Hwnd,UINT message,WPARAM wParam, LPARAM lParam)
{static TUchenik NameFile;
static RECT Rect = {0L}; //будем определять разммер рабочей области
static SCROLLINFO sinfo;
int off, tmp;
static int FlagPaint=0;
 
 switch(message)
         {
         case  WM_CREATE:
                  {  
                    GetClientRect(Hwnd,&Rect);
                    SetRect(&Rect,Rect.top+10, Rect.left+10, Rect.bottom-10, Rect.right-10);
                    NameFile.NameFile(HProg, Hwnd);
                    NameFile.Jornal();
                   break;}                                                             //подготовит файл настроек
 
         case  WM_PAINT:
                  { 
                     if (FlagPaint == 0) NameFile.ReadFile(Hwnd);  
                     if(FlagPaint==1) NameFile.FormTalon(Hwnd);
                   break;}
 
         case  WM_DESTROY:
                {
                    //NameFile.~TUchenik();
                    PostQuitMessage(0);
                break;}
 
         case  WM_COMMAND:
             {switch(LOWORD(wParam))
                 {
                     case IDM_PRAVKA:    { CreateChildWindow(); break; }
                     case IDM_LIST:      {FlagPaint = 0; InvalidateRect(Hwnd,NULL,1);break;}
 
                     case IDM_TALON:     {FlagPaint=1;  InvalidateRect(Hwnd,NULL,1);break;}
     
                     case IDM_SET_DATE:  {CreateDateWindow();break;}
                }
             break;} //close CASE of WM_COMMAND
    
             case WM_VSCROLL :
                {
                    sinfo.cbSize=sizeof(SCROLLINFO); //получаем размер структуры
                    sinfo.fMask = SIF_ALL; // Определяет параметры полосы прокрутки для установки или получения. Этот элемент может быть комбинацией следующих значений:
                    GetScrollInfo(Hwnd,SB_VERT, &sinfo);
                    off = sinfo.nPos;RECT rcUpdate;
                       switch (LOWORD (wParam))
                            {
                                case SB_THUMBTRACK:
                                sinfo.nPos = sinfo.nTrackPos;
                            break;
 
                                case SB_LINEUP:
                                    sinfo.nPos -= 2;
                                    ScrollWindowEx(Hwnd,0,15,&Rect,NULL,NULL,&rcUpdate,0 );
                                break;
 
                                case SB_LINEDOWN:
                                    sinfo.nPos += 2;
                                    ScrollWindowEx(Hwnd,0,-15,&Rect,NULL,NULL,&rcUpdate,0 );
                                break;
                             }
               
                       SetScrollInfo(Hwnd, SB_VERT,&sinfo,TRUE);
                       tmp=GetScrollInfo(Hwnd,SB_VERT,&sinfo);
                       GetClientRect(Hwnd,&Rect);
                       //ScrollWindowEx(Hwnd,0,1,&Rect,NULL,NULL,&rcUpdate,0 );
                       InvalidateRect(Hwnd,&rcUpdate,TRUE);
                       UpdateWindow(Hwnd);
                break;}
             default:{return(DefWindowProc(Hwnd,message,wParam,lParam));}
        }
 
    return 0L;
}



В чем ошибка?
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
18.02.2014, 21:47     Переполнение стека во время освобождения памяти
Посмотрите здесь:

Нюансы выделения и освобождения памяти (структуры) - C++
struct point { int a; point *next; } point *p=new point Я освобожу всю память которая отводилась под динам. структуру? delete...

Функциии динамического выделения и освобождения памяти - C++
Необходим написать функции выделения и освобождения памяти под двумерный динамический массив. Причем освобождение должн быть не сразу для...

Использование __try __finaly для освобождения памяти - C++
Здравствуйте. Подскажите, как возможно использовать конструкцию __try, __finaly для освобождения ресурсов в одном месте. Пример: ...

Переполнение стека из-за передачи струтуры - C++
у меня происходит переполнение стека из-за передачи большой структуры. и не только из-за структуры пытался передать большую строку тоже...

Почему возникает переполнение стека??? - C++
У меня есть программа которая выводит все возможные перестановки заданного слова(числа) в данном случае числа 123, но почему то возникает...

Диалоговое окно, переполнение стека - C++
У меня создается диалоговое окно (через ресурсы и DialogBox). Я к нему леплю ф-ию обработки сообщений. Все норм. Но стоит дописать в конце...

После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
Kuzia domovenok
1890 / 1745 / 118
Регистрация: 25.03.2012
Сообщений: 5,924
Записей в блоге: 1
18.02.2014, 22:58     Переполнение стека во время освобождения памяти #2
Не программа, а адский бред. Ты не показал описание класса, но я догадываюсь, что у тебя там
C++
1
2
class TUchenik{
  TUchenik* Uchenik;//массив
видно же, что это бесконечная рекурсия! Деструктор класса вызывает деструкторы членов, и.т.д. до бесконечности!
DrOffset
7058 / 4199 / 949
Регистрация: 30.01.2014
Сообщений: 6,965
18.02.2014, 23:16     Переполнение стека во время освобождения памяти #3
Пока не увижу весь код предполагать ничего не буду. Спрошу только вот что:

Не по теме:


Кто вас всех учит вот этой вот проверке переде delete/free?

C++
1
2
3
4
if(p)
{
    delete p;
}
Чесслово какой-то тотальный пипец

Kuzia domovenok
1890 / 1745 / 118
Регистрация: 25.03.2012
Сообщений: 5,924
Записей в блоге: 1
18.02.2014, 23:42     Переполнение стека во время освобождения памяти #4
DrOffset, телепатия подсказывает, что у него там что-то вроде:
C++
1
2
3
4
5
6
class TUchenik{
  TUchenik* Uchenik;
  public:
    ~TUchenik(){
      delete[] Uchenik;
//рекурсия в деструкторе!
casper007
67 / 67 / 22
Регистрация: 12.12.2013
Сообщений: 400
19.02.2014, 08:58  [ТС]     Переполнение стека во время освобождения памяти #5
Цитата Сообщение от Kuzia domovenok Посмотреть сообщение
DrOffset,
//рекурсия в деструкторе![/CPP]
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
class TUchenik
{
public:
    
    TUchenik(void);
    ~TUchenik(void);
    int ReadFile(HWND);
    int Write_and_Out(HWND);
    void NameFile(HINSTANCE, HWND);
    void FormTalon(HWND);
    void Jornal(void);
private:
    WCHAR numb[4];  wchar_t name[40];   wchar_t surname[60]; wchar_t klass[4];   WCHAR bukva[2];   WCHAR identifier[100]; // puple data
    int NNN; 
    wchar_t kol_tal[1]; wchar_t NameOfFile[50]; string Name_Sys_File;// job of file
};


Кликните здесь для просмотра всего текста
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
static HFONT HF; static LOGFONT LF; static HDC DC; static PAINTSTRUCT ps;
 
TUchenik *Uchenik = NULL;
 
 
 
int TUchenik::ReadFile(HWND Hwnd)
{   
    /*//_wsetlocale(LC_ALL, L"Russian"); locale loc ("rus_rus.866");
    //int i=1;
    wifstream File;
    locale loc("rus_rus.1251");
    File.imbue(loc);
    File.open(NameOfFile);
    //File.open(NameOfFile);
    if(File.fail()) {MessageBoxW(HWND_DESKTOP,L"Файл не существует или пустой!",L"Ошибка открытия файла!", MB_OK); return 1;}
    else
    {
        while(!File.eof())
        { 
            File>>name; 
            File>>surname;
            File>>klass;
            File>>bukva;
            if(wcslen(name)==0) return 1;
            else */
 
    DC=BeginPaint(Hwnd,&ps);
    SelectObject(DC,HF);
    TextOutW(DC, 10,20,L"Список класса", 13);
    MoveToEx(DC, 10,38, NULL); LineTo(DC,650,38);
 
    TextOutW(DC, 30 , 42, L"N п-п"    , 5);
    TextOutW(DC,    165,    42, L"Имя"      ,3);
    TextOutW(DC, 380, 42, L"Фамилия", 7);
    TextOutW(DC, 515, 42, L"Класс"  , 5);
    TextOutW(DC, 600, 42,L"Буква",    5);
    MoveToEx(DC,  10, 60  , NULL); LineTo(DC,650,60);
    for(int i=0;i<NNN;i++)
            {
                _itow(i+1,numb,10);
                TextOutW(DC, 40, 46+(i+1)*20-2,numb, wcslen(numb));
                TextOutW(DC,150, 46+(i+1)*20-2,  Uchenik[i].name   , wcslen(Uchenik[i].name)  );
                TextOutW(DC,380, 46+(i+1)*20-2, Uchenik[i].surname, wcslen(Uchenik[i].surname));
                TextOutW(DC,530, 46+(i+1)*20-2, Uchenik[i].klass,   wcslen(Uchenik[i].klass)  );
                TextOutW(DC, 615,46+(i+1)*20-2,Uchenik[i].bukva,   wcslen(Uchenik[i].bukva)  );
 
                MoveToEx(DC, 10, 40+(i+1)*20, NULL); LineTo(DC,650,40+(i+1)*20);
 
                MoveToEx(DC,  10, 40, NULL); LineTo(DC,  10, 40+(i+2)*20);
                MoveToEx(DC, 95, 40, NULL); LineTo(DC, 95, 40+(i+2)*20);
                MoveToEx(DC, 295, 40, NULL); LineTo(DC,295, 40+(i+2)*20);
                MoveToEx(DC, 495, 40, NULL); LineTo(DC, 495, 40+(i+2)*20);
                MoveToEx(DC,570,  40,NULL); LineTo(DC,570,   40+(i+2)*20);
                MoveToEx(DC,650,  40,NULL); LineTo(DC,650,   40+(i+2)*20);
                MoveToEx(DC,10,40+(i+2)*20,NULL);   LineTo(DC,65040+(i+2)*20);
            }
        
    return 0;
}
 
//////////////////////////////////////////////////////////////////////////////////
 
int TUchenik::Write_and_Out(HWND HWin1)
{
    TUchenik Uchenik;   
    {wofstream File; locale loc("rus_rus.1251");
    File.imbue(loc);
    File.open(NameOfFile, ios::app);
    GetDlgItemText(HWin1,IDD_EDIT1,Uchenik.name    ,52);
    GetDlgItemTextW(HWin1,IDD_EDIT2,Uchenik.surname ,70);
    GetDlgItemTextW(HWin1,IDD_EDIT3,Uchenik.klass, 4);
    GetDlgItemText(HWin1,IDD_EDIT4,Uchenik.bukva,2);
 
    File<<Uchenik.name<<"\t";
    File<<Uchenik.surname<<"\t";
    File<<Uchenik.klass<<"\t";
    File<<Uchenik.bukva<<"\n";
    File.close();
    }
    Jornal();
    return 0;
}
 
//////////////////////////////////////////////////////////////////////////
 
void TUchenik::NameFile(HINSTANCE HProg, HWND HW)
{
    wifstream SysFile;
    SysFile.open(Name_Sys_File);
    if (SysFile.fail())
    {
        //SysFile.close();
        DialogBox(HProg, L"My_Dialog_File",HW, (DLGPROC) My_Dialog_Proc);//create system file
    }
    else 
    {//wifstream SysFile(*Name_Sys_File);
        SysFile>>NameOfFile>>kol_tal;
        SysFile.close();
    }
    SysFile.open(Name_Sys_File);
    SysFile>>NameOfFile>>kol_tal;
    SysFile.close();
}
 
///////////////////////////////////////////////////////////////
 
 
void TUchenik::FormTalon(HWND Hwnd)
{
    /*TUchenik *Uchenik;
    Uchenik = new TUchenik[NNN];wifstream File;
    locale loc("rus_rus.1251");
    File.imbue(loc);
    File.open(NameOfFile);*/
    DC=BeginPaint(Hwnd,&ps);
    SelectObject(DC,HF);
    RECT Rect;
    int JJJ=0; _itow(JJJ,kol_tal,10);
    for (int i=0;i<NNN;i++)
        {
            //File>>Uchenik[i].name>>Uchenik[i].surname>>Uchenik[i].klass>>Uchenik[i].bukva;
        for (int j=0;j<7;j++)
        {
            GetClientRect(Hwnd,&Rect);
            MoveToEx(DC,10+(j*150), Rect.top+(i*150),NULL);     LineTo(DC,10+(j+1)*150,Rect.top+(i*150));// ----
            MoveToEx(DC,10+(j*150), Rect.top+(i*150),NULL);     LineTo(DC,10+(j*150),Rect.top+((i+1)*150)); // |
            MoveToEx(DC,10+(j*150), Rect.top+(150+(i*150)),NULL);   LineTo(DC, 10+(j+1)*150,Rect.top+(150+(i*150)));// ___
            MoveToEx(DC,10+((j+1)*150), Rect.top+(i*150),NULL); LineTo(DC,10+((j+1)*150),Rect.top+((i+1)*150)); // ||
            TextOutW(DC,30+(j*150), Rect.top+30+(i*150),Uchenik[i].name,wcslen(Uchenik[i].name));
            TextOutW(DC,30+(j*150), Rect.top+50+(i*150),Uchenik[i].surname,wcslen(Uchenik[i].surname));
            TextOutW(DC,30+(j*150), Rect.top+70+(i*150),Uchenik[i].klass,wcslen(Uchenik[i].klass));
            TextOutW(DC,60+(j*150), Rect.top+70+(i*150),Uchenik[i].bukva,wcslen(Uchenik[i].bukva));
 
            /*MoveToEx(DC,10+(i*100), Rect.top+(j*100),NULL);       LineTo(DC,10+(i+1)*100,Rect.top+(j*100));// ----
            MoveToEx(DC,10+(i*100), Rect.top+(j*100),NULL);     LineTo(DC,10+(i*100),Rect.top+((j+1)*100)); // |
            MoveToEx(DC,10+(i*100), Rect.top+(100+(j*100)),NULL);   LineTo(DC, 10+(j+1)*100,Rect.top+(100+(j*100)));// ___
            */
            //MoveToEx(DC,Rect.left+i, Rect.top+j,NULL); LineTo(DC,Rect.left+i,Rect.top+j);
        }
    }
        EndPaint(Hwnd,&ps);
        //delete []Uchenik;
}
 
 
 
 
 
//////////////////////////////////////////////////////////////
TUchenik::TUchenik()
{
 
    LF.lfHeight = 20;
    LF.lfWidth = 8;
    LF.lfEscapement = 0;
    LF.lfOrientation =0;
    LF.lfWeight = FW_MEDIUM;
    LF.lfItalic=0;
    LF.lfUnderline=0;
    LF.lfStrikeOut=0;
    LF.lfCharSet=DEFAULT_CHARSET;
    LF.lfOutPrecision = OUT_DEVICE_PRECIS;
    LF.lfClipPrecision = CLIP_STROKE_PRECIS;
    LF.lfQuality= PROOF_QUALITY;
    LF.lfPitchAndFamily = VARIABLE_PITCH | FF_MODERN;
    wcscpy_s(LF.lfFaceName,L"Calibry");//MS_Sans_Serif   wcscpy вместо strcpy
    //LF.lfFaceName=MS_Sans_Serif;
    HF=CreateFontIndirect(&LF);
    Name_Sys_File = "setting.tal";
    NNN=0;
}
 
 
TUchenik::~TUchenik()
{
    if (HF!=NULL)       {DeleteObject(HF); HF=NULL;}
    if (DC!=NULL)       {DeleteObject(DC); DC=NULL;}
    //if (Uchenik!=NULL)
    //{delete []Uchenik; Uchenik=NULL;};
}
 
 
 
void TUchenik::Jornal(void)
{
    //if(Uchenik != NULL) {delete []Uchenik; Uchenik = NULL;};
    wifstream File;
    locale loc("rus_rus.1251");
    File.imbue(loc);
    File.open(NameOfFile);
    while(!File.eof())
    {
        
        File>>name; File>>surname; File>>klass; File>>bukva;
        if ((wcslen(name))==0) break;
        NNN++;
    }
    File.close();
    Uchenik =new TUchenik[NNN];
    File.open(NameOfFile);
    for (int i=0; i<NNN;i++)
    {
        File>>Uchenik[i].name;
        File>>Uchenik[i].surname;
        File>>Uchenik[i].klass;
        File>>Uchenik[i].bukva;
        if (wcslen(Uchenik[i].name)==0) 
            break;
    }
    File.close();
}


Добавлено через 17 минут
Проблему решил простым созданием структуры. Всем спасибо!!!

Добавлено через 22 секунды
Проблему решил простым созданием структуры. Всем спасибо!!!

Добавлено через 7 часов 42 минуты
Цитата Сообщение от DrOffset Посмотреть сообщение
Кто вас всех учит вот этой вот проверке переде delete/free?

Преподаватели)))
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
19.02.2014, 14:38     Переполнение стека во время освобождения памяти
Еще ссылки по теме:

Быстрая сортировка. Переполнение стека - C++
Написал программу быстрой сортировки происходит переполнение стека, при большом количестве значений. Кто чем может помочь исправить эту...

Не могу разобраться с указателями(не понимаю сути освобождения памяти) - C++
Вот я тоже дуб в указателях... помогите пожалуйста кому не лениво... с горем пополам заработал Enter (ноуту пора в утиль), и вот (вопросы...

Адреса памяти, в которых происходит выделение, и нет освобождения - C++
Добрый день форумчане. Уже несколько недель ломаю себе голову над одной задачей. За меня делать ничего не прошу, так как в принципе проект...

Необходим совет при освобождения памяти динамического массива - C++
Имеется код: -создание трехмерного динамического массива float ***Kadr; Kadr = new float **; for (int i = 0; i &lt; D-&gt;Np; i++)...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
Tulosba
:)
Эксперт С++
4393 / 3236 / 297
Регистрация: 19.02.2013
Сообщений: 9,045
19.02.2014, 14:38     Переполнение стека во время освобождения памяти #6
Цитата Сообщение от casper007 Посмотреть сообщение
Преподаватели)))
Пора создавать раздел с доской позора.
Yandex
Объявления
19.02.2014, 14:38     Переполнение стека во время освобождения памяти
Ответ Создать тему
Опции темы

КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2017, vBulletin Solutions, Inc.
Рейтинг@Mail.ru