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

[alexey-malov]

TEST_CASE("Set gear on onEngine")
{
    Car car;
    car.TurnOnEngine();
    car.SetGear(1);
    REQUIRE(car.TurnOffEngine());
}

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

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());
    ...
}

[alexey-malov]

class Car
{
public:
    bool IsEngineOn();
    bool TurnOnEngine();
    bool TurnOffEngine();
    bool SetGear(int gear);
    std::string GetEror();
    bool IsACorrectGear();

private:
    bool isEngineTurnOn = false;
    std::string error;
    int m_speed = 0;
    int m_gear = 0;
};

• [x] IsACorrectGear не должно быть в публичной части класса. Да и без параметров от него пользы нет.
• [x] Методы, не изменяющие состояние объекта, должны быть константными.

[alexey-malov]

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

[alexey-malov]

private:
    bool isEngineTurnOn = false;
    std::string error;
    int m_speed = 0;
    int m_gear = 0;
};

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

[alexey-malov]

• [x] Имена тестов следует задавать в виде требований

Ознакомьте со средствами для написания тестов в bdd-стиле в Catch2: https://github.com/catchorg/Catch2/blob/master/docs/tutorial.md#bdd-style

[alexey-malov]

Enter command:
> EngineOff
Engine is turned of
engine allready stoped
> EngineOff
Engine is turned of
engine allready stoped
> EngineOn
Engine is turned on
engine allready stoped
> Info
Engine is On
Gear : 0
Speed : 0
>

• [x] Странный вывод

• [ ] В тестах надо проверять и сообщение о последней ошибке

[alexey-malov]

> Info
Engine is On
Gear : 0
Speed : -20
Direction : backward
> SetSpeed 10
Not a correct gear
Speed not change
>

• [x] При движении задом на нейтрали не получается снизить скорость.

[alexey-malov]

• [x] При движении вперед на нейтрали не получается снизить скорость Воспроизвести проблемные ситуации в тесте и исправить ошибку.

[alexey-malov]

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

[alexey-malov]

• [ ] На нейтрали должно быть запрещено ускоряться. Добавить тест для этой ситуации

[alexey-malov]

• [ ] Покрыть тестами Remote control

[alexey-malov]

1>c:\teaching\ips\2018\oop\dorohov\oop\lab3\car\remotecontrol.cpp(105): error C3861: 'is_numeric': identifier not found

• [ ]Программа не компилируется