Обработка исключительных ситуаций

@Nike1995 · Регистрация: 14.10.2012

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

Правильно ли тут я обрабатываю исключения?

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
#pragma once
#include <iostream>
#include "stdio.h"
#include "stdlib.h"
#include <stdexcept>
using namespace std;
 
const char *filename = "testfile.txt";
FILE *fout;
void debug(char* str);
 
template <typename T>
class MyArray
{
    T *arr;
    size_t size;
public:
    MyArray();
    MyArray(int n);
    MyArray(const MyArray &copy_arr);
    ~MyArray();
    void bubble_sort();
    MyArray concat_array(const MyArray &copy_arr);
    void add_elem(T el);
    T operator [](int i) const;
    T get_elem(int i);
    MyArray& operator = (const MyArray & right);
    template <typename T1>
    friend ostream& operator <<(ostream &s, const MyArray<T1> &right);
    void show();
};
 
template <typename T>
MyArray<T>::MyArray()
{
    debug("->MyArray()");
    arr = NULL;
    size = 0;
    debug("MyArray()->");
}
 
template <typename T>
MyArray<T>::MyArray(int n)
{
    size = n;
    arr = new T[n];
    for(int i = 0; i < n; i++)
            arr[i] = 0;
}
 
template <typename T>
MyArray<T>::MyArray(const MyArray & copy_arr)
{
    size = copy_arr.size;
    arr = new T[size];
    for(int i = 0; i < size; i++)
        arr[i] = copy_arr.arr[i];
}
 
template <typename T>
MyArray<T>::~MyArray()
{
    delete []arr;
    size = 0;
}
 
template <typename T>
void MyArray<T>::bubble_sort()
{
    debug("->bubble_sort()");
    T buf_el;
    bool swap = false;
    for(int i = 0; i < size - 1; i++)
    {
        swap = false;
        for(int j = 0; j < size - 1 - i; j++)
            if(arr[j] > arr[j + 1])
            {
                buf_el = arr[j];
                arr[j] = arr[j + 1];
                arr[j + 1] = buf_el;
                swap = true;
            }
            if(!swap)
                break;
    }
    debug("bubble_sort()->");
}
 
template <typename T>
void MyArray<T>::add_elem(T el)
{
    debug("->add_elem(T el)");
    try{
        try{
            T* temp_arr = new T[size + 1];
            for(int i = 0; i < size; i++)
                temp_arr[i] = arr[i];
            delete [] arr;
            try{
                arr = new T[size + 1];
                for(int i = 0; i < size; i++)
                    arr[i] = temp_arr[i];
                delete []temp_arr;
                arr[size] = el;
                size++;
            }
            catch(bad_alloc &x){
                debug(" error : bad_alloc");
                cout << "Memory problem#3.1!\n";
                exit(1);
            }
        }
        catch(bad_alloc &x){
            debug(" error : bad_alloc");
            cout << "Memory problem#3.2!\n";
            exit(1);
        }
    }
    catch(T){
        debug(" error type");
        cout << "error type#3.!\n";
        exit(1);
    }
 
    debug("add_elem(T el)->");
}
 
template <typename T>
T MyArray<T>::get_elem(int i)
{
    debug("->get_elem(int i)");
    if(i < 0 || i > size)
    {
        debug("going beyond the bounds of the array!");
        cout << "Going beyond the bounds of the array!" << endl;
        throw i;
    }
    debug("get_elem(int i)->");
    return arr[i];
}
 
template <typename T>
MyArray<T> MyArray<T>::concat_array(const MyArray &copy_arr)
{
    debug("->concat_array(const MyArray &copy_arr)");
    MyArray cur;
    try{
        cur.arr = new T[copy_arr.size + size];
        cur.size = copy_arr.size + size;
        for(int i = 0; i < size; i++)
            cur.arr[i] = arr[i];
 
        int j = 0;
        for(int i = size; i < copy_arr.size + this->size; i++, j++)
            cur.arr[i] = copy_arr.arr[j];
    }
    catch(bad_alloc &x){
        debug(" error : bad_alloc");
        cout << "Memory problem#3.5!\n";
        exit(1);
    }
 
    debug("concat_array(const MyArray &copy_arr)->");
 
    return cur;
}
 
