Утечка памяти

@Buildog · Регистрация: 29.04.2012

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

Доброго времени суток, форумчане. Помогите справиться с утечкой памяти, не понятно где зараза засела. Это класс представления числа в СОК (система остаточных классов) На вход подается лишь одно число и возникает проблема, если это (число < 2310) то всё нормально и отлично, при этом получается 6 остатков, но как только (число >= 2310), тогда количество остатков становится больше 6, и выбивает исключение об утечки памяти, после оператора return в main.cpp

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
#include "SOK.h"
#include <iostream>
 
using namespace std;
 
void main ()
{
    int number=0;
    cout<<"Enter number=";
    cin>>number;
    SOK numb (number);
    numb.Show();
    system ("pause");
    return;
}

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
#ifndef _SOK_H
#define _SOK_H
 
class SOK
{
public:
    SOK(int number);
    SOK ();
    ~SOK();
    int Show () const;
    void ChangeNumb (int Numb);
    SOK (const SOK &);
 
private:
    void Create (int n);
    int *pOsnova;
    int *pOstatok;
    int NumbCount;
};
 
#endif

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
#include "SOK.h"
#include <iostream>
 
using namespace std;
 
SOK::SOK()
{
    int NumbC = 1;
    ChangeNumb (NumbC);
    pOstatok = new int [NumbC];
    pOsnova = new int [NumbC];
    pOsnova[0]=0;
    pOstatok[0]=0;
}
 
SOK::SOK (int n)
{
    Create(n);
}
 
SOK::~SOK ()
{
    delete []pOstatok;
    pOstatok = 0;
    delete []pOsnova;
    pOsnova=0;
}
 
int SOK::Show () const
{
    if (pOsnova[0] == 0 && pOstatok[0]== 0)
    {
        cout << 0 << endl;
        return 0;
    }
    int P=1;
    int NUMBER = 0;
    int Count = 0;
    Count = NumbCount;
    for (int i = 0; i<Count; i++)
    {
        P *= pOsnova[i];
    }
    int *B = new int (Count);
    int z = 0;
    for (int i = 0; i < Count; i++)
    {
        z = 0;
        z = B[i] = P/pOsnova[i];
        while (B[i]%pOsnova[i] != 1)
        {
            B[i] += z;
        }
    }
    for (int i = 0; i < Count; i++)
    {
        NUMBER += B[i] * pOstatok [i];
    }
    while (true)
    {
        if (NUMBER < 0) 
        {
            NUMBER += P;
            cout << NUMBER << endl;
            return NUMBER;
        }
 
        NUMBER -= P;
    }
}
 
void SOK::ChangeNumb (int Numb)
{
    NumbCount = Numb;
}
 
void SOK::Create (int n)
{
    if (n==0) 
    SOK();
    int A[]={2, 3, 5, 7, 11, 13, 17, 19};
    int NumbC = 0;
    int count = 1; 
    for (int i=0; i<sizeof(A)/sizeof(int); i++)
    {
        if (n>count)
        {
            count *= A[i];
            NumbC++;
        }
    }
    NumbC++;
    ChangeNumb (NumbC);
    pOstatok = new int [NumbC];
    pOsnova = new int [NumbC];
    cout<<"Ostatok { ";
    for (int i=0; i<NumbC; i++)
    {
        pOstatok[i]=n%A[i];
        pOsnova[i] = A[i];
        cout<<pOstatok[i]<<" ";
    }
    cout<<"}"<<endl;
}

@Issues · 12.11.2012, 18:45

Мне кажется, или нужно сначала присваивать 0, а потом удалять?

@Buildog · 12.11.2012, 18:50 **[ТС]**

Дело не в деструкторе, я уверен в этом на 90%, а в функции Show, поскольку если я запускаю без неё, то всё нормально проходит.

@Issues · 12.11.2012, 18:54

C++
1
 int *B = new int (Count);

где удаление?

@Buildog · 12.11.2012, 18:57 **[ТС]**

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

@Issues · 12.11.2012, 19:03

Ну не знаю... Вот вы два раза виделяете память для:

C++
1
2
pOstatok = new int [NumbC];
pOsnova = new int [NumbC];

@Buildog · 12.11.2012, 19:04 **[ТС]**

Ну так память выделяется на 2 разных будущих массива и она потом в деструкторе освобождается, ошибку выбивает уже после того как прошел return, то есть после выполнения деструктора

