Проверка кода на ошибки (описать класс Time на С++)

[igoreshka3333]

Действительно, очень интересно) Сейчас доработаю и выложу

Плюс у меня еще 1 вопрос напрашивается:

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

Если не делать функцию перегрузки дружественной, то увеличивается число методов, которые имеют доступ к защищенным полям!

int sec() { return m_sec; }

[8Observer8]

Сообщение от igoreshka3333

Плюс у меня еще 1 вопрос напрашивается:
Если не делать функцию перегрузки дружественной, то увеличивается число методов, которые имеют доступ к защищенным полям!

int sec() { return m_sec; }

Eсли доступ к внутренним закрытым переменным при печати необходим, то надо делать оператор дружественным, чтобы не плодить лишние методы доступа. Но если есть возможность этого не делать, что нужно следовать правилу. И автор объясняет почему нарушение инкапсуляции это плохо, и почему надо к ней стремиться. Но в классе Time есть в открытом интерфейсе метод для доступа к закрытой переменной "m_sec" и поэтому есть возможность соблюдать правило выше и не нарушать инкапсуляцию (внутренний мир объекта, как я её понимаю)

[igoreshka3333]

Ну вот, что у меня вышло:

Time.h

#ifndef TIME_H
#define TIME_H
#include <iostream>
#include <windows.h>
#include <process.h>
using namespace std;

class Time
{
private:
    int hour;
    int min;
    int sec;
public:
    int r_hour() const { return hour; }
    int r_min() const { return min; }
    int r_sec() const { return sec; }
    void get_time();
    bool isHour () const { return (hour >= 0 && hour < 24); }
    bool isMin () const { return (min >= 0 && min < 60); }
    bool isSec () const { return (sec >= 0 && sec < 60); }
    Time& operator-- ();
    void sw () const
    {
        cout.fill('0');
        cout.width(2);
    };
};
#endif // TIME_H

Timer.cpp

#include "time.h"
Time& Time::operator-- ()
{
    if (sec == 0)
    {
        if (min == 0 &&  hour == 0)
        {
            cout << endl;
            system ("pause");
            exit (1);
        }
        if (min == 0 && hour>0)
        {
            min = 60;
            hour--;
        }
        sec = 60;
        min--;
    }
    sec--;
    return *this;
}

//------------------------------------------------------
void Time::get_time ()
{
    while (true)
    {
        cout << "\nEnter the hours: ";
        cin.unsetf(ios::skipws);
        cin >> hour;
        if (isHour () && cin.good() )
        { break; }
        cin.clear(); cin.ignore(10, '\n');
        cerr << "Wrong format!!!";
    }
    cin.clear(); cin.ignore(10, '\n');
    while (true)
    {
        cout << "Enter the minutes: ";
        cin.unsetf(ios::skipws);
        cin >> min;
        if (isMin() && cin.good())
        {  break; }
        cin.clear(); cin.ignore(10, '\n');
        cout << "Wrong format!!!\n";
    }
    cin.clear(); cin.ignore(10, '\n');
    while (true)
    {
        cout << "Enter the seconds: ";
        cin.unsetf(ios::skipws);
        cin >> sec;
        if (isSec() && cin.good())
        {  break; }
        cin.clear(); cin.ignore(10, '\n');
        cout << "Wrong format!!!\n";
    }
}

main.cpp

#include "time.h"
ostream& operator<< (ostream& stream, Time t)
{
    t.sw();
    cout << t.r_hour();
    cout << " : ";
    t.sw();
    cout << t.r_min();
    cout << " : ";
    t.sw();
    cout << t.r_sec();
    cout << '\r';
    return stream;
}
//--------------------------------------------------------------------
int main ()
{
    Time t;
    cout << "Start the timer: \n";
    t.get_time();
    cout << "\nTimer set: \n\n";
    while (true)
    {
      cout << --t;
      Sleep(1000);
    }
    cout << endl;
    system ("pause");
}

[8Observer8]

--------------------------
Когда делаете отдельный файл, то никогда в нём не пишить "использовать std, как глобальное пространство имён":

using namespace std;

Так как у вас могут возникнуть конфликты имён. Когда вы подключаете файл, в котором есть "using namespace std" то вы с этого момента во всём коде будете иметь это пространство и вероятность конфликта между вашими переменными\функциями и переменными\функциями из std:: повышается

