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

Ambiguous overload for 'operator+' (operand types are 'int' and 'Rational')

31.08.2019, 22:04. Показов 11526. Ответов 3

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

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
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
#include <iostream>
struct Rational
{
 
    Rational(int numerator = 0, int denominator = 1)
    {
        this->mNumerator = numerator;
        this->mDenominator = denominator;
    }
 
    int gcd(int a, int b)
    {
        int c;
        if (a > b)
        {
            std::swap(a, b);
        }
        while ((b % a) != 0)
        {
            c = b % a;
            b = a;
            a = c;
        }
        return a;
    }
 
    int getNumerator() const
    {
        return mNumerator;
    }
 
    unsigned getDenominator() const
    {
        return mDenominator;
    }
 
    void println()
    {
        std::cout << mNumerator << "/" << mDenominator << std::endl;
    }
 
    void add(Rational rational)
    {
        int c = (mDenominator * rational.mDenominator) / gcd(mDenominator, rational.mDenominator);
        mNumerator = mNumerator * (c / mDenominator);
        rational.mNumerator = rational.mNumerator * (c / rational.mDenominator);
        mNumerator = mNumerator + rational.mNumerator;
        int f = gcd(mNumerator, c);
        mNumerator = mNumerator / f;
        mDenominator = c / f;
    }
 
    void sub(Rational rational)
    {
        int c = (mDenominator * rational.mDenominator) / gcd(mDenominator, rational.mDenominator);
        mNumerator = mNumerator * (c / mDenominator);
        rational.mNumerator = rational.mNumerator * (c / rational.mDenominator);
        mNumerator = mNumerator - rational.mNumerator;
        int f = gcd(mNumerator, c);
        mNumerator = mNumerator / f;
        mDenominator = c / f;
    }
 
    void mul(Rational rational)
    {
        mNumerator = mNumerator * rational.mNumerator;
        mDenominator = mDenominator * rational.mDenominator;
        int c = gcd(mNumerator, mDenominator);
        mNumerator = mNumerator / c;
        mDenominator = mDenominator / c;
    }
 
    void div(Rational rational)
    {
        int c = rational.mDenominator;
        rational.mDenominator = rational.mNumerator;
        rational.mNumerator = c;
        mNumerator = mNumerator * rational.mNumerator;
        mDenominator = mDenominator * rational.mDenominator;
        c = gcd(mNumerator, mDenominator);
        mNumerator = mNumerator / c;
        mDenominator = mDenominator / c;
    }
 
    void neg()
    {
        mNumerator = (-1) * mNumerator;
    }
 
 
    double toDouble() const
    {
        return double(mNumerator) / mDenominator;
    }
 
    operator double() const
    {
        return toDouble();
    }
 
    Rational &operator~()
    {
        this->neg();
        return *this;
    }
 
    Rational &operator=(Rational rational)
    {
        this->mNumerator = rational.mNumerator;
        this->mDenominator = rational.mDenominator;
        return *this;
    }
 
    friend std::istream &operator>>(std::istream &in, Rational &rational)
    {
        in >> rational.mNumerator >> rational.mDenominator;
        return in;
    }
 
    friend std::ostream &operator<<(std::ostream &out, Rational &rational)
    {
        out << rational.mNumerator << " \\ " << rational.mDenominator;
        return out;
    }
 
    Rational &operator+=(const Rational &rational)
    {
        this->add(rational);
        return *this;
    }
 
 
    Rational &operator+=(const int d)
    {
        this->add(Rational(d));
        return *this;
    }
 
    Rational &operator-=(const Rational &rational)
    {
        this->sub(rational);
        return *this;
    }
 
    Rational &operator-=(const int d)
    {
        this->sub(Rational(d));
        return *this;
    }
 
    Rational &operator*=(const Rational &rational)
    {
        this->mul(rational);
        return *this;
    }
 
    Rational &operator*=(const int d)
    {
        this->mul(Rational(d));
        return *this;
    }
 