template <typename T>
MyArray<T>& MyArray<T>::operator = (const MyArray &right)
{
    debug("->operator = (const MyArray &right)");
    if(this == &right)
        return *this;
 
    delete []arr;
    size = right.size;
    try{
        if(size == 0)
            throw size;
        arr = new T[size];
        for(int i = 0; i < size; i++)
            arr[i] = right.arr[i];
    }
    catch(bad_alloc &x){
        debug(" error : bad_alloc");
        cout << "Memory problem#4!\n";
        exit(1);
    }
 
    debug("operator = (const MyArray &right)->");
    return *this;
 
}
 
template <typename T>
T MyArray<T>::operator [] (int i) const
{
    debug("->operator [] (int i) const");
    if(i < 0 || i > size)
    {
        debug("going beyond the bounds of the array!");
        cout << "Going beyond the bounds of the array!" << endl;
        throw i;
    }
    debug("operator [] (int i) const->");
    return arr[i];
}
 
template <typename T>
ostream& operator <<(ostream &s, const MyArray<T> &right)
{
    debug("->operator <<(ostream &s, const MyArray<T> &right)");
    for(int i = 0; i < right.size; i++)
        s  << right[i] << " ";
    debug("operator <<(ostream &s, const MyArray<T> &right)->");
    return s;
}
 
void debug(char* str){
    int ecode;
    int encode = fprintf(fout, "%s  ", str);
    fprintf(fout, " in code %d\n", encode);
}

Добавлено через 34 минуты
Немного исправил

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
#pragma once
#include <iostream>
#include "stdio.h"
#include "stdlib.h"
#include <stdexcept>
using namespace std;
 
const char *filename = "testfile.txt";
FILE *fout;
void debug(char* str);
 
template <typename T>
class MyArray
{
    T *arr;
    size_t size;
public:
    MyArray();
    MyArray(int n);
    MyArray(const MyArray &copy_arr);
    ~MyArray();
    void bubble_sort();
    MyArray concat_array(const MyArray &copy_arr);
    void add_elem(T el);
    T operator [](int i) const;
    T get_elem(int i);
    MyArray& operator = (const MyArray & right);
    template <typename T1>
    friend ostream& operator <<(ostream &s, const MyArray<T1> &right);
    void show();
};
 
template <typename T>
MyArray<T>::MyArray()
{
    debug("->MyArray()");
    arr = NULL;
    size = 0;
    debug("MyArray()->");
}
 
template <typename T>
MyArray<T>::MyArray(int n)
{
    size = n;
    arr = new T[n];
    for(int i = 0; i < n; i++)
            arr[i] = 0;
}
 
template <typename T>
MyArray<T>::MyArray(const MyArray & copy_arr)
{
    size = copy_arr.size;
    arr = new T[size];
    for(int i = 0; i < size; i++)
        arr[i] = copy_arr.arr[i];
}
 
template <typename T>
MyArray<T>::~MyArray()
{
    delete []arr;
    size = 0;
}
 
template <typename T>
void MyArray<T>::bubble_sort()
{
    debug("->bubble_sort()");
    T buf_el;
    bool swap = false;
    for(int i = 0; i < size - 1; i++)
    {
        swap = false;
        for(int j = 0; j < size - 1 - i; j++)
            if(arr[j] > arr[j + 1])
            {
                buf_el = arr[j];
                arr[j] = arr[j + 1];
                arr[j + 1] = buf_el;
                swap = true;
            }
            if(!swap)
                break;
    }
    debug("bubble_sort()->");
}
 
template <typename T>
void MyArray<T>::add_elem(T el)
{
    debug("->add_elem(T el)");
    try{
        try{
            T* temp_arr = new T[size + 1];
            for(int i = 0; i < size; i++)
                temp_arr[i] = arr[i];
            delete [] arr;
            try{
                arr = new T[size + 1-100];
                for(int i = 0; i < size; i++)
                    arr[i] = temp_arr[i];
                delete []temp_arr;
                arr[size] = el;
                size++;
            }
            catch(bad_alloc &x){
                cerr << x.what() << endl;
                debug(" error : bad_alloc");
                exit(1);
            }
        }
        catch(bad_alloc &x){
            cerr << x.what();
            debug(" error : bad_alloc");
            exit(1);
        }
    }
    catch(T){
        debug(" error type");
        cout << "error type#3.!\n";
        exit(1);
    }
 
    debug("add_elem(T el)->");
}
 
