ultimatum424 / OOP

0 stars 0 forks source link

Замечания по программе Car #5

Open alexey-malov opened 8 years ago

alexey-malov commented 8 years ago

Класс CCar ничего не должен выводить в stdout и ничего не должен считывать из stdin Это задача кода, выполняющего взаимоедйствие с пользователем

Добавьте методы, позволяющие получить состояние двигателя, текущую передачу и скорость автомобиля. В тестах надо проверять не только, что методы, модифицирующие состояние, вернули true/false, но и сравнивать состояние с ожидаемым. Посмотрете комментарии к работе Чугунова - там полезные советы

alexey-malov commented 8 years ago

m_gearSpeedRangesMap отображает передачу на диапазон скоростей, задаваемый вектором. Вектор - не самое лучший выбор для задания диапазона:

Либо pair, либо структуру с полями minSpeed и maxSpeed

Т.к. для всех автомобилей характеристики одинаковые, лучше сделать статическую мапу

alexey-malov commented 8 years ago

Имена тестов должны описывать объект, а не тесты Тесты должны читаться так, чтобы было понятно, какое поведение ждать от автомобиля. В комментариях то ли у Аникина, то ли у Илдаркина была описана структура тестов

alexey-malov commented 8 years ago

В тестах тоже можно использовать цикл и массивы