    Rational &operator/=(const Rational &rational)
    {
        this->div(rational);
        return *this;
    }
 
    Rational &operator/=(const int d)
    {
        this->div(Rational(d));
        return *this;
    }
 
    Rational operator-() const
    {
        Rational rational(this->mNumerator, this->mDenominator);
        rational.neg();
        return rational;
    }
 
    Rational operator+() const
    {
        Rational rational(this->mNumerator, this->mDenominator);
        return rational;
    }
 
private:
    int mNumerator;
    int mDenominator;
 
};
 
Rational operator+(Rational r1, Rational const &r2)
{
    return r1 += r2;
}
 
Rational operator-(Rational r1, Rational const &r2)
{
    return r1 -= r2;
}
 
Rational operator*(Rational r1, Rational const &r2)
{
    return r1 *= r2;
}
 
Rational operator/(Rational r1, Rational const &r2)
{
    return r1 /= r2;
}
 
bool operator==(Rational const &r1, Rational const &r2)
{
    if (r1.getNumerator() * r2.getDenominator() == r2.getNumerator() * r1.getDenominator())
    {
        return true;
    } else
    {
        return false;
    }
}
 
bool operator!=(Rational const &r1, Rational const &r2)
{
    return !(r1 == r2);
}
 
bool operator<(Rational const &r1, Rational const &r2)
{
    int check = r1.getNumerator() * r2.getDenominator() - r2.getNumerator() * r1.getDenominator();
    if (check < 0)
    {
        return true;
    } else
    {
        return false;
    }
}
 
bool operator>(Rational const &r1, Rational const &r2)
{
    return !(r1 < r2) && !(r1 == r2);
}
 
bool operator<=(Rational const &r1, Rational const &r2)
{
    return (r1 < r2) || (r1 == r2);
}
 
bool operator>=(Rational const &r1, Rational const &r2)
{
    return (r1 > r2) || (r1 == r2);
}

