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

[igoreshka3333]

Собственно вот:

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

class Time
{
protected:
    int hour;
    int min;
    int sec;
    
public:
    void get_time ();
    void show_time ();
    Time& operator-- ();
    bool isHour () {return (hour >= 0 && hour < 24);}
    bool isMin () {return (min >= 0 && min < 60);}
    bool isSec () {return (sec >= 0 && sec < 60);}
};
//------------------------------------------------------
Time::Time& operator-- ()
{
   // body
    return *this;
}
//------------------------------------------------------
void Time::get_time ()
{
   //body
}
//----------------------------------------------------
void Time::show_time ()
{
    while (true)
    {
        Time t;
        cout << endl  << '\t' << '\t' << '\t' << '\t';
        if (t.hour<10) cout << '0';
        cout << t.hour  << " : ";
        if (t.min<10) cout << '0';
        cout << t.min  << " : ";
        if (t.sec<10) cout << '0';
        cout << t.sec;
        t--;
        Sleep (1000);
    }
    
}
//========================================================
int main ()
{
    Time t;
    t.get_time();
    t.show_time();
}

Скрин в builder 1.6!

[8Observer8]

Вообще-то для каждой темы свой вопрос. Нужно было придумать хорошее название. Код и скрин, которые вы привели выше. В следующие раз так и делайте

По поводу проблемы, лично я пока не могу ответить. Занят. Попозже посмотрю. Может до этого ещё кто поможет

[igoreshka3333]

Хорошо )

[8Observer8]

Сейчас компилируется. Подумайте почему. Я просто внёс определение оператора внутрь класса.

Отмечу, что такого оператора у вас нет: t--;

У вас есть такой: --t;

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

class Time
{
protected:
    int hour;
    int min;
    int sec;

public:
    void get_time( );
    void show_time( );
    Time& operator--( )
    {
        return *this;
    }

    bool isHour( )
    {
        return (hour >= 0 && hour < 24 );
    }

    bool isMin( )
    {
        return (min >= 0 && min < 60 );
    }

    bool isSec( )
    {
        return (sec >= 0 && sec < 60 );
    }
};
//------------------------------------------------------

//Time::Time& operator--( )
//{
//    // body
//    return *this;
//}
//------------------------------------------------------

void Time::get_time( )
{
    //body
}
//----------------------------------------------------

void Time::show_time( )
{
    while ( true ) {
        Time t;
        cout << endl << '\t' << '\t' << '\t' << '\t';
        if ( t.hour < 10 ) cout << '0';
        cout << t.hour << " : ";
        if ( t.min < 10 ) cout << '0';
        cout << t.min << " : ";
        if ( t.sec < 10 ) cout << '0';
        cout << t.sec;
        --t;
        Sleep( 1000 );
    }

}
//========================================================

int main( )
{
    Time t;
    t.get_time( );
    t.show_time( );
}

Сразу не объяснишь проблему. Ответьте на вопрос. Это префиксный оператор или постфиксный:

    Time& operator--()
    {
        m_sec--;
        return *this;
    }

[igoreshka3333]

Сообщение от 8Observer8

Сразу не объяснишь проблему. Ответьте на вопрос. Это префиксный оператор или постфиксный:

    Time& operator--()
        m_sec--;
    }

Это постфиксный! Я пробовал и тот и другой! Все та же ошибка

[8Observer8]

Это постфиксный!

А вот и 1 вам за это! Учите мат. часть!

По коду же видно, что сначала меняется состояние объекта, а потом возвращается ссылка на него. То есть в коде это будет выгдядеть так:

Time t;
++t;

Пока не напишете реализацию для постфикного, не отстану

Признаю, это моя вина, что я не с того начал:

Замените этот код:

Time::Time& operator-- ()
{
   // body
    return *this;
}

На этот:

Time& Time::operator-- ()
{
   // body
    return *this;
}

