Замечания по CCar

[alexey-malov]

• [x] Класс "Автомобиль" не должен выполнять ввод-вывод. О наличии ошибок он может сообщать либо кодом возврата, либо выбрасыванием исключения, либо хранить внутри себя описание последней ошибки, предоставляемое через метод, вроде GetLastErrorDescription()

[alexey-malov]

• [x] Ветка не покрыта тестами

[alexey-malov]

void CCar::Info()
{
    string engineStatus = m_isEngineOn ? "on" : "off";
    string direction;
    switch (m_direction) 
    {
    case Direction::stop:
            direction = "stop";
            break;
    case Direction::forward:
            direction = "forward";
            break;
    case Direction::backward:
            direction = "backward";
            break;
    }
    cout << "Car engine is " << engineStatus << endl
        << "Current direction: " << direction << endl
        << "current gear: " << m_gear << endl
        << "current speed: " << m_speed << endl;
}

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

[alexey-malov]

• [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]

Please, enter command for car: SetGear<55>

• [x] При попытке включить недопустимую передачу программа падает. Исправить, в тесты внести необходимые проверки

[alexey-malov]

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

[alexey-malov]

• [x] Ветка не покрыта тестами

[alexey-malov]

class CCar
{
private:
    bool m_isEngineOn = false;
    int m_gear = 0;
    unsigned m_speed = 0;
    Direction m_direction = Direction::stop;
    std::string m_lastErrorDescription;
    void SetDirection();

public:
    const bool IsEngineOn();
    const Direction GetDirection();
    const int GetGear();
    const unsigned GetSpeed();
    const std::string GetLastErrorDescription();
    bool EngineOn();
    bool EngineOff();
    bool SetGear(int gear);
    bool SetSpeed(unsigned speed);
};

• [x] Публичную часть класса лучше разместить до приватной
• [ ] методы класса не меняющие его состояние, должны быть константными
• [ ] возвращать const излишне, т.к. примитивные r-value значения компилятор изменить не позволит. Для строки возврат константы помешает выполнить перемещение.

[alexey-malov]

• [x] Если класс хранит last error description, то при отсутствии ошибок это описание должно опустошаться.

[alexey-malov]

• [x] Не проверяется поведение автомобиля при движении задом:

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

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

• Нельзя ускориться, двигаясь на нейтрали.

• [x] Нет проверок, что заднюю передачу можно включить только при полной остановке (либо с нейтрали, либо с первой).

[alexey-malov]

Please, enter command for car: Info
Car engine is on
Current direction: backward
current gear: -1
current speed: 10
Please, enter command for car: SetGear<0>
Please, enter command for car: Info
Car engine is on
Current direction: forward
current gear: 0
current speed: 10

• [x] Автомобиль не должен менять резко направление движения при переключении на нейтраль на задней передаче. Написать тест для этой ситуации

[alexey-malov]

    bool canSetGear = m_isEngineOn &&
        m_gear != gear &&
        (gear == 0 || (gear == -1) && (m_direction == Direction::stop) ||
        ((m_direction == Direction::stop || m_direction == Direction::forward) &&
            m_speed <= currentGearMaxSpeed && m_speed >= currentGearMinSpeed && gear != -1));

• [ ] рекомендуется явно в скобки заключить условие, объедияемое по &&, чтобы исключить неоднозначность прочтения. Как вариант - ввести логические константы, упрощающие условие. Код кажется сложным и, вероятно, какие-то условия избыточные, либо недостающие

[alexey-malov]

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

Please, enter command for car: SetSpeed<20>
Speed at gear 0 must be between 0 and 150!