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

@marakesh · Регистрация: 23.10.2018

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

Не могу найти ошибку, при вызове деструктора от класса, который был создан при помощи конструктора с параметром const String &, возникает ошибка - не работает delete[]. Остальные деструкторы для других классов работают нормально.

Код:

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
#include <iostream>
 
using namespace std;
 
//------------------------------------------//
 
int strLen(char *str)
{
    int i = 0;
    for (; str[i] != 0; i++);
    return i;
}
 
void strCopy(char *to_str, char *from_str)
{
    int l = strLen(to_str);
    for (int i = 0; i < l; i++)
    {
        to_str[i] = from_str[i];
    }
    to_str[l] = '\0';
}
 
//------------------------------------------//
 
class String
{
private:
    char *data;
    int length;
public:
    String();
    String(char *);
    String(const String &);
    ~String();
    String& operator=(const String &);
    String& operator+=(const String &);
    friend String operator+(const String &, const String &);
    void getDataStr()
    {
        cout << data << endl;
    }
};
 
//------------------------------------------//
 
String::String()
{
    data = new char[1];
    data[0] = '\0';
    length = 1;
}
 
String::String(char *str)
{
    data = new char[strLen(str) + 1];
    strCopy(data, str);
    length = strLen(data);
}
 
String::String(const String &orig)
{
    data = new char[strLen(orig.data)];
    strCopy(data, orig.data);
    length = strLen(data);
}
 
String::~String()
{
    delete[] this->data;
    length = 0;
}
 
String& String::operator=(const String &orig)
{
    delete[] this->data;
    this->data = new char[strLen(orig.data) + 1];
    strCopy(this->data, orig.data);
    length = strLen(this->data);
    return *this;
}
 
String& String::operator+=(const String &orig)
{
    int size = strLen(this->data) + strLen(orig.data);
    char* tmp = new char[size + 1];
    int l = strLen(this->data);
    for (int i = 0; i < l; i++)
    {
        tmp[i] = this->data[i];
    }
    for (int j = 0, k = l; k < size; j++, k++)
    {
        tmp[k] = orig.data[j];
    }
    tmp[size] = '\0';
    delete[] this->data;
    this->data = tmp;
    length = strLen(this->data);
    return *this;
}
 
String operator+(const String &a, const String &b)
{
    int size = strLen(a.data) + strLen(b.data);
    char *tmp = new char[size + 1];
    for (int i = 0; i < a.length; i++)
    {
        tmp[i] = a.data[i];
    }
    for (int j = 0, k = a.length; k < size; j++, k++)
    {
        tmp[k] = b.data[j];
    }
    tmp[size] = '\0';
    return tmp;
}
 
//------------------------------------------//
 
int main()
{
    setlocale(LC_ALL, "rus");
    char* b = new char[10];
    for (int i = 0; i < 9; i++)
    {
        cin >> b[i];
    }
    b[9] = '\0';
    char* c = new char[10];
    for (int i = 0; i < 9; i++)
    {
        cin >> c[i];
    }
    c[9] = '\0';
    String q(c);
    String A;
    String B(b);
    String C(q);
 
    cout << endl << "/--------0---------/" << endl << endl;
 
    B.getDataStr();
    C.getDataStr();
 
    cout << endl << "/--------1---------/" << endl << endl;
 
    A = B;
    A.getDataStr();
 
    cout << endl << "/--------2---------/" << endl << endl;
 
    A = B + C;
    A.getDataStr();
 
    cout << endl << "/--------3---------/" << endl << endl;
 
    B += C;
    B.getDataStr();
 
    q.~String();
    A.~String();
    B.~String();
    C.~String();
 
    system("pause");
    return 0;
}

@aLarman · 25.10.2018, 14:08

Сообщение от Azazel-San

C++
1
2
3
int l = strLen(to_str); // получаем длину == 3
for (int i = 0; i < l; ++i) // в цикле переписали все до '\0' он остался 
to_str[l] = '\0'; // итого не имеет смысла

Думаю чтоит начать с того что должно быть не

C++
1
int l = strLen(to_str);

а

C++
1
int l = strLen(from_str);

Ну и вот мне интересно а кто гарантирует что в to_str[3] уже лежит '\0', по-моему никто

@Azazel-San · 25.10.2018, 14:39

aLarman,

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

C++
1
int l = strLen(to_str);//Зачем Вы определяете длину строки, которую нужно заменить?

Об этом уже сказал Croessmah..

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

Думаю чтоит начать с того что должно быть

Может да, может нет.. Код не мой, а точного задания не дано, как именно копировать..

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

Ну и вот мне интересно а кто гарантирует что в to_str[3] уже лежит '\0', по-моему никто

Ну, тогда, возможно, стоит это проверить, прежде чем бездумно туда что-то писать?

Croessmah · 25.10.2018, 21:49

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

а кто гарантирует что в to_str[3] уже лежит '\0', по-моему никто

В коде ТС это гарантировал strLen, когда вернул управление.

Новые блоги и статьи Все статьи Все блоги /
Doom для терминала без стрельбы и монстров. 3D Raycasting на ascii. dcc0 05.07.2026 Попросил нейронную сеть deepai. org написать рейкастинг 3D с библиотекой ncurses для Linux. Чтобы можно было ходить на стрелочки. Чтобы стены были отрисованы символами. Справилась. Первый вариант. . .	Установка статуса документа по условию Maks 05.07.2026 Алгоритм из решения ниже реализован на нетиповом документе "НарядПутевка" разработанного в КА2. Задача: в табличной части "Материалы" документа при записи автоматически устанавливать статус. . .	Сезонность и суточность закисления почв anaschu 04.07.2026 200 часов это все равно моловато. Есть ситуации, но нестандартные, когда смена происходит за 5 лет. Но обычно это 50 лет и более. Наверное, закисление почвы происходит сезонно в средней. . .	В чем ценность человеческого опыта в глобальном смысле? kumehtar 03.07.2026 Возможно, ценность человека не в том, что он однажды достигает мудрости, а в том, что он становится носителем карты пути. Он знает не только истину, но и последовательность внутренних изменений,. . .
интеграция AnyLogic с самописным REST API и переход на Odoo anaschu 03.07.2026 Успешная интеграция AnyLogic с самописным REST API и переход на промышленную Odoo WMS Сегодня проделал огромный путь от простой симуляции физических процессов до построения полноценной. . .	Поиск всех путей на ориентированном графе. Linux dcc0 02.07.2026 Переработка старого кода из моей статьи. Через несколько переработок от PHP кода к C89 (надеюсь, 89). Но довольно запутанно получилось. Код для Linux. Но если убрать time и то, что с ним. . .	Сам себя обучал rest api anaschu 02.07.2026 Педагогический лайфхак: Почему чистый REST API для ученика намного круче, чем готовые библиотеки Когда мы отказались от капризного JAR-файла AnyLogic и переписали код на стандартный HttpClient,. . .	rest api anylogic - выполнение модели на своём русском сайте anaschu 02.07.2026 Как подружиться с AnyLogic Cloud API, победить провайдеров и развернуться Java-бэкенд в Docker на бесплатном хостинге: Двухдневный лог борьбы Всем привет! Хочу поделиться свежим (и довольно. . .