search

RattleInGlasses / ps_oop

0 stars 0 forks source link

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

Open alexey-malov opened 9 years ago

alexey-malov commented 9 years ago

Вместо

        BOOST_CHECK_EQUAL(car.EngineIsTurnedOn(), false);

можно писать

        BOOST_CHECK(!car.EngineIsTurnedOn());
alexey-malov commented 9 years ago

Избыточные проверки можно явно не указывать, если подобный сценарий проверялся выше

    BOOST_AUTO_TEST_CASE(TurningOnError)
    {
        CCar car;
        car.TurnOnEngine();
        BOOST_CHECK_EQUAL(car.EngineIsTurnedOn(), true); // Лишняя

        BOOST_CHECK_EQUAL(car.TurnOnEngine(), false);
        BOOST_CHECK_EQUAL(car.EngineIsTurnedOn(), true);
    }
alexey-malov commented 9 years ago

посмотри примеры в oop2015, как можно улучшить тесты

alexey-malov commented 9 years ago

Лучше, чтобы каждый тест проверял лишь один небольшой сценарий, тогда их будет легче читать

alexey-malov commented 9 years ago 
    std::map<int, CCar::Range> const static SPEED_RANGE;

static лучше писть перед типом, CCar внутри класса можно не указывать

alexey-malov commented 9 years ago 
CCar::CCar()
{
    m_engineIsTurnedOn = false;
    m_speed = 0;
    m_gear = 0;
    m_direction = Direction::NOWHERE;
}

Для инициализации полей класса в конструкторе лучше использоват списки инициализации

alexey-malov commented 9 years ago

Деструктор можно не писать, если в нем нет никакого кода и он не виртуальный. Компилятор сгенерирует сам

alexey-malov commented 9 years ago

Ключевые замечания: -Тесты сделать более простыми 140*0,8=112