template <typename T>
T MyArray<T>::get_elem(int i)
{
    debug("->get_elem(int i)");
    if(i < 0 || i > size)
    {
        debug("going beyond the bounds of the array!");
        cout << "Going beyond the bounds of the array!" << endl;
        throw i;
    }
    debug("get_elem(int i)->");
    return arr[i];
}
 
template <typename T>
MyArray<T> MyArray<T>::concat_array(const MyArray &copy_arr)
{
    debug("->concat_array(const MyArray &copy_arr)");
    MyArray cur;
    try{
        cur.arr = new T[copy_arr.size + size];
        cur.size = copy_arr.size + size;
        for(int i = 0; i < size; i++)
            cur.arr[i] = arr[i];
 
        int j = 0;
        for(int i = size; i < copy_arr.size + this->size; i++, j++)
            cur.arr[i] = copy_arr.arr[j];
    }
    catch(bad_alloc &x){
        debug(" error : bad_alloc");
        cerr << x.what();
        exit(1);
    }
 
    debug("concat_array(const MyArray &copy_arr)->");
 
    return cur;
}
 
template <typename T>
MyArray<T>& MyArray<T>::operator = (const MyArray &right)
{
    debug("->operator = (const MyArray &right)");
    if(this == &right)
        return *this;
 
    delete []arr;
    size = right.size;
    try{
        if(size == 0)
            throw size;
        arr = new T[size];
        for(int i = 0; i < size; i++)
            arr[i] = right.arr[i];
    }
    catch(bad_alloc &x){
        debug(" error : bad_alloc");
        cerr << x.what();
        exit(1);
    }
 
    debug("operator = (const MyArray &right)->");
    return *this;
 
}
 
template <typename T>
T MyArray<T>::operator [] (int i) const
{
    debug("->operator [] (int i) const");
    if(i < 0 || i > size)
    {
        debug("going beyond the bounds of the array!");
        cout << "Going beyond the bounds of the array!" << endl;
        throw i;
    }
    debug("operator [] (int i) const->");
    return arr[i];
}
 
template <typename T>
ostream& operator <<(ostream &s, const MyArray<T> &right)
{
    debug("->operator <<(ostream &s, const MyArray<T> &right)");
    for(int i = 0; i < right.size; i++)
        s  << right[i] << " ";
    debug("operator <<(ostream &s, const MyArray<T> &right)->");
    return s;
}
 
void debug(char* str){
    int ecode;
    int encode = fprintf(fout, "%s  ", str);
    fprintf(fout, " in code %d\n", encode);
}

Добавлено через 18 минут
также есть main, в котором нужно обрабатывать ошибки работы с файлами (потоками) при считывании исходных
данных, но как это сделать?
Вот мой main

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
int main()
{
    // файлы, из которых я беру данные
    FILE *source_1, *source_2;
    // файл, в который я вписываю информацию(вход в функцию(ошибку, если она есть))
    fout = fopen(filename, "w");
    if (!fout)
    {
        cout << "File not found!" << endl;
        return 1;
    }
    MyArray<int> *data = new MyArray<int>[3];
    source_1 = fopen("1.txt","r");
 
    int id = 0;
    while(fscanf(source_1,"%d", &id ) == 1)
    {
            data[0].add_elem(id);
    }
 
    source_2 = fopen("2.txt","r");
    id = 0;
    while(fscanf(source_2,"%d", &id ) == 1)
    {
        data[1].add_elem(id);
    }
 
    cout << data[0] << endl;
    cout << data[1] << endl;
 
    data[2] = data[0].concat_array(data[1]);
    data[2].bubble_sort();
 
    cout << data[2] << endl;
 
    return 0;
}

Добавлено через 27 минут
В задании сказано:

C++
1
2
3
4
5
6
7
8
9
Реализовать в соответствующем шаблонном классе-контейнере обработку
ошибочных ситуаций с помощью аппарата исключений (try-throw-catch).
Необходимо отслеживать следующие ошибки:
 Ошибки работы с файлами (потоками) при считывании исходных
данных
 Ошибки работы с динамической памятью
 Выход за границы массивов
 Добавление в уже полный контейнер (если размер контейнера
ограничен), извлечение из пустого контейнера.

Нужно написать свой класс ошибок или делать так, как я это делал?

