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

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

Восстановить пароль Регистрация
 
casper007
61 / 61 / 21
Регистрация: 12.12.2013
Сообщений: 375
18.02.2014, 21:47     Переполнение стека во время освобождения памяти #1
Здравствуйте! СТолкнулся с такой проблемой, что при работе деструктора программа аварийно завершается, а в дебаггере появляется сообщение о том, что стек заполнен. Оказалось, что дело в освобождении памяти оператором 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++
C++ Функциии динамического выделения и освобождения памяти
Переполнение стека C++
переполнение стека C++
Переполнение стека C++
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
Kuzia domovenok
 Аватар для Kuzia domovenok
1883 / 1738 / 116
Регистрация: 25.03.2012
Сообщений: 5,907
Записей в блоге: 1
18.02.2014, 22:58     Переполнение стека во время освобождения памяти #2
Не программа, а адский бред. Ты не показал описание класса, но я догадываюсь, что у тебя там
C++
1
2
class TUchenik{
  TUchenik* Uchenik;//массив
видно же, что это бесконечная рекурсия! Деструктор класса вызывает деструкторы членов, и.т.д. до бесконечности!
DrOffset
6460 / 3834 / 885
Регистрация: 30.01.2014
Сообщений: 6,629
18.02.2014, 23:16     Переполнение стека во время освобождения памяти #3
Пока не увижу весь код предполагать ничего не буду. Спрошу только вот что:

Не по теме:


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

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

Kuzia domovenok
 Аватар для Kuzia domovenok
1883 / 1738 / 116
Регистрация: 25.03.2012
Сообщений: 5,907
Записей в блоге: 1
18.02.2014, 23:42     Переполнение стека во время освобождения памяти #4
DrOffset, телепатия подсказывает, что у него там что-то вроде:
C++
1
2
3
4
5
6
class TUchenik{
  TUchenik* Uchenik;
  public:
    ~TUchenik(){
      delete[] Uchenik;
//рекурсия в деструкторе!
casper007
61 / 61 / 21
Регистрация: 12.12.2013
Сообщений: 375
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?

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

Текущее время: 04:47. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru