Перегруженные операторы - ошибка в коде

@peva · Регистрация: 12.02.2017

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

Здравствуйте. Подскажите, пожалуйста, в чем ошибка.
Нужно описать класс динамического массива и перегрузить операторы. С перегрузкой унарных операторов разобралась. Проблема возникла с перегрузкой оператора + для поэлементного сложения массивов и =.
Вот код. Правильно работает только если массивы одного размера.

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
#include <iostream>
#include <time.h>
using namespace std;
 
class DynArray
{
    int len;
    int *array;
 
public:
 
    DynArray()
    {
        len = 0;
        array = NULL;
    }
    DynArray(int size) //конструктор, заполнение массива случайными значениями от -20 до 20
    {
        len = size;
        array = new int[size];
        for (int i = 0; i < size; i++)
            array[i] = -20 + rand() % 41;
    }
 
    DynArray(const DynArray &newArr) //Конструктор копирования
    {
        len = newArr.len;
        array = new int[len];
        for (int i = 0; i < len; i++)
        {
            array[i] = newArr.array[i];
        }
    }
    ~DynArray() //деструктор
    {
        delete[] array;
    }
 
    int getLen() const
    {
        return len;
    }
 
    void showDynArray()
    {
        for (int i = 0; i < len; i++)
        {
            cout << array[i] << " ";
        }
    }
 
    int &operator[](int i)
    {
        return array[i];
    }
 
    DynArray operator+(const DynArray &newArr)
    {
        int i;
        DynArray temp(len);
        if (len < newArr.getLen())
        {
            for (size_t i = 0; i < len; i++)
            {
                temp[i] = array[i] + newArr.array[i];
            }
            for (size_t i = 0; i >= len, i < newArr.getLen(); i++)
            {
                temp[i] = newArr.array[i];
            }
        }
 
        else if (len > newArr.getLen())
        {
            for (size_t i = 0; i < newArr.getLen(); i++)
            {
                temp[i] = array[i] + newArr.array[i];
            }
            for (size_t i = 0; i >= newArr.getLen(), i < len; i++)
            {
                temp[i] = array[i];
            }
        }
        else
        {
            for (size_t i = 0; i < len; i++)
            {
                temp[i] = array[i] + newArr.array[i];
            }
        }
    return temp;
    }
    
    DynArray operator=(const DynArray &newArr)
    {
        len = newArr.len;
        array = new int[len];
        for (int i = 0; i < len; i++)
        {
            array[i] = newArr.array[i];
        }
        return *this; 
    }
};
 
void main()
{
    srand(time(NULL));
    setlocale(LC_ALL, "rus");
 
    DynArray mas1(5);
    DynArray mas2(5);
    DynArray mas3(0);
 
    cout << "Созданы 3 массива: " << endl;
    cout << "Размер массива 1: " << mas1.getLen() << endl;
    cout << "Массив 1: ";
    mas1.showDynArray();
    cout << endl << endl;
 
    cout << "Размер массива 2: " << mas2.getLen() << endl;
    cout << "Массив 2: ";
    mas2.showDynArray();
    cout << endl << endl;
 
    cout << "Размер массива 3: " << mas3.getLen() << endl;
    cout << "Массив 3 пустой. ";
    mas3.showDynArray();
    cout << endl << endl;
 
    cout << "Сложение двух массивов поэлементно:" << endl;
    mas3 = mas1 + mas2;
 
    mas3.showDynArray();
    cout << endl << endl;
}

@palva · 19.06.2017, 22:17

Наверно так:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
if (len < newArr.getLen())
{
    for (size_t i = 0; i < newArr.getLen(); i++)
    {
        if(i < len)
            temp[i] = array[i] + newArr.array[i];
        else
            temp[i] = newArr.array[i];
    }
}
 
else if (len > newArr.getLen())
{
    for (size_t i = 0; i < len; i++)
        if(i < newArr.getLen())
            temp[i] = array[i] + newArr.array[i];
        else
            temp[i] = newArr.array[i];
    }
}

@peva · 19.06.2017, 22:35 **[ТС]**

palva, спасибо, но, к сожалению, нет.

Если первый массив больше - и в Вашем и в моем вариантах корректно прибавляются только элементы до размера меньшего массива. Далее - мусор.
Если второй массив больше - оба варианта не работают вообще.

@gray_fox · 19.06.2017, 23:24

peva, как то так думаю:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
#include <algorithm>
#include <functional>
 
// ...
 
DynArray operator+(const DynArray &newArr) const {
   int const minLen = std::min(len, newArr.len);
   int const maxLen = std::max(len, newArr.len);
        
   DynArray temp(maxLen);
                
   std::transform(array, array + minLen, newArr.array, temp.array, std::plus<int>());
        
   int const* const maxArray = len < newArr.len ? newArr.array : array;
   std::copy(maxArray + minLen, maxArray + maxLen, temp.array + minLen);
        
   return temp;
}

transform и copy можно заменить на обычные циклы, если нужно.

nd2 · 19.06.2017, 23:25

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

