Замечания по Rational

[alexey-malov]

1>c:\teaching\ips\2019\smirnov\cpp_oop_practice\lab_5\c_rational\crational.h(75): warning C4227: anachronism used: qualifiers on reference are ignored
1>c:\teaching\ips\2019\smirnov\cpp_oop_practice\lab_5\c_rational\crational.h(76): warning C4227: anachronism used: qualifiers on reference are ignored

• [x] Программа должна компилироваться без предупреждений компилятора во всех конфигурациях и платформах.

[alexey-malov]

• [x] Покрыть тестами эту ситуацию

[alexey-malov]

• [x] Не нужен пользовательский конструктор копирования

[alexey-malov]

• [x] Конструктор по умолчанию, с одним и с двумя параметрами может быть реализован при помощи одного конструктора с параметрами по умолчанию

[alexey-malov]

• [x] Лучше сразу после конструирования число нормализовать. Так будет меньше шансов, что при выполнении вычислений произойдёт переполнение

[alexey-malov]

void CRational::Normalize()
{
    const int gcd = GCD(m_numerator, m_denominator);
    m_numerator /= gcd;
    m_denominator /= gcd;
}

• [x] Добавить защиту от деления на 0. Покрыть тестами

[alexey-malov]

CRational const operator+(const int n1, const CRational& n2)
{
    return CRational(n1) + n2;
}

CRational const operator+(const CRational& n1, const int n2)
{
    return CRational(n2) + n1;
}

• [x] Если оператор + объявлен для двух рациональных чисел, то компилятор может сам догадаться, как сложить целое с рациональным и рациональное с целым, вызвав предварительно конструктор рационального из целого

• [x] То же самое касается операторов сравнения

[alexey-malov]

SCENARIO("check boolean !=")
{
    WHEN("check inequality of 1/2 and 1/2")
    {
        CRational n1(1, 2);
        CRational n2(1, 2);

        THEN("it returns false")

• [x] В именах тестов нужно указывать суть, а не детали. В данном случае, речь идёт о сравнении равных чисел

[alexey-malov]

CRational::CRational(const int numerator, const int denominator)
    : m_numerator(numerator)
    , m_denominator(denominator)
{
    if (m_denominator == 0)
    {
        throw std::overflow_error("Divide by zero exception");
    }
    Normalize();
}

• [x] Тут больше подходит invalid argument

[alexey-malov]

• [x] adjuct - нет такого слова

[alexey-malov]

std::pair<int, CRational> CRational::ToCompoundFraction() const
{
    const int intPart = (int)(floor(abs(ToDouble())));
    const int remainder = intPart * GetDenominator();

• [ ] Можно решить задачу целых числах

[alexey-malov]

    WHEN("convert negative retional number with intPart")
    {
        CRational n(-9, 5);
        THEN("get intPart and negative normal rational number")
        {
            REQUIRE(n.ToCompoundFraction() == make_pair<int, CRational>(1, CRational(-4, 5)));
        }
    }

• [x] Неверный знак целой части