Форум программистов, компьютерный форум, киберфорум
Наши страницы
С++ для начинающих
Войти
Регистрация
Восстановить пароль
 
 
Рейтинг 4.83/6: Рейтинг темы: голосов - 6, средняя оценка - 4.83
petuz
1 / 1 / 1
Регистрация: 26.06.2015
Сообщений: 55
1

Написать перегрузку operator+() для класса "Массив"

27.12.2015, 21:51. Просмотров 1021. Ответов 20
Метки нет (Все метки)

Добрый день,

Подскажите, как выйти правильно из ситуации:
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
Array& Array::operator+(const Array& obj)
{
    Array temp;
    
    if (size > 0 && obj.size > 0)
        temp.size = size + obj.size;
 
    temp.array = new int[temp.size];
 
    for (unsigned int i = 0; i < size; ++i)
        temp.array[i] = array[i];
 
    for (unsigned int i = 0; i < obj.size; ++i)
        temp.array[i + size] = obj.array[i];
    
    return temp;
}
Получается, что при выходе из функции я теряю адрес возвращаемого (и уничтожаемого) объекта temp.

А как мне в С получить это самое temp, если я хочу сделать такое для объектов:
C++
1
C = A + B;
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
27.12.2015, 21:51
Ответы с готовыми решениями:

Написать перегрузку оператора "минус" для строк
Такой вопрос, для строк нужно написать оператор &quot;минус&quot;, что это значит? Если...

Перегрузка "operator =" для шаблона класса
как правильно перегрузить operator= для щаблона, по идее я принимаю 2 ссылки на...

Реализовать перегрузку operator+() для пользователского класса
Есть класс ДРОБЬ, его члены ЧИСЛИТЕЛЬ И ЗНАМЕНАТЕЛЬ, а методы - конструктор и...

Написать конструктор для пользовательского класса "Двусвязный список"
Попытался построить двусвязный список. Как написать этот конструктор ? Может...

Написать шаблон для класса "Рабочий"
Всем доброго время суток! У меня возникла следующая проблема. Препод дал...

20
Croessmah
++Ͻ
14740 / 8422 / 1597
Регистрация: 27.09.2012
Сообщений: 20,714
Записей в блоге: 2
Завершенные тесты: 1
27.12.2015, 21:53 2
Лучший ответ Сообщение было отмечено petuz как решение

Решение

C++
1
Array Array::operator+(const Array& obj)
2
petuz
1 / 1 / 1
Регистрация: 26.06.2015
Сообщений: 55
27.12.2015, 22:20  [ТС] 3
Спасибо, понял.

Я уж еще понаглею, а здесь что не так (error C2676: binary '++': 'Array' does not define this operator or a conversion to a type acceptable to the predefined operator):

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
Array& Array::operator++()
{
    if (size > 0)
    {
        Array temp;
        temp.size = size + 1;
        temp.array = new int[temp.size];
        for (unsigned int i = 0; i < size; ++i)
        {
            temp.array[i] = array[i];
        }
        temp.array[size] = 3;
        size++;
        delete[]array;
        array = temp.array;
 
        return *this;
    }
    else
        return *this;
}
0
Ferrari F1
792 / 522 / 156
Регистрация: 27.01.2015
Сообщений: 3,025
Записей в блоге: 1
Завершенные тесты: 1
27.12.2015, 22:29 4
petuz, а вы постфиксный инкремент перегрузить не забыли?
1
petuz
1 / 1 / 1
Регистрация: 26.06.2015
Сообщений: 55
27.12.2015, 22:32  [ТС] 5
А нужно обязательно оба?
0
Ferrari F1
792 / 522 / 156
Регистрация: 27.01.2015
Сообщений: 3,025
Записей в блоге: 1
Завершенные тесты: 1
27.12.2015, 22:34 6
petuz, покажите код в main(), чтобы можно было понять, какие операции с объектом вы производите.
0
petuz
1 / 1 / 1
Регистрация: 26.06.2015
Сообщений: 55
27.12.2015, 22:37  [ТС] 7
Тут я уже понял, что вызывал постфиксный инкримент для объекта, а такой перегрузки не было.
Теперь вызываю, допустим
C++
1
2
Array C = B;
++С;
Но все равно не работает - что-то с динамической памятью не то...
0
Ferrari F1
792 / 522 / 156
Регистрация: 27.01.2015
Сообщений: 3,025
Записей в блоге: 1
Завершенные тесты: 1
27.12.2015, 22:41 8
Цитата Сообщение от petuz Посмотреть сообщение
что-то с динамической памятью не то...
Я давно не прогал на С++, но наверное нужно перегружать оператор=, либо копирующий конструктор.
0
hoggy
Заблокирован
Эксперт С++
27.12.2015, 22:42 9
http://rextester.com/PWTVI57955

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
// --- пример-иллюстрация
// изготовления канонического вида
// операторов инкремента и декремента
 
