C++, объединяющий перегруженные операторы в выражении

Проблема

Я пишу дробный класс с перегруженными операторами. Моя проблема в том, что когда я пытаюсь объединить два выражения перегрузки, например, объединение постинкремента ++ и ostream ‹‹, компилятор выдает ошибку.

Рабочий тест

Следующие тестовые примеры работают отлично — проблема в объединении перегрузок

Fraction testFraction(1,2);
Fraction anotherFraction = testFraction ++;
Fraction yetAnotherFraction = - anotherFraction;
Fraction woahSoManyFractions = anotherFraction + yetAnotherFraction;

Пример 1

Fraction testFraction(1,3)
cout << testFraction++;

Ошибка перегрузки

Пример 2

Fraction testFraction(1,6);
cout << "Should be \"-1/6\": " << -testFraction << endl;

Ошибка перегрузки номер два

Пример 3

 Fraction test14 = 2 + test3++;

ошибка арифметической перегрузки

Заголовочный файл

#pragma once
#include <string>
#include <iostream>
using namespace std;
//Fractions Class
class Fraction{
public:
    Fraction();
    Fraction(int wholeNum);
    Fraction(int numer, int denom);
    int getNumerator();
    int getDenominator();
    Fraction operator - ();    // Member Overloading of unary -
    Fraction  operator ++ ();    // Member Overloading of prefix ++
    Fraction  operator ++ (int);    // Member Overloading of postfix ++

    Fraction operator + (const Fraction& rhs) const; //Member Overloading of binary + (Fraction + Fraction)
    Fraction operator - (const Fraction& rhs) const; //Member Overloading of binary - (Fraction - Fraction)
    Fraction operator * (const Fraction& rhs) const; //Member Overloading of binary * (Fraction * Fraction)
    Fraction operator / (const Fraction& rhs) const; //Member Overloading of binary / (Fraction / Fraction)

    friend ostream& operator << (ostream& os, Fraction& fr);
private:
    int numerator;
    int denominator;


};
//Non-member overloads
Fraction operator + (int num, Fraction &rhs); //non-member Overloading of binary + (Num + Fraction) or (Fraction + Num)
Fraction operator - (int num, Fraction &rhs); //non-member Overloading of binary - (Num - Fraction) or (Fraction - Num)
Fraction operator * (int num, Fraction &rhs); //non-member Overloading of binary * (Num * Fraction) or (Fraction * Num)

//GCD Helper Function
int findGCD(int num, int denom); // update num and denom values passed by reference to normalized array

//Custom exception class to division by zero fractions
class FractionException {
public:
    FractionException(const string&);
    string what() const;
private:
    string message;
};

Файл реализации C++

#include "fraction_14cc78.h"
#include <string>
#include <iostream>
using namespace std;
// ---------------- Exceptions ---------------------
FractionException::FractionException(const string& m){
    message = m;
}

string FractionException::what() const {
    return message;
}
// ------------------------------ Fraction Class ----------------------------
//----- << Overload-----
ostream& operator<<(ostream& os, Fraction& fr)
{
    os << fr.getNumerator() << '/' << fr.getDenominator() << endl;
    return os;
}
//----- Unary (-) Overload----
Fraction Fraction::operator -(){
    return Fraction(-numerator, denominator);
}
//----- Prefix ++ Overload----
Fraction Fraction::operator ++ (){;
//increment object
    return Fraction(numerator + denominator, denominator);
}
//----- Postfix ++ Overload----
Fraction Fraction::operator ++ (int){
    // save the orignal value
    Fraction F(numerator, denominator);
    // increment object
    numerator = numerator + denominator;
    // return old original value
    return F;
}
//----- Binary + Overload (Fraction + Fraction) ----
Fraction Fraction::operator + (const Fraction& rhs) const{
    return Fraction((numerator * rhs.denominator) + (rhs.numerator * denominator), (denominator * rhs.denominator));
}
//----- Binary + Overload (num + Fraction) ----
Fraction operator + (int num, Fraction &rhs){
    return Fraction((rhs.getDenominator() * num) + rhs.getNumerator(), rhs.getDenominator());
}
//----- Binary - Overload (Fraction - Fraction) ----
Fraction Fraction::operator - (const Fraction& rhs) const{
    return Fraction((numerator * rhs.denominator) - (rhs.numerator * denominator), (denominator * rhs.denominator));
}
//----- Binary - Overload (num - Fraction) ----
Fraction operator - (int num, Fraction &rhs){
    return Fraction((rhs.getDenominator() * num) - rhs.getNumerator(), rhs.getDenominator());
}
//----- Binary * Overload (Fraction * Fraction) ----
Fraction Fraction::operator * (const Fraction& rhs) const{
    return Fraction((numerator * rhs.numerator), (denominator * rhs.denominator));
}
//----- Binary * Overload (num * Fraction) ----
Fraction operator * (int num, Fraction &rhs){
    return Fraction((rhs.getNumerator() * num), rhs.getDenominator());
}
//----- Binary / Overload (Fraction / Fraction) ----
Fraction Fraction::operator / (const Fraction& rhs) const{
    return Fraction((numerator * rhs.denominator), (denominator * rhs.numerator));
}



