Ошибка при вызове деструктора

@Rastrizh · Регистрация: 30.12.2015

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

Здравствуйте, помогите разобраться с проблемой, она довольно распространенная и на форуме куча сообщений про это но в моей программе я никак не могу найти где я ошибся, нужен ваш взгляд со стороны.
Вот код:

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
#include <cstring>
class vec
{
public:
    vec()
    {
        m_size = offset;
        m_len = 0;
        m_vector = new int[m_size];
        memset(m_vector, 0, sizeof(m_vector));
    }
    vec(size_t size)
    {
        m_size = size;
        m_len = 0;
        m_vector = new int[m_size];
        memset(m_vector, 0, sizeof(m_vector));
    }
    vec(const vec& orig)
        :
        m_size(orig.m_size),
        m_len(orig.m_len),
        m_vector(nullptr)
    {
        m_vector = new int[m_size];
        for (size_t i = 0; i < m_len; ++i)
        {
            m_vector[i] = orig.m_vector[i];
        }
    }
    ~vec()
    {
        if (m_vector != nullptr)
        {
            delete[] m_vector;
            m_vector = nullptr;
        }
    }
    size_t GetSize()
    {
        return m_size;
    }
    size_t GetLength()
    {
        return m_len;
    }
    int* GetValue()
    {
        return m_vector;
    }
    vec& operator= (const vec& rhs)
    {
        if (m_vector)
        {
            delete[] m_vector;
            m_vector = nullptr;
            m_len = rhs.m_len;
            m_vector = new int[m_size];
            for (size_t i = 0; i < m_len; ++i)
            {
                m_vector[i] = rhs.m_vector[i];
            }
        }
        return *this;
    }
    int& operator[] (size_t index)
    {
        return m_vector[index];
    }
    bool operator== (const vec& rhs)
    {
        if (m_len != rhs.m_len)
            return false;
        for (unsigned int i = 0; i < m_len; ++i)
        {
            if (m_vector[i] != rhs.m_vector[i])
                return false;
        }
        return true;
    }
    bool operator!= (const vec& rhs)
    {
        if (!(this == &rhs))
            return true;
        return false;
    }
    void push_back(int value)
    {
        if (m_len == m_size)
        {
            vec v = vec(m_size * (m_size / 2));
            v = *this;
            this->m_size = v.m_size;
            *this = v;
        }
        m_vector[m_len] = value;
        m_len++;
    }
    void erase(unsigned int pos)
    {
        m_vector[pos] = m_vector[m_len - 1];
    }
    void erase(unsigned int begin, unsigned int end)
    {
        for (unsigned int i = begin; i <= end; ++i)
        {
            m_vector[i] = m_vector[m_len - i - 1];
        }
    }
    void pop_back()
    {
        m_vector[m_len - 1] = 0;
    }
private:
    static constexpr size_t offset = 8;
    size_t m_size;
    size_t m_len;
    int* m_vector;
};

,

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
#include "Vec.h"
int main()
{
    vec v0;
    v0.push_back(2);
    v0.push_back(22);
    v0.push_back(234);
    v0.push_back(2325);
    printf("\n");
    for (unsigned int i = 0; i < v0.GetLength(); ++i)
    {
        printf("%d ", v0[i]);
    }
    vec v1(v0);
    v1.push_back(532);
    v1.push_back(2324);
    v1.push_back(1321);
    v1.push_back(6532);
    v1.push_back(42);
    printf("\n");
    for (unsigned int i = 0; i < v1.GetLength(); ++i)
    {
        printf("%d ", v1[i]);
    }
    vec v2;
    v2 = v1;
    v2.push_back(456);
    v2.push_back(452);
    v2.push_back(224);
    v2.push_back(12);
    v2.push_back(698);
    printf("\n");
    for (unsigned int i = 0; i < v2.GetLength(); ++i)
    {
        printf("%d ", v2[i]);
    }
    std::cin.get();
    return 0;
}

