cin.peek() & cin.seekg() in a for loop??

Why doesn't the for loop advance the read position of the cin stream via the seekg()line, so that I can test each character via cin.peek()???

bool success = true;
charar invalidChar = '\0';
double operand = NULL;
string discard;
cout<< "Please enter an operand --> ";

for(int n = 0; success == true && cin.peek() != '\n'; n++)
{
   invalidChar = cin.peek();
   if(!isdigit(invalidChar))
         if(!(invalidChar == '.'))
            success = false;
   cin.seekg((n+1), ios::beg);
}
smconveyAsked:
Who is Participating?
I wear a lot of hats...

"The solutions and answers provided on Experts Exchange have been extremely helpful to me over the last few years. I wear a lot of hats - Developer, Database Administrator, Help Desk, etc., so I know a lot of things but not a lot about one thing. Experts Exchange gives me answers from people who do know a lot about one thing, in a easy to use platform." -Todd S.

jkrCommented:
>>advance the read position of the cin stream

Err, why would you want to do that? 'Advancing' cin is sort of an oxymoron. Why don't you just

for(int n = 0; success == true && invalidChar != '\n'; n++)
{
  cin >> invalidChar;
  if(!isdigit(invalidChar))
        if(!(invalidChar == '.'))
           success = false;
}
0
smconveyAuthor Commented:
I'm trying to read in a double, but I wanted to test it first. By peeking at and testing each character, I still have it in the cin stream to be able to read it into operand if it is valid. If I do it your way, the potential operand is gone (dicarded a character at a time). I am a student, and I am trying to learn to use the peek and seekg methods.
0
jkrCommented:
Ah, OK, that makes sense. 'seekg()' won't be able to advance 'cin' as there is no further input past the current stream position So, using

for(int n = 0; success == true && cin.peek() != '\n'; n++)
{
  invalidChar = cin.peek();
  if(!isdigit(invalidChar))
        if(!(invalidChar == '.'))
           success = false;
 cin >> invalidChar;
}

will make it work.
0
Cloud Class® Course: MCSA MCSE Windows Server 2012

This course teaches how to install and configure Windows Server 2012 R2.  It is the first step on your path to becoming a Microsoft Certified Solutions Expert (MCSE).

smconveyAuthor Commented:
cout<< "Please enter an operand --> ";

If I would have entered 35.68 to the above prompt, why wouldn't seekg have moved the read position incrementally with each iteration of the for loop so that peek could validate each of the characters until the newline is reached, and if success cin>>operand?

for(int n = 0; success == true && cin.peek() != '\n'; n++)
{
   invalidChar = cin.peek();
   if(!isdigit(invalidChar))
         if(!(invalidChar == '.'))
            success = false;
   cin.seekg((n+1), ios::beg);
}
0
jkrCommented:
>>If I would have entered 35.68 to the above prompt, why wouldn't seekg have moved the read position
>>incrementally

Because you are not giving it a chance to do so - your code is executed even *before* the '3' is entered.
0
jkrCommented:
Hum, if you want to learn "learn to use the peek and seekg methods", try using a stringstream for that purpose:

#include <sstream>
#include <iostream>
using namespace std;

//...

bool success = true;
char invalidChar = '\0';
double operand = NULL;
string discard;
cout<< "Please enter an operand --> ";

char ac [ 256];
cin >> ac;

stringstream ss;

ss << ac;

for(int n = 0; success == true && cin.peek() != '\n'; n++)
{
  invalidChar = ss.peek();
  if(!isdigit(invalidChar))
        if(!(invalidChar == '.'))
           success = false;
  ss.seekg((n+1), ios::beg);
}
0

Experts Exchange Solution brought to you by

Your issues matter to us.

Facing a tech roadblock? Get the help and guidance you need from experienced professionals who care. Ask your question anytime, anywhere, with no hassle.

Start your 7-day free trial
smconveyAuthor Commented:
I just tried emailing you at XRKFNDZEMORB@spammotel.com as indicated in your profile, but I'm not sure if it is current. Can I call you to discuss? I'm confused on a few points.
0
jkrCommented:
The email address is correct. Are you sure you want to make long distance call to good ole europe? BTW, gonna leave for lunch soon anyway :o)
0
havman56Commented:

there are two seekg functions are there

basic_istream<charT,traits>& seekg(off_type, ios_base::seekdir);
basic_istream<charT,traits>& seekg(off_type);

you are using the first one . so std::ios::beg will initialise the stream pointer ate beg
so seekg(n+1,std::ios:beg)
will make ur pointer to not to move i feel so

firts initialise with std::ios::beg but later use only seekg(n+1)
use only seekg(n+1)

i hope it will work





0
havman56Commented:
check also std::isspace
0
smconveyAuthor Commented:
Thanks, but I had to move on. I have already changed my code to accomplish my goal another way. My project was due.
0
tinchosCommented:
No comment has been added lately, so it's time to clean up this TA.
I will leave the following recommendation for this question in the Cleanup topic area:

Accept: jkr {http:#9690300}

Please leave any comments here within the next seven days.
PLEASE DO NOT ACCEPT THIS COMMENT AS AN ANSWER!

Tinchos
EE Cleanup Volunteer
0
It's more than this solution.Get answers and train to solve all your tech problems - anytime, anywhere.Try it for free Edge Out The Competitionfor your dream job with proven skills and certifications.Get started today Stand Outas the employee with proven skills.Start learning today for free Move Your Career Forwardwith certification training in the latest technologies.Start your trial today
C++

From novice to tech pro — start learning today.

Question has a verified solution.

Are you are experiencing a similar issue? Get a personalized answer when you ask a related question.

Have a better answer? Share it in a comment.