#include <iostream>
 
 
struct iterator
{
    iterator(int* colletion): m_pointer(colletion){}
    
    // преффиксная форма ++it
    iterator& operator++() { return ++m_pointer, *this; }
    iterator& operator--() { return --m_pointer, *this; }
 
    // постфиксная форма it++
    iterator operator++(int) 
    {
        const auto copy = *this; 
        operator++(); 
        return copy;
    }
 
    iterator operator--(int)
    {
        const auto copy = *this; 
        operator--(); 
        return copy;
    }
    
    int& operator*() { return *m_pointer; }
 
    
private:
    
    int* m_pointer;
 
};
 
int main()
{
    std::cout << "Hello, world!\n";
    
    int arr[]={1,2,3};
    
    iterator it(arr);
    std::cout << "value = " << *it <<'\n';
    
    ++it;
    std::cout << "value = " << *it <<'\n';
    
    it++;
    std::cout << "value = " << *it <<'\n';
    
    --it;
    std::cout << "value = " << *it <<'\n';
    
    it--;
    std::cout << "value = " << *it <<'\n';
    
}
1
petuz
1 / 1 / 1
Регистрация: 26.06.2015
Сообщений: 55
27.12.2015, 22:44  [ТС] 10
Цитата Сообщение от Ferrari F1 Посмотреть сообщение
Нужно перегружать оператор =
уже есть:
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
Array& Array::operator=(const Array& obj)
{
    if (this == &obj)
        return *this;
 
    delete[]array;
 
    if (obj.size)
        size = obj.size;
    else
        size = 1;
    
    if (obj.array)
    {
        array = new int[size];
        for (unsigned int i = 0; i < size; ++i)
            array[i] = obj.array[i];
    }
    else
        array = nullptr;
}
0
petuz
1 / 1 / 1
Регистрация: 26.06.2015
Сообщений: 55
27.12.2015, 22:47  [ТС] 11
hoggy, спасибо, я теорию понимаю, а с моим случаем, что не так?
Написать перегрузку operator+() для класса "Массив"
0
petuz
1 / 1 / 1
Регистрация: 26.06.2015
Сообщений: 55
27.12.2015, 22:50  [ТС] 12
Цитата Сообщение от Ferrari F1 Посмотреть сообщение
либо копирующий конструктор.
C++
1
2
3
4
5
6
7
8
9
10
11
12
Array::Array(const Array &obj)
{
    if (obj.size > 0)
        size = obj.size;
 
    array = new int[size];
 
    for (unsigned int i = 0; i < size; i++)
    {
        array[i] = obj.array[i];
    }
}
0
Ferrari F1
792 / 522 / 156
Регистрация: 27.01.2015
Сообщений: 3,025
Записей в блоге: 1
Завершенные тесты: 1
27.12.2015, 22:50 13
hoggy, а вот у вас в методе:
C++
1
2
3
4
5
6
iterator operator++(int) 
    {
        const auto copy = *this; 
        operator++(); 
        return copy;
    }
