Перегруженный оператор +

@DeveloperS73 · Регистрация: 18.11.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
#include <iostream>
#include <string> 
using namespace std;
 
class mwork
{
public: 
    int m[]={5,6,7};
    int n[]={2,3,8};
    
public: 
    int operator + (const int & other) {
    int temp[];    
    for(int i=0;i<3;i++)
    temp[i]=m[i]+n[i];
    }
};
 
int main()
{
 mwork ooo;
 ooo.m = {5,6,7};
 ooo.n = {2,3,8};
 ooo.p = ooo.m+ooo.n;
}

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

с++ = плюсы = кресты = могила = кладбище = смерть

@zayats80888 · 03.01.2020, 00:45

DeveloperS73,

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
#include <iostream>
 
using namespace std;
 
struct mwork
{
    int arr[3];
 
    mwork operator + (const mwork& other) const
    {
        mwork temp;
        for (int i = 0; i < 3; i++)
            temp.arr[i] = arr[i] + other.arr[i];
        return temp;
    }
};
 
int main()
{
    mwork m = { 5,6,7 };
    mwork n = { 2,3,8 };
    mwork p = m + n;
}

@TheCalligrapher · 03.01.2020, 00:48

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

int m[]={5,6,7};

Такое в С++ не разрешается.

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

Я взял оператор + и пытаюсь сделать плюсик, который складывает массивы

В С++ не существует пользовательского "перегруза операторов" для встроенных/фундаментальных типов. Перегрузка операторов возможна только для пользовательских типов: для классов или для enum-типов. Поэтому никакой перегрузки "для складывания массивов" быть не может.

Вы в своем коде написали [бессмысленную] перегрузку для операндов типа mwork и int. Зачем - не ясно.

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

Пожалуйста помогите.

Вам сначала нужно вменяемо объяснить, что вы пытаетесь сделать.

@AndryS1 · 03.01.2020, 00:54

Мне кажется если вам нужно сложение массивов, то можете использовать std::vector
Поскольку он не является фундаментальным типом, для него можно определить оператор суммы

@AndryS1 · 03.01.2020, 00:56

Мне кажется если вам нужно сложение массивов, то можете использовать std::vector
Поскольку он не является фундаментальным типом, для него можно определить оператор суммы

Добавлено через 1 минуту
Прошу прощения, баг форума

@fao · 03.01.2020, 01:18

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

Мне кажется если вам нужно сложение массивов, то можете использовать std::vector
Поскольку он не является фундаментальным типом, для него можно определить оператор суммы

С языка снял ))

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
#include <iostream>
#include <vector>
 
using namespace std;
 
typedef vector<int> vec;
vec operator+(const vec& a, const vec& b)
{
    vec result;
 
    const size_t size = a.size();
    if (size == b.size())
    {
        for (size_t i = 0; i < size; i++)
        {
            result.push_back(a[i] + b[i]);
        }
    }
    return result;
}
 
    
int main()
{
    vec A = { 1, 2, 3 };
    vec B = { 4, 5, 6 };
 
    vec C = A + B + B;
 
    for (const int& c : C)
    {
        cout << c << " ";
    }
}

Добавлено через 11 минут
Есть еще valarray. Он подобные операции от рождения умеет делать.

@AndryS1 · 03.01.2020, 02:20

У меня для этих операторов вообще отдельный хедер есть

Кликните здесь для просмотра всего текста

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
#pragma once
 
#include <vector>
#include <algorithm>
 
template<typename T>
std::vector<T> operator + (const std::vector<T> left, const std::vector<T> right)
{
    std::vector<T> max_size(left.size() > right.size() ? left : right);
    std::vector<T> min_size(left.size() < right.size() ? left : right);
    for (size_t i = 0; i < min_size.size(); i++)
    {
        max_size[i] += min_size[i];
    }
    return max_size;
}
template<typename T>
std::vector<T> operator - (const std::vector<T> left, const std::vector<T> right)
{
    std::vector<T> max_size(left.size() > right.size() ? left : right);
    std::vector<T> min_size(left.size() < right.size() ? left : right);
    for (size_t i = 0; i < min_size.size(); i++)
    {
        max_size[i] -= min_size[i];
    }
    return max_size;
}
 
