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

[igoreshka3333]

#include <iostream>
using namespace std;

class Time
{
  private:
    int hour;
    int min;
    int sec;
  public:
    void get_time ();
    void show_time ()
    {cout <<'\n' << hour << ' : ' << min << ' : ' << sec;}
    bool isHour (int t){
      if (t< 0 || t>23)
      return 0;
    }
    bool isMinSec (int t)
      {if (t<0 || t >59) return 0;}
};
//----------------------------------------------------
void Time::get_time ()
{
  Time t;
  while (true)
  {
    cout << "\nVvedite chasy: ";
    cin.unsetf(ios::skipws);
    cin >> t.hour; int ti = t.hour;
    if (isHour (ti) && cin.good())
      {cin.ignore (10, '\n'); break; }
    cin.clear();
    cin.ignore(10, '\n');
    cout << "Nevernyi format!!!";
  }
  while (true)
  {
    cout << "Vvedite minuty: ";
    cin.unsetf(ios::skipws);
    cin >> t.min; int mi = t.min;
    if (isMinSec(mi) && cin.good())
      { cin.ignore(10, '\n'); break; }
    cin.clear();
    cin.ignore(10, '\n');
    cout << "\nNe pravil'nyi format vremeni!";
  }
  while (true)
  {
    cout << "Vvedite minuty: ";
    cin.unsetf(ios::skipws);
    cin >> t.sec; int se = t.sec;
    if (isMinSec(se) && cin.good())
      { cin.ignore(10, '\n'); break; }
    cin.clear();
    cin.ignore(10, '\n');
    cout << "\nNe pravil'nyi format vremeni!";
  }
}
//-----------------------------------------------------
int main ()
{
  Time t;
  t.get_time();
  t.show_time();
  cout << endl;
  system ("pause");
  return 0;
}

Привет, ребят? Прошу помочь. Вроде с ошибками в порядке все, а с выводом времени на екран проблемы

void show_time ()
    {cout <<'\n' << hour << ' : ' << min << ' : ' << sec;}

[Stilet]

void Time::get_time ()
{
Time t;

Time t отсюда выкинь.
и еще:

cin >> t.hour; int ti = t.hour;

t. тоже не пиши. Ибо тут должны заноситься в поля этого класса.

[igoreshka3333]

void Time::get_time ()
{
  while (true)
  {
    cout << "\nVvedite chasy: ";
    cin.unsetf(ios::skipws);
    cin >> hour; int ti = hour;
    if (isHour (ti) && cin.good())
      {cin.ignore (10, '\n'); break; }
    cin.clear();
    cin.ignore(10, '\n');
    cout << "Nevernyi format!!!";
  }

Убрал, но все равно адреса выводит!

И еще: компилируется нормально, но подводит строку на уровне метода ШОУТАЙМ с замечанием:

Multi-character character constant

[Stilet]

Не знаю что ты там с стандартными потоками навертел, но вот так работает:

#include <iostream>

using namespace std;

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

  public:
    void get_time ();

    void show_time ()
    {cout <<"\n" << hour  << " : " << min  << " : " << sec;}
    bool isHour (){ return (hour> 0 && hour<23);   }
    bool isMinSec (){return (min>=0 && min <=59);}
};
//----------------------------------------------------
void Time::get_time ()
{
  while (true)
  {
    cout << "\nVvedite chasy: ";
    cin >> hour;
    if (isHour ())
      { break; }
    cout << "Nevernyi format!!!";
  }
  while (true)
  {
    cout << "Vvedite minuty: ";
    cin >> min;
    if (isMinSec())
      {  break; }
    cout << "\nNe pravil'nyi format vremeni!";
  }
  while (true)
  {
    cout << "Vvedite Sec: ";
    cin >> sec;
    if (isMinSec())
      {  break; }
    cout << "\nNe pravil'nyi format vremeni!";
  }
}
//-----------------------------------------------------

int main()
{
Time *t=new Time();
  t->get_time();
  t->show_time();
  delete t;
  cout << endl;
  cin.get();
  return 0;
}

[igoreshka3333]

Спасибо, Стил!

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

Не знаю что ты там с стандартными потоками навертел

Я ж новичек Мне можно вертеть!

[Stilet]

Ну смотри демонов своими заклятьями не вызови )

