Ad:

C++ Discussion Thread
Marked Solved
Views: 852

Nov 5th, 2009

Input time as hh:mm and no other way

Expand Post »

So my program is coming along, but now I am writing in the range checking to ensure that the inputs are actually what they should be. My code works fine for time, except for the fact that white space gets ignored. I need to ensure that a user cannot input "10: 09 AM". It must be in the format hh:mm

My code:

C++ Syntax (Toggle Plain Text)
void Time::makeAppt()
{
  Time startTime, endTime;
 
  cout << "Start time >>";
  cin >> startTime;
  cout << "End time >> ";
  cin >> endTime;
}
 
istream &operator>>( istream &input, Time &time)
{
  input >> time.hour;
  input.ignore();
  input >> time.minute;
  input.ignore();
  input >> time.period;
 
  return input;
}
void Time::makeAppt()
{
  Time startTime, endTime;

  cout << "Start time >>";
  cin >> startTime;
  cout << "End time >> ";
  cin >> endTime;
}

istream &operator>>( istream &input, Time &time)
{
  input >> time.hour;
  input.ignore();
  input >> time.minute;
  input.ignore();
  input >> time.period;

  return input;
}

Thank you for any suggestions you might have. =]

Similar Threads

Time Difference in VB 6.0 in Visual Basic 4 / 5 / 6
Time validation code in C
Keep time and wait for input in Assembly
How can I use a delimter to take sevral variables from the same input in C++
check is time and insert null time in PHP

dotnabox

Reputation Points: 10

Solved Threads: 0

Newbie Poster

Offline

18 posts
since Oct 2009

Permalink

Nov 5th, 2009

Re: Input time as hh:mm and no other way

How about using cin.get() to get one character at a time, verifying that the order is digit, digit [convert to int and check range], colon, digit, digit [convert to int and check range], end-of-line. Maybe ignore leading or trailing white space to be nice. Not sure how you're going to indicate an error -- throw an exception maybe, or just exit() after an error message?

boblied

Reputation Points: 36

Solved Threads: 9

Newbie Poster

Offline

24 posts
since Mar 2009

Permalink

Nov 5th, 2009

Re: Input time as hh:mm and no other way

How would I use the cin.get() within my overloaded operator? As for the errors, well, I have to build certain messages into it and loop it back around to handle asking for the input again. I'm not worried about that part.

dotnabox

Reputation Points: 10

Solved Threads: 0

Newbie Poster

Offline

18 posts
since Oct 2009

Permalink

Nov 5th, 2009

Re: Input time as hh:mm and no other way

Click to Expand / Collapse Quote originally posted by dotnabox ...

How would I use the cin.get() within my overloaded operator? As for the errors, well, I have to build certain messages into it and loop it back around to handle asking for the input again. I'm not worried about that part.

If you read it in as a c-style string, you can use getline and specify 6 characters (5 + 1) maximum. Or use get () five times.

http://www.cplusplus.com/reference/i...tream/getline/

isdigit could come in handy from cctype.

http://www.cplusplus.com/reference/c...ctype/isdigit/

To use get or getline using a c-string, declare a C-string of size 6. If you want to use get, loop through, specifying the character with the loop control variables.

C++ Syntax (Toggle Plain Text)
char c[6];
for (int i = 0; i < 5; i++)
{
   cin.get (c[i]);
}
c[5] = 0;
char c[6];
for (int i = 0; i < 5; i++)
{
   cin.get (c[i]);
}
c[5] = 0;

Don't have a compiler handy , but I think this should work. One potential problem is if the user doesn't type in at least 5 characters. That's why getline may be superior here.

http://www.cplusplus.com/reference/i...m/istream/get/

[edit]
Replace "cin" with "input" above. I didn't pay attention to the fact that you had an istream, not necessarily cin.
[/edit]

Last edited by VernonDozier; Nov 5th, 2009 at 6:25 pm. Reason: Assumed cin as the stream earlier.

VernonDozier

Featured Poster

Reputation Points: 2614

Solved Threads: 687

Posting Expert

Offline

5,375 posts
since Jan 2008

Permalink

Nov 5th, 2009

Re: Input time as hh:mm and no other way

getline() seems to be doing the trick, but for some reason, it will only process once, not twice.

C++ Syntax (Toggle Plain Text)
istream &operator>>( istream &input, Time &time)
{
  char charHour[60], charMinute[60];
  input.getline(charHour, 3, ':');
  input.getline(charMinute, 3, ' ');
  input.getline(time.period, 3, ' ');
  time.hour = atoi(charHour);
  time.minute = atoi(charMinute);
}
istream &operator>>( istream &input, Time &time)
{
  char charHour[60], charMinute[60];
  input.getline(charHour, 3, ':');
  input.getline(charMinute, 3, ' ');
  input.getline(time.period, 3, ' ');
  time.hour = atoi(charHour);
  time.minute = atoi(charMinute);
}

From my original code, it should run through it once, store it in startTime, then run through it again (when I call the operator once more) and store it in endTime. Instead, it just runs through it for startTime, then leaves, but the data is correctly stored in startTime.

dotnabox

Reputation Points: 10

Solved Threads: 0

Newbie Poster

Offline

18 posts
since Oct 2009

Permalink

Nov 5th, 2009

Re: Input time as hh:mm and no other way

So I ended up figuring out how to do it to allow for white space and get the information as needed. Problem is, I cannot seem to store the AM/PM portion.