Исправляйте ошибку и выкладывайте код. Дальше будем улучшать. Я всё таки дал вам задание, и есть ещё опыт, которым я могу поделиться на этом примере. Добавьте оператор вывода на экран << для вашего Time по такому примеру:

main.cpp

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

int main( )
{
    SecondsCounter sc( 100 );

    for ( int i = 0; i < 10; i++ ) {
        ++sc;
        std::cout << sc << std::endl;
    }

    return 0;
}

SecondsCounter.h

#ifndef SECONDSCOUNTER_H
#define SECONDSCOUNTER_H

#include <ostream>

class SecondsCounter
{
public:

    SecondsCounter( int sec = 0 ) :
        m_sec( sec )
    {
    }

    int sec( ) const
    {
        return m_sec;
    }

    SecondsCounter &operator++( )
    {
        m_sec++;
        return *this;
    }

    SecondsCounter operator++( int )
    {
        SecondsCounter sc = *this;
        m_sec++;
        return sc;
    }

private:
    int m_sec;
};

std::ostream &operator<<( std::ostream &stream,
                          SecondsCounter sc )
{
    std::cout << sc.sec( );
}

#endif // SECONDSCOUNTER_H

Я исправил ошибку: зря сделал оператор << дружественной к классу. Оператор вывода на экран должен быть отдельно, то есть должен быть свободной функцией.

Объявление оператора дружественным я убрал:

private:
    friend std::ostream &operator<<( std::ostream &stream,
                                     SecondsCounter sc );
    int m_sec;
};

Дружественность означает, что из тела функции можно обращаться к private членам класса

[igoreshka3333]

Спасибо) Буду улучшать!

Так а ка же я буду иметь доступ к полям класса если функция не дружественная?

std::ostream &operator<<( std::ostream &stream,
                          SecondsCounter sc )
{
    std::cout << sc.sec( );
}

[8Observer8]

Ответ на вопрос. Не нужна дружественность, так как вам не нужны private члены. Вы все данные можете получить через интерфейс

А тепер внимательно и вдумчиво прочитайте следующий параграф из книги "Изучаем C++", автор А. Крупник:

[igoreshka3333]

Не нужна дружественность, так как вам не нужны private члены. Вы все данные можете получить через интерфейс

Вы имеете ввиду заголовочный файл?

[8Observer8]

Оператор вывода на экран не должен иметь доступ к закрытым членам объекта класса Time. Обязательно прочтите это правило: Правило 23: Предпочитайте функциям-членам функции, не являющиеся ни членами, ни друзьями класса

Можно конечно сделать оператор вывода другом, чтобы получить доступ к закрытой переменной внутри оператора, вот так: time.m_sec. Как в примере:

#include <iostream>
#include <string>

class Time
{
public:
    Time( int sec = 100 ) : m_sec( sec ) {}
    int sec() { return m_sec; }
    friend std::ostream &operator<<( std::ostream &stream, Time time );

private:
    int m_sec;
};

std::ostream &operator<<( std::ostream &stream, Time time )
{
    stream << time.m_sec;
    return stream;
}

int main()
{
    Time t;
    std::cout << t << std::endl;
    return 0;
}

Но зачем? Мы ведь можем получить секунды и через открытый интерфейс класса, вот так: time.sec(). Согласно правилу выше, оператор вывода на экран лучше сделать свободной функцией, а не другом (сделайте пространство имён, как в правиле). Пользователь вашего класса может захочет свой оператор вывода написать. Вот так лучше сделать:

#include <iostream>
#include <string>

class Time
{
public:
    Time( int sec = 100 ) : m_sec( sec ) {}
    int sec() { return m_sec; }

private:
    int m_sec;
};

std::ostream &operator<<( std::ostream &stream, Time time )
{
    stream << time.sec();
    return stream;
}

int main()
{
    Time t;
    std::cout << t << std::endl;
    return 0;
}

P.S. Список правил: ссылка

Покажите ваш класс Time, может там ещё какие-то ошибки есть