template<typename T>
std::vector<T> operator /(const std::vector<T> left, const std::vector<T> right)
{
    std::vector<T> max_size(left.size() > right.size() ? left : right);
    std::vector<T> min_size(left.size() < right.size() ? left : right);
    for (size_t i = 0; i < min_size.size(); i++)
    {
        max_size[i] /= min_size[i];
    }
    return max_size;
}
 
template<typename T>
std::vector<T> operator * (const std::vector<T> left, const std::vector<T> right)
{
    std::vector<T> max_size(left.size() > right.size() ? left : right);
    std::vector<T> min_size(left.size() < right.size() ? left : right);
    for (size_t i = 0; i < min_size.size(); i++)
    {
        max_size[i] *= min_size[i];
    }
    return max_size;
}
 
template<typename T>
void operator -=(std::vector<T>& left, const std::vector<T> right)
{
    left = left - right;
}
 
template<typename T>
void operator /=(std::vector<T>& left, const std::vector<T> right)
{
    left = left / right;
}
 
template<typename T>
void operator *=(std::vector<T>& left, const std::vector<T> right)
{
    left = left * right;
}
 
template<typename T>
void operator +=(std::vector<T>& left, const std::vector<T> right)
{
    left = left + right;
}

@zayats80888 · 03.01.2020, 02:37

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

У меня для этих операторов вообще отдельный хедер есть

Хмм... многвато копирований, можно было и ссылками обойтись

@TheCalligrapher · 03.01.2020, 02:39

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

У меня для этих операторов вообще отдельный хедер есть

В которых вы упорно передаете потенциально тяжелые объекты по значению (!), но при этом никак не пользуетесь преимуществами такого способа передачи.

Причем из кода как-будто следует, что такая передача по значению выполнена намеренно... Зачем?

@AndryS1 · 03.01.2020, 02:41

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

Причем из кода как-будто следует, что такая передача по значению выполнена намеренно... Зачем?

Спустя определённое время, я понял что компилятор в релизе это оптимизирует, и да, нужно заменить на взятие по адресу

@TheCalligrapher · 03.01.2020, 02:46

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

Спустя определённое время, я понял что компилятор в релизе это оптимизирует

Это как это? Индивидуальные элементы, хранимые в векторе, могут обладать какой-то нетривиальной семантикой копирования. Компилятор не может/не имеет права устранить копирование в таком случае.

@zayats80888 · 03.01.2020, 02:47

AndryS1,

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
template<typename T>
std::vector<T> operator + (const std::vector<T>& left, const std::vector<T>& right)
{   
    using vec_t = std::vector<T>;
    const vec_t& max_size(left.size() > right.size() ? left : right);
    const vec_t& min_size(left.size() < right.size() ? left : right);
    vec_t result(max_size); // только одно копирование
    for (size_t i = 0; i < min_size.size(); i++)
    {
        result[i] += min_size[i];
    }
    return result;
}
 
template<typename T>
std::vector<T> operator + (std::vector<T> left, std::vector<T> right)
{   // от 0 до 2 копирований при передаче параметров
    std::vector<T> max_size(std::move(left.size() > right.size() ? left : right));
    std::vector<T> min_size(std::move(left.size() < right.size() ? left : right));
 
    for (size_t i = 0; i < min_size.size(); i++)
    {
        max_size[i] += min_size[i];
    }
    return max_size;
}

@TheCalligrapher · 03.01.2020, 03:23

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

Это как это? Индивидуальные элементы, хранимые в векторе, могут обладать какой-то нетривиальной семантикой копирования. Компилятор не может/не имеет права устранить копирование в таком случае.

... да и навскидку не вижу я что-то никакого оптимизирования копирования в этом случае ни в GCC, ни в MSVC...

@fao · 03.01.2020, 03:25

Здесь есть множество полезных рекомендаций, в. т.ч. и на этот счет: http://isocpp.github.io/CppCor... ines#Rf-in

@zayats80888 · 03.01.2020, 03:28

AndryS1, кстати, в случае равноразмерных векторов будет складываться(делится, умножатся...) right и right (нужно в тернарном операторе > и <=)

@TheCalligrapher · 03.01.2020, 03:32

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

Здесь есть множество полезных рекомендаций, в. т.ч. и на этот счет: http://isocpp.github.io/CppCor... ines#Rf-in

Не сказал бы, что подобная категоричность

C++
1
void f2(string s);         // bad: potentially expensive

всегда оправдана. Да, наиболее эффективным вариантом поддержки семантики перемещения является предоставление двух версий функции const & и &&, однако за это приходится платить размножением кода. А когда параметров становится много - то еще и комбинаторным взрывом количества вариантов.

