Форум программистов, компьютерный форум, киберфорум
С++ для начинающих
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
Рейтинг 4.67/6: Рейтинг темы: голосов - 6, средняя оценка - 4.67
0 / 0 / 1
Регистрация: 12.02.2017
Сообщений: 55

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

19.06.2017, 22:02. Показов 1125. Ответов 8

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

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;
}
0
Лучшие ответы (1)
IT_Exp
Эксперт
34794 / 4073 / 2104
Регистрация: 17.06.2006
Сообщений: 32,602
Блог
19.06.2017, 22:02
Ответы с готовыми решениями:

Перегруженные функции и операторы
вопрос может быть тривиальный, но все же: как влияет количество вариантов перегруженных функций и операторов на скорости обработки? то...

Указатели и перегруженные операторы
Когда перегружается какой нибудь оператор, допустим квадратные скобки, то для использования оператора нужно писать такую конструкцию: ...

Не работают перегруженные операторы
проблема такова: есть перегруженный оператор &quot;=&quot;, при вызове которого выполняется присвоение значений массива, однако же, результат...

8
 Аватар для palva
4276 / 2969 / 692
Регистрация: 08.06.2007
Сообщений: 9,922
Записей в блоге: 4
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];
    }
}
1
0 / 0 / 1
Регистрация: 12.02.2017
Сообщений: 55
19.06.2017, 22:35  [ТС]
palva, спасибо, но, к сожалению, нет.

Если первый массив больше - и в Вашем и в моем вариантах корректно прибавляются только элементы до размера меньшего массива. Далее - мусор.
Если второй массив больше - оба варианта не работают вообще.
0
What a waste!
 Аватар для gray_fox
1610 / 1302 / 180
Регистрация: 21.04.2012
Сообщений: 2,733
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 можно заменить на обычные циклы, если нужно.
1
nd2
3438 / 2817 / 1249
Регистрация: 29.01.2016
Сообщений: 9,427
19.06.2017, 23:25
Цитата Сообщение от peva Посмотреть сообщение
C++
1
2
3
4
DynArray operator=(const DynArray &newArr) 
{ 
      len = newArr.len; 
      array = new int[len];
Старую память нужно освобождать.
1
0 / 0 / 1
Регистрация: 12.02.2017
Сообщений: 55
19.06.2017, 23:35  [ТС]
gray_fox,
Я еще очень мало знаю, и даже о таких заголовочных файлах не слышала. Но тем интереснее будет разобраться и понять. Спасибо.
0
What a waste!
 Аватар для gray_fox
1610 / 1302 / 180
Регистрация: 21.04.2012
Сообщений: 2,733
19.06.2017, 23:42
Лучший ответ Сообщение было отмечено peva как решение

Решение

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;
}
1
0 / 0 / 1
Регистрация: 12.02.2017
Сообщений: 55
20.06.2017, 01:02  [ТС]
gray_fox, огромное спасибо!
Да, так понятнее, но с первоначальным вашим вариантом тоже обязательно разберусь.
0
Вездепух
Эксперт CЭксперт С++
 Аватар для TheCalligrapher
12930 / 6798 / 1819
Регистрация: 18.10.2014
Сообщений: 17,198
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.
1
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
BasicMan
Эксперт
29316 / 5623 / 2384
Регистрация: 17.02.2009
Сообщений: 30,364
Блог
20.06.2017, 01:07
Помогаю со студенческими работами здесь

Перегруженные логические операторы И, ИЛИ
Учу по Шилдту плюсы. Дошел до перегрузки операторов. С некоторыми понятно, но вот некоторые вызывают проблемы. Напишите, пожалуйста,...

Класс Эллипс. Перегруженные операторы
#include &lt;iostream&gt; using namespace std; class Ellipse { private: double bigRadius; //большая полуось double...

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

Классы. Перегруженные операторы ввода-вывода
Есть класс book, который описывает 3 типа данных, то есть автор - char, название книги - char, год - int. Нужно использовать...

Класс вектор с арифметическими операциями, перегруженные операторы
Добрый день! Нужна помощь. Реализовываю класс вектор с арифметическими операторами с помощью контейнера &lt;vector&gt;. У меня имеются...


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

Или воспользуйтесь поиском по форуму:
9
Ответ Создать тему
Новые блоги и статьи
Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост.
Programma_Boinc 28.12.2025
Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Налог на собак: https:/ / **********/ gallery/ V06K53e Финансовый отчет в Excel: https:/ / **********/ gallery/ bKBkQFf Пост отсюда. . .
Кто-нибудь знает, где можно бесплатно получить настольный компьютер или ноутбук? США.
Programma_Boinc 26.12.2025
Нашел на реддите интересную статью под названием Anyone know where to get a free Desktop or Laptop? Ниже её машинный перевод. После долгих разбирательств я наконец-то вернула себе. . .
Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка.
Programma_Boinc 23.12.2025
Рецензия / Мнение/ Перевод Нашел на реддите интересную статью под названием The Thinkpad X220 Tablet is the best budget school laptop period . Ниже её машинный перевод. Thinkpad X220 Tablet —. . .
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
Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2025, CyberForum.ru