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

Неправильная работа сложения в пользовательском классе

05.07.2017, 22:42. Показов 1145. Ответов 3
Метки нет (Все метки)

Студворк — интернет-сервис помощи студентам
Вечер добрый. Написал свой класс Money. Решил протестировать, и наткнулся на ошибку.
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
#include <iostream>
#include "Money.h"
using namespace std;
 
int main() {
    . . .
    Money a(1, 4);
    Money b(2, 7);
    Money c;
    c = a + b;
    cout << c << endl;
    . . .
    return 0;
}
Вывод следующий "-858993460 р. -858993460 коп." Не совсем понятно из-за чего оно так работает. Вот файл h и cpp.
Money.h
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
//Money.h
#pragma once
#include <iostream>
using namespace std;
class Money
{
private:
    int rubl, kopeyk;
    void reduce();
public:
    //конструкторы и деструктор
    Money();
    Money(const int&);
    Money(const int&, const int&);
    Money(const Money&);
    //перегрузки
    Money& operator=(const Money&);
    //Money& operator+(const Money&);
    //Money& operator+(const Money&);
    friend const Money& operator+(const Money &left, const Money &right);
    Money& operator/(const int&);
    Money& operator*(const int&);
    friend const bool operator<(const Money&, const Money&);
    friend const bool operator>(const Money&, const Money&);
    friend const bool operator==(const Money&, const Money&);
    //Вывод 
    friend ostream& operator<<(ostream&, const Money&);
    ////Ввод 
    //friend istream& operator>>(istream&, Money&);
    ~Money();
};
Money.cpp
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
//Money.cpp
#include "Money.h"
 
 
 
void Money::reduce()
{
    if (kopeyk >= 100) {
        rubl += kopeyk / 100;
        kopeyk %= 100;
    }
}
 
Money::Money()
{
    rubl = 0;
    kopeyk = 0;
}
 
Money::Money(const int &in)
{
    if (in < 0)
        rubl = 0;
    else
        rubl = in;
    kopeyk = 0;
}
 
Money::Money(const int &r, const int &k)
{
    if (r < 0)
        rubl = 0;
    else
        rubl = r;
    if (k < 0)
        kopeyk = 0;
    else
        kopeyk = k;
    reduce();
}
 
Money::Money(const Money &in)
{
    rubl = in.rubl;
    kopeyk = in.kopeyk;
}
 
 
 
Money& Money::operator=(const Money &in)
{
    rubl = in.rubl;
    kopeyk = in.kopeyk;
    return *this;
}
 
 
const Money& operator+(const Money &left, const Money &right) {
    Money res;
    res.rubl = left.rubl + right.rubl;
    res.kopeyk = left.kopeyk + right.kopeyk;
    res.reduce();
    return res;
}
 
Money & Money::operator/(const int& del)
{
    int tmp = rubl * 100 + kopeyk;
    rubl = tmp / 100;
    kopeyk = tmp % 100;
    kopeyk *= 100;
    kopeyk /= 100;
    reduce();
    return (*this);
}
 
Money & Money::operator*(const int &in)
{
    if (in >= 0) {
        kopeyk *= in;
        rubl *= in;
        reduce();
    }
    return (*this);
}
 
Money::~Money()
{
}
const bool operator>(const Money& left, const Money& right) {
    if (left.rubl > right.rubl)
        return true;
    else if (left.rubl < right.rubl)
        return false;
    else if (left.kopeyk > right.kopeyk)
        return true;
    else 
        return false;
}
const bool operator<(const Money& left, const Money& right) {
    if (left.rubl < right.rubl)
        return true;
    else if (left.rubl > right.rubl)
        return false;
    else if (left.kopeyk < right.kopeyk)
        return true;
    else
        return false;
}
const bool operator==(const Money& left, const Money& right) {
    if (left.rubl == right.rubl && left.kopeyk==right.kopeyk)
        return true;
    else
        return false;
}
//Вывод 
ostream &operator<<(ostream &out, const Money &c) {
    out << c.rubl << " р. " << c.kopeyk << " коп.";
    return out;
}
0
Лучшие ответы (1)
cpp_developer
Эксперт
20123 / 5690 / 1417
Регистрация: 09.04.2010
Сообщений: 22,546
Блог
05.07.2017, 22:42
Ответы с готовыми решениями:

Неправильная работа счетчика в классе
Здравствуйте, появилась проблема, мое задание-это поставить на счетчик кол-во раз, когда переменная равна 2-ке, вся суть в том что нужно...

Некорректная работа операции сложения векторов в классе
Объясните мне пожалуйста, почему операция сложения векторов работает неккоректно! Дело в том, что после выполнения операции, каким-то...