теперь main:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
#include "Rational.cpp"
 
 
int main() {
    Rational r (5, 7);
    std::cout << r;
    Rational r2 (6, 8);
    r += r2;
    r.println();
    Rational r1;
    r1 = 2;
    r1 = 2+2;
    r1 += r + r;
    r1 += 2 + r;  //здесь ругается
    r1 += r + 2;   // и здесь
    r1.println();
    r1->println();
    if (r > r2) {
        std::cout << (double)r1;
    }
    return 0;
Подскажите пожалуйста в чем проблема...
Миниатюры
Ambiguous overload for 'operator+' (operand types are 'int' and 'Rational')  
0
IT_Exp
Эксперт
34794 / 4073 / 2104
Регистрация: 17.06.2006
Сообщений: 32,602
Блог
31.08.2019, 22:04
Ответы с готовыми решениями:

Error: ambiguous overload for 'operator<<' (operand types are 'std::ofstream {aka std::basic_ofstream<char>}'
на #define LOG(Variable) Log&lt;&lt;&quot;&lt;&quot;;...

Error: no match for 'operator-' (operand types are 'Space' and 'int')
Не знаю с чем связано, но не с SFML, решил попробовать создать симуляцию космоса и так-как если космических объектов будет много то и...

Ошибка в коде "no match for 'operator[]' (operand types are 'Student' and 'int')"
Помогите пожалуйста разобраться, в чём проблема? Весь код: #include &quot;windows.h&quot; #include &lt;stdio.h&gt; #include &lt;string.h&gt;...

3
Неэпический
 Аватар для Croessmah
18149 / 10731 / 2067
Регистрация: 27.09.2012
Сообщений: 27,035
Записей в блоге: 1
31.08.2019, 22:42
Лучший ответ Сообщение было отмечено Rabbit8 как решение

Решение

Цитата Сообщение от Rabbit8 Посмотреть сообщение
Подскажите пожалуйста в чем проблема...
Произвожу тебя в ранг компилятора. Вот тебе твой код и вопрос.
C++
1
2
3
4
5
6
//...
    Rational(int numerator = 0, int denominator = 1) {/*...*/}
//...
    operator double() const {/*...*/} 
//...
Rational operator+(Rational r1, Rational const &r2) {/*...*/}
А теперь скажи мне, дорогой компилятор, глядя на код
C++
1
2
//...
/*r1 += */ 2 + r;
ты сделаешь так:
operator+(Rational(2, 1), r);
или так
2 + (double)r;
???
Почему?
1
2 / 2 / 1
Регистрация: 22.05.2019
Сообщений: 33
01.09.2019, 11:46  [ТС]
Понял ошибку, спасибо! А можно это как-то исправить, не перегружая знаки для каждого случая отдельно и при этом оставить оператор приведения типа?
0
Модератор
Эксперт С++
 Аватар для zss
13773 / 10966 / 6491
Регистрация: 18.12.2011
Сообщений: 29,244
01.09.2019, 13:29
Лучший ответ Сообщение было отмечено Rabbit8 как решение

Решение

Кроме имеющегося оператора сложения добавьте еще
C++
1
2
3
4
5
6
7
8
Rational operator+(int r1,  const Rational &r2) 
{ 
   return Rational(r1)+r2;
}
Rational operator+(const Rational& r1, int r2) 
{ 
   return Rational(r2)+r1;
}
1
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
BasicMan
Эксперт
29316 / 5623 / 2384
Регистрация: 17.02.2009
Сообщений: 30,364
Блог
01.09.2019, 13:29
Помогаю со студенческими работами здесь

Неявные приведения типов, error: ambiguous overload for ‘operator=’
Всем привет! Как заставить код работать без костыля? https://rextester.com/SHUYH87895 #include &lt;iostream&gt; ...

Почему возникает ошибка? error: ambiguous overload for 'operator>>'
class planet { protected: double distance; int revolve; public: planet(double d, int r) {distance=d; revolve=r;} };...

Странная перегрузка операторов (Error: ambiguous overload for 'operator[]')
Что-то я туплю. Есть код: #include&lt;string&gt; struct test { int operator(const std::string&amp;)const{return 0;} operator...

No match for 'operator'(operand types are rapidjson:: )
эта ошибка возникает в цикле for. указывает на итератор iter=document.Begin(). Суть такая : мне нужно парсить json файл. дела. по...

Ошибка: bad operand types for binary operator '>'
Найти минимальное из двух чисел пишут вот так: import java.io.BufferedReader; import java.io.InputStreamReader; public...


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

Или воспользуйтесь поиском по форуму:
4
Ответ Создать тему
Новые блоги и статьи
Ритм жизни
kumehtar 27.02.2026
Иногда приходится жить в ритме, где дел становится всё больше, а вовлечения в происходящее — всё меньше. Плотный график не даёт вниманию закрепиться ни на одном событии. Утро начинается с быстрых,. . .
SDL3 для Web (WebAssembly): Сборка SDL3 из исходников с помощью CMake и Emscripten
8Observer8 27.02.2026
Недавно вышла версия 3. 4. 2 библиотеки SDL3. На странице официальной релиза доступны исходники, готовые DLL (для x86, x64, arm64), а также библиотеки для разработки под Android, MinGW и Visual Studio. . . .
SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами
8Observer8 20.02.2026
Содержание блога Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .
Конвертировать закладки radiotray-ng в m3u-плейлист
damix 19.02.2026
Это можно сделать скриптом для PowerShell. Использование . \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json> Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом. # Check if. . .
Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI
Eddy_Em 18.02.2026
Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .
Камера Toupcam IUA500KMA
Eddy_Em 12.02.2026
Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .
И ясному Солнцу
zbw 12.02.2026
И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.
«Знание-Сила»
zbw 12.02.2026
«Знание-Сила» «Время-Деньги» «Деньги -Пуля»
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2026, CyberForum.ru