Замечания по классу рациональных чисел

gagik12 / RationalNumbers

0 stars 0 forks source link

Замечания по классу рациональных чисел #1

Open alexey-malov opened 7 years ago

alexey-malov commented 7 years ago

    BOOST_AUTO_TEST_CASE(must_have_a_valid_addition_assignment)
    {
        int x = 3, y = 5, z = 6;
        (x += y) += z;

        CRational rx(3);
        CRational ry(5);
        CRational rz(6);
        (rx += ry) += rz;
        BOOST_CHECK(rx == x);
    }

[x] Оператор += и иже с ним реализован некорректно. Вышеупомянутый код не компилируется

alexey-malov commented 7 years ago

        BOOST_AUTO_TEST_CASE(can_be_greater_than)
        {
            BOOST_CHECK(CRational(1, 2) > CRational(1, 3));
            BOOST_CHECK(CRational(7, 2) > 2);
            BOOST_CHECK(!(CRational(-5, 2) > CRational(1, 3)));
        }

[x] Не хватает проверки, что число не больше самого себя

alexey-malov commented 7 years ago

        BOOST_AUTO_TEST_CASE(can_be_less_than)
        {
            BOOST_CHECK(CRational(1, 4) < CRational(1, 2));
            BOOST_CHECK(CRational(-1, 2) < 4);
            BOOST_CHECK(!(CRational(1, 4) < CRational(-1, 6)));
        }

[x] Не хватает проверки, что числе не меньше самого себя

alexey-malov commented 7 years ago

        BOOST_AUTO_TEST_CASE(can_be_greater_or_equal_to)
        {
            BOOST_CHECK(CRational(6, 3) >= 2);
            BOOST_CHECK(CRational(6, 5) >= CRational(6, 7));
            BOOST_CHECK(!(CRational(6, 5) >= CRational(7, 2)));
        }

[x] Не хватает проверки, что число >= самого себя

alexey-malov commented 7 years ago

        BOOST_AUTO_TEST_CASE(can_input_rational_number)
        {
            std::istringstream stream("4/3");
            CRational rational;
            stream >> rational;
            VerifyRational(rational, 4, 3);
        }

[x] Тут надо проверить еще и обработку ошибочных ситуаций

            {
                std::istringstream stream("v4/4");
                CRational rational;
                stream >> rational;
                BOOST_CHECK(stream.fail());
            }

[x] Надо убедиться, что когда бы ни произошла ошибка (числитель, дробь, знаменатель), рациональное число останется неизменным, а обновит свое значение только при отсутствии ошибок чтения

alexey-malov commented 7 years ago

[ ] Объяснить, почему в данном случае предпочтительнее перегрузить арифметические операторы и операторы сравнения вне класса, а не внутри.