Оператор присваивания

[Superlotles]

Почему тут

Fraction& operator=(const Fraction& src){
        set(src.num,src.den);
        cout << "bla-bla-bla" << endl;
        return *this;
    }

не выполняется эта строка

cout << "bla-bla-bla" << endl;

Вот весь код:

#include <stdafx.h>
#include <conio.h>
using namespace std;
class Fraction {
public:
    int num,den;
private:
    void normalize(){
    if (den == 0 || num == 0) {
        num = 0;
        den = 1;
    }
    if (den < 0) {
        num *= -1;
        den *= -1;
    }
    int n = gcf(num, den);
    num = num / n;
    den = den / n;
}
    int gcf(int a, int b){
         if (a % b == 0)
        return abs(b);
    else
        return gcf(b, a % b);
}
    int lcm(int a, int b){
        return (a/gcf(a,b))*b;
    }
public:
    Fraction(){
        set(0,1);
    }
    Fraction(int n, int d){
        set(n,d);
    }
    Fraction(int n){
        set(n,1);
    }
    Fraction(const Fraction& src){
        num=src.num;
        den=src.den;
    }
    void set(int n, int d){
        num=n;
        den=d;
        normalize();
    }
    int get_num() const{
        return num;
    }
    int get_den() const{
        return den;
    }
    Fraction add(const Fraction& other){
    Fraction fract;
    int lcd = lcm(den, other.den);
    int quot1 = lcd/den;
    int quot2 = lcd/other.den;
    fract.set(num * quot1 + other.num * quot2, lcd);
    fract.normalize();
    return fract;
}
    Fraction mult(const Fraction& other) {
    Fraction fract;
    fract.set(num * other.num, den * other.den);
    fract.normalize();
    return fract;
}
    Fraction operator*(const Fraction& other){
        return mult(other);
    }
    Fraction& operator=(const Fraction& src){
        set(src.num,src.den);
        cout << "bla-bla-bla" << endl;
        return *this;
    }
    bool operator==(const Fraction& other){
        return(num==other.num && den==other.den);
    }
    Fraction operator+(const Fraction& other){
        return add(other);
    }
    friend ostream& operator << (ostream& os, Fraction& fr){
        os << fr.num << fr.den;
        return os;
    }
    bool operator>(const Fraction &other){
        return ((num * other.den) > (den * other.num));
}
    bool operator<(const Fraction &other){
        return ((num * other.den) < (den * other.num));
}  
};
int main() {
    Fraction f1(1,2);
    Fraction f2(1,3);
    Fraction f3(5);
    Fraction f4 = f1+f2+f3;
    cout << "1/2 + 1/3 + 5 = " << f4.num << "/" << f4.den << endl;
    Fraction f5 = f1*f2*f3;
    cout << "1/2 * 1/3 * 5 = " << f5.num << "/" << f5.den << endl;
    if (f3>f2==true && f3>f2==true && f1>f2==true)
        cout << f2.num << "/" << f2.den << " < " << f1.num << "/" << f1.den << " < " << f3.num << "/" << f3.den << endl;
        return 0;
    }

[Stilet]

не выполняется эта строка

Да у тебя в принципе этот перегруженный оператор не выполняется

[Superlotles]

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

Да у тебя в принципе этот перегруженный оператор не выполняется

Если так сделать, то выполняется

Fraction f4; 
f4= f1+f2+f3;

Разница в чем ?

[Stilet]

Разница в чем ?

Я Си не знаю так хорошо но хелп сказал что оператор = можно перегружать только как бинарный оператор для сравнений.

[netrino]

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

Если так сделать, то выполняется

Fraction f4; 
f4= f1+f2+f3;

Разница в чем ?

Очевидно это одна из оптимизаций. Результат выражения пишется сразу в f4, без создания промежуточного объекта, что видно по дизассемблированному коду. Во втором же случае вы сначала создаёте f4, для него вызывается стандартный конструктор, а уже потом происходит присваивание новому значению

[Stilet]

netrino

а почему в хелпе написана та ерунда что я сказал?
В чем подвох?

[netrino]

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

а почему в хелпе написана та ерунда что я сказал?
В чем подвох?

Не знаю, видимо там ошибка )
Для класса должен быть перегружен оператор копирующего присваивания и он должен именно копировать "правый" объект в "левый"(относительно оператора). Наверное они хотели сказать "=="

[Superlotles]

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

Очевидно это одна из оптимизаций. Результат выражения пишется сразу в f4, без создания промежуточного объекта, что видно по дизассемблированному коду. Во втором же случае вы сначала создаёте f4, для него вызывается стандартный конструктор, а уже потом происходит присваивание новому значению

#include <stdafx.h>
#include <conio.h>
using namespace std;
class Fraction {
public:
        int num,den;
        Fraction(int n, int d){
                set(n,d);
        }
        void set(int n, int d){
                num=n;
                den=d;
        }
        Fraction operator=(const Fraction& src){
                cout << "kkkk" << endl;
                set(src.num,src.den);
                return *this;
        }
};
int main() {
    Fraction f1(1,2);
        Fraction f2 = f1; 
        }

Здесь я не написал конструктор по умолчанию, компилятор свой тоже убрал,- ошибки нет, значит конструктор не создается

[Stilet]

Здесь я не написал конструктор по умолчанию

А Fraction(int n, int d) это разве не конструктор?

[mrChester]

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

А Fraction(int n, int d) это разве не конструктор?

Конструктор, но не конструктор по умолчанию, т.к. есть параметры