Некорректное удаление динамического массива внутри объекта

@Jass · Регистрация: 31.01.2016

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

Здравствуйте, встретился вот такой проблемой: скрин

Краткое введение в курс дела, захотел сделать создание собственных массивов на основе int. Сделал автоматическое увеличение массива при передачи в него аргумента (будет происходить только, если массив заполнен полностью до отвала, но цикл продолжает пичкать его). При удалении массива из объекта, встретился с данной проблемой, но попробовал её обойти путем добавления ещё одного метода Delete() в class, но уверен что так не должно быть, однако помогло.
При создании ещё одного массива в объекте и последующем удалении вот такой косяк повторился и попытался исправить как и тот, но не пашет.
(Если будет вариант создать в объекте дин. указатель на ещё один массив в области private, то не предлагать)

Вот код:

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
#include <iostream>
#include <cstring>
#include <conio.h>
#include <ctime>
 
using namespace std;
 
class Test {
public:
    Test(int Lenght);
 
    void operator+(int Numeric);
    void operator-();
    int operator[](int Index);
    bool operator<(Test T);
 
    
private:
    int Lenght;
    int *Array;
    int Index;
    int UseLenght;
 
    void Delete();
    void Delete(int *Arr);
    
};
 
Test::Test(int Lenght) {
    this->Lenght = Lenght;
    Array = new int[Lenght];
    UseLenght = 0;
}
 
void Test::operator+(int Numeric) {
    if (Numeric <= this->Lenght) {
        Array[UseLenght] = Numeric;
        UseLenght++;
    }
    else
    {
        int *Arr = new int[this->Lenght + 1];
 
        for (register int i = 0; i < this->Lenght; i++) {
            Arr[i] = this->Array[i];
        }
        void Delete();
        //delete[] this->Array;
        Arr[Lenght + 1] = Numeric;
        Lenght++;
 
        this->Array = new int[Lenght];
    
        for (register int i = 0; i < Lenght; i++) {
            this->Array[i] = Arr[i];
            UseLenght = i;
        }
        this->Delete(Arr);
    }
}
 
void Test::operator-() {
 
}
 
int Test::operator[](int Index) {
    return Array[Index];
}
 
bool Test::operator<(Test T) {
    int CountUnit = 0;
 
    if (this->Lenght <= T.Lenght)
    {
        for (register int i = 0; i < Lenght; i++) {
            if (this->Array[i] == T.Array[i]) CountUnit++;
        }
        if (CountUnit == this->Lenght) return true;
        else return false;
    }
}
 
void Test::Delete() {
    delete[] this->Array;
}
 
void Test::Delete(int *Arr) {
    delete[] Arr;
}
 
 
 
 
int main()
{
    setlocale(0, "");
 
    Test T(5);
 
    for (int i = 0; i < 10; i++) {
        T + i;
        cout << "\n[" << i << "]: " << T[i];
    }
    cout << "g";
    //T.Delete();
 
    system("PAUSE");
    return 0;
}

@schdub · 29.11.2017, 12:37

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

void Test::operator+(int Numeric)

Подход к написанию этого оператора в корне неверен. Выражения как в строке 101 недопустимы. Кстати, что он должен делать по вашему? Добавлять новый элемент? Для этого нужно создать функцию, например, с именем apeend(), а не использовать operator+()

[url="http://courses.cms.caltech.edu/cs11/material/cpp/donnie/cpp-ops.html"]C++ Operator Overloading Guidelines[url]

@Jass · 29.11.2017, 12:45 **[ТС]**

Я с Вами согласен, но я делаю для личной практики и хочу использовать операторы.
А от данного оператора хочу чтобы он добавлял если есть место в массиве, а если нет то расширить его и добавить новый элемент

@schdub · 29.11.2017, 13:12

Jass,

C++
1
2
3
4
5
6
7
8
9
10
11
12
void Test::operator+(int Numeric) {
    if (this->UseLenght >= this->Lenght) {
        const int append_const = 1;
        int *Arr = new int[this->Lenght + append_const];
        for (int i = 0; i < this->Lenght; i++)
            Arr[i] = this->Array[i];
        delete[] this->Array;
        this->Array = Arr;
        this->Lenght += append_const;
    }
    this->Array[this->UseLenght++] = Numeric;
}