Буфер в пользовательском классе из стринг
Выполняю задание из учебника Страуструпа. Столкнулся с 3-мя проблемами. Комментарии на английском из другой программы. 1) Непонятно как...

3
Заблокирован
05.07.2017, 22:46
Лучший ответ Сообщение было отмечено ApocFel как решение

Решение

Цитата Сообщение от ApocFel Посмотреть сообщение
Не совсем понятно из-за чего оно так работает.
Всё сразу понятно. Возвращаешь из operator+ ссылку на локальную переменную.
1
 Аватар для ApocFel
0 / 0 / 0
Регистрация: 25.12.2016
Сообщений: 39
05.07.2017, 22:51  [ТС]
Хм, в таком случае я могу возвращать переменную, но не будет ли это костылем? Не надо ли мне как-то по-другому перегрузить?
0
Заблокирован
05.07.2017, 23:06
ApocFel, возвращать надо по значению, больше никак.

И ещё тебе не нужны сделанные вручную (неправильно) конструктор копирования и оператор присваивания. Объяви их дефолтными. Как и их перемещающих партнёров.

Цитата Сообщение от ApocFel Посмотреть сообщение
Не надо ли мне как-то по-другому перегрузить?
Можно немного по-другому, см. http://en.cppreference.com/w/c... _operators
Но не обязательно.

Добавлено через 11 минут
Цитата Сообщение от ApocFel Посмотреть сообщение
Не совсем понятно из-за чего оно так работает.
Вообще, предупреждения надо читать https://wandbox.org/permlink/YWJVVgymDeytXIig
Code
1
2
warning: reference to stack memory associated with local variable 'res' returned [-Wreturn-stack-address]
    return res;
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
raxper
Эксперт
30234 / 6612 / 1498
Регистрация: 28.12.2010
Сообщений: 21,154
Блог
05.07.2017, 23:06
Помогаю со студенческими работами здесь

Ошибки в пользовательском классе String
Помогите пожалуйста разобраться с ошибками. Вроде все правильно написал, но все равно куча ошибок. #include...

Использование компонентов в пользовательском классе
Это код формы #pragma once #include &quot;List.h&quot; namespace Laba1 { using namespace System; using namespace...

Error C2040 в пользовательском классе
Функция main: int main( void ) { const char str = &quot;message&quot;; String( str ); std::cin.get(); std::cin.get(); return 0; } ...

Обращение к компонентам формы в пользовательском классе
Друзья, изучаю C# по путно теория с практикой и сейчас уткнулся в маленькую проблему и не могу ее разрешить, что-то не допонял похоже: ...

Почему не срабатывает конструктор копирования в пользовательском классе
вроде со всем разобралась, но не заходит в конструктор копирования. В чём ошибка? #include &lt;iostream&gt; #include...


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

Или воспользуйтесь поиском по форуму:
4
Ответ Создать тему
Новые блоги и статьи
Модель микоризы: классовый агентный подход 3
anaschu 06.01.2026
aa0a7f55b50dd51c5ec569d2d10c54f6/ O1rJuneU_ls https:/ / vkvideo. ru/ video-115721503_456239114
Owen Logic: О недопустимости использования связки «аналоговый ПИД» + RegKZR
ФедосеевПавел 06.01.2026
Owen Logic: О недопустимости использования связки «аналоговый ПИД» + RegKZR ВВЕДЕНИЕ Введу сокращения: аналоговый ПИД — ПИД регулятор с управляющим выходом в виде числа в диапазоне от 0% до. . .
Модель микоризы: классовый агентный подход 2
anaschu 06.01.2026
репозиторий https:/ / github. com/ shumilovas/ fungi ветка по-частям. коммит Create переделка под биомассу. txt вход sc, но sm считается внутри мицелия. кстати, обьем тоже должен там считаться. . . .
Расчёт токов в цепи постоянного тока
igorrr37 05.01.2026
/ * Дана цепь постоянного тока с сопротивлениями и напряжениями. Надо найти токи в ветвях. Программа составляет систему уравнений по 1 и 2 законам Кирхгофа и решает её. Последовательность действий:. . .
Новый CodeBlocs. Версия 25.03
palva 04.01.2026
Оказывается, недавно вышла новая версия CodeBlocks за номером 25. 03. Когда-то давно я возился с только что вышедшей тогда версией 20. 03. С тех пор я давно снёс всё с компьютера и забыл. Теперь. . .
Модель микоризы: классовый агентный подход
anaschu 02.01.2026
Раньше это было два гриба и бактерия. Теперь три гриба, растение. И на уровне агентов добавится между грибами или бактериями взаимодействий. До того я пробовал подход через многомерные массивы,. . .
Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост.
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? Ниже её машинный перевод. После долгих разбирательств я наконец-то вернула себе. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2026, CyberForum.ru