SashaOcheev / OOP

0 stars 0 forks source link

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

Open alexey-malov opened 8 years ago

alexey-malov commented 8 years ago

В тестах операций сравнения использовать рациональные числа, заданные в самих тестах, либо дать им имена, из которых было бы понятно, какое значение с каким сравнивается.

Для других тестов то же самое

alexey-malov commented 8 years ago

Реализовать автозапуск тестов после их сборки с выводом структуры тестов в виде лесенки. Имена тестам дать такие, чтобы при чтении названии было понятно, какие требования предъявляются к классу CRational

alexey-malov commented 8 years ago

ToCompoundFraction() работает неправильно для отрицательных чисел

alexey-malov commented 8 years ago

Если хотите проверить, что - + * / не изменяют значений своих аргументов, то:

const CRational a(1, 2);
const CRational b(1, 6);
VerifyRational(a + b, 2, 3);

либо

VerifyRational(MakeRational(1, 2) + MakeRational(1, 6), 2, 3);

либо

VerifyRational(CRational(1, 2) + CRational(1, 6), 2, 3);