Добавлено через 2 минуты
Если я не могу открыть файл, то как обрабатывать?

C++
1
2
3
4
5
6
source_1 = fopen("11.txt","r");
    if(!source_1)
    {
        throw "Can not open file";
        exit(1);
    }

Это не работает

C++
1
2
3
4
5
6
source_1 = fopen("11.txt","r");
    if(!source_1)
    {
        cout << "Can not open file";
        exit(1);
    }

а это скорее всего не подойдет, так как нужно использовать механизм исключений

Добавлено через 1 час 13 минут
Весь код привел, может кто-то да подскажет?

@Nike1995 · 23.05.2016, 01:22 **[ТС]**

Вот еще исправление

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
#pragma once
#include <iostream>
#include "stdio.h"
#include "stdlib.h"
#include <stdexcept>
using namespace std;
 
const char *filename = "testfile.txt";
FILE *fout;
void debug(char* str);
 
template <typename T>
class MyArray
{
    T *arr;
    size_t size;
public:
    MyArray();
    MyArray(int n);
    MyArray(const MyArray &copy_arr);
    ~MyArray();
    void bubble_sort();
    MyArray concat_array(const MyArray &copy_arr);
    void add_elem(T el);
    T operator [](int i) const;
    MyArray& operator = (const MyArray & right);
    template <typename T1>
    friend ostream& operator <<(ostream &s, const MyArray<T1> &right);
    void show();
};
 
template <typename T>
MyArray<T>::MyArray()
{
    arr = NULL;
    size = 0;
}
 
template <typename T>
MyArray<T>::MyArray(int n)
{
    size = n;
    arr = new T[n];
    for(int i = 0; i < n; i++)
            arr[i] = 0;
}
 
template <typename T>
MyArray<T>::MyArray(const MyArray & copy_arr)
{
    size = copy_arr.size;
    arr = new T[size];
    for(int i = 0; i < size; i++)
        arr[i] = copy_arr.arr[i];
}
 
template <typename T>
MyArray<T>::~MyArray()
{
    delete []arr;
    size = 0;
}
 
template <typename T>
void MyArray<T>::bubble_sort()
{
    debug("->bubble_sort()");
    T buf_el;
    bool swap = false;
    for(int i = 0; i < size - 1; i++)
    {
        swap = false;
        for(int j = 0; j < size - 1 - i; j++)
            if(arr[j] > arr[j + 1])
            {
                buf_el = arr[j];
                arr[j] = arr[j + 1];
                arr[j + 1] = buf_el;
                swap = true;
            }
            if(!swap)
                break;
    }
    debug("bubble_sort()->");
}
 
template <typename T>
void MyArray<T>::add_elem(T el)
{
    debug("->add_elem(T el)");
    T* temp_arr;
    try{
        temp_arr = new T[size];
    }catch(bad_alloc x){
        cerr << x.what();
        debug(" error : bad_alloc");
        exit(1);
    }
    for(int i = 0; i < size; i++)
        temp_arr[i] = arr[i];
    delete []arr;
    try{
        arr = new T[size + 1];
    }catch(bad_alloc x){
        delete []temp_arr;
        cerr << x.what() << endl;
        debug(" error : bad_alloc");
        exit(1);
    }
    for(int i = 0; i < size; i++)
        arr[i] = temp_arr[i];
    delete []temp_arr;
    arr[size] = el;
    size++;
    debug("add_elem(T el)->");
}
 
template <typename T>
MyArray<T> MyArray<T>::concat_array(const MyArray &copy_arr)
{
    debug("->concat_array(const MyArray &copy_arr)");
    MyArray cur;
    try{
        cur.arr = new T[copy_arr.size + size];
        cur.size = copy_arr.size + size;
    }
    catch(bad_alloc &x){
        debug(" error : bad_alloc");
        cerr << x.what();
        exit(1);
    }
    for(int i = 0; i < size; i++)
        cur.arr[i] = arr[i];
 
    int j = 0;
    for(int i = size; i < copy_arr.size + this->size; i++, j++)
        cur.arr[i] = copy_arr.arr[j];
 
    debug("concat_array(const MyArray &copy_arr)->");
 
    return cur;
}
 