[8Observer8]

Для начала очень даже неплохо! Молодец!

Очень хорошая задача! Вы её только начали решать. Это первый этап - добиться, чтобы работало. А теперь нужно рефакторить и рефакторить (то есть, улучшать и улучшать). Не упускайте возможности извлечь из этой пещеры, как можно больше сокровищ!

Для начала переведите на английский. Если не знаете, то имейте ввиду, что https://translate.google.com очень хорошо переводит. Например, вы вводите: "Введите часы" И получаете "Enter hours"

Класс уберите в отдельный файл, чтобы другие программисты могли себе скопировать файлы "Time.h" и "Time.cpp" и подключить с помощью #include "Time.h"

[igoreshka3333]

Спасибо) Английский, как понимаю, очень необходим! Буду немного ИТишный английский осваивать)
Кстати, не могли бы подсказать план действий насчет следующей задачи?

Создайте класс name. Создайте методы для этого класса,осуществляющие 
файловый ввод/вывод данных указанного класса (с использованием ofstream и ifstream).
 Используйте форматирование данных (операторы << и >>). Функции чтения и записи 
должны быть независимыми: в них необходимо внести выражения для открытия 
соответствующего потока, а также чтения и записи данных.

1) То есть, под "форматированием данных" надо подразумевать перегрузку операторов << и >>?
2) Функции чтения и записи сделать методами и назвать write () и read()?

[8Observer8]

Тоже отличное задание и направление мысли! Я завтра в течении дня напишу свои соображения

Несколько замечаний.

1. Почти в любой IDE есть средства для форматирования кода по нажатию горячих клавиш. Эти средства можно гибко настроить под себя. Это очень удобно. Например, когда вы начнёте помогать другим, то увидите, как удобно скопировать к себе чужой неотформатированный код, и в момент отформатировать

2. У вас уже большая программа, а не демо-пример, поэтому вынесите ваш класс в отдельные файлы. Вы потом сами осознаете, как это удобно, когда у вас появится 2-3 класса и более

Time.h

#ifndef TIME_H
#define TIME_H

#include <iostream>
using namespace std;

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

  public:
    void get_time ();

    void show_time ()
    {cout <<"\n" << hour  << " : " << min  << " : " << sec;}
    bool isHour (){ return (hour> 0 && hour<23);   }
    bool isMinSec (){return (min>=0 && min <=59);}
};

#endif // TIME_H

Time.cpp

#include "Time.h"

void Time::get_time ()
{
  while (true)
  {
    cout << "\nVvedite chasy: ";
    cin >> hour;
    if (isHour ())
      { break; }
    cout << "Nevernyi format!!!";
  }
  while (true)
  {
    cout << "Vvedite minuty: ";
    cin >> min;
    if (isMinSec())
      {  break; }
    cout << "\nNe pravil'nyi format vremeni!";
  }
  while (true)
  {
    cout << "Vvedite Sec: ";
    cin >> sec;
    if (isMinSec())
      {  break; }
    cout << "\nNe pravil'nyi format vremeni!";
  }
}

main.cpp

#include "Time.h"
#include <iostream>
using namespace std;

int main()
{
  Time *t=new Time();
  t->get_time();
  t->show_time();
  delete t;
  cout << endl;
  cin.get();
  return 0;
}

P.S. Не всё сразу, потом ещё напишу. Пока эту информацию переварите. Вопросов побольше задавайте

2. У вас уже большая программа, а не демо-пример, поэтому вынесите ваш класс в отдельные файлы. Вы потом сами осознаете, как это удобно, когда у вас появится 2-3 класса и более

Правда проблему лучше всего выкладывать в коротких демо-примерах, когда нет лишнего кода, а только код, который демонстрирует проблему

[Stilet]

То есть, под "форматированием данных" надо подразумевать перегрузку операторов << и >>?

Кажись для cout есть какие-то методы установок флагов форматирования вывода. Не помню точно какие нуна в МСДН копаться.

Функции чтения и записи сделать методами и назвать write () и read()?

Да это как хочешь )