@Kuzia domovenok · 12.11.2012, 19:04

Наверняка выход за пределы массива.
Я ставил точки останова в конструкторе, Create и деструкторе
вводил 6553 - вылетает, но
Удаляются точно те адреса, что были выделены. Значит память не терялась.
Наверняка выход за пределы массива.
К таким вещам:

C++
1
2
3
4
5
6
7
8
 for (int i=0; i<sizeof(A)/sizeof(int); i++)
    {
        if (n>count)
        {
            count *= A[i];
            NumbC++;
        }
    }

К таким вещам я крайне подозрительно отношусь. Проблема наверняка тут

@Buildog · 12.11.2012, 19:08 **[ТС]**

Сообщение от Kuzia domovenok

Наверняка выход за пределы массива.
Я ставил точки останова в конструкторе, Create и деструкторе
вводил 6553 - вылетает, но
Удаляются точно те адреса, что были выделены. Значит память не терялась.
Наверняка выход за пределы массива.
К таким вещам:

C++
1
2
3
4
5
6
7
8
 for (int i=0; i<sizeof(A)/sizeof(int); i++)
    {
        if (n>count)
        {
            count *= A[i];
            NumbC++;
        }
    }

К таким вещам я крайне подозрительно отношусь. Проблема наверняка тут

Что именно тут может быть не так? И если ошибка здесь, то она почему то проявляется только тогда, когда если функция show ? Если это функции нет, то всё нормально с любыми числами в пределах диапазона. Убрал функцию Show из main.cpp, запустил с числом 6553, ничего не выбило, отлично отработало, а как только обратно решил использовать Show, тут же получил исключение

@Issues · 12.11.2012, 19:13

Buildog,

C++
1
int *B = new int (Count);

находится именно в Show, попробуйте в конце освободить память для него.

Добавлено через 54 секунды
Ведь там утечка тоже происходит.

@Buildog · 12.11.2012, 19:13 **[ТС]**

Сообщение от SeregaC++

Buildog,

C++
1
int *B = new int (Count);

находится именно в Show, попробуйте в конце освободить память для него.

Если бы в этом дело было, я бы давно исправил, строки освобождения памяти

C++
1
2
    delete []B;
    B = 0;

уже добавлены, а ошибка всё равно не исчезла

@Kuzia domovenok · 12.11.2012, 19:15

а как насчёт delete[] B; в функции Show() ?
опоздал

Croessmah · 12.11.2012, 19:20