MrGluck · 09.08.2018, 15:23

Проверьте написание оператора присваивания

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
vec& operator= (const vec& rhs)
    {
        if (this != &rhs)
        {
            delete[] m_vector;
            m_len = rhs.m_len;
            m_size = rhs.m_size;
            m_vector = new int[m_size];
            for (size_t i = 0; i < m_len; ++i)
            {
                m_vector[i] = rhs.m_vector[i];
            }
        }
        return *this;
    }

Добавлено через 7 минут
А ещё выкиньте строки

C++
1
memset(m_vector, 0, sizeof(m_vector));

1. элементы в дин. массиве итак равны 0.
2. sizeof(m_vector) - размер указателя.
3. В С++ используют std::fill

@Rastrizh · 09.08.2018, 15:36 **[ТС]**

Внес предложенные вами изменения, не помогло.
Кстати почему Release версия не ругается? Я так понимаю очистка памяти возлагается на ОС?

@QuakerRUS · 09.08.2018, 15:38

Не забываем очищать массивы перед очисткой указателя.

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
vec(const vec& orig)
        :
        m_size(orig.m_size),
        m_len(orig.m_len)
    {
        if (m_vector != nullptr)
            delete[] m_vector;
 
        m_vector = new int[m_size];
        for (size_t i = 0; i < m_len; ++i)
        {
            m_vector[i] = orig.m_vector[i];
        }
    }

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
vec& operator= (const vec& rhs)
    {
        if (m_vector != nullptr)
            delete[] m_vector;
    
        m_size = rhs.m_size;
        m_len = rhs.m_len;
        m_vector = new int[m_size];
        for (size_t i = 0; i < m_len; ++i)
        {
            m_vector[i] = rhs.m_vector[i];
        }
        return *this;
    }

Метод push_back у вас неправильно работает.

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
void push_back(int value)
    {
        if (m_len == m_size)
        {
            m_size = m_size * (m_size / 2);
            int* temp_vector = m_vector;
            m_vector = new int[m_size];
            for (size_t i = 0; i < m_len; ++i)
            {
                m_vector[i] = temp_vector[i];
            }
            delete[] temp_vector;
        }
        m_vector[m_len] = value;
        m_len++;
    }

@zss · 09.08.2018, 15:38

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

C++
1
2
3
4
5
m_vector = new int[m_size]; 
for (size_t i = 0; i < m_len; ++i);
{
   m_vector[i] = rhs.m_vector[i];
}

Выделяем память размером m_size
А заполняем m_len элементов.
Если m_len>m_size, то приплыли...

@QuakerRUS · 09.08.2018, 15:43

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

Если m_len>m_size, то приплыли...

По логике m_len не должна быть больше чем m_size. Вот только в методе push_back ошибка, из-за которой m_size не увеличивается корректно.

@Rastrizh · 09.08.2018, 15:57 **[ТС]**

QuakerRUS, я внес изменения которые вы предложили, и все равно вылазит ошибка

Croessmah · 09.08.2018, 15:57

C++
1
2
        m_vector = new int[m_size];
        memset(m_vector, 0, sizeof(m_vector));

Зануляются первые sizeof(m_vector) байт.
Наверное, имелось ввиду m_size * sizeof(*m_vector).

@QuakerRUS · 09.08.2018, 15:59

Rastrizh, у меня без ошибок работает. Покажите весь код, который у вас сейчас получился.

@Rastrizh · 09.08.2018, 16:00 **[ТС]**

Вот так сейчас выглядит код:

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
#include <cstring>
 
