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

eligantRU / oop

This is repository for labs of OOP

0 stars 2 forks source link

Замечания по Car #13

Open alexey-malov opened 8 years ago

alexey-malov commented 8 years ago

[ ] Корявый английский в именах тестов
[ ] Имена и структура тестов должны играть роль спецификации на тестируемую функцию или класс, а не просто перечислить ситуации, в которых вы проводите испытание класса или функции.

alexey-malov commented 8 years ago

inmotion
- speed_can_be_changed (within gear speed range)

alexey-malov commented 8 years ago

>>EngineOn
Unknown command

>>Engine On
engine was turn on

>>Set Gear 0
Unknown command

>>SetGear -1
gear now at -1

>>SetSpeed 10
speed now at 10

>>SetGear 0
gear now at 0

>>Info
Engine status: turn on
Speed: 10
Direction: back
Gear: neutral

>>SetSpeed 8
speed now at 8

>>SetGear -1
gear now at -1

[ ] Заднюю передачу должно быть можно включить только, когда машина стоит. В движении (даже задом) включить заднюю передачу включить нельзя

alexey-malov commented 8 years ago

>>Info
Engine status: turn on
Speed: 1
Direction: back
Gear: reverse

>>Engine Off
engine is already disabled

>>Info
Engine status: turn on
Speed: 1
Direction: back
Gear: reverse

[ ] странное сообщение при попытке заглушить двигатель во время движения

alexey-malov commented 8 years ago

const std::pair<int, int> REVERSE_SPEED_RANGE(0, 20);
const std::pair<int, int> NEUTRAL_SPEED_RANGE(0, 150);
const std::pair<int, int> FIRST_SPEED_RANGE(0, 30);
const std::pair<int, int> SECOND_SPEED_RANGE(20, 50);
const std::pair<int, int> THIRD_SPEED_RANGE(30, 60);
const std::pair<int, int> FOURTH_SPEED_RANGE(40, 90);
const std::pair<int, int> FIFTH_SPEED_RANGE(50, 150);

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

alexey-malov commented 8 years ago

[ ] GetDirection можно упростить, учитывая толко знак скорости движения (без учета передачи)