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

alexey-malov commented 7 years ago

[x] CCar::SetGear покрыта тестами неполностью

alexey-malov commented 7 years ago

[x] CCar::SetSpeed покрыта тестами не полностью

alexey-malov commented 7 years ago

Алгоритмы + Структуры данных = Программы

[x] Задав структуру данных, определяющую допустимые диапазоны скорости движения на разных передачах, мы сможем сократить объем и дублирование кода

alexey-malov commented 7 years ago

Engine is turned on
Direction is: forward
Gear is: 0
Speed is: 3
> SetGear 1
Engine is turned off or gear does not match current speed

[x] На нейтральной передаче при движении прямо со скоростью от 0 до 30 должна иметься возможность включить первую передачу. Написать тест для проверки этого функционала. UPD: 30 сентября это невыполнено, в тестах тоже нет доработки

alexey-malov commented 7 years ago

> Info
Engine is turned on
Direction is: forward
Gear is: 5
Speed is: 90
> EngineOff
> Info
Engine is turned on
Direction is: forward
Gear is: 5
Speed is: 90

[x] При попытке выключить двигатель во время движения, либо не на нулевой передаче, должно сообщаться о невозможности этой операции

alexey-malov commented 7 years ago

[x] Для CarController разработать тесты

    // Вспомогательная функция для проверки работы команды command
    // Она принимает ожидаемый номер канала expectedChannel и expectedOutput
    void VerifyCommandHandling(const string& command, const string& expectedOutput)
    {
        // Предварительно очищаем содержимое выходного потока
        output = stringstream();
        input = stringstream();
        BOOST_CHECK(input << command);
        BOOST_CHECK(remoteControl.HandleCommand());
        BOOST_CHECK(input.eof());
        BOOST_CHECK_EQUAL(output.str(), expectedOutput);
    }

[ ] Комментарий от не той программы. Функция должна проверять состояние объекта после выполнения над ним команды. С этой целью оригинальная функция принимал ожидаемые значения состояния телевизора

alexey-malov commented 7 years ago

        canSetGear = (gear == -1 && m_speed == 0 && m_direction <= Direction::none) || (gear == 1 && m_speed == 0 && m_direction>= Direction::none) || (gear == 0)||(gear>=2);

[ ] отформатировать код, чтобы было понятнее

alexey-malov commented 7 years ago

bool CCar::SpeedIsInRange(int const &gear, int speed) const

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

alexey-malov commented 7 years ago

bool CCar::SpeedIsInRange(int const &gear, int speed) const
{
    auto it = speedRange.find(gear);
    auto speedRange = it->second;
    return  (speedRange.second >= speed) && (speedRange.first <= speed);
}

[ ] map::find возвращает итератор, который может указывать на end() и разыменовывать его нельзя. Можно использовать map::at(), который бросает исключение, если не найдет

aBuTaMuHo3 / oop

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