Приучите себя сразу писать std::cout. Так делают все среднички (вроде меня и вас) и профи. В очень простых демо-примера можно написать "using namespace std;". Но я даже в демо-примерах пришу std::

Упражнение. Уберите строку "using namespace std;"



--------------------------
Никогда не встречал таких имён функций:

int r_hour() const { return hour; }

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

int hour() const { return hour; }

Некоторые ещё пишут get:

int getHour() const { return hour; }

Упражнение. Замените на hour()



--------------------------
В конце функции не нужно ставить точку с запятой:

void sw () const
    {
        cout.fill('0');
        cout.width(2);
    };

--------------------------
Защищённые переменные необходимо снабжать отличительным признаком. Есть множество вариантов, вот несколько из них:

private:
    int _hour;
    int _min;
    int _sec;

private:
    int mHour;
    int mMin;
    int mSec;

Я предпочитаю этот:

private:
    int m_hour;
    int m_min;
    int m_sec;

Упражнение. Замените имена закрытых переменных класса (m_)



--------------------------

Упражнение. Добавьте в класс конструктор, чтобы пользователь вашего класса мог использовать его так:

#include <iostream>
#include "Time.h"

int main ()
{
    Time t( 15, 50, 28 );

    std::cout << "\nTimer set: \n\n";
    while (true)
    {
      std::cout << --t;
      Sleep(1000);
    }
    std::cout << std::endl;
    system ("pause");
}

P.S. Выложете исправленный код

[igoreshka3333]

Спасибо за советы)
Time.h

#ifndef TIME_H
#define TIME_H
#include <iostream>
#include <windows.h>
#include <process.h>

class Time
{
private:
    int _hour;
    int _min;
    int _sec;
public:
    Time(int h, int m, int s)
    {_hour = h, _min = m, _sec = s;}
    int hour() const { return _hour; }
    int min() const { return _min; }
    int sec() const { return _sec; }
    void get_time();
    bool isHour () const { return (_hour >= 0 && _hour < 24); }
    bool isMin () const { return (_min >= 0 && _min < 60); }
    bool isSec () const { return (_sec >= 0 && _sec < 60); }
    Time& operator-- ();
    void sw () const
    {
        std::cout.fill('0');
        std::cout.width(2);
    }
};
#endif // TIME_H

Time.cpp

#include "time.h"
Time& Time::operator-- ()
{
    if (_sec == 0)
    {
        if (_min == 0 && _hour == 0)
        {
            std::cout << std::endl;
            system ("pause");
            exit (1);
        }
        if (_min == 0 && _hour>0)
        {
            _min = 60;
            _hour--;
        }
        _sec = 60;
        _min--;
    }
    _sec--;
    return *this;
}

//------------------------------------------------------
void Time::get_time ()
{
    while (true)
    {
        std::cout << "\nEnter the hours: ";
        std::cin.unsetf(std::ios::skipws);
        std::cin >> _hour;
        if (isHour () && std::cin.good() )
        { break; }
        std::cin.clear(); std::cin.ignore(10, '\n');
        std::cerr << "Wrong format!!!";
    }
    std::cin.clear(); std::cin.ignore(10, '\n');
    while (true)
    {
        std::cout << "Enter the minutes: ";
        std::cin.unsetf(std::ios::skipws);
        std::cin >> _min;
        if (isMin() && std::cin.good())
        {  break; }
        std::cin.clear(); std::cin.ignore(10, '\n');
        std::cerr << "Wrong format!!!\n";
    }
    std::cin.clear(); std::cin.ignore(10, '\n');
    while (true)
    {
        std::cout << "Enter the seconds: ";
        std::cin.unsetf(std::ios::skipws);
        std::cin >> _sec;
        if (isSec() && std::cin.good())
        {  break; }
        std::cin.clear(); std::cin.ignore(10, '\n');
        std::cerr << "Wrong format!!!\n";
    }
}

main.cpp

