0

我试图在问题中为已经给出的类（@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 构造函数？

4

1 回答 1

1

看看你的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（“我已经为该类编写了“移动”构造函数，[...]但是，由于以下错误，我无法使用它”），您可以在此页面中看到

如果以下任何一项为真，则为类 T 隐式声明的复制赋值运算符定义为已删除：

  • T 有一个用户声明的移动构造函数；
  • T 有一个用户声明的移动赋值运算符。

因此，当您定义移动构造函数时，您删除了隐式复制运算符。

您可以解决添加的问题

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

重新激活隐式复制构造函数

于 2018-03-17T21:20:57.943 回答