Перегрузка оператора *

[Даниил]

Здравствуйте.

Пишу класс, для работы с простыми дробями. Застрял на перегрузке операторов.
Стандартная ситуация: необходимо выполнять математический оператор и возвращать полученное значение.
Класс:

Код

C++ (Qt)
class TFrac
{
public:
    TFrac();
    TFrac(int a, int b);
    TFrac(QString drob);
 
    void operator =      (TFrac &other);
    TFrac& operator *      (TFrac &other);
 
private:
    int chislitel;
    int znamenatel;
};
 

Реализация:

Код

C++ (Qt)
void TFrac::operator =(TFrac &other)
{
    this->chislitel = other.chislitel;
    this->znamenatel = other.znamenatel;
}
TFrac& TFrac::operator *(TFrac const &other)
{
        this->chislitel *= this->chislitel;
        this->znamenatel *= this->znamenatel;
        return *this;
}
 

Main:

Код

C++ (Qt)
...
TFrac first(1,3);
TFrac second(2,3);
TFrac third;
third = first * second;
...
 

Внимание, вопрос: как осуществить умножение, не изменяя первый операнд?

[Nimbus]

>TFrac& operator *      (TFrac &other);
>TFrac& TFrac::operator *(TFrac const &other)
А оно вообще "конпелируется"? 

Надо так:

Код

C++ (Qt)
//Declaration
TFrac& operator *      (const TFrac &other) const;
//Implementation
TFrac& TFrac::operator *(const TFrac  &other) const {
    return TFrac(chislitel * other.chislitel, 
        znamenatel * other.znamenatel);
}
 

Если конечно твой конструктор TFrac(int a, int b) инициализирует числитель и знаменатель

[Даниил]

>TFrac& operator *      (TFrac &other);
>TFrac& TFrac::operator *(TFrac const &other)
А оно вообще "конпелируется"?  
Если конечно твой конструктор TFrac(int a, int b) инициализирует числитель и знаменатель

Оно компилириуется, а вот так как ты указал - нет.

Код:

TFrac.cpp:194: ошибка: invalid initialization of non-const reference of type 'TFrac&' from a temporary of type 'TFrac'

[Nimbus]

Оно компилириуется, а вот так как ты указал - нет.

Код:

TFrac.cpp:194: ошибка: invalid initialization of non-const reference of type 'TFrac&' from a temporary of type 'TFrac'

Пардон. Убери из объявления возвращаемого значения ссылку (&), т. е.:
TFrac operator *      (const TFrac &other) const;
и
TFrac TFrac::operator *(const TFrac  &other) const

[kambala]

потому что в коде JC присутствует & - ссылка на временную переменную, что и сказано в ошибке. убирание амперсанда всё решит.

оператор присваивания тоже весело объявлен у тебя

Код

C++ (Qt)
TFrac &TFrac::operator =(const TFrac &other)
{
    if (this != &other)
    {
        this->chislitel = other.chislitel;
        this->znamenatel = other.znamenatel;
    }
    return *this;
}

[Даниил]

потому что в коде JC присутствует & - ссылка на временную переменную, что и сказано в ошибке. убирание амперсанда всё решит.

оператор присваивания тоже весело объявлен у тебя

Код

C++ (Qt)
TFrac &TFrac::operator =(const TFrac &other)
{
    if (this != &other)
    {
        this->chislitel = other.chislitel;
        this->znamenatel = other.znamenatel;
    }
    return *this;
}

Лять ... из-за тебя еще и оператор != перегружать  
Да, кстати такой код работает ТОЛЬКО в таком виде:

Код

C++ (Qt)
...
TFrac third = first * second;
...
 

А так, как у меня написано:

Код

C++ (Qt)
...
TFrac fhird();
third = first * second;
...
 

Вываливается ошибка:

Код:

main.cpp:13: ошибка: no match for 'operator=' in 'third = first.TFrac::operator*(((const TFrac&)((const TFrac*)(& second))))'

[kambala]

там сравнение указателей

[Nimbus]

Код

C++ (Qt)
...
TFrac third = first * second;
...
 

Логично, потому что это дефолтный конструктор копирования вызывается.

[Даниил]

Логично, потому что это дефолтный конструктор копирования вызывается.

Спасибо, за справку.
А как недефолтный сделать?  

[Nimbus]

А как недефолтный сделать? 

Перегрузи, если ещё не перегружен. И реализуй его так же как operator=. Только там не нужно проверять адрес и возвращать значение.
И вообще, это незачем, если нет нужды в глубоком копировании.
Проблема в чём-то другом. Почему он указал ((const TFrac&)((const TFrac*)(& second))), в частности звёздочку - вот это интересно. И почему там second, а не other?

[Igors]

Возвращение ссылки лучше оставить для оператора *= а умножение (и др. арифметику) сделать так 

Код

C++ (Qt)
class TFrac
{
public:
 ..  
 friend TFrac operator * ( const TFrac & f1, const TFrac & f2 )  { return TFrac(f1.a * f2.a, f1.b * f2.b); } 
 ... 
 int a, b;
};

Также

Код

C++ (Qt)
    int chislitel;
    int znamenatel;
 

Обычно члены таких классов float или double, ну да ладно. А вот длина имен совершенно неподходящая, при интенсивном обращении Вы быстро устанете их писать. В данном конкретном случае короткие имена лучше, из одной буквы - еще лучше. Также нечего стесняться сделать все public - нет правил без исключений

[Даниил]

Собственно проблему не решает, по прежнему ругается на присваивание:

Код:

main.cpp:13: ошибка: no match for 'operator=' in 'third = operator*(((const TFrac&)((const TFrac*)(& first))), ((const TFrac&)((const TFrac*)(& other))))'

[Nimbus]

Собственно проблему не решает, по прежнему ругается на присваивание:

Код:

main.cpp:13: ошибка: no match for 'operator=' in 'third = operator*(((const TFrac&)((const TFrac*)(& first))), ((const TFrac&)((const TFrac*)(& other))))'

Выложи уже свой код полностью

[Даниил]

Выложи уже свой код полностью

Не думаю, что вы там что-то новое увидите.

[Nimbus]

Не думаю, что вы там что-то новое увидите.

То-то и оно, что проблема в operator=, потому что ты оставил свой вариант, а не сделал как посоветовал kambala тут
Он должен возвращать TFrac&, а у тебя он void возвращает