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

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

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

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

18.02.2014, 21:47. Просмотров 340. Ответов 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;
}



В чем ошибка?
0
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
18.02.2014, 21:47
Здравствуйте! Я подобрал для вас темы с ответами на вопрос Переполнение стека во время освобождения памяти (C++):

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

Деструкторы, аналоги освобождения памяти - C++
Доброго времени суток. Возник следующий вопрос. Какие, помимо деструкторов, способы очистки памяти существуют. Спасибо за уделенное...

Стратегия выделения/освобождения памяти - C++
здравствуйте господа. есть такой вопрос. пусть есть класс например Bitmap - несжатое растровое изображение. Пусть необходимо уметь...

Ошибка освобождения памяти new/delete - C++
При выполнении оператора deleterez вылетает ошибка BLOCK_TYPE_IS_VALID(pHead-&gt;nBlockUse) #include &lt;iostream&gt; using namespace std;...

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

Указатель после освобождения памяти - C++
Стивен Прата, глава 4, раздел &quot;Освобождение памяти с помощью операции delete&quot; Я пытаюсь понять. Написал: #include &lt;iostream&gt; ...

Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
Kuzia domovenok
1891 / 1746 / 118
Регистрация: 25.03.2012
Сообщений: 5,925
Записей в блоге: 1
18.02.2014, 22:58 #2
Не программа, а адский бред. Ты не показал описание класса, но я догадываюсь, что у тебя там
C++
1
2
class TUchenik{
  TUchenik* Uchenik;//массив
видно же, что это бесконечная рекурсия! Деструктор класса вызывает деструкторы членов, и.т.д. до бесконечности!
1
DrOffset
7146 / 4287 / 966
Регистрация: 30.01.2014
Сообщений: 7,090
18.02.2014, 23:16 #3
Пока не увижу весь код предполагать ничего не буду. Спрошу только вот что:

Не по теме:


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

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

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

Преподаватели)))
0
Tulosba
:)
Эксперт С++
4396 / 3239 / 297
Регистрация: 19.02.2013
Сообщений: 9,045
19.02.2014, 14:38 #6
Цитата Сообщение от casper007 Посмотреть сообщение
Преподаватели)))
Пора создавать раздел с доской позора.
0
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
19.02.2014, 14:38
Привет! Вот еще темы с ответами:

Переполнение стека - C++
Всем добрый вечер. Я создаю вектор, который имеет большой размер: порядка 256000000. Этот вектор имеет тип float, т.к. функция, куда я...

Переполнение стека - C++
Привет народ. Такой вопрос: Если в общем виде: if (условие) double d else double d почему компилятор отказывается выполнять такое с...

Переполнение стека - C++
Всем привет. Помогите, пожалуйста с решением одной проблемы. Мне нужно в программе обрабатывать большие объемы текста. 10 000 000 символов....

Переполнение стека - C++
Есть функция f(): void f() { //... std::make_pair&lt;size_t, size_t&gt; pairReturner(); //... } Если пускать ее в цикле от...


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

Или воспользуйтесь поиском по форуму:
Yandex
Объявления
19.02.2014, 14:38
Ответ Создать тему
Опции темы

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