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

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

Войти
Регистрация
Восстановить пароль
 
 
Buildog
40 / 40 / 2
Регистрация: 29.04.2012
Сообщений: 185
#1

Утечка памяти - C++

12.11.2012, 18:39. Просмотров 1036. Ответов 27
Метки нет (Все метки)

Доброго времени суток, форумчане. Помогите справиться с утечкой памяти, не понятно где зараза засела. Это класс представления числа в СОК (система остаточных классов) На вход подается лишь одно число и возникает проблема, если это (число < 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;
}
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
12.11.2012, 18:39
Здравствуйте! Я подобрал для вас темы с ответами на вопрос Утечка памяти (C++):

Утечка памяти - C++
Подскажите, где здесь может возникать утечка памяти? Вроде везде, где можно - чищу... ///////////////////////////////////// // It's...

Утечка памяти - C++
Либо я себе мозг запудрила, либо помогите мне :) есть у меня вектор vector&lt;char*&gt; names_variable; дополняю его таким образом ...

Утечка памяти - C++
Здравствуйте. Есть программа (разбитая на функции) постоянно работающая в цикле и выводящая информацию на консоль. И получается что она...

Утечка памяти - C++
Взял листинг кода из книги Стивен Прата Язык программирования С++. Меня терзают смутные сомнения нет ли здесь утечки памяти, указатель ...

Утечка памяти - C++
Доброго времени суток! Столкнулся с проблемой утечки памяти! Будь у меня маленькая программка, выловил бы легко, но в 2000 строк кода все...

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

Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
Buildog
40 / 40 / 2
Регистрация: 29.04.2012
Сообщений: 185
12.11.2012, 19:26  [ТС] #16
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
Модератор
Эксперт CЭксперт С++
13154 / 7417 / 831
Регистрация: 27.09.2012
Сообщений: 18,254
Записей в блоге: 3
Завершенные тесты: 1
12.11.2012, 19:34 #17
Переделал немного конструктор с параметром и функцию 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
430 / 365 / 37
Регистрация: 06.08.2012
Сообщений: 961
12.11.2012, 19:35 #18
Buildog,
C++
1
  Count = NumbCount;
а NumbCount = ???
Croessmah
Модератор
Эксперт CЭксперт С++
13154 / 7417 / 831
Регистрация: 27.09.2012
Сообщений: 18,254
Записей в блоге: 3
Завершенные тесты: 1
12.11.2012, 19:38 #19
Цитата Сообщение от SeregaC++ Посмотреть сообщение
а NumbCount = ???
а что NumbCount?
Issues
430 / 365 / 37
Регистрация: 06.08.2012
Сообщений: 961
12.11.2012, 19:41 #20
Croessmah, когда я убрал эту строку, ошибка пропала.

Добавлено через 1 минуту
NumbCount ведь не инициализирован.
Croessmah
Модератор
Эксперт CЭксперт С++
13154 / 7417 / 831
Регистрация: 27.09.2012
Сообщений: 18,254
Записей в блоге: 3
Завершенные тесты: 1
12.11.2012, 19:41 #21
Цитата Сообщение от SeregaC++ Посмотреть сообщение
Croessmah, когда я убрал ету строку, ошибка пропала.
Потому что получилось:
C++
1
int *B = new int (0);
Buildog
40 / 40 / 2
Регистрация: 29.04.2012
Сообщений: 185
12.11.2012, 19:42  [ТС] #22
NumbCount в классе
Croessmah
Модератор
Эксперт CЭксперт С++
13154 / 7417 / 831
Регистрация: 27.09.2012
Сообщений: 18,254
Записей в блоге: 3
Завершенные тесты: 1
12.11.2012, 19:43 #23
Цитата Сообщение от SeregaC++ Посмотреть сообщение
NumbCount ведь не инициализирован.
Смотрите код внимательнее. В каждом конструкторе есть вызов:
C++
1
ChangeNumb (NumbC);
Добавлено через 51 секунду
Цитата Сообщение от Buildog Посмотреть сообщение
NumbCount в классе
Вы делали так:
C++
1
int *B = new int (Count);
а нужны квадратные скобки
Issues
430 / 365 / 37
Регистрация: 06.08.2012
Сообщений: 961
12.11.2012, 19:43 #24
Croessmah, сорри, я просто уже не знаю что и думать.
Croessmah
Модератор
Эксперт CЭксперт С++
13154 / 7417 / 831
Регистрация: 27.09.2012
Сообщений: 18,254
Записей в блоге: 3
Завершенные тесты: 1
12.11.2012, 19:44 #25
Проблема решена?
Buildog
40 / 40 / 2
Регистрация: 29.04.2012
Сообщений: 185
12.11.2012, 19:45  [ТС] #26
Ё моё... ахринеть... спасибо большое. Уже второй раз на этом попался... Получается я удалял за границами массива
Issues
430 / 365 / 37
Регистрация: 06.08.2012
Сообщений: 961
12.11.2012, 19:47 #27
Цитата Сообщение от Croessmah Посмотреть сообщение
Вы делали так:
C++
1
int *B = new int (Count);
а нужны квадратные скобки
Блииин, я тоже сначала подумал, что ошибка здесь. Но потом передумал. Вот я лошара.
Buildog
40 / 40 / 2
Регистрация: 29.04.2012
Сообщений: 185
12.11.2012, 19:48  [ТС] #28
Всем большое спасибо за участие
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
12.11.2012, 19:48
Привет! Вот еще темы с ответами:

Утечка памяти - C++
Господа подскажите как узнать какая п̶а̶д̶л̶а̶ переменная жрет память? Или как посмотреть сколько вообще переменных и объектов находятся в...

Утечка памяти - C++
Помогите с кодом, вот написал реализацию стека по заданию. Преподаватель попросил проверить, верно ли я использую delete, ибо mas в классе...

Утечка памяти - C++
Здравствуйте! Я пишу длинную арифметику для целых чисел. Ниже - функция произведения чисел X и Y. Вопрос: произойдет ли утечка памяти, и...

Утечка памяти - C++
Где я допустил ее(утечку)? #define _CRT_SECURE_NO_WARNINGS #include &lt;iostream&gt; #include &lt;cstring&gt; #include &lt;string&gt; class...


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

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

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