#include "time.h"
std::ostream& operator<< (std::ostream& stream, Time t)
{
    t.sw();
    std::cout << t.hour();
    std::cout << " : ";
    t.sw();
    std::cout << t.min();
    std::cout << " : ";
    t.sw();
    std::cout << t.sec();
    std::cout << '\r';
    return stream;
}
//--------------------------------------------------------------------
int main ()
{
    Time t;
    std::cout << "Start the timer: \n";
    t.get_time();
    std::cout << "\nTimer set: \n\n";
    while (true)
    {
      std::cout << --t;
      Sleep(1000);
    }
    std::cout << std::endl;
    system ("pause");
}

[8Observer8]

Хорошо! Оставлю тут памятку: конструкторы, сеттеры. Позже к ним вернёмся

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

int main ()
{
    Time t;
    std::cout << "Start the timer: \n";
    t.get_time();
    std::cout << "\nTimer set: \n\n";
    while (true)
    {
      std::cout << t--;
      Sleep(1000);
    }
    std::cout << std::endl;
    system ("pause");
}

P.S Выложете код только оператора постфикного декремента

[8Observer8]

[8Observer8]

Рекомендую вместо Builder

Установка Qt на Windows

Переходим по ссылке: http://qt-project.org/downloads

Нажимаем кнопку “Show downloads”

Выбираем “Qt 5.3.2 for Windows 32-bit (MinGW 4.8.2, OpenGL, 737 MB)”

Запускаем скаченный файл и следуем инструкции установки

Запуск консольного приложения

Запускаем Qt Creator

Если у вас Qt Creator на русском, то переключите его на английский по этой инструкции:

Переключение Qt на английский

В меню выбираем: “Инструменты” -> “Параметры…”

В окне “Параметры” слева выбираем “Среда” -> открываем вкладку “Основные” -> выбираем в выпадающем списке “English” -> нажимаем кнопку “OK” -> перезапускаем Qt Creator

Запуск консольного приложения (продолжение)

Создадим новый проект. Для этого в меню Qt Creator’а выбираем “File” -> “New File or Project…”

В разделе “Projects” выбираем “Other Project” -> во второй колонке выбираем “Empty Qt Project” -> нажимаем кнопку “Choose…”

В поле “Name” пишем: HelloWorld -> нажимаем кнопку “Browse…” и указываем путь, где будет создан наш проект -> нажимаем два раза “Next” -> нажимаем кнопку “Finish”

Нажимаем правой кнопкой мыши по названию проекта (на панели слева) -> выбираем “Add New…” -> в первом разделе выбираем “C++” -> во втором разделе выбираем “C++ Source File” -> нажимаем кнопку “Choose…” -> водим имя: main -> нажимаем кнопку “Next” -> нажимаем кнопку “Finish”

Отрываем файл HelloWorld.pro и добавляем в него строку: CONFIG += console

Теперь файл HelloWorld.pro выглядит так:

HelloWorld.pro

CONFIG += console
 
SOURCES += \
    main.cpp

Вставляем в файл main.cpp следующий код:

main.cpp

#include <iostream>
 
int main()
{
    std::cout << "Hello, World!" << std::endl;
    return 0;
}

Запускаем приложение. Для этого нажимаем зелёный треугольник слева внизу (или Ctrl+R)

Примечание. Чтобы программа запустилась в отдельном терминальном окне необходимо на панели слева нажать кнопку “Projects” -> в открывшемся окне, нажать кнопку “Run” -> найти и поставить галочку “Run in terminal” -> нажать кнопку “Edit” (чтобы вернуться к файлам проекта)

Есть ещё замечания к классу Time, но сначала покажите операторную функцию для постфикного декремента

[igoreshka3333]

Есть ещё замечания к классу Time

Возможно вы имеете ввиду дополнительный конструктор, который задает значение по умолчанию?
Насчет постфиксной декрементации усек!
Вот, собственно:

Time.h

#ifndef TIME_H
#define TIME_H
#include <iostream>
#include <windows.h>
#include <process.h>
class Time
{
private:
    int _hour;
    int _min;
    int _sec;
public:
    Time(int h, int m, int s)
    {_hour = h, _min = m, _sec = s;}
    Time ()
    {_hour = 0; _min = 0; _sec = 0;}
    int hour() const { return _hour; }
    int min() const { return _min; }
    int sec() const { return _sec; }
    void get_time();
    bool isHour () const { return (_hour >= 0 && _hour < 24); }
    bool isMin () const { return (_min >= 0 && _min < 60); }
    bool isSec () const { return (_sec >= 0 && _sec < 60); }
    Time& operator--();
    Time operator--( int );
    void sw () const
    {
        std::cout.fill('0');
        std::cout.width(2);
    }
    void tick();
};
#endif // TIME_H