class vec
{
public:
    vec()
    {
        m_size = offset;
        m_len = 0;
        m_vector = new int[m_size];
    }
    vec(size_t size)
    {
        m_size = size;
        m_len = 0;
        m_vector = new int[m_size];
    }
    vec(const vec& orig)
        :
        m_size(orig.m_size),
        m_len(orig.m_len),
        m_vector(nullptr)
    {
        if (m_vector != nullptr)
            delete[] m_vector;
 
        m_vector = new int[m_size];
        for (size_t i = 0; i < m_len; ++i)
        {
            m_vector[i] = orig.m_vector[i];
        }
    }
    ~vec()
    {
        if (m_vector != nullptr)
        {
            delete[] m_vector;
        }
    }
    size_t GetSize()
    {
        return m_size;
    }
    size_t GetLength()
    {
        return m_len;
    }
    int* GetValue()
    {
        return m_vector;
    }
    vec& operator= (const vec& rhs)
    {
        if (m_vector != nullptr)
            delete[] m_vector;
 
        m_len = rhs.m_len;
        m_vector = new int[m_size];
        for (size_t i = 0; i < m_len; ++i)
        {
            m_vector[i] = rhs.m_vector[i];
        }
        return *this;
    }
    int& operator[] (size_t index)
    {
        return m_vector[index];
    }
    bool operator== (const vec& rhs)
    {
        if (m_len != rhs.m_len)
            return false;
        for (unsigned int i = 0; i < m_len; ++i)
        {
            if (m_vector[i] != rhs.m_vector[i])
                return false;
        }
        return true;
    }
    bool operator!= (const vec& rhs)
    {
        if (!(this == &rhs))
            return true;
        return false;
    }
    void push_back(int value)
    {
        if (m_len == m_size)
        {
            m_size = m_size * (m_size / 2);
            int* temp_vector = m_vector;
            m_vector = new int[m_size];
            for (size_t i = 0; i < m_len; ++i)
            {
                m_vector[i] = temp_vector[i];
            }
            delete[] temp_vector;
        }
        m_vector[m_len] = value;
        m_len++;
    }
    void erase(unsigned int pos)
    {
        m_vector[pos] = m_vector[m_len - 1];
        --m_len;
    }
    void erase(unsigned int begin, unsigned int end)
    {
        for (unsigned int i = begin; i <= end; ++i)
        {
            m_vector[i] = m_vector[m_len - i - 1];
 
        }
        m_len -= end - begin;
    }
    void pop_back()
    {
        m_vector[m_len - 1] = 0;
        --m_len;
    }
private:
    static constexpr size_t offset = 8;
    size_t m_size;
    size_t m_len;
    int* m_vector;
};

И он по прежнему выдает ту же ошибку

@QuakerRUS · 09.08.2018, 16:06

Rastrizh, вы неправильно вставили те фрагменты, которые я вам написал. Откуда, например, 22я строка?

Добавлено через 2 минуты
В operator= тоже строки не хватает из моего кода.

@Rastrizh · 09.08.2018, 16:07 **[ТС]**

В оператор присваивания не вставил строку:

C++
1
m_size = rhs.m_size;

Все работает, спасибо вам QuakerRUS большое

@QuakerRUS · 09.08.2018, 16:09

Rastrizh, 22ю строку не забудьте удалить, иначе у вас утечка памяти будет.

@Kuzia domovenok · 09.08.2018, 16:35

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

Не забываем очищать массивы перед очисткой указателя.

имелось в виду очищать память m_vector в конструкторе? Ну зачееем? Конструктор он гарантированно создаёт новый объект, под поля которого память до этого никакая не выделялась!

@QuakerRUS · 09.08.2018, 16:41

Kuzia domovenok, да, это я затупил что то, спасибо.

