Ошибка при освобождении памяти (block type is valid) и неправильный вывод структуры данных

@mishula · Регистрация: 01.10.2014

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

Доброго времени суток.
У меня есть класс вектор для реализации длинной арифметики. Возникли проблемы с умножением. Я устроил эту функцию (как и вычитание) таким образом, что в качестве параметра по значению передается вектор, на который нужно умножить (или который нужно вычесть). Вектор, от которого вызван метод, умножается на вектор-параметр и результат в него же записывается. В случае умножения (метод multiply) мне пришлось воспользоваться вспомогательный вектором. После выполнения умножения, если вывести вспомогательный вектор, то все правильно выполняется. Но если присвоить текущему вектору вспомогательный вектор (*this = tmp

, то выводятся какие то отрицательные числа.

Вторая проблема - то же сообщении при срабатывании деструктора на этапе выхода из программы, если .

Последняя проблема тоже связана с деструктором - в методе power (возведение в степень) после захода в функцию умножения на этапе присваивания вектора tmp нашему вектору в деструкторе

C++
1
2
3
std::cout << "I am die!" << std::endl;
    delete[]numbers;
    std::cout << "Here." << std::endl;

выводится
I am die!
Here.
I am die!
в то время как при умножении, вызванном из функции main, а не power, 2 деструктора срабатывают нормально.
описание класса

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
#include <string>
#include <iostream>
#include <stdbool.h>
    class TVector {
    public:
        TVector();
        TVector(int size);
        ~TVector();
 
        void push_back(int n);
        void pop_back();
        void resize(int cap);
        int Size();
        int last();
        int& back();
        int Capacity();
        void Delete();
        bool Empty();
        int &operator[](int i);
        
        void plus(TVector v2);
        void plus2(TVector v2, int l, int r);
        void minus(TVector v2);
        void minus2(TVector *v2, int l, int r);
        void multiply(TVector v2);
        void power(TVector v2);
        void division(TVector v2, TVector *res);
 
        void push(int i, int x);
        void repair();
        TVector &operator=(TVector &v2);
        
        friend bool operator == (TVector &v, TVector &v2);
        friend bool operator <(TVector &v, TVector &v2);
        friend bool operator >(TVector &v, TVector &v2);
 
        friend std::ostream& operator<<(std::ostream& os, TVector& v);
        
        TVector *reverse();
 
        int size;
 
    private:
        int *numbers;
        int capacity;
        
        //int base;
    };

Вот функция умножения реализация методов и операторов, которые используются

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
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
#define b 10
 
TVector::TVector() {
    numbers = new int[8](); 
    size = 0;
    capacity = 8;
    std::cout << "I am alive!" << std::endl;
}
 
TVector::TVector(int newsize) {
    try {
        numbers = new int[newsize]();
    }
    catch (std::bad_alloc &e) {
        std::cerr << "Bad alloc caught:" << e.what() << std::endl;
        std::exit(0);
    }
    size = 0;
    capacity = newsize;
    
}
 
TVector::~TVector() {
    std::cout << "I am die!" << std::endl;
    delete[]numbers;
    std::cout << "Here." << std::endl;
 
}
 
void TVector::resize(int newcap) {
    capacity = newcap;
    try {
        int* temp = new int[newcap]();
        for (int i = 0; i < size; i++)
            temp[i] = numbers[i];
        
        delete [] numbers;
        numbers = temp;
    }
    catch (std::bad_alloc &e) {
        std::cerr << "Bad alloc caught:" << e.what() << std::endl;
        std::exit(0);
    }
    
 
}
 
void TVector::push_back(int n) {
    if (capacity == size + 1) {
        resize(capacity * 2);
    }
    numbers[size] = n;
    size++;
}
void TVector::repair() {
    int i = capacity-1;
    while (i>=0 && numbers[i] == 0){
        i--;
    }
    size = i+1;
}
 
void TVector::push(int i, int x) {
    if (i < capacity) {
        numbers[i] = x;
    }
    repair();
}
 
void TVector::pop_back() {
    size--;
}
 
bool TVector::Empty() {
    return size == 0;
}
int& TVector::back() {
    return numbers[size - 1];
}
 
int TVector::Size() {
    return size;
}
 
int TVector::Capacity() {
    return capacity;
}
 
int TVector::last() {
    return numbers[size - 1];
}
void TVector::Delete() {
    
    try {
                delete[] numbers;
        numbers = new int[8]();
    }
    catch(std::bad_alloc& e) {
        std::cerr << "Bad alloc caught:" << e.what() << std::endl;
        std::exit(0);
    }
    size = 0;
    capacity = 8;
}
 
TVector* TVector::reverse() {
    try {
        std::cout << "size = "<<size << std::endl;
        int* temp = new int[capacity]();
        for (int i = 0; i < size; i++)
            temp[i] = numbers[size-i-1];
        
        delete[] numbers;
        numbers = temp;
        
    }
    catch (std::bad_alloc &e) {
        std::cerr << "Bad alloc caught:" << e.what() << std::endl;
        std::exit(0);
    }
}
 
void TVector::minus(TVector v2) {
    if (*this < v2) {
        std::cout << "Error" << std::endl;
        return;
    }
    if (size > v2.Size()) {
        v2.resize(capacity);
    }
    int k = 0;
    int tmp;
    for (int i = 0; i < size; i++) {
        tmp = (numbers[i] < v2[i] ? -1 : 0);
        push(i, ost(numbers[i] - v2[i] + k, b));
        k = tmp;
    }
    
    while (!(*this).Empty() && (*this).back() == 0)
        (*this).pop_back();
    if ((*this).Empty())
        (*this).push_back(0);
}
 
void TVector::multiply(TVector v2) {
    TVector tmp;
    tmp.resize(size + v2.Size());
    for (int j = 0; j < v2.Size(); j++) {
        if (v2[j] != 0) {
            int k = 0;
            int t;
            int i;
            for (i = 0; i < size; i++) {
                t = numbers[i] * v2[j] + tmp[i + j] + k;
                tmp.push(i + j, t % b);
                std::cout << "v[" << i << "=" << tmp[i] << "v2[" << j << "=" << v2[j] << " t = " << t%b << std::endl;
                k = t / b;
            }
            if (k > 0)
                tmp.push(j + size, k);
        }
    }
    tmp.repair();
    std::cout << tmp;
    *this = tmp;
    std::cout << *this; //здесь выводит какие-то большие отрицательные числа
}
 
void TVector::power(TVector v2) {
    TVector one; //вспомогательный вектор, который отнимается у вектора степени после очередного умножения на само себя
    one.push_back(1);
    TVector tmp_res = *this;
    while (!v2.Empty()) {
        multiply(tmp_res);
        v2.minus(one);
    }
    std::cout << tmp_res;
}
 
int& TVector::operator[](int i) {
    return numbers[i];
}
 
TVector& TVector::operator=(TVector &v2) {
    std::cout << "100" << std::endl;
    if (&v2 != this) {
        std::cout << "101 cap = " << std::endl;
        
        try {
            delete[] numbers;
            size = v2.Size();
            capacity = v2.capacity;
            numbers = v2.numbers;
        }
        catch (std::bad_alloc &e) {
            std::cerr << "Bad alloc caught:" << e.what()<<std::endl;
            std::exit(0);
        }
    }
    return *this;
}
 
std::ostream& operator<<(std::ostream& os, TVector& v)
{
    int i = 0;
    while (i < v.Size()) {
        os << v[i]<< " " << std::endl;
        i++;
    }
    return os;
}
 
bool operator ==(TVector &v, TVector &v2) {
    if (v.Size() != v2.Size())
        return false;
    else {
        int i;
        for (i = 0; i < v.Size() && v[i] == v2[i]; i++)
            ;
        if (i == v.Size())
            return true;
        else
            return false;
    }
}
 
bool operator <(TVector &v, TVector &v2) {
    if (v.Size() < v2.Size()) {
        return true;
    }
    else if (v.Size() > v2.Size())
        return false;
    else {
        if (v == v2)
            return false;
        for (int i = v.Size() - 1; i>=0; i--) {
            if (v[i] > v2[i])
                return false;
        }
        return true;
    }
}
 
bool operator >(TVector &v, TVector &v2) {
    if (v.Size() > v2.Size())
        return true;
    else if (v.Size() < v2.Size())
        return false;
    else if (v == v2)
        return false;
    else {
        for (int i = v.Size(); i >= 0; i--) {
            if (v[i] > v2[i])
                return false;
            return true;
        }
    }
}

Помогите пожалуйста разобраться, в чем проблема.

nd2 · 19.02.2016, 22:35

Нужен копирующий конструктор.

Добавлено через 1 минуту
Распространенные ошибки

@mishula · 20.02.2016, 10:19 **[ТС]**

nd2, спасибо. Я создал такой конструктор

C++
1
2
3
4
5
6
7
8
9
10
TVector::TVector(TVector &v) :size(v.size), capacity(v.capacity){
    try { numbers = new int[v.Size()]; }
    catch (std::bad_alloc &e) {
        std::cerr << "Bad alloc: " << e.what() << std::endl;
    }
    std::cout << "Here is copy constructor" << std::endl;
    for (int i = 0; i < v.Size(); i++) {
        numbers[i] = v.numbers[i];
    }
}

При вызове вычитания на выходе из программы все стало нормально, однако при вызове произведения после выхода из программы все та же проблема. И у меня осталась все та же проблема с выведением отрицательных чисел при умножении после присваивания *this tmp (165 строчка). Что у меня еще может быть неправильно?

nd2 · 20.02.2016, 11:36

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

Что у меня еще может быть неправильно?

Твой оператор присваивания делает то же, что и оператор присваивания по умолчанию: копирует указатель. Нужно новую память выделять, и копировать туда данные.

Новые блоги и статьи Все статьи Все блоги /
http://iceja.net/ математические сервисы iceja 20.01.2026 Обновила свой сайт http:/ / iceja. net/ , приделала Fast Fourier Transform экстраполяцию сигналов. Однако предсказывает далеко не каждый сигнал (см ограничения http:/ / iceja. net/ fourier/ docs ). Также. . .	http://iceja.net/ сервер решения полиномов iceja 18.01.2026 Выкатила http:/ / iceja. net/ сервер решения полиномов (находит действительные корни полиномов методом Штурма). На сайте документация по API, но скажу прямо VPS слабенький и 200 000 полиномов. . .	Расчёт переходных процессов в цепи постоянного тока igorrr37 16.01.2026 / * Дана цепь постоянного тока с R, L, C, k(ключ), U, E, J. Программа составляет систему уравнений по 1 и 2 законам Кирхгофа, решает её и находит переходные токи и напряжения на элементах схемы. . . .	Восстановить юзерскрипты Greasemonkey из бэкапа браузера damix 15.01.2026 Если восстановить из бэкапа профиль Firefox после переустановки винды, то список юзерскриптов в Greasemonkey будет пустым. Но восстановить их можно так. Для этого понадобится консольная утилита. . .
Сукцессия микоризы: основная теория в виде двух уравнений. anaschu 11.01.2026 https:/ / rutube. ru/ video/ 7a537f578d808e67a3c6fd818a44a5c4/	WordPad для Windows 11 Jel 10.01.2026 WordPad для Windows 11 — это приложение, которое восстанавливает классический текстовый редактор WordPad в операционной системе Windows 11. После того как Microsoft исключила WordPad из. . .	Classic Notepad for Windows 11 Jel 10.01.2026 Old Classic Notepad for Windows 11 Приложение для Windows 11, позволяющее пользователям вернуть классическую версию текстового редактора «Блокнот» из Windows 10. Программа предоставляет более. . .	Почему дизайн решает? Neotwalker 09.01.2026 В современном мире, где конкуренция за внимание потребителя достигла пика, дизайн становится мощным инструментом для успеха бренда. Это не просто красивый внешний вид продукта или сайта — это. . .

nd2 3438 / 2817 / 1249 Регистрация: 29.01.2016 Сообщений: 9,427
	19.02.2016, 22:35
	Нужен копирующий конструктор. Добавлено через 1 минуту Распространенные ошибки 0