Идиома "если вам все равно нужна копия - передавайте по значению" - ценная и полезная идиома: https://josephmansfield.uk/art... value.html (плюс ссылка на лекцию Герба Саттера в тексте)

Поэтому ни о каком безусловном "bad" в таком случае речи быть не может. Проблема с кодом AndryS1 не в том, что он передает по значению, а в том, что он передает по значению там, где это ему совсем не нужно. То есть он не пользуется возможностями, предоставляемыми таким способом передачи.

@fao · 03.01.2020, 03:37

Ну так это же не приказы, а рекомендации. Я думаю Страуструп здесь просто выражал свое мнение )

... как и Герб Саттер

Новые блоги и статьи Все статьи Все блоги /
Знаешь почему 90% людей редко бывают счастливыми? kumehtar 14.04.2026 Потому что они ждут. Ждут выходных, ждут отпуска, ждут удачного момента. . . а удачный момент так и не приходит.	Фиксация колонок в отчете СКД Maks 14.04.2026 Фиксация колонок в СКД отчета типа Таблица. Задача: зафиксировать три левых колонки в отчете. Процедура ПриКомпоновкеРезультата(ДокументРезультат, ДанныеРасшифровки, СтандартнаяОбработка) / / . . .	Настройки VS Code Loafer 13.04.2026 { "cmake. configureOnOpen": false, "diffEditor. ignoreTrimWhitespace": true, "editor. guides. bracketPairs": "active", "extensions. ignoreRecommendations": true, . . .	Оптимизация кода на разграничение прав доступа к элементам формы Maks 13.04.2026 Алгоритм из решения ниже реализован на нетиповом документе, разработанного в конфигурации КА2. Задачи, как таковой, поставлено не было, проделанное ниже исключительно моя инициатива. Было так:. . .
Контроль заполнения и очистка дат в зависимости от значения перечислений Maks 12.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "ПланированиеПерсонала", разработанного в конфигурации КА2. Задача: реализовать контроль корректности заполнения дат назначения. . .	Архитектура слоя интернета для сервера-слоя. Hrethgir 11.04.2026 В продолжение https:/ / www. cyberforum. ru/ blogs/ 223907/ 10860. html Знаешь что я подумал? Раз мы все источники пишем в голове ветки, то ничего не мешает добавить в голову такой источник, который сам. . .	Подстановка значения реквизита справочника в табличную часть документа Maks 10.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "ПланированиеПерсонала", разработанного в конфигурации КА2. Задача: при выборе сотрудника (справочник Сотрудники) в ТЧ документа. . .	Очистка реквизитов документа при копировании Maks 09.04.2026 Алгоритм из решения ниже применим как для типовых, так и для нетиповых документов на самых различных конфигурациях. Задача: при копировании документа очищать определенные реквизиты и табличную. . .

@AndryS1 653 / 466 / 183 Регистрация: 23.04.2019 Сообщений: 1,987
	03.01.2020, 00:54
	Мне кажется если вам нужно сложение массивов, то можете использовать std::vector Поскольку он не является фундаментальным типом, для него можно определить оператор суммы 1

@AndryS1 653 / 466 / 183 Регистрация: 23.04.2019 Сообщений: 1,987
	03.01.2020, 00:56
	Мне кажется если вам нужно сложение массивов, то можете использовать std::vector Поскольку он не является фундаментальным типом, для него можно определить оператор суммы Добавлено через 1 минуту Прошу прощения, баг форума 0

@fao 199 / 155 / 45 Регистрация: 11.11.2019 Сообщений: 348
	03.01.2020, 03:25
	Здесь есть множество полезных рекомендаций, в. т.ч. и на этот счет: http://isocpp.github.io/CppCor... ines#Rf-in 0

@zayats80888 6352 / 3523 / 1428 Регистрация: 07.02.2019 Сообщений: 8,995
	03.01.2020, 03:28
	AndryS1, кстати, в случае равноразмерных векторов будет складываться(делится, умножатся...) right и right (нужно в тернарном операторе > и <=) 1

@fao 199 / 155 / 45 Регистрация: 11.11.2019 Сообщений: 348
	03.01.2020, 03:37
	Ну так это же не приказы, а рекомендации. Я думаю Страуструп здесь просто выражал свое мнение ) ... как и Герб Саттер 0