Добавлено через 3 минуты
Rastrizh, конструктор vec(const vec& orig) у вас изначально правильный был, проверка на непустоту m_vector там лишняя, извиняюсь за дезынформацию.

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): Обработчик клика мыши в браузере ПК и касания экрана в браузере на мобильном устройстве 8Observer8 02.02.2026 Содержание блога Для начала пошагово создадим рабочий пример для подготовки к экспериментам в браузере ПК и в браузере мобильного устройства. Потом напишем обработчик клика мыши и обработчик. . .	Философия технологии iceja 01.02.2026 На мой взгляд у человека в технических проектах остается роль генерального директора. Все остальное нейронки делают уже лучше человека. Они не могут нести предпринимательские риски, не могут. . .	SDL3 для Web (WebAssembly): Вывод текста со шрифтом TTF с помощью SDL3_ttf 8Observer8 01.02.2026 Содержание блога В этой пошаговой инструкции создадим с нуля веб-приложение, которое выводит текст в окне браузера. Запустим на Android на локальном сервере. Загрузим Release на бесплатный. . .	SDL3 для Web (WebAssembly): Сборка C/C++ проекта из консоли 8Observer8 30.01.2026 Содержание блога Если вы откроете примеры для начинающих на официальном репозитории SDL3 в папке: examples, то вы увидите, что все примеры используют следующие четыре обязательные функции, а. . .
SDL3 для Web (WebAssembly): Установка Emscripten SDK (emsdk) и CMake для сборки C и C++ приложений в Wasm 8Observer8 30.01.2026 Содержание блога Для того чтобы скачать Emscripten SDK (emsdk) необходимо сначало скачать и уставить Git: Install for Windows. Следуйте стандартной процедуре установки Git через установщик. . . .	SDL3 для Android: Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 29.01.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами. Версия v3 была полностью переписана на Си, в. . .	Инструменты COM: Сохранение данный из VARIANT в файл и загрузка из файла в VARIANT bedvit 28.01.2026 Сохранение базовых типов COM и массивов (одномерных или двухмерных) любой вложенности (деревья) в файл, с возможностью выбора алгоритмов сжатия и шифрования. Часть библиотеки BedvitCOM Использованы. . .	SDL3 для Android: Загрузка PNG с альфа-каналом с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 28.01.2026 Содержание блога SDL3 имеет собственные средства для загрузки и отображения PNG-файлов с альфа-каналом и базовой работы с ними. В этой инструкции используется функция SDL_LoadPNG(), которая. . .

@Rastrizh 0 / 0 / 0 Регистрация: 30.12.2015 Сообщений: 7
	09.08.2018, 15:36 [ТС]
	Внес предложенные вами изменения, не помогло. Кстати почему Release версия не ругается? Я так понимаю очистка памяти возлагается на ОС? 0

@Rastrizh 0 / 0 / 0 Регистрация: 30.12.2015 Сообщений: 7
	09.08.2018, 15:57 [ТС]
	QuakerRUS, я внес изменения которые вы предложили, и все равно вылазит ошибка 0

@QuakerRUS 1469 / 1010 / 456 Регистрация: 30.10.2017 Сообщений: 2,799
	09.08.2018, 15:59
	Rastrizh, у меня без ошибок работает. Покажите весь код, который у вас сейчас получился. 0

@QuakerRUS 1469 / 1010 / 456 Регистрация: 30.10.2017 Сообщений: 2,799
	09.08.2018, 16:06
	Rastrizh, вы неправильно вставили те фрагменты, которые я вам написал. Откуда, например, 22я строка? Добавлено через 2 минуты В `operator=` тоже строки не хватает из моего кода. 1

@QuakerRUS 1469 / 1010 / 456 Регистрация: 30.10.2017 Сообщений: 2,799
	09.08.2018, 16:09
	Rastrizh, 22ю строку не забудьте удалить, иначе у вас утечка памяти будет. 0

@QuakerRUS 1469 / 1010 / 456 Регистрация: 30.10.2017 Сообщений: 2,799
	09.08.2018, 16:41
	Kuzia domovenok, да, это я затупил что то, спасибо. Добавлено через 3 минуты Rastrizh, конструктор `vec(const vec& orig)` у вас изначально правильный был, проверка на непустоту `m_vector` там лишняя, извиняюсь за дезынформацию. 0

Ошибка при вызове деструктора

Решение