2

Сразу прошу не писать что этот код плохой.

Я и сам знаю. Это является лабораторной работой. Поэтому прошу помочь с этим кодом: может Вы исправите или добавите что-то или скажите что добавить. На всякий случай задание лаблраторной работы:

Создать шаблон класса Время и сделать 2 перегрузки операторов:

  1. >> (ввод новых значений полей)
  2. += (добавить заданное количество секунд)

Обе перегрузки работают, но 2 с костылем немного.

И чтобы бы вы добавили для работы с исключительными ситуациями?

И собственно сам код:

#include <iostream>
#include <iomanip>
#include <stdexcept>

using namespace std;

template <class T>
class Time {
private:
    T hour;
    T minute;
    T second;
public:
    Time();
    Time(T h, T m, T s);

    T getHour() const;
    void setHour(T h);

    T getMinute() const;
    void setMinute(T m);

    T getSecond() const;
    void setSecond(T s);

    void print() const;

    Time& nextSecond();

    friend istream& operator>>(istream& input, Time<T>& T);

    friend const Time<T> operator+(const Time<T>& rv) {
        return Time<T>(second + rv.second);
    }

    Time<T>& operator+=(const Time<T>& rv) {
        cout << "operator+=" << endl;
        second += rv.second;
        return *this;
    }
};

istream& operator>>(istream& input, Time<int>& T) {

    try {
        input >> T.hour >> T.minute >> T.second;
        if (T.hour < 0 || T.hour > 23) {
            throw invalid_argument("Invalid hour! 0 <= Hour <= 23");
        }
        if (T.minute < 0 || T.minute > 59) {
            throw invalid_argument("Invalid minute! 0 <= minute <= 59.");
        }
        if (T.second < 0 || T.second > 59) {
            throw invalid_argument("Invalid second! 0 <= second <= 59.");
        }
    }
    catch (invalid_argument& ex) {
        cout << "Exception: " << ex.what() << endl;
    }

    return input;
}

template <class T>
Time<T>::Time() {
    hour = 0;
    minute = 0;
    second = 0;
}

template <class T>
Time<T>::Time(T h, T m, T s) {
    setHour(h);
    setMinute(m);
    setSecond(s);
}

template <class T>
T Time<T>::getHour() const {
    return hour;
}

template <class T>
void Time<T>::setHour(T h) {
    if (h >= 0 && h <= 23) {
        hour = h;
    }
    else {
        throw invalid_argument("Invalid hour! 0 <= Hour <= 23");
    }
}

template <class T>
T Time<T>::getMinute() const {
    return minute;
}

template <class T>
void Time<T>::setMinute(T m) {
    if (m >= 0 && m <= 59) {
        minute = m;
    }
    else {
        throw invalid_argument("Invalid minute! 0 <= minute <= 59.");
    }
}

template <class T>
T Time<T>::getSecond() const {
    return second;
}

template <class T>
void Time<T>::setSecond(T s) {
    if (s >= 0 && s <= 59) {
        second = s;
    }
    else {
        throw invalid_argument("Invalid second! 0 <= second <= 59.");
    }
}

template <class T>
void Time<T>::print() const {
    cout << setfill('0');
    cout << setw(2) << hour
            << ":" << setw(2) << minute
            << ":" << setw(2) << second << endl;
}

template <class T>
Time<T>& Time<T>::nextSecond() {
    if (++second == 60) {
        second = 0;
        if (++minute == 60) {
            minute = 0;
            if (++hour == 24) {
                hour = 0;
            }
        }
    }
    return *this;
}

int main() {
    try {
        Time<int> t1(19, 0, 0);

        int i = 0;

        cout << "Before adding second !" << endl;
        t1.print();

        cout << "Enter your second: ";
        cin >> i;

        Time<int> summator(0, 0, i);

        t1 += summator;
        t1.nextSecond();

        cout << "After adding " << i << " seconds !"<< endl;
        t1.print();
    }
    catch( invalid_argument& ex ) {
        cout << "Exception: " << ex.what() << endl;
    }

    try {
        Time<int> t2;
        cout << "Enter full time please !" << endl;
        cin >> t2;
    }
    catch (invalid_argument& ex) {
        cout << "Exception: " << ex.what() << endl;
    }

    system("pause");
    return 0;
}
4
  • Могу сказать, что во-первых для новичка очень неплохо, каких-то значительных замечаний в стиле кода сделать не могу. А во вторых - здесь не место для подобных вопросов, в том числе и потому что некоторые советы по этому поводу могут быть основаны лишь на субъективном мнении. Здесь следует задавать вопрос если вы можете четко сформулировать 3 вещи - что вы хотели получить? что вы пытались сделать? какие возникли ошибки? 23 ноя 2015 в 17:42
  • 1
    @YuriyOrlov, +20, что таким вопросам здесь место: meta.ru.stackoverflow.com/a/1762/178988
    – Qwertiy
    23 ноя 2015 в 17:44
  • @Egor Voziyanov попробуйте конкретизировать вопрос. что именно вам кажется костылем в +=? почему вам кажется, что в работе с исключениями чего-то не хватает?
    – user177221
    23 ноя 2015 в 17:47
  • Хорошо. По поводу костыля в том что если добавлю 59 секунд там будет 60 секунд а должно перевестись в 1 минуту, то есть там 19 00 01 и добавить 59 секунд должно быть 19 01 00 а получается 19 00 60 поэтому использую метод nextsecond для сброса в минуту. 24 ноя 2015 в 15:19

