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

GreatHorizon / oop

Object Oriented Programming Course

1 stars 0 forks source link

Замечания по программе Rational #11

Open alexey-malov opened 4 years ago

alexey-malov commented 4 years ago

https://github.com/GreatHorizon/oop-labs/blob/e78f448c6a1540dd6987da661d7f72752a7a9cd7/lab5/Rational/Rational/CRational.cpp#L105-L112

[ ] Переменной number дайте более удачное имя

alexey-malov commented 4 years ago

https://github.com/GreatHorizon/oop-labs/blob/e78f448c6a1540dd6987da661d7f72752a7a9cd7/lab5/Rational/Rational/CRational.cpp#L1-L2

[ ] stdafx.h следует подключать ДО подключения других заголовочных файлов

alexey-malov commented 4 years ago

[ ] После выполнения арифметических операций следует выполнять нормализацию дроби.
[ ] Покрыть тестами

alexey-malov commented 4 years ago

[ ] Переменные firstNumber, secondNumber, thirdNumber в тестах объявлены в fixture неудачным образом. Потому как читая код теста сложно понять, что должно получиться, не перелистывая всякий раз к месту инициализации этих переменных. Лучше, когда все можно увидеть прямо внутри теста. В fixture лучше помещать те объекты, о состоянии которых можно узнать, не глядя на инициализацию.

GreatHorizon commented 4 years ago

[ ] После выполнения арифметических операций следует выполнять нормализацию дроби.

[ ] Покрыть тестами Нормализация дроби происходит в конструкторе, который вызывается во время выполнения арифметических операций.