Time.cpp

#include "Time.h"
void Time::tick()
{
    if (_sec == 0)
    {
        if (_min == 0 && _hour == 0)
        {
            std::cout << std::endl;
            system ("pause");
            exit (1);
        }
        if (_min == 0 && _hour>0)
        {
            _min = 60;
            _hour--;
        }
        _sec = 60;
        _min--;
    }
    _sec--;
}
//------------------------------------------------------
Time& Time::operator-- ()
{
    tick();
    return *this;
}
//------------------------------------------------------
Time Time::operator--( int )
{
    Time t = *this;
    tick();
    return t;
}
//------------------------------------------------------
void Time::get_time ()
{
    while (true)
    {
        std::cout << "\nEnter the hours: ";
        std::cin.unsetf(std::ios::skipws);
        std::cin >> _hour;
        if (isHour () && std::cin.good() )
        { break; }
        std::cin.clear(); std::cin.ignore(10, '\n');
        std::cerr << "Wrong format!!!";
    }
    std::cin.clear(); std::cin.ignore(10, '\n');
    while (true)
    {
        std::cout << "Enter the minutes: ";
        std::cin.unsetf(std::ios::skipws);
        std::cin >> _min;
        if (isMin() && std::cin.good())
        {  break; }
        std::cin.clear(); std::cin.ignore(10, '\n');
        std::cerr << "Wrong format!!!\n";
    }
    std::cin.clear(); std::cin.ignore(10, '\n');
    while (true)
    {
        std::cout << "Enter the seconds: ";
        std::cin.unsetf(std::ios::skipws);
        std::cin >> _sec;
        if (isSec() && std::cin.good())
        {  break; }
        std::cin.clear(); std::cin.ignore(10, '\n');
        std::cerr << "Wrong format!!!\n";
    }
}

main.cpp

#include "Time.h"
std::ostream& operator<< (std::ostream& stream, Time t)
{
    t.sw();
    std::cout << t.hour();
    std::cout << " : ";
    t.sw();
    std::cout << t.min();
    std::cout << " : ";
    t.sw();
    std::cout << t.sec();
    std::cout << '\r';
    return stream;
}
//--------------------------------------------------------------------
int main ()
{
    Time t;
    std::cout << "Start the timer: \n";
    t.get_time();
    std::cout << "\nTimer set: \n\n";
    while (true)
    {
      std::cout << t--;
      Sleep(1000);
    }
    std::cout << std::endl;
    system ("pause");
}

Вопрос:

Time Time::operator--( int )
{
    Time t = *this;
    tick();
    return t;
}

Метод принимает в качестве аргумента ИНТ как фейковый что б компилятор не ругался насчет множественного обьявления одной и той же функции?

[8Observer8]

Эти два конструктора можно объединить в один, тогда число строк кода уменьшится:

Time(int h, int m, int s)
    {_hour = h, _min = m, _sec = s;}
    Time ()
    {_hour = 0; _min = 0; _sec = 0;}

Вот так:

    Time(int h = 0, int m = 0, int s = 0)
    {_hour = h, _min = m, _sec = s;}

Вам теперь будет интересно это правило прочитать про список инициализации (только там в коде ошибки есть, лучше, конечно, книжку скачать) Правило 4: Прежде чем использовать объекты, убедитесь, что они инициализированы

Ошибку в коде правила заметил такую: двоеточие ставится перед каждой переменной, а нужно написать его один раз, как в примере:

    Time(int h = 0, int m = 0, int s = 0) :
        _hour( h ),  _min( m ), _sec( s ) {}

Списки инициализации пишут все среднички и профи, так что привыкайте

В вашем коде есть недостаток. Вот допустим пользователь вашего класса создал объект класса Time с помощью конструктора. А в процессе работы захотел изменить время. Короче говоря, чаще всего, там где есть геттеры - есть и сеттеры. Геттеры у вас есть, теперь напишите сеттеры. Если непонятно, то я уточню

После исправлений скопируйте весь код. Ещё подумаем, что можно улучшить