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

[alexey-malov]

• [ ] Не реализован и не протестирован оператор умножения числа на время

[alexey-malov]

• [x] Тесты операторов сравнения на деле проверяют не их работу, а работу метода GetTimestamp

[alexey-malov]

• [x] Каждый оператор сравнения должен сравнивать равные и неравные комбинации временных позиций и проверять в каких условиях должны возвращать true и false
• [ ] Тесты должны проверять граничные условия (значения на самой границе и в ее окрестностях).

[alexey-malov]

• [x] Покрыте тестами операцию деления на 0

[alexey-malov]

unsigned CTime::GetMinutes() const
{
    return m_timeStamp / SECONDS_IN_A_MINUTE - 
        ((m_timeStamp / SECONDS_IN_A_HOUR) * SECONDS_IN_A_MINUTE);
}

• [ ] Успростить с использованием остатка от деления

[alexey-malov]

CTime const CTime::operator++(int)
{
    unsigned tempCopy = m_timeStamp;
    m_timeStamp++;
    if (m_timeStamp >= SECONDS_IN_A_DAY)
    {
        m_timeStamp = 0;
    }
    return CTime(tempCopy);
}

• [ ] Реализовать через префиксную версию. Для декремента сделать так же

[alexey-malov]

CTime CTime::operator-(CTime const & time2) const
{
    unsigned resultOfSubTimes = 0;
    if (m_timeStamp < time2.GetTimeStamp())
    {
        resultOfSubTimes = SECONDS_IN_A_DAY - (time2.GetTimeStamp() - m_timeStamp);
    }
    else
    {
        resultOfSubTimes = m_timeStamp - time2.GetTimeStamp();
    }
    return CTime(resultOfSubTimes);
}

• [ ] Упростить с использованием остатка от деления (по аналогии с оператором сложения)

[alexey-malov]

CTime & CTime::operator+=(CTime const & sumTimeValue) 
{
    unsigned firstTime = m_timeStamp;
    unsigned timeStamp = (firstTime + sumTimeValue.m_timeStamp) % SECONDS_IN_A_DAY;
    m_timeStamp = timeStamp;
    return *this;
}

• [ ] Реализовать += через +, либо наоборот. Избегать дублирования кода. Аналогично поступить с -= и -

[alexey-malov]

unsigned CTime::operator/(CTime const & time) const
{
    if (time.m_timeStamp <= 0)
    {
        throw std::invalid_argument("Invalid argument");
    }
    return (m_timeStamp / time.m_timeStamp);
}

• [ ] timestamp не может быть меньше 0. Поэтому оставить только сравнение с нулём

[alexey-malov]

bool CTime::operator >(CTime const & time) const
{
    return (m_timeStamp >= time.m_timeStamp);
}

bool CTime::operator <(CTime const & time) const
{
    return (m_timeStamp <= time.m_timeStamp);
}

bool CTime::operator <=(CTime const & time) const
{
    return (m_timeStamp < time.m_timeStamp);
}

• [ ] Неполная проверка граничных условий. Тесты проходят и вот такие дефектные реализации

[alexey-malov]

• [ ] Нет оператора умножения числа на время

[alexey-malov]

• [ ] Нет операторов ввода-вывода в поток