Форум программистов, компьютерный форум, киберфорум
Наши страницы
С++ для начинающих
Войти
Регистрация
Восстановить пароль
 
Рейтинг 4.75/4: Рейтинг темы: голосов - 4, средняя оценка - 4.75
crazy1ua
2 / 2 / 1
Регистрация: 09.12.2012
Сообщений: 65
1

Программа вылетает при добавлении элемента

11.03.2014, 16:47. Просмотров 703. Ответов 15
Метки нет (Все метки)

Здравствуйте ув. форумчане.
Есть у меня в программе массив первого и второго типа. При начальном создание (мы выбераем сами количество элементов) массивов деструктор срабатывает нормально. Но как только стоит добавить элемент туда ещё один элемент - программа вылетает при роботе деструктора.
Прикрепил скрин ошибки и сам проект.
0
Миниатюры
Программа вылетает при добавлении элемента  
Вложения
Тип файла: rar projs_to_kolokvium.rar (189.1 Кб, 3 просмотров)
Лучшие ответы (1)
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
11.03.2014, 16:47
Ответы с готовыми решениями:

Программа вылетает при добавлении элемента
Написал программу Car, в которой реализовал функции Add и Delete нового авто,...

Вылетает программа при удалении элемента вектора
Всех с новым годом, уважаемые киберфорумцы! Сделал я на днях класс Button,...

При удалении элемента из списка вылетает программа
Здравствуйте, помогите, пожалуйста. У меня есть программа по работе с классами...

После ввода первого элемента программа вылетает
Добрый день. Я студент, пытаюсь сделать лабораторную работу. #include...

