Thread: Conversion functions

Originally posted by Bones

Code:

      int hrs;                       //1 to 23

Hours go from zero to 23, as you were using them in the rest of your class.

Code:

      void display() const           //format: 11:59 p.m.
         {
         cout << hrs << ':' << mins << ':' << secs;
         }

Instead of this, have you thought about overloading the << operator? Then, instead of

Code:

t.display();
//you could do this:
cout << t;

Okay, and about those increment and decrement operators, your implementation is wrong. Time does not progress by an hour, minute, and second. It progresses by one second at a time. The implementation of your increment operator needs to reflect that basic understanding of time. I'm not sure why you need a decrement operator, unless of course, you've discovered time travel in which case, your talents are far greater in physics than programming

Code:

void main(void)

No. Noooooooooo. Bad coder. Here's what you should do:

Code:

int main()
{
     // statements and such
     return 0;
}

Code:

   cout << "Enter 24-hour time: \n";
   cout << "   Hours (0 to 23): "; cin >> h;
   cout << "   Minutes: ";  cin >> m;
   cout << "   Seconds: ";  cin >> s;

Please put seperate expressions on seperate lines. That code there is difficult to read.

And, instead of having them enter the hours, minutes, and seconds seperately, you could overload the >> operator so that the user could enter time all at once in the form HH:MM:SS.

I hope that helps

Whoops, I didn't exactly answer your question. Silly me.

You can write a constructor that takes a single argument of seconds and then convert that into minutes and seconds and then hours, minutes, and seconds.

Code:

Time(int seconds)
{
     secs = seconds % 60;
     mins = seconds / 60;
     hrs = mins / 60;
     mins = mins % 60;
}

// Prove to myself that this works:
time(7000);
secs = 7000 % 60;
secs = 40;
mins = 7000 / 60;
mins = 101;
hrs = 1;
mins = 101 / 60;
mins = 41;

time(7000) == 1:41:40;

Okay, that looks like a solution to me.

Originally posted by joshdick
// Prove to myself that this works:
time(7000) == 1:41:40;
Okay, that looks like a solution to me.

You might want to find the 15 minutes that are missing.

[edit]
7000 seconds != 1:41:40
7000 seconds == 1:56:40
[/edit]

Originally posted by Dave_Sinkula
You might want to find the 15 minutes that are missing.

He just discovered time travel?

Originally posted by swoopy
He just discovered time travel?

Hmmm, I always wanted to do that.