template <typename T>
MyArray<T>& MyArray<T>::operator = (const MyArray &right)
{
    debug("->operator = (const MyArray &right)");
    if(this == &right)
        return *this;
 
    delete []arr;
    size = right.size;
    try{
        if(size == 0)
            throw size;
        arr = new T[size];
        for(int i = 0; i < size; i++)
            arr[i] = right.arr[i];
    }
    catch(bad_alloc &x){
        debug(" error : bad_alloc");
        cerr << x.what();
        exit(1);
    }
 
    debug("operator = (const MyArray &right)->");
    return *this;
 
}
 
template <typename T>
T MyArray<T>::operator [] (int i) const
{
    debug("->operator [] (int i) const");
    try {
        if (i < 0 || i >= size)
            throw out_of_range("Index was out of range");
    }
    catch(const out_of_range& e) {
        cerr << e.what() << endl;
        debug("out of range!");
    }
    debug("operator [] (int i) const->");
    return arr[i];
}
 
template <typename T>
ostream& operator <<(ostream &s, const MyArray<T> &right)
{
    debug("->operator <<(ostream &s, const MyArray<T> &right)");
    for(int i = 0; i < right.size; i++)
        s  << right[i] << " ";
    debug("operator <<(ostream &s, const MyArray<T> &right)->");
    return s;
}
 
void debug(char* str){
    fprintf(fout, "%s \n", str);
}
#include <iostream>
#include "stdio.h"
#include "stdlib.h"
#include "MyArray.h"
#include "errors.h"
#include <stdexcept>
using namespace std;
 
int main()
{
    FILE *source_1,
         *source_2;
 
    fout = fopen(filename, "w");
    if (!fout)
    {
        cout << "File not found!" << endl;
        return 1;
    }
 
    MyArray<int> *data;
    try{
        data = new MyArray<int>[3];
    }
    catch(bad_alloc &x)
    {
        cerr << x.what();
        debug("bad alloc in main()!");
    }
 
    try {
        source_1 = fopen("1.txt","r");
        if(!source_1)
            throw 1;
        int id = 0;
        while(fscanf(source_1,"%d", &id ) == 1)
        {
            data[0].add_elem(id);
        }
    }
    catch (int)
    {
        debug("Can't open FILE!");
        cout <<"Can't open FILE!";
    }
    catch(...)
    {
        debug("Unknow error in main()");
        cout << "Unknow error in main()" << endl;
    }
 
 
    try{
        source_2 = fopen("2.txt","r");
        if(!source_2)
            throw 2;
        int id = 0;
        while(fscanf(source_2,"%d", &id ) == 1)
        {
            data[1].add_elem(id);
        }
    }
    catch (int) {
        debug("Can't open FILE!");
        cout <<"Can't open FILE!";
    }
    catch(...)
    {
        debug("Unknow error in main()");
        cout << "Unknow error in main()" << endl;
    }
 
    cout << data[0] << endl;
    cout << data[1] << endl;
 
    data[2] = data[0].concat_array(data[1]);
    data[2].bubble_sort();
 
    cout << data[2] << endl;
 
    return 0;
}

@Renji · 23.05.2016, 02:03

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

Правильно ли тут я обрабатываю исключения?

Не правильно.
1) Я не знаю что хотел сказать автор задания, но в случае любого сбоя шаблонный класс должен кинуть исключение. Конец предложения. Ловить и обрабатывать это исключение не его забота. Иногда все же требуется catch секция для алгоритма "поймал исключение, закрыл вон тот ставший ненужным файлик, кинул исключение дальше". Но это вроде как не ваш случай.
2) Кидать надо потомков std::exception (std::runtime_error, например), ловить std::exception. Опять же, исключения из правила есть, но это не ваш случай.
3) try/catch секцией должен озаботиться пользователь MyArray. Где-то в районе кода для "выберите необходимое действие". Содержимое catch секции - код в духе cout<<"опс... Что-то сбойнуло: "<<error.what()<<endl;
4) И извините, но что за хрень у вас в add_elem? На 99 строчке должно быть что-то вроде:

C++
1
2
3
4
delete arr;
arr=temp_arr;
arr[size++]=el;
return;

IGPIGP · 23.05.2016, 09:06

