Рациональный класс и семантика Move не работают

Question

Рациональный класс и семантика Move не работают

Я попытался добавить больше функций к уже заданному классу (ответ @Aak) в задаче: Как вывести дробь вместо десятичного числа?
для печати фракций в numerator / denominator ,

Прежде всего, данный код не работал без каких-либо изменений в коде. Затем я сделал это после внесения некоторых изменений. Тем не менее, моя реализация дает мне неправильный вывод.

например:

input: A = 3;
B = 3;
Output: 9/1
9
instead of: 1

вот полная реализация:

#include <iostream>
/********************** Rational class **********************************/
class Rational
{
private:
int m_numerator, m_denominator;
private:
inline void simplificate()
{
int commondivisor = 1;
for(int i=2;i<= std::min(abs(m_numerator), abs(m_denominator));i++)
if( m_numerator%i == 0 && m_denominator%i == 0 )
commondivisor = i;
m_numerator /= commondivisor;
m_denominator /= commondivisor;
}

public:
Rational()                                  // Defualt
:m_numerator(1), m_denominator(1)
{}

Rational(const int& num, const int& den=1)  // Parameterized
:m_numerator(num), m_denominator(den)
{}

Rational(const Rational& other)             // Copy
:m_numerator(other.m_numerator), m_denominator(other.m_denominator)
{}

/*Rational(Rational&& other)                  // Move
:m_numerator(other.m_numerator), m_denominator(other.m_denominator)
{}*/

~Rational(){}

Rational& operator/ (const int& divisor)
{
m_denominator *= divisor;
simplificate();
return *this;
}
Rational& operator/ (const Rational &divisor)
{
m_numerator *= divisor.m_numerator;
m_denominator *= divisor.m_denominator;
simplificate();
return *this;
}
const double getrealformat()const
{
return  static_cast<double>(m_numerator)/
static_cast<double>(m_denominator);
}

friend double operator/ (Rational& obj, const int& divisor);
friend void printRational(Rational& obj, const int& A, const int& B);
friend void printRational(Rational& obj, const int&& A, const int&& B);
};
/************************** Friend functions ********************************/
double operator/ (Rational& obj, const int& divisor)
{
obj.m_denominator *= divisor;
obj.simplificate();
return obj.getrealformat();
}
void printRational(Rational& obj, const int& A, const int& B)   // lvalue
{
Rational r1(A), r2(B);
obj = r1/r2;
std::cout<<obj.m_numerator<<'/'<<obj.m_denominator<<std::endl;
std::cout<<obj.getrealformat()<<std::endl;
}
void printRational(Rational& obj, const int&& A, const int&& B)  // rvalue
{
Rational r1(A), r2(B);
obj = r1/r2;
std::cout<<obj.m_numerator<<'/'<<obj.m_denominator<<std::endl;
std::cout<<obj.getrealformat()<<std::endl;
}
/*****************************************************************************/
int main()
{
Rational obj;
printRational(obj, 3,3);
return 0;
}

Вопрос — 1: логика выглядит хорошо, но я не знаю, почему я получаю неправильный ответ. Кто-нибудь может найти проблему?

Вопрос 2: я написал конструктор Move для класса, который вы можете найти в разделе с комментариями. Однако я не смог использовать его из-за следующей ошибки:

D:\Programming\C++\CPP Programs\Class - Fractions\Class - Fractions.cpp|70|error: use of deleted function 'Rational& Rational::operator=(const Rational&)'|

D:\Programming\C++\CPP Programs\Class - Fractions\Class - Fractions.cpp|77|error: use of deleted function 'Rational& Rational::operator=(const Rational&)'|

(насколько мне известно, перемещаемый объект / экземпляр уничтожается, насколько мне известно.)

Кто-нибудь может мне помочь реализовать конструктор Move для этого класса?

0

c++c++14 move-semantics operator-overloading rational-numbers

Решение

Другие решения

Других решений пока нет …

Источник

Accepted Answer

Посмотри operator/()

Rational& operator/ (const Rational &divisor)
{
m_numerator *= divisor.m_numerator;
m_denominator *= divisor.m_denominator;
simplificate();
return *this;
}

Этот код правильный для operator*(), не для operator/(),

Может быть

        m_numerator *= divisor.m_denominator;
m_denominator *= divisor.m_numerator;

Но хуже того, что ты operator/() изменить объект.

Ваш код (исправлено переключение числителя и знаменателя) должно быть правильным для operator/=(), не для operator/() это должно вернуть новый объект.

Я предлагаю что-то следующее

Rational& operator/= (const Rational &divisor)
{
m_numerator *= divisor.m_denominator;
m_denominator *= divisor.m_numerator;
simplificate();
return *this;
}

friend Rational operator/ (Rational A, Rational const & B);

и вне класса,

Rational operator/ (Rational A, Rational const & B)
{ return A/=B; }

Что касается вопроса 2 («Я написал конструктор« Move »для класса, […] Однако я не смог использовать его из-за следующей ошибки»), вы можете увидеть в эта страница тот

Неявно объявленный оператор присваивания копии для класса T определяется как удаленный, если выполняется любое из следующих условий:

T имеет объявленный пользователем конструктор перемещения;

T имеет объявленный пользователем оператор присваивания перемещения.

Итак, когда вы определяете конструктор перемещения, вы удаляете неявный оператор копирования.

Вы можете решить проблему, добавив

 Rational & operator= (Rational const &) = default;

реактивация неявного конструктора копирования

1