// ---------------- Constructors------------------
//TODO normalize fractions and find greates common denom in this constructor
Fraction::Fraction(int num, int denom){
    //Handle 0 denom case
    if(denom == 0){
        throw FractionException("Cannot create fraction with zero denominator");
    }
    //Find GCD
    int GCD = findGCD(abs(num), abs(denom));
    num = num / GCD;
    denom = denom / GCD;
    //Handle Negative Fractions
    if(num < 0 && denom < 0){
        num = num * -1;
        denom = denom * -1;
    }else if(num < 0 || denom < 0){
        num = abs(num) * -1;
        denom = abs(denom);
    }
    numerator = num;
    denominator = denom;
    //cout << "Fraction Created: " << numerator << "/" << denominator <<endl;
}
Fraction::Fraction(int num){
    numerator = num;
    denominator = 1;
    //cout << "Fraction Created: " << numerator << "/" << denominator <<endl;
}
Fraction::Fraction(){
    numerator = 0;
    denominator = 1;
    //cout << "Fraction Created: " << numerator << "/" << denominator <<endl;
}
// ---------------- Accessors ------------------
int Fraction::getNumerator(){
    return numerator;
}
int Fraction::getDenominator(){
    return denominator;
}
// ---------------- GCD Helper Function ------------------
int findGCD(int num, int denom){
    if((denom <= num) && (num % denom == 0))
        return denom;
    else if(num < denom){
        return findGCD(denom, num);
    } else{
        return findGCD(denom, num % denom);
    }
}

person Cory Crowley    schedule 09.10.2017    source источник
comment
related/dupe: stackoverflow.com/questions/46549734/   -  person NathanOliver    schedule 09.10.2017


Ответы (1)


ostream& operator << (ostream& os, Fraction& fr);
Fraction operator + (int num, Fraction &rhs);
Fraction operator - (int num, Fraction &rhs);
Fraction operator * (int num, Fraction &rhs);

Должны ли все принимать Fraction как константную ссылку! Стандарт не позволяет брать временную ссылку (даже если msvc компилирует такой код...).

редактировать:

Обновленные части заголовочного файла:

#pragma once
#include <string>
#include <iostream>
using namespace std;
//Fractions Class
class Fraction{
    public:
    Fraction();
    Fraction(int wholeNum);
    Fraction(int numer, int denom);
    int getNumerator();
    int getDenominator();
    Fraction operator - ();    // Member Overloading of unary -
    Fraction  operator ++ ();    // Member Overloading of prefix ++
    Fraction  operator ++ (int);    // Member Overloading of postfix ++

    Fraction operator + (const Fraction& rhs) const;
    Fraction operator - (const Fraction& rhs) const;
    Fraction operator * (const Fraction& rhs) const;
    Fraction operator / (const Fraction& rhs) const;

    friend ostream& operator << (ostream& os, const Fraction& fr);
private:
    int numerator;
    int denominator;
};
//Non-member overloads
Fraction operator + (int num, const Fraction &rhs);
Fraction operator - (int num, const Fraction &rhs);
Fraction operator * (int num, const Fraction &rhs);

И адаптированный код реализации:

//----- << Overload-----
ostream& operator<<(ostream& os, const Fraction& fr)
{
    os << fr.getNumerator() << '/' << fr.getDenominator() << endl;
    return os;
}
//----- Binary + Overload (num + Fraction) ----
Fraction operator + (int num, const Fraction &rhs){
    return Fraction((rhs.getDenominator() * num) + rhs.getNumerator(), rhs.getDenominator());
}
//----- Binary - Overload (num - Fraction) ----
Fraction operator - (int num, const Fraction &rhs){
     return Fraction((rhs.getDenominator() * num) - rhs.getNumerator(), 
rhs.getDenominator());
}
//----- Binary * Overload (num * Fraction) ----
Fraction operator * (int num, const Fraction &rhs){
    return Fraction((rhs.getNumerator() * num), rhs.getDenominator());
}
person MichaelE1000    schedule 09.10.2017
comment
Чтобы сделать этот ответ полным, вы должны показать фиксированный код. - person Mark Ransom; 09.10.2017
comment
Спасибо, превращение Fraction в константную ссылку в функциях перегрузки помогло! Можете ли вы уточнить, почему это решило проблему. Почему передача в качестве константной ссылки позволяет комбинировать операции перегрузки? В чем именно разница между передачей в качестве константной ссылки и передачей в качестве обычного адреса объекта? - person Cory Crowley; 09.10.2017
comment
Я думаю, что это довольно хорошо подводит итог: stackoverflow.com/questions/1565600/ - person MichaelE1000; 09.10.2017