Nike1995, до того как разбираться с обработкой исключений имеет смысл перечитать выделение и освобождение памяти. Renji, пишет о функции add_elem и он прав.

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
try{
        try{
            T* temp_arr = new T[size + 1];//временный массив указателей на выделенную память
            for(int i = 0; i < size; i++)
                temp_arr[i] = arr[i];// установили на старые адреса и потеряли все новые
            delete [] arr;//удалили память старых адресов теперь и temp_ar смотрит в память формально уже недоступную
//дальше не смотрю
            try{
                arr = new T[size + 1];
                for(int i = 0; i < size; i++)
                    arr[i] = temp_arr[i];
                delete []temp_arr;
                arr[size] = el;
                size++;
            }

Что касается исключений то в конструкторах больнее. То есть: если недобавление элемента приводит к останову, то несоздание объекта и подавно ведь?
Совет: Nike1995 у Вас есть над чем поработать до темы "исключения"(имхо).

@Nike1995 · 23.05.2016, 17:43 **[ТС]**

IGPIGP, Так?

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
template <typename T>
void MyArray<T>::add_elem(T el)
{
    debug("->add_elem(T el)");
    T* temp_arr;
    try{
        temp_arr = new T[size];
    }catch(bad_alloc x){
        cerr << x.what();
        debug(" error : bad_alloc");
        exit(1);
    }
 
    for(int i = 0; i < size; i++)
        temp_arr[i] = arr[i];
 
    delete []arr;
 
    try
    {
        arr = new T[size + 1];
    }
    catch(bad_alloc x){
        delete []temp_arr;
        cerr << x.what() << endl;
        debug(" error : bad_alloc");
        exit(1);
    }
 
    for(int i = 0; i < size; i++)
        arr[i] = temp_arr[i];
 
    delete []temp_arr;
    arr[size++] = el;
 
    debug("add_elem(T el)->");
}

IGPIGP · 23.05.2016, 18:59

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

IGPIGP, Так?

Формально, исключение из конструктора лучше. Обработку имеет смысл делать если нужно освободит вручную какие-то ресурсы или сохранить какие-то данные из прерванного кода. Что касается исключения от выделения памяти, то оно и само вывалит программу, если не обработать. То есть если у Вас есть что передать внешнему коду из прерванного - передайте. Пусть ловит и обрабатывает. Вашими руками конечно.

Я сутра написал не разобрав. Подумал что это двумерный массив... За тот комментарий про выделение памяти - мои извинения.
Однако непонятно зачем:

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

C++
1
for(int i = 0; i < size; i++) arr[i] = temp_arr[i]; delete []temp_arr;

Вы выделили массив temp_ar. Потом скопировали туда содержимое arr
Теперь запишите в конец добавляемый элемент:

C++
1
 temp_ar[size/*неудачное имя*/]=el;

после чего освободите arr и установите arr в temp_ar:

C++
1
2
delete []arr;
arr=temp_ar;

Не нужно опять выделять под arr и копировать туда из области temp_ar.

@Nike1995 · 23.05.2016, 20:23 **[ТС]**

IGPIGP, ругается у меня компилятор на arr = temp_arr

Добавлено через 21 минуту
а перегружать оператор для массива не хочется

IGPIGP · 23.05.2016, 20:30

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

ругается у меня компилятор на arr = temp_arr

что говорит?

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

а перегружать оператор для массива не хочется

Там есть массив только для удобства в речи. На самом деле там указатель. Он в private секции и это не препятствие для методов-членов, в т.ч. конструкторов. Никакой сеттер а тем более оператор не нужен.

@Nike1995 · 23.05.2016, 21:23 **[ТС]**

Программа просто зависает. у меня Code::Blocks

IGPIGP · 24.05.2016, 00:06

Добавлено через 18 секунд

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

Программа просто зависает.

Ну и пусть. Напишите как-то иначе. Вот что я имел ввиду:

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
#include <iostream>
using namespace std;
template <typename T>
class MyArray
{
    T *arr;
    size_t size;
public:
    MyArray();
    MyArray(int n);
    MyArray(const MyArray &copy_arr);
    ~MyArray();
    void bubble_sort();
    MyArray concat_array(const MyArray &copy_arr);
    void add_elem(T el);
    T &operator [](int i) const;//&
    
    MyArray& operator = (const MyArray & right);
    template <typename T1>
    friend ostream& operator <<(ostream &s, const MyArray<T1> &right);
    void show();
};
 
template <typename T>
MyArray<T>::MyArray()
{
   
    arr = NULL;
    size = 0;
    
}
 
template <typename T>
MyArray<T>::MyArray(int n)
{
    size = n;
    arr = new T[n];
    for(int i = 0; i < n; i++)
            arr[i] = 0;
}
 
template <typename T>
MyArray<T>::MyArray(const MyArray & copy_arr)
{
if(arr!=NULL)delete [] arr;
size = copy_arr.size;
arr = new T[size];
for(int i = 0; i < size; i++)
arr[i] = copy_arr.arr[i];
 
}
 
template <typename T>
MyArray<T>::~MyArray()
{
    if(arr!=NULL)delete []arr;   
}
 
template <typename T>
void MyArray<T>::add_elem(T el)
{
 
            size_t old_size=size++;  
            T* temp_arr = new T[size];
 
            for(int i = 0; i < old_size; i++)
                temp_arr[i] = arr[i];
 
                temp_arr[old_size] = el;
 
            if(arr!=NULL)delete [] arr;
 
            arr=temp_arr;
                
}
template <typename T>
T& MyArray<T>::operator [] (int i) const
{
    
    if(i < 0 || i > size)
    {       
        cout << "Going beyond the bounds of the array!" << endl;
       cin.get();
    }
   
    return arr[i];
}
template <typename T>
ostream& operator <<(ostream &s, const MyArray<T> &right)
{    
    for(int i = 0; i < right.size; i++)
        s  << right[i]<< " ";    
    return s;
}
 
 
int main(int argc, char* argv[])
{
    const int sz=3;
    int arr[]={1,2,3};
MyArray<int> myArray;
for(int i=0; i<sz; i++)
myArray.add_elem(arr[i]);
cout<<myArray<<endl;
system("pause");
return 0;
}

У меня не кодблок и не зависает. Хотя я думаю это не связанные вещи.

@Nike1995 · 24.05.2016, 04:44 **[ТС]**

IGPIGP, все равно виснет)) С исключениями более-менее разобрался, спасибо