C++ Syntax (Toggle Plain Text)
istream &operator>>( istream &input, Time &time)
{
  char s[256];
  time.period = new char[3];
  input.getline(s, 256);
  time.hour = atoi(strtok(s,":"));
  time.minute = atoi(strtok(NULL," "));
  time.period = strtok(NULL, " ");
}
istream &operator>>( istream &input, Time &time)
{
  char s[256];
  time.period = new char[3];
  input.getline(s, 256);
  time.hour = atoi(strtok(s,":"));
  time.minute = atoi(strtok(NULL," "));
  time.period = strtok(NULL, " ");
}

When I output time.hour and time.minute, they work just fine, but time.period doesn't seem to store anything.

Edit: Take that back. It still has a problem with white space (It's part of the assignment that people can't follow instructions to input a time...) as well.

Last edited by dotnabox; Nov 5th, 2009 at 7:07 pm.

dotnabox

Reputation Points: 10

Solved Threads: 0

Newbie Poster

Offline

18 posts
since Oct 2009

Permalink

Nov 6th, 2009

Re: Input time as hh:mm and no other way

Without seeing the function calls and the class and the exact assumptions regarding what you can assume as far as data, plus whatever else may be going on with the stream, I'm not 100% sure what the problem is. Note that "time" is highlighted. It's a C++ keyword. I doubt that's the problem, but possibly consider renaming the variable just in case.

I assume this is the criteria, in order:

Any amount of white space.
0-5 (index 0)
0-9 (index 1)
: (index 2)
0-5 (index 3)
0-9 (index 4)
Exactly one blank space (index 5)
"AM", "PM", "am" or "pm" (indexes 6 and 7)

If this is it, you can use getline to get a maximum of 8 characters, then just go character by character. Use no character delimiter.

C++ Syntax (Toggle Plain Text)
// read in and throw away all white-space, character by character, till you get to first non-white space.  "Put back" into the stream any non-white-space character if you have read it in.
 
char s[9];
getline (s, 9);
// check length of s using strlen.  If it isn't 8, flag as invalid input.
 
// Go through s[] character by character, testing using the criteria above.  If any test fails, flag as invalid input.
// read in and throw away all white-space, character by character, till you get to first non-white space.  "Put back" into the stream any non-white-space character if you have read it in.

char s[9];
getline (s, 9);
// check length of s using strlen.  If it isn't 8, flag as invalid input.

// Go through s[] character by character, testing using the criteria above.  If any test fails, flag as invalid input.

isspace and isdigit from cctype could come in handy. strlen and strcmp from cstring could also come in handy. peek, get, getline, ignore, unget, and putback from the istream library could be useful.

http://www.cplusplus.com/reference/iostream/istream/

What are you supposed to do if the person enters bad data and what data do you need to be able to handle? Depending on that, you may need to do a variety of things to the stream, including setting or clearing failure bits, or clearing the stream. I'm not trying to over-complicate things, so if I am, I'm sorry. It's just good to know as much as possible what data might be thrown at you, what you need to be able to handle and recover from, and what you don't. If it's simply a cin stream where you ask people to enter times and press enter, then say "Sorry, try again" if it's bad, that's less complicated than if you have to set failure bits at the exact point of failure and allow the stream to be cleared and used again, stuff like that.

This page, as well as the next several pages in the link, might be useful.

http://www.linuxtopia.org/online_boo...mming_074.html

Last edited by VernonDozier; Nov 6th, 2009 at 1:22 am.

VernonDozier

Featured Poster

Reputation Points: 2614

Solved Threads: 687

Posting Expert

Offline

5,375 posts
since Jan 2008

Permalink

Nov 6th, 2009

Re: Input time as hh:mm and no other way

Click to Expand / Collapse Quote originally posted by VernonDozier ...

... Note that "time" is highlighted. It's a C++ keyword.

Sorry for the tangent, but "time" is not a C++ keyword. The fact that it's highlighted is maybe because the syntax highlighter handles multiple languages, and "time" is a keyword in some of them (for instance, in the bash shell); or maybe because there is a Unix/POSIX/C standard function called time() from time.h . But "time" is not a C or C++ keyword, and the global function time() wouldn't conflict with a member variable called "time."

boblied

Reputation Points: 36

Solved Threads: 9

Newbie Poster

Offline

24 posts
since Mar 2009

Permalink

Nov 6th, 2009

Re: Input time as hh:mm and no other way

Click to Expand / Collapse Quote originally posted by boblied ...

Sorry for the tangent, but "time" is not a C++ keyword. The fact that it's highlighted is maybe because the syntax highlighter handles multiple languages, and "time" is a keyword in some of them (for instance, in the bash shell); or maybe because there is a Unix/POSIX/C standard function called time() from time.h . But "time" is not a C or C++ keyword, and the global function time() wouldn't conflict with a member variable called "time."

Yes, you are correct. Thanks for pointing that out. I was thinking of the time(time_t*) function. I was pretty sure it would not be a problem, but should not have called it a keyword. Here is a list of keywords.

http://www.cppreference.com/wiki/keywords/start

VernonDozier

Featured Poster

Reputation Points: 2614

Solved Threads: 687

Posting Expert

Offline

5,375 posts
since Jan 2008

This thread is solved

Either the thread starter or a moderator has marked this thread as solved. You can most likely trust the responses and answers given. There is most likely no reason for any further responses to be posted here. If you have a related question, please start a new thread in this forum instead.

This thread is more than three months old

No one has posted to this discussion for at least three months. Please let old threads die and do not reply to them unless you feel you have something new and valuable to contribute that absolutely must be added to make the discussion complete. Otherwise, please start a new thread in this forum instead.

Previous Thread in C++ Forum Timeline: overloading assignment operator problem

Next Thread in C++ Forum Timeline: Simple Winsock Question?

DaniWeb Message

Cancel Changes

User Name
Password