Замечания по Car about oop HOT 9 OPEN

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

from oop.

> Info
Engine: on
Gear: -1
Speed: 10
Direction: backward
> SetGear 0
Gear was switched on 0
> Info
Engine: on
Gear: 0
Speed: 10
Direction: forward

• Автомобиль резко меняет направление движения, если включить нейталь на задней передаче

from oop.

• Нет тестов, проверяющих диапазоны скоростей на каждой передаче, а также допустимые для переключения передач скорости движения

from oop.

• Скорость и направление движения связаны друг с другом. Поддержка их в согласованном состоянии требует бОльших усилий

from oop.

• зачем объявлять таблицу скоростей в .h файле, если она используется исключительно внутри Car.cpp

from oop.

	int GetDirection();

• методы класса не меняющие его наблюдаемое состояние, должны быть константными

from oop.

• зачем нужны публичные методы CheckGear и IsSpeedInRange

from oop.

	bool m_isTurnOn = false;

• is turned on

from oop.

• Тесты должны не только проверять успешность или неудачу при выполнении модифицирующей состояние объекта операции, но еще и сравнивать состояние объекта после операции с ожидаемым. Если в разных тестах используются однотипные проверки состояния (состоящие из нескольких действий), то полезным будет выделить их в функции вида:

void ExpectOperationSuccess(SomeObj & obj, const function<bool(SomeObj & obj)> & op, int expectedProperty1, const string & expectedProperty2, ...)
{
    BOOST_REQUIRE(op(obj)); // ожидаем, что операция вернет true (успех)
    // Сравниваем состояние свойства объект с ожидаемым
    BOOST_CHECK_EQUAL(obj.GetProperty1(), expectedProperty1);
    BOOST_CHECK_EQUAL(obj.GetProperty2(), expectedProperty2);
    ...
}

void ExpectOperationFailure(const SomeObj & obj, const function<bool(SomeObj & obj)> & op);
{
    auto clone(obj); // сделали клон объекта
    BOOST_REQUIRE(!op(clone)); // ожидаем, что операция завершится неуспешно (передаем клон)
    // проверяем, что после выполнения операции состояние клона не отличается от оригинала 
    // (операция в случае неудачи оставляет объект в состоянии, в котором он пребывал до операции)
    BOOST_CHECK_EQUAL(clone.GetProperty1(), obj.GetProperty1());
    BOOST_CHECK_EQUAL(clone.GetProperty2(), obj.GetProperty2());
    ...
}

from oop.