IGPIGP · 24.05.2016, 11:55

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

C++
1
if(i < 0 || i > size)

У Вас Nike1995, скатал не глядя. А надо бы:

C++
1
if(i < 0 || i > size-1)

@Nike1995 · 24.05.2016, 12:21 **[ТС]**

IGPIGP, Да, Вы правы. Не учёл.

Новые блоги и статьи Все статьи Все блоги /
PhpStorm 2025.3: WSL Terminal всегда стартует в ~ and_y87 14.12.2025 PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .	Access VikBal 11.12.2025 Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.	Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro	Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .	От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .
Поиск "дружественных имён" СОМ портов Argus19 22.11.2025 Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .	Сколько Государство потратило денег на меня, обеспечивая инсулином. Programma_Boinc 20.11.2025 Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .	Ломающие изменения в C#.NStar Alpha Etyuhibosecyu 20.11.2025 Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .	Мысли в слух kumehtar 18.11.2025 Кстати, совсем недавно имел разговор на тему медитаций с людьми. И обнаружил, что они вообще не понимают что такое медитация и зачем она нужна. Самые базовые вещи. Для них это - когда просто люди. . .	Создание Single Page Application на фреймах krapotkin 16.11.2025 Статья исключительно для начинающих. Подходы оригинальностью не блещут. В век Веб все очень привыкли к дизайну Single-Page-Application . Быстренько разберем подход "на фреймах". Мы делаем одну. . .

@Nike1995 3 / 3 / 8 Регистрация: 14.10.2012 Сообщений: 288
	23.05.2016, 20:23 [ТС]
	IGPIGP, ругается у меня компилятор на arr = temp_arr Добавлено через 21 минуту а перегружать оператор для массива не хочется 0

@Nike1995 3 / 3 / 8 Регистрация: 14.10.2012 Сообщений: 288
	23.05.2016, 21:23 [ТС]
	Программа просто зависает. у меня Code::Blocks 0

@Nike1995 3 / 3 / 8 Регистрация: 14.10.2012 Сообщений: 288
	24.05.2016, 04:44 [ТС]
	IGPIGP, все равно виснет)) С исключениями более-менее разобрался, спасибо 0

@Nike1995 3 / 3 / 8 Регистрация: 14.10.2012 Сообщений: 288
	24.05.2016, 12:21 [ТС]
	IGPIGP, Да, Вы правы. Не учёл. 0

Обработка исключительных ситуаций

Решение