C++
1
2
3
4
void SOK::Create (int n)
{
    if (n==0) 
    SOK();

Добавлено через 2 минуты
Уберите Вызов конструктора в методе!!!

@Buildog · 12.11.2012, 19:22 **[ТС]**

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

C++
1
2
3
4
void SOK::Create (int n)
{
    if (n==0) 
    SOK();

Добавлено через 2 минуты
Уберите Вызов конструктора в методе!!!

Убрал, всё равно исключение есть

HEAP[Lab1 SOK.exe]: Invalid address specified to RtlValidateHeap( 006B0000, 006BA070 )
Lab1 SOK.exe вызвал срабатывание точки останова.

Croessmah · 12.11.2012, 19:22

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

Убрал, всё равно исключение есть

Киньте весь код, а то Вы так и будете её ловить!

@Buildog · 12.11.2012, 19:26 **[ТС]**

main.cpp

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
#include "SOK.h"
#include <iostream>
 
using namespace std;
 
void main ()
{
    int number=0;
    cout<<"Enter number=";
    cin>>number;
    SOK numb (number);
    numb.Show();
    system ("pause");
    return;
}

SOK.h

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
#ifndef _SOK_H
#define _SOK_H
 
class SOK
{
public:
    SOK(int number);
    SOK ();
    ~SOK();
    int Show () const;
    void ChangeNumb (int Numb);
    SOK (const SOK &);
 
private:
    void Create (int n);
    int *pOsnova;
    int *pOstatok;
    int NumbCount;
};
 
#endif

SOK.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
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
#include "SOK.h"
#include <iostream>
 
using namespace std;
 
SOK::SOK()
{
    int NumbC = 1;
    ChangeNumb (NumbC);
    pOstatok = new int [NumbC];
    pOsnova = new int [NumbC];
    pOsnova[0]=0;
    pOstatok[0]=0;
}
 
SOK::SOK (int n)
{
    Create(n);
}
 
SOK::~SOK ()
{
    delete []pOstatok;
    pOstatok = 0;
    delete []pOsnova;
    pOsnova=0;
}
 
SOK::SOK (const SOK & rhs)
{
    NumbCount = rhs.NumbCount;
    pOsnova = new int [NumbCount];
    pOstatok = new int [NumbCount];
    for (int i=0; i<NumbCount; i++)
    {
        pOsnova[i] = rhs.pOsnova[i];
        pOstatok[i] = rhs.pOstatok[i];
    }
}
 
int SOK::Show () const
{
    if (pOsnova[0] == 0 && pOstatok[0]== 0)
    {
        cout << 0 << endl;
        return 0;
    }
    int P=1;
    int NUMBER = 0;
    int Count = 0;
    Count = NumbCount;
    for (int i = 0; i<Count; i++)
    {
        P *= pOsnova[i];
    }
    int *B = new int (Count);
    int z = 0;
    for (int i = 0; i < Count; i++)
    {
        z = 0;
        z = B[i] = P/pOsnova[i];
        while (B[i]%pOsnova[i] != 1)
        {
            B[i] += z;
        }
    }
    for (int i = 0; i < Count; i++)
    {
        NUMBER += B[i] * pOstatok [i];
    }
    while (true)
    {
        if (NUMBER < 0) 
        {
            NUMBER += P;
            cout << NUMBER << endl;
            return NUMBER;
        }
 
        NUMBER -= P;
    }
    delete []B;
    B = 0;
}
 
void SOK::ChangeNumb (int Numb)
{
    NumbCount = Numb;
}
 
void SOK::Create (int n)
{
    //if (n==0) 
    //SOK();
    int A[]={2, 3, 5, 7, 11, 13, 17, 19};
    int NumbC = 0;
    int count = 1; 
    for (int i=0; i<sizeof(A)/sizeof(int); i++)
    {
        if (n>count)
        {
            count *= A[i];
            NumbC++;
        }
    }
    NumbC++;
    ChangeNumb (NumbC);
    pOstatok = new int [NumbC];
    pOsnova = new int [NumbC];
    cout<<"Ostatok { ";
    for (int i=0; i<NumbC; i++)
    {
        pOstatok[i]=n%A[i];
        pOsnova[i] = A[i];
        cout<<pOstatok[i]<<" ";
    }
    cout<<"}"<<endl;
}

Croessmah · 12.11.2012, 19:34

Переделал немного конструктор с параметром и функцию show
sok.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
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
#include "SOK.h"
#include <iostream>
 
using namespace std;
 
SOK::SOK()
{
    int NumbC = 1;
    ChangeNumb (NumbC);
    pOstatok = new int [NumbC];
    pOsnova = new int [NumbC];
    pOsnova[0]=0;
    pOstatok[0]=0;
}
 
SOK::SOK (int n)
{
    if (n==0){
        int NumbC = 1;
        ChangeNumb (NumbC);
        pOstatok = new int [NumbC];
        pOsnova = new int [NumbC];
        pOsnova[0]=0;
        pOstatok[0]=0;
    }else{
        Create(n);
    }
}
 
SOK::~SOK ()
{
    delete []pOstatok;
    pOstatok = 0;
    delete []pOsnova;
    pOsnova=0;
}
 
SOK::SOK (const SOK & rhs)
{
    NumbCount = rhs.NumbCount;
    pOsnova = new int [NumbCount];
    pOstatok = new int [NumbCount];
    for (int i=0; i<NumbCount; i++)
    {
        pOsnova[i] = rhs.pOsnova[i];
        pOstatok[i] = rhs.pOstatok[i];
    }
}
 
int SOK::Show () const
{
    if (pOsnova[0] == 0 && pOstatok[0]== 0)
    {
        cout << 0 << endl;
        return 0;
    }
    int P=1;
    int NUMBER = 0;
    int Count = 0;
    Count = NumbCount;
    for (int i = 0; i<Count; i++)
    {
        P *= pOsnova[i];
    }
    int *B = new int [Count];//Квадратные скобки должны быть!!!
    int z = 0;
    for (int i = 0; i < Count; i++)
    {
        z = 0;
        z = B[i] = P/pOsnova[i];
        while (B[i]%pOsnova[i] != 1)
        {
            B[i] += z;
        }
    }
    for (int i = 0; i < Count; i++)
    {
        NUMBER += B[i] * pOstatok [i];
    }
    while (true)
    {
        if (NUMBER < 0) 
        {
            NUMBER += P;
            cout << NUMBER << endl;
            return NUMBER;
        }
 
        NUMBER -= P;
    }
    delete [] B;
    B = 0;
}
 
void SOK::ChangeNumb (int Numb)
{
    NumbCount = Numb;
}
 
 
 
 
void SOK::Create (int n)
{
    int A[]={2, 3, 5, 7, 11, 13, 17, 19};
    int NumbC = 0;
    int count = 1; 
    for (int i=0; i<sizeof(A)/sizeof(int); i++)
    {
        if (n>count)
        {
            count *= A[i];
            NumbC++;
        }
    }
    NumbC++;
    ChangeNumb (NumbC);
    pOstatok = new int [NumbC];
    pOsnova = new int [NumbC];
    cout<<"Ostatok { ";
    for (int i=0; i<NumbC; i++)
    {
        pOstatok[i]=n%A[i];
        pOsnova[i] = A[i];
        cout<<pOstatok[i]<<" ";
    }
    cout<<"}"<<endl;
}

@Issues · 12.11.2012, 19:35

Buildog,

C++
1
  Count = NumbCount;

а NumbCount = ???

Croessmah · 12.11.2012, 19:38

Сообщение от SeregaC++

а NumbCount = ???

а что NumbCount?

@Issues · 12.11.2012, 19:41

Croessmah, когда я убрал эту строку, ошибка пропала.

Добавлено через 1 минуту
NumbCount ведь не инициализирован.

Новые блоги и статьи Все статьи Все блоги /
Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro	Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .	От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .	Поиск "дружественных имён" СОМ портов Argus19 22.11.2025 Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .	Сколько Государство потратило денег на меня, обеспечивая инсулином. Programma_Boinc 20.11.2025 Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .
Ломающие изменения в C#.NStar Alpha Etyuhibosecyu 20.11.2025 Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .	Мысли в слух kumehtar 18.11.2025 Кстати, совсем недавно имел разговор на тему медитаций с людьми. И обнаружил, что они вообще не понимают что такое медитация и зачем она нужна. Самые базовые вещи. Для них это - когда просто люди. . .	Создание Single Page Application на фреймах krapotkin 16.11.2025 Статья исключительно для начинающих. Подходы оригинальностью не блещут. В век Веб все очень привыкли к дизайну Single-Page-Application . Быстренько разберем подход "на фреймах". Мы делаем одну. . .	Фото: Daniel Greenwood kumehtar 13.11.2025	Расскажи мне о Мире, бродяга kumehtar 12.11.2025 — Расскажи мне о Мире, бродяга, Ты же видел моря и метели. Как сменялись короны и стяги, Как эпохи стрелою летели. - Этот мир — это крылья и горы, Снег и пламя, любовь и тревоги, И бескрайние. . .

@Issues 433 / 368 / 149 Регистрация: 06.08.2012 Сообщений: 961
	12.11.2012, 18:45
	Мне кажется, или нужно сначала присваивать 0, а потом удалять? 0

@Buildog 41 / 41 / 5 Регистрация: 29.04.2012 Сообщений: 185
	12.11.2012, 18:50 [ТС]
	Дело не в деструкторе, я уверен в этом на 90%, а в функции Show, поскольку если я запускаю без неё, то всё нормально проходит. 0

@Buildog 41 / 41 / 5 Регистрация: 29.04.2012 Сообщений: 185
	12.11.2012, 18:57 [ТС]
	Строку это добавить не помешает, но дело не в этом указателе, я объявлял даже статичным массив этот, всё равно не помогло. 0

@Buildog 41 / 41 / 5 Регистрация: 29.04.2012 Сообщений: 185
	12.11.2012, 19:04 [ТС]
	Ну так память выделяется на 2 разных будущих массива и она потом в деструкторе освобождается, ошибку выбивает уже после того как прошел return, то есть после выполнения деструктора 0

@Kuzia domovenok 4268 / 3327 / 926 Регистрация: 25.03.2012 Сообщений: 12,531 Записей в блоге: 1
	12.11.2012, 19:15
	а как насчёт delete[] B; в функции Show() ? опоздал 0

@Issues 433 / 368 / 149 Регистрация: 06.08.2012 Сообщений: 961
	12.11.2012, 19:41
	Croessmah, когда я убрал эту строку, ошибка пропала. Добавлено через 1 минуту NumbCount ведь не инициализирован. 0