оператор
C++
1
operator++();
интерпритируется компилятором как:
C++
1
this->operator++();
?
0
hoggy
Заблокирован
Эксперт С++
27.12.2015, 23:03 14
Цитата Сообщение от petuz Посмотреть сообщение
а с моим случаем, что не так?
покараптилась куча.

ваш кривой алгоритм приводит к повреждению кучи.
срабатывает дебаг-защита.
0
Ferrari F1
792 / 522 / 156
Регистрация: 27.01.2015
Сообщений: 3,025
Записей в блоге: 1
Завершенные тесты: 1
27.12.2015, 23:04 15
petuz, это сложно, помочь вам, не имея полностью всего кода.
Не знаете, где ошибка? Брейкпоинтов понаставьте в сомнительных местах, и проблема найдется.
0
hoggy
Заблокирован
Эксперт С++
27.12.2015, 23:05 16
Цитата Сообщение от Ferrari F1 Посмотреть сообщение
оператор
operator++();
интерпритируется компилятором как:
this->operator++();
?
да.
0
petuz
1 / 1 / 1
Регистрация: 26.06.2015
Сообщений: 55
28.12.2015, 10:04  [ТС] 17
Цитата Сообщение от Ferrari F1 Посмотреть сообщение
это сложно, помочь вам, не имея полностью всего кода.
Да я вижу где ошибка, а как исправить... Вот сейчас, например, получается при перегрузке префиксной формы ++ должен добавляться еще один элемент массива и туда записаться число 33, а вместо него мусор...

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
#include <iostream>
using namespace std;
 
class Array
{
    int* array;
    unsigned int size;
public:
    Array();
    Array(unsigned int, int);
    Array(const Array&);
    ~Array();
    int& operator[](unsigned int);
    Array& operator=(const Array&);
    Array operator+(const Array&);
    Array operator-(const Array&);
    Array operator++();
    void Print();
};
 
 
 
 
Array::Array()
{
    array = nullptr;
    size = 1;
}
 
Array::Array(unsigned int size, int number)
{
    if (size == 0)
        this->size = 1;
    else
        this->size = size;
 
    array = new int[this->size];
 
    for (unsigned int i = 0; i < this->size; ++i)
        array[i] = number;
}
 
Array::Array(const Array &obj)
{
    if (obj.size > 0)
        size = obj.size;
 
    array = new int[size];
 
    for (unsigned int i = 0; i < size; i++)
    {
        array[i] = obj.array[i];
    }
}
 
Array::~Array()
{
    if (array != nullptr)
    {
        delete[]array;
        array = nullptr;
    }
}
 
int & Array::operator[](unsigned int index)
{
    return array[index];
}
 
Array& Array::operator=(const Array& obj)
{
    if (this == &obj)
        return *this;
 
    delete[]array;
 
    if (obj.size)
        size = obj.size;
    else
        size = 1;
    
    if (obj.array)
    {
        array = new int[size];
        for (unsigned int i = 0; i < size; ++i)
            array[i] = obj.array[i];
    }
    else
        array = nullptr;
}
 
Array Array::operator+(const Array& obj)
{
    Array temp;
    
    if (size > 0 && obj.size > 0)
        temp.size = size + obj.size;
 
    temp.array = new int[temp.size];
 
    for (unsigned int i = 0; i < size; ++i)
        temp.array[i] = array[i];
 
    for (unsigned int i = 0; i < obj.size; ++i)
        temp.array[i + size] = obj.array[i];
    
    return temp;
}
 
Array Array::operator-(const Array& obj)
{
    if ((size - obj.size) > 0)
    {
        Array temp;
 
        if (size > 0 && size > obj.size)
            temp.size = size - obj.size;
 
        temp.array = new int[temp.size];
 
        for (unsigned int i = 0; i < temp.size; ++i)
            temp.array[i] = array[i];
 
        return temp;
    }
    else
        return *this;   
}
 
Array Array::operator++()
{
    Array temp;
    temp.size = size + 1;
    temp.array = new int[temp.size];
 
    for (unsigned int i = 0; i < size; ++i)
    {
        temp.array[i] = array[i];
    }
    temp.array[size] = 33;
    
    return temp;
}
 
void Array::Print()
{
    if (size > 0 && array != nullptr)
    {
        for (unsigned int i = 0; i < size; ++i)
        {
            cout << array[i] << ' ';
        }
        cout << endl;
    }
}
 
 
void main()
{
    Array A(7, 1);
    A.Print();
    Array B(5, 2);
    B.Print();
 
    A[3] = 3;
    cout << A[3] << endl;
    A.Print();
 
    Array C = B;
    C.Print();
 
    C = A;
    C.Print();
 
    C = A + B;
    C.Print();
    
    C = A - B;
    C.Print();
    
    ++C;
    C.Print();  
 
 
 
}
Добавлено через 10 часов 29 минут
Почему там мусор?
0
Mr.X
Эксперт С++
3182 / 1709 / 435
Регистрация: 03.05.2010
Сообщений: 3,867
28.12.2015, 15:25 18
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
#define _SCL_SECURE_NO_WARNINGS
#include <algorithm>
#include <iostream>
#include <iterator>
///////////////////////////////////////////////////////////////////////////////
class T_array
{
    //-------------------------------------------------------------------------
    int     *   array_;
    size_t      size_;
    //-------------------------------------------------------------------------
public:
    //-------------------------------------------------------------------------
    T_array                     ();
    T_array                     (size_t, int);
    T_array                     (const T_array&);
    ~T_array                    ();
    int         &   operator[]  ( size_t );
    T_array     &   operator=   ( const T_array& );
    T_array         operator+   ( const T_array& );
    T_array         operator-   ( const T_array& );
    T_array     &   operator++  ();
    void            print       ();
    //-------------------------------------------------------------------------
};
///////////////////////////////////////////////////////////////////////////////
T_array::T_array()
    :
    array_  (),
    size_   ()
{}
///////////////////////////////////////////////////////////////////////////////
T_array::T_array
    (
        size_t  size, 
        int     val     =   0
    )
    :
    size_( size )
{
    array_ = new int[ size_ ];
 
    std::fill
        (
            array_,
            array_ + size_,
            val
        ); 
}
///////////////////////////////////////////////////////////////////////////////
T_array::T_array( T_array   const   &   obj )
{
    size_   =   obj.size_;
    array_  =   new int[ size_ ];
 
    std::copy
        (
            obj.array_,
            obj.array_ + obj.size_,
            array_
        );
}
///////////////////////////////////////////////////////////////////////////////
T_array::~T_array()
{
    if( size_ )
    {
        delete[]    array_;
    }   
}
///////////////////////////////////////////////////////////////////////////////
int     &   T_array::operator[]     ( size_t    index )
{
    if( index   >=  size_ )
    {
        throw   "Выход за пределы массива.";
    }
 
    return  array_[ index ];
}
///////////////////////////////////////////////////////////////////////////////
T_array     &   T_array::operator=  ( T_array    const   &   obj )
{
    if( this    ==  &obj )
    {
        return  *this;
    }
    
    delete[]    array_;
 
    size_   =   obj.size_;
    array_  =   new     int[ size_ ];
 
    std::copy
        (
            obj.array_,
            obj.array_ + obj.size_,
            array_
        ); 
}
///////////////////////////////////////////////////////////////////////////////
T_array     T_array::operator+  ( T_array   const   &   obj )
{
    T_array     res( size_  +   obj.size_ );
 
    std::copy
        (
            array_,
            array_  +   size_,
            res.array_
        );
 
    std::copy
        (
            obj.array_,
            obj.array_  +   obj.size_,
            res.array_  +   size_ 
        );
    
    return  res;
}
///////////////////////////////////////////////////////////////////////////////
T_array     T_array::operator-  ( T_array   const   &   obj )
{
    if  (
            obj.size_   >=  size_
        )
    {
        return  T_array();
    }
 
    T_array     res( size_ - obj.size_ );
 
    std::copy
        (
            array_,
            array_  +   res.size_,
            res.array_
        );
 
    return  res;
}
///////////////////////////////////////////////////////////////////////////////
T_array     &   T_array::operator++()
{
    auto    res     =       *this
                        +   T_array( 1,     33  );
 
    std::swap
        (
            res,
            *this
        );
 
    return  *this;
}
///////////////////////////////////////////////////////////////////////////////
void T_array::print()
{
    std::copy
        (
            array_,
            array_  +   size_,
            std::ostream_iterator< int >    ( std::cout, " " )
        );
 
    std::cout   <<  std::endl;
}
///////////////////////////////////////////////////////////////////////////////
void main()
{
    std::locale::global(std::locale(""));
 
    for(;;)
    {
        std::cout   <<  "\n\n\n\nНовый тест:"
                    <<  std::endl;
 
        auto    A   =   rand() % 2
                            ?   T_array(7, 1)
                            :   T_array();
 
        A.print();
 
        auto    B   =   rand() % 2
                            ?   T_array(5, 2)
                            :   T_array();
 
        B.print();
 
        try
        {
            A[3] = 3;
            std::cout << A[3] << std::endl;
            A.print();
        }
        catch(...)
        {
            std::cout   <<  "Что-то с индексами!"
                        <<  std::endl;
        }
 
        T_array C = B;
        C.print();
 
        C = A;
        C.print();
 
        C = A + B;
        C.print();
 
        C = A - B;
        C.print();
 
        ++C;
        C.print();  
 
        ++++++C;
        C.print();  
 
        system("pause");
    }//fof
}
1
petuz
1 / 1 / 1
Регистрация: 26.06.2015
Сообщений: 55
29.12.2015, 12:59  [ТС] 19
Спасибо! Да, мне еще учиться и учиться.
А скажите, считается нормальным использование перегруженного оператора внутри описания перегрузки другого? Это же более "памятиёмко" выходит или нет?
И еще, как все-таки сделать чистую перегрузку ++С? (без использования других перегруженных операторов)
0
Mr.X
Эксперт С++
3182 / 1709 / 435
Регистрация: 03.05.2010
Сообщений: 3,867
29.12.2015, 13:39 20
Лучший ответ Сообщение было отмечено petuz как решение