Удаление последнего элемента динамического массива (программа вылетает с ошибкой)
for (vector<P>::iterator i = p.begin(); i != p.end(); ++i) { ...

15
crazy1ua
2 / 2 / 1
Регистрация: 09.12.2012
Сообщений: 65
11.03.2014, 17:12  [ТС] 2
Извиняюсь не то вложение. Вот то что надо!
0
Вложения
Тип файла: rar lab2.rar (8.6 Кб, 5 просмотров)
crazy1ua
2 / 2 / 1
Регистрация: 09.12.2012
Сообщений: 65
11.03.2014, 18:16  [ТС] 3
Ни одного просмотра.
0
0x10
11.03.2014, 19:16
  #4

Не по теме:

Цитата Сообщение от crazy1ua Посмотреть сообщение
Ни одного просмотра
Потому что обычно людям лень качать какие-то архивы, собирать непонятно что, разбираться в куче файлов...
Воспроизведите проблему с минимумом кода и выложите сюда. Чтобы можно было скопировать, вставить, скомпилировать и увидеть проблему. Все, что не имеет отношения к проблеме - удалить.
В общем, курите http://www.sscce.org/

0
crazy1ua
2 / 2 / 1
Регистрация: 09.12.2012
Сообщений: 65
11.03.2014, 21:46  [ТС] 5
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
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
#include <iostream>
#include <cstring>
#include <algorithm>
#include <cstdlib>
#include <vector>
 
 
using std::cout;
using std::endl;
using std::cin;
using std::sort;
 
 
#pragma warning(disable : 4996)
 
class Mosquito
{
private:
    int Number;
    double Speed;
    double Weight;
    char *Type;
public:
    Mosquito();
    Mosquito(int, double, double, char*);
    Mosquito(const Mosquito&);
    ~Mosquito();
    void Print() const;
    int Mosquito::GetNumber() const;
    double GetSpeed() const;
    double GetWeight() const;
    char * GetType() const;
    void SetNumber(int);
    void SetSpeed(double);
    void SetWeight(double);
    void SetType(char *);
    friend std::ostream & operator<<(std::ostream &, Mosquito &);
    friend std::istream & operator>>(std::istream &, Mosquito &);
    Mosquito & operator=(const Mosquito &);
    bool operator==(const Mosquito &);
    bool operator<(const Mosquito&);
};
 
using std::cout;
using std::endl;
 
enum SORT_TYPE{ BY_ID = 1, BY_CALIBER, BY_TYPE };
SORT_TYPE sort_type = BY_ID;
 
Mosquito::Mosquito()
{
    Number = 0;
    Speed = 0;
    Weight = 0;
    Type = NULL;
    cout << "Mosquito default constructor" << endl;
}
 
Mosquito::Mosquito(int Number, double Caliber, double Weight, char *Type)
{
    this->Number = Number;
    this->Speed = Speed;
    this->Weight = Weight;
    this->Type = new char[strlen(Type) + 1];
    strcpy(this->Type, Type);
    cout << "Mosquito param constructor" << endl;
}
 
Mosquito::Mosquito(const Mosquito& mosq)
{
    Type = NULL;
    *this = mosq;
    cout << "Copy constructor " << endl;
}
 
Mosquito::~Mosquito()
{
    cout << "Mosquito destructor" << endl;
    if (Type)
    delete[] Type;
}
 
void Mosquito::Print() const
{
    cout << "Number of Mosquito - " << Number << "." << endl;
    cout << "Speed of Mosquito - " << Speed << " km/h." << endl;
    cout << "Weight of Mosquito - " << Weight << " gram." << endl;
    cout << "Type of Mosquito - " << Type << "." << endl;
}
 
int Mosquito::GetNumber() const
{
    return Number;
}
 
double Mosquito::GetSpeed() const
{
    return Speed;
}
 
double Mosquito::GetWeight() const
{
    return Weight;
}
 
char * Mosquito::GetType() const
{
    return Type;
}
 
void Mosquito::SetNumber(int Number)
{
    this->Number = Number;
}
 
 
void Mosquito::SetSpeed(double Speed)
{
    this->Speed = Speed;
}
 
void Mosquito::SetWeight(double Weight)
{
    this->Weight = Weight;
}
 
 
void Mosquito::SetType(char *Type)
{
    delete[] this->Type;
    this->Type = new char[strlen(Type) + 1];
    strcpy(this->Type, Type);
}
 
 
//перегрузка >> - с потока
std::istream & operator>>(std::istream &is, Mosquito &mosq)
{
    char buf[10];
    is >> mosq.Number;
    is >> mosq.Speed >> mosq.Weight;
    is >> buf;
    mosq.Type = new char[strlen(buf) + 1];
    strcpy(mosq.Type, buf);
    return is;
}
 
//перегрузка оператора << - в поток
std::ostream & operator<<(std::ostream &os, Mosquito &mosq)
{
    os << mosq.Number << " ";
    os << mosq.Speed << " " << mosq.Weight << " ";
    os << mosq.Type;
    return os;
}
 
 
void menu(Mosquito *array_1, Mosquito **array_2, int& len_1, int& len_2);
void case_num_3(Mosquito **array_1, Mosquito ***array_2, int& len_1, int& len_2, const bool active_arr);
bool case_num_6(bool& active_arr);
 
 
 
int main()
{
    int amount_1, amount_2;
    cout << "Input amount arrays by space: " << endl;
    cin >> amount_1 >> amount_2;
    Mosquito *dyn_arr_1 = new Mosquito[amount_1];
    Mosquito **dyn_arr_2 = new Mosquito*[amount_2];
    for (int i = 0; i < amount_2; i++)
    {
        dyn_arr_2[i] = new Mosquito;
    }
    for (int i = 0; i < amount_1; i++)
    {
        dyn_arr_1[i].SetNumber(i);
        dyn_arr_1[i].SetSpeed(rand() % 10 / (double)1 * (0.86 - 0.25) + 0.86);
        dyn_arr_1[i].SetWeight(rand() % 5 / (double)1 * (0.74 - 0.36) + 0.074);
        dyn_arr_1[i].SetType("Culex pipiens");
    }
    for (int i = 0; i < amount_2; i++)
    {
        dyn_arr_2[i]->SetNumber(i);
        dyn_arr_2[i]->SetSpeed(rand() % 10 / (double)1 * (0.98 - 0.54) + 0.98);
        dyn_arr_2[i]->SetWeight(rand() % 5 / (double)1 * (0.65 - 0.15) + 0.65);
        dyn_arr_2[i]->SetType("Anopheles");
    }
 
    menu(dyn_arr_1, dyn_arr_2, amount_1, amount_2);
    delete[] dyn_arr_1;
    for (int i = 0; i < amount_2; i++)
    {
        delete dyn_arr_2[i];
    }
    delete[] dyn_arr_2;
    cout << "Finish" << endl;
    system("pause");
    return 0;
}
 
void menu(Mosquito *array_1, Mosquito **array_2, int& len_1, int& len_2)
{
    char command;
    bool active_arr;
    case_num_6(active_arr);
    for (;;)
    {
 
        cout << "Press 3 - to add object " << endl;
        cout << "Press 6 - to choose array " << endl;
        cout << "Press 7 - exit " << endl;
        cin >> command;
        switch (command)
        {
        case '3':
            case_num_3(&array_1, &array_2, len_1, len_2, active_arr); break;
        case '6':
            case_num_6(active_arr); break;
        case '7':
            return;
 
        }
    }
}
 
 
 
 
void case_num_3(Mosquito **array_1, Mosquito ***array_2, int& len_1, int& len_2, const bool active_arr)
{
    int obj_num_add;
    Mosquito *new_obj = new Mosquito;
 
    cout << "Enter number of object: " << endl;
    cin >> obj_num_add;
 
    if (active_arr == 1 && obj_num_add > len_1 + 1 || obj_num_add < 1)
    {
        cout << "Incorrect number" << endl;
        return;
    }
 
    if (active_arr == 0 && obj_num_add > len_2 + 1 || obj_num_add < 1)
    {
        cout << "Incorrect number" << endl;
        return;
    }
 
    cout << "Enter parameters of object by space(Number, Speed, Weight, Type): " << endl;
    cin >> *new_obj;
 
    if (active_arr == 1)
    {
        Mosquito *buf = new Mosquito[len_1 + 1];
        for (int i = 0; i < obj_num_add - 1; i++)
        {
            buf[i] = (*array_1)[i];
        }
        buf[obj_num_add - 1] = *new_obj;
        for (int i = obj_num_add - 1; i < len_1; i++)
        {
            buf[i + 1] = (*array_1)[i];
        }
        delete[](*array_1);
        *array_1 = &(*buf);
        len_1++;
    }
    else
    {
        Mosquito **buf = new Mosquito*[len_2 + 1];
        for (int i = 0; i < len_2 + 1; i++)
        {
            buf[i] = new Mosquito;
        }
 
        for (int i = 0; i < obj_num_add - 1; i++)
        {
            *buf[i] = *(*array_2)[i];
        }
        *buf[obj_num_add - 1] = *new_obj;
        for (int i = obj_num_add; i < len_2 + 1; i++)
        {
            *buf[i] = *(*array_2)[i - 1];
        }
        for (int i = 0; i < len_2; i++)
        {
            delete (*array_2)[i];
        }
        delete[] * array_2;
        *array_2 = &(*buf);
        len_2++;
    }
    delete new_obj;
}
 
bool case_num_6(bool& active_arr)
{
    char active;
    cout << "Choose your array (1 or 2): " << endl;
    cin >> active;
    if (active < '1' || active > '2'){ cout << "Incorrect number" << endl; case_num_6(active_arr); }
    if (active == '1') active_arr = 1;
    else active_arr = 0;
    return active_arr;
}
 
bool Mosquito::operator==(const Mosquito &mosq)
{
    if (this->Number == mosq.Number && this->Speed == mosq.Speed && this->Weight == mosq.Weight && !strcmp(this->Type, mosq.Type)) return true;
    else return false;
}
Mosquito & Mosquito::operator=(const Mosquito &mosq)
{
    if (this == &mosq) return *this;
    if (*this == mosq) return *this;
    Number = mosq.GetNumber();
    Speed = mosq.GetSpeed();
    Weight = mosq.GetWeight();
    if (Type) delete[] Type;
    Type = new char[strlen(mosq.Type) + 1];
    strcpy(Type, mosq.GetType());
    return *this;
}

Укоротил больше чем в половину. Под спойлером не открывало.

Добавлено через 1 час 6 минут
Хоть бы какие-то идеи..

Добавлено через 58 минут
ап....
0
Alex5
1123 / 784 / 232
Регистрация: 12.04.2010
Сообщений: 2,012
11.03.2014, 21:58 6
Цитата Сообщение от crazy1ua Посмотреть сообщение
Хоть бы какие-то идеи..
А у Вас есть какие-нибудь идеи?
0
crazy1ua
2 / 2 / 1
Регистрация: 09.12.2012
Сообщений: 65
11.03.2014, 22:23  [ТС] 7
Да, были. Пробовал через цикл - постиг неудач.
0
Alex5
1123 / 784 / 232
Регистрация: 12.04.2010
Сообщений: 2,012
11.03.2014, 23:01 8
Предлагаю такую идею. Перед вызовом delete[] выводим на экран содержимое массива, который хотим удалить. (Чтобы убедиться, что пытаемся удалить то, что нужно.)
Цитата Сообщение от crazy1ua Посмотреть сообщение
Пробовал через цикл
Не понял. Что пробовал?
0
crazy1ua
2 / 2 / 1
Регистрация: 09.12.2012
Сообщений: 65
11.03.2014, 23:15  [ТС] 9
Удалял в в цикле каждый элемент массива.
0
alsav22
5442 / 4837 / 831
Регистрация: 04.06.2011
Сообщений: 13,587
11.03.2014, 23:21 10
В каком месте кода вылет происходит - определили?

Добавлено через 1 минуту
C++
1
cin >> amount_1 >> amount_2;
Тут значения какие вводите?
0
crazy1ua
2 / 2 / 1
Регистрация: 09.12.2012
Сообщений: 65
11.03.2014, 23:29  [ТС] 11
Количество элементов массивов.
Вылет происходит на деструкторе элемента, если не ошибаюсь последнего.
0
alsav22
5442 / 4837 / 831
Регистрация: 04.06.2011
Сообщений: 13,587
11.03.2014, 23:38 12
Цитата Сообщение от crazy1ua Посмотреть сообщение
Количество элементов массивов.
Вылет происходит на деструкторе элемента, если не ошибаюсь последнего.
Конкретные цифры скажите того, что вводите, какой пункт меню выбираете, дальше что конкретно вводите. Все значения ввода. Мне некогда разбираться: где и что там нужно вводить. Отладчиком пользоваться умеете? Пошагово идите по коду, и найдите конкретную строку кода, где происходит вылет.
0
crazy1ua
2 / 2 / 1
Регистрация: 09.12.2012
Сообщений: 65
11.03.2014, 23:58  [ТС] 13
Создаем 2 массива по 5 элементов. Выбераем первый массив - вводим 3(добавления элементов) - вводим номер добавляемого элемента(припустим 1) - вводим параметры обьекта(припустим: 1 1 1 1) - вводим 7 (выход) - ошибак на строке
C++
1
    delete[] dyn_arr_1;
0
Alex5
1123 / 784 / 232
Регистрация: 12.04.2010
Сообщений: 2,012
12.03.2014, 00:21 14
Цитата Сообщение от crazy1ua Посмотреть сообщение
C++
1
2
3
4
5
6
7
8
9
 void menu(Mosquito *array_1, Mosquito **array_2, int& len_1, int& len_2);
 
int main()
{
    // ...
    Mosquito *dyn_arr_1 = new Mosquito[amount_1];
    // ... 
    menu(dyn_arr_1, dyn_arr_2, amount_1, amount_2); /* В menu() передаётся копия dyn_arr_1 */
    // ...
Что бы ни произошло в функции menu(), значение dyn_arr_1 всегда остаётся неизменным.
Посмотрим, какие значения указателей мы получаем при вызове new[]
и какие передаём оператору delete[].

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
// int main()
 
    Mosquito *dyn_arr_1 = new Mosquito[amount_1];
    cout << "\n                                               dyn_arr_1 = new[...]   " << dyn_arr_1 << endl;
 
    cout << "\n                                                       delete[]    " << dyn_arr_1 << endl;
    delete[] dyn_arr_1;
 
// void case_num_3 ( ... )
 
        Mosquito *buf = new Mosquito[len_1 + 1];
        cout << "\n                                                     buf = new[...]   " << buf << endl;
 
        cout << "\n                                                       delete[]    " << (*array_1) << endl;
        delete[](*array_1);
1
Миниатюры
Программа вылетает при добавлении элемента  
alsav22
5442 / 4837 / 831
Регистрация: 04.06.2011
Сообщений: 13,587
12.03.2014, 00:25 15
Лучший ответ Сообщение было отмечено crazy1ua как решение

Решение

Вот так меню измените:
C++
1
2
3
4
void menu(Mosquito* &array_1, Mosquito** &array_2, int& len_1, int& len_2);
...
void menu(Mosquito* &array_1, Mosquito** &array_2, int& len_1, int& len_2)
{...
1
crazy1ua
2 / 2 / 1
Регистрация: 09.12.2012
Сообщений: 65
12.03.2014, 00:32  [ТС] 16
Огромное спасибо Вам.
0
12.03.2014, 00:32
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
12.03.2014, 00:32

Ошибка при добавлении элемента в список
Первый элемент всегда выводится правильно, у второго почему-то портится...

Ошибка при добавлении элемента в связный список
Помогите найти ошибку в программе,добавляю элементы в связный список,и если...

Ошибка при добавлении элемента в динамический массив
Всем привет! Создал класс для работы с дин.массивом. Нормально компилируется и...


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

Или воспользуйтесь поиском по форуму:
16
Ответ Создать тему
Опции темы

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