@Jass · 29.11.2017, 15:39 **[ТС]**

А что дает константа при том, что создается массив и без константы нормально ?

Ошибка появляется при удалении массива уже не встроенного в объект, а временного:

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
void Test::operator+(int Numeric) {
    if (this->UseLenght < this->Lenght) {
        Array[UseLenght] = Numeric;
        UseLenght++;
    }
    else
    {
        int *array;
        array = new int[ this->Lenght + 1 ];    // Создаем новый массив на 1 единицу больше 
 
        for (register int i = 0; i < this->Lenght; i++) 
        {   array[ i ] = this->Array[ i ];  }
 
        this->Lenght++;
        array[ this->Lenght ] = Numeric;
        delete[] this->Array;   // Удаляем старый массив
 
        this->Array = array;
    
        for (register int i = 0; i < this->Lenght; i++ ) 
        {
            this->Array[ i ] = array[ i ];
            UseLenght = i;
        }
        delete[] array; <------------------------ ошибка выполнения тут
    }
}

Добавлено через 16 минут
И есть вопрос, я создаю указатель типа int (к примеру int *T), но не знаю будет ли это обычная переменная или массив
Сам вопрос, могу ли я сделать так:

C++
1
2
3
4
5
6
7
8
9
10
11
...
 
int *T;
// 1 вариант, когда мне нужен массив
int T = new int[i];
 
// 2 вариант, когда я хочу переменную
int T = new int;
T = 10;
 
...

Можно ли так ?

Новые блоги и статьи Все статьи Все блоги /
Модель заражения группы наркоманов alhaos 17.04.2026 Условия задачи сформулированы тут Суть: - Группа наркоманов из 10 человек. - Только один инфицирован ВИЧ. - Колются одной иглой. - Колются раз в день. - Колются последовательно через. . .	Мысли в слух. Про "навсегда". kumehtar 16.04.2026 Подумалось тут, что наверное очень глупо использовать во всяких своих установках понятие "навсегда". Это очень сильное понятие, и я только начинаю понимать край его смысла, не смотря на то что давно. . .	My Business CRM MaGz GoLd 16.04.2026 Всем привет, недавно возникла потребность создать CRM, для личных нужд. Собственно программа предоставляет из себя базу данных клиентов, в которой можно фиксировать звонки, стадии сделки, а также. . .	Знаешь почему 90% людей редко бывают счастливыми? kumehtar 14.04.2026 Потому что они ждут. Ждут выходных, ждут отпуска, ждут удачного момента. . . а удачный момент так и не приходит.
Фиксация колонок в отчете СКД Maks 14.04.2026 Фиксация колонок в СКД отчета типа Таблица. Задача: зафиксировать три левых колонки в отчете. Процедура ПриКомпоновкеРезультата(ДокументРезультат, ДанныеРасшифровки, СтандартнаяОбработка) / / . . .	Настройки VS Code Loafer 13.04.2026 { "cmake. configureOnOpen": false, "diffEditor. ignoreTrimWhitespace": true, "editor. guides. bracketPairs": "active", "extensions. ignoreRecommendations": true, . . .	Оптимизация кода на разграничение прав доступа к элементам формы Maks 13.04.2026 Алгоритм из решения ниже реализован на нетиповом документе, разработанного в конфигурации КА2. Задачи, как таковой, поставлено не было, проделанное ниже исключительно моя инициатива. Было так:. . .	Контроль заполнения и очистка дат в зависимости от значения перечислений Maks 12.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "ПланированиеПерсонала", разработанного в конфигурации КА2. Задача: реализовать контроль корректности заполнения дат назначения. . .

@Jass 8 / 8 / 10 Регистрация: 31.01.2016 Сообщений: 84
	29.11.2017, 12:45 [ТС]
	Я с Вами согласен, но я делаю для личной практики и хочу использовать операторы. А от данного оператора хочу чтобы он добавлял если есть место в массиве, а если нет то расширить его и добавить новый элемент 0

Опции темы