2 ответа 2

6
using namespace std;

using namespace std; считается плохой практикой и рекомендуется использовать только в примерах кода чтобы было меньше букв.

template <class T>
class Time {
private:
    T hour;
    T minute;
    T second;

private блок традиционно уносят в конец, потому что для читающего публичные функции обычно более интересны. Но это не очень-то важно т.к. в большинстве IDE эти блоки можно сворачивать.

public:
    Time();

Вынос тела функций из класса - это имеет смысл для раздельной компиляции, но довольно бессмысленно для шаблонов. Аргумент "так лучше видно API" довольно сомнителен, т.к. тела функций можно свернуть в IDE.

    void print() const;

Очень странная функция. Куда print? Дайте ей лучше аргумент (std::ostream& out=std::cout)

    Time& nextSecond();

Почему только секунды? И зачем оно что-то возвращает (нарушение CQSP)?

    friend istream& operator>>(istream& input, Time<T>& T);

Ок, но так ли ей нужно быть другом?

    friend const Time<T> operator+(const Time<T>& rv) {
        return Time<T>(second + rv.second);
    }

Это унарный плюс чтоли? А что за конструктор? И зачем у возвращаемого типа const?

    Time<T>& operator+=(const Time<T>& rv) {

А что если мы хотим к Time<int> добавить Time<long> ?

        cout << "operator+=" << endl;

Для этого есть отладчик.

        second += rv.second;

Это правильно что время надо хранить в наименьшей единице измерения, но тогда не надо членов minute, hour - их надо вычислять.

        if (T.hour < 0 || T.hour > 23) {
            throw invalid_argument("Invalid hour! 0 <= Hour <= 23");
        }

должна быть отдельная функция bool Time::isValid() const

    catch (invalid_argument& ex) {

Исключения не для того чтобы их тут же ловить и игнорировать

template <class T>
Time<T>::Time() {
    hour = 0;
    minute = 0;
    second = 0;
}

Для инициализации есть список инициализации. Либо сразу можно писать T second = 0;

template <class T>
Time<T>::Time(T h, T m, T s) {
    setHour(h);
    setMinute(m);
    setSecond(s);
}

Еще раз, в С++ для инициализации полей есть список инициализации.

template <class T>
void Time<T>::setHour(T h) {
    if (h >= 0 && h <= 23) {
        hour = h;
    }
    else {
        throw invalid_argument("Invalid hour! 0 <= Hour <= 23");
    }
}

std::invalid_argument обычно используют как std::invalid_argument("hour"). Контекст виден из стектрейса, условия на аргументы описаны в документации или в коде класса, нет смысла их дублировать.

    system("pause");

Зачем это? IDE сама делает паузу, если руками запускать из консоли, пауза тоже не нужна.

    return 0;

return 0; в main() не нужен

11
  • 2
    Про return 0 принципиально не согласен.
    – Qwertiy
    23 ноя 2015 в 20:22
  • 1
    Потому что его отсутствие нелогично, хотя и разрешено стандартом.
    – Qwertiy
    23 ноя 2015 в 21:21
  • 1
    По поводу using namespace std; совет весьма странный, как и его обоснование некоторыми пуристами(?). Во всех языках используется «втаскивание» имен без проблем, и только некоторые C++ программисты откуда-то взяли, что это плохо...
    – ixSci
    24 ноя 2015 в 18:11
  • 3
    @ixSci помимо кол-ва файлов растет и кол-во имен, чем больше имен, тем больше шанс их пересечения. И когда в одном файле нужно использовать сущности из разных namespace использовать using-директиву не лучший выход. Т.к. это смешает все имена в одну кучу, что равносильно вообще отсутствию пространств имен. Спрашивается, зачем тогда было их вообще вводить? В таком случае разумнее или обойтись using-декларациями, как я уже говорил, или ввести алиасы для исключения упоминания длинных имен.
    – αλεχολυτ
    24 ноя 2015 в 19:09
  • 1
    @alexolut, ещё немного поразмыслив, я пришёл к выводу, что Ваше с Abyx мнению верно, тогда как мои придирки ошибочны и вот почему: C++ никак не регламентирует внутреннюю зависимость между заголовками, поэтому подключая string и делая using namespace std программист не знает, какие ещё имена, помимо string, он «втащил» в файл, там может быть всё, что угодно, начиная с functional и заканчивая vector. Это принципиально отличает C++ от C#(в худшую сторону), в котором using Something означает импортировать лишь те имена, что были запрошены. Спасибо за дискуссию, я был не прав.
    – ixSci
    25 ноя 2015 в 7:30
3
friend const Time<T> operator+(const Time<T>& rv) {
    return Time<T>(second + rv.second);
}

Не вижу такого конструктора. В любом случае, есть подозрение, что тут теряется информация о минутах и часах.

Time<T>& operator+=(const Time<T>& rv) {
    cout << "operator+=" << endl;
    second += rv.second;
    return *this;
}

Проверка на 60 и перенос где?

istream& operator>>(istream& input, Time<int>& T) {

Кидается исключение и в том же методе ловится. Да ещё и выводится на консоль.

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

PS: Временами создаётся ощущение, что это гибрид двух классов - один хранит время в трёх полях, а другой исключительно в секундах и должен делать перевод при необходимости.

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.