C++
1
2
3
4
DynArray operator=(const DynArray &newArr) 
{ 
      len = newArr.len; 
      array = new int[len];

Старую память нужно освобождать.

@peva · 19.06.2017, 23:35 **[ТС]**

gray_fox,
Я еще очень мало знаю, и даже о таких заголовочных файлах не слышала. Но тем интереснее будет разобраться и понять. Спасибо.

@gray_fox · 19.06.2017, 23:42

peva, вот без них, если так понятнее:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
DynArray operator+(const DynArray &newArr) const {
    // int const minLen = std::min(len, newArr.len);
    int const minLen = len < newArr.len ? len : newArr.len;
    // int const minLen = std::max(len, newArr.len);
    int const maxLen = len < newArr.len ? newArr.len : len;
        
    DynArray temp(maxLen);
                
    // std::transform(array, array + minLen, newArr.array, temp.array, std::plus<int>());
    for (int i = 0; i != minLen; ++i) {
       temp.array[i] = array[i] + newArr.array[i];   
    }
        
    int const* const maxArray = len < newArr.len ? newArr.array : array;
    // std::copy(maxArray + minLen, maxArray + maxLen, temp.array + minLen);
    for (int i = minLen; i != maxLen; ++i) {
       temp.array[i] = maxArray[i]; 
    }
        
    return temp;
}

@peva · 20.06.2017, 01:02 **[ТС]**

gray_fox, огромное спасибо!
Да, так понятнее, но с первоначальным вашим вариантом тоже обязательно разберусь.

@TheCalligrapher · 20.06.2017, 01:07

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

C++
1
2
3
4
5
6
7
8
9
10
DynArray operator=(const DynArray &newArr)
{
  len = newArr.len;
  array = new int[len];
  for (int i = 0; i < len; i++)
  {
    array[i] = newArr.array[i];
  }
  return *this; 
}

Оператор присваивания реализован неправильно. Задача оператора присваивания (в простейшем варианте): освободить старые данные объекта, скопировать в объект новые данные. Я не вижу тут освобождения старых данных.

Лучше было бы применить идиому Copy-And-Swap.

Новые блоги и статьи Все статьи Все блоги /
делаю науч статью по влиянию грибов на сукцессию anaschu 13.03.2026 прикрепляю статью	SDL3 для Desktop (MinGW): Создаём пустое окно с нуля для 2D-графики на SDL3, Си и C++ 8Observer8 10.03.2026 Содержание блога Финальные проекты на Си и на C++: hello-sdl3-c. zip hello-sdl3-cpp. zip Результат:	Установка CMake и MinGW 13.1 для сборки С и C++ приложений из консоли и из Qt Creator в EXE 8Observer8 10.03.2026 Содержание блога MinGW - это коллекция инструментов для сборки приложений в EXE. CMake - это система сборки приложений. Здесь описаны базовые шаги для старта программирования с помощью CMake и. . .	Как дизайн сайта влияет на конверсию: 7 решений, которые реально повышают заявки Neotwalker 08.03.2026 Многие до сих пор воспринимают дизайн сайта как “красивую оболочку”. На практике всё иначе: дизайн напрямую влияет на то, оставит человек заявку или уйдёт через несколько секунд. Даже если у вас. . .
Модульная разработка через nuget packages DevAlt 07.03.2026 Сложившийся в . Net-среде способ разработки чаще всего предполагает монорепозиторий в котором находятся все исходники. При создании нового решения, мы просто добавляем нужные проекты и имеем. . .	Модульный подход на примере F# DevAlt 06.03.2026 В блоге дяди Боба наткнулся на такое определение: В этой книге («Подход, основанный на вариантах использования») Ивар утверждает, что архитектура программного обеспечения — это структуры,. . .	Управление камерой с помощью скрипта OrbitControls.js на Three.js: Вращение, зум и панорамирование 8Observer8 05.03.2026 Содержание блога Финальная демка в браузере работает на Desktop и мобильных браузерах. Итоговый код: orbit-controls-threejs-js. zip. Сканируйте QR-код на мобильном. Вращайте камеру одним пальцем,. . .	SDL3 для Web (WebAssembly): Синхронизация спрайтов SDL3 и тел Box2D 8Observer8 04.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-sync-physics-sprites-sdl3-c. zip На первой гифке отладочные линии отключены, а на второй включены:. . .

@peva 0 / 0 / 1 Регистрация: 12.02.2017 Сообщений: 55
	19.06.2017, 22:35 [ТС]
	palva, спасибо, но, к сожалению, нет. Если первый массив больше - и в Вашем и в моем вариантах корректно прибавляются только элементы до размера меньшего массива. Далее - мусор. Если второй массив больше - оба варианта не работают вообще. 0

@peva 0 / 0 / 1 Регистрация: 12.02.2017 Сообщений: 55
	19.06.2017, 23:35 [ТС]
	gray_fox, Я еще очень мало знаю, и даже о таких заголовочных файлах не слышала. Но тем интереснее будет разобраться и понять. Спасибо. 0

@peva 0 / 0 / 1 Регистрация: 12.02.2017 Сообщений: 55
	20.06.2017, 01:02 [ТС]
	gray_fox, огромное спасибо! Да, так понятнее, но с первоначальным вашим вариантом тоже обязательно разберусь. 0

Перегруженные операторы - ошибка в коде

Решение