Решение

Цитата Сообщение от petuz Посмотреть сообщение
А скажите, считается нормальным использование перегруженного оператора внутри описания перегрузки другого?
Ну, повторное использование кода - это основной принцип программирования.
Цитата Сообщение от petuz Посмотреть сообщение
Это же более "памятиёмко" выходит или нет?
Мне даже интересно какую такую память вы собрались экономить в своей микроскопической программе!
Вообще-то преждевременная оптимизация считается большим злом.
В современном программировании понятная и легкочитаемая программа ценится выше, чем более быстрая, или использующая меньше памяти, так как и компьютерное время, и память стремительно дешевеют, а время программиста, который будет разбираться в вашей программе, как стоило, так и будет стоить.
1
29.12.2015, 13:39
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
29.12.2015, 13:39

Реализовать перегрузку операторов сравнения для пользовательского типа "Обыкновенная дробь"
Структура. дробное сравнение 1. Создайте структуру дробей, состоящую из...

Разработать класс "Массив больших чисел", который состоит из объектов класса "Большие целые числа". Найти сумму элементов массива.
Разработать класс &quot;Массив больших чисел&quot;, который состоит из объектов класса...

Перегрузить операторы operator+() и operator*() в пользовательском классе "Комплексное число"
Здравствуйте. Предлагаю заняться арифметикой. Создал прослейший класс,...


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

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

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