Thread: string help.

I am trying to find the number of words in a line via getline(istream,line,'\n') by counting white spaces. What I am using to count white spaces is this:

int pos = Line.find(' ',0);
while(pos != string::npos)
{
pos = Line.find(' ',pos);
nwords++;
cout << pos << endl;
}

Is what im doing with find() legal in regard to white spaces? When this loop is executed,
cout for pos = 6, which is where my first white space is, however I cannot get it to proceed....

So you mean a line like this will have
10 words

Code:

a          a

hmmm... dont understand... here is void main, and the calling function:

void analyzeLine( string Line, string & longest, int & nwords, int & nchars)
{
int length = Line.size();
int pos = Line.find(' ',0);
while(pos != string::npos)
{
pos = Line.find(' ',pos);
nwords++;
cout << pos << endl;
}
nchars = length + nchars;
cout << length << " " << nchars << endl;
}

void main ()
{
WritePoem();
ifstream poem;
poem.open("poem.txt");
const int NPOS = string::npos;
int nchars = 0, nwords = 0;
string Line, longest;
while(!poem.eof())
{
getline(poem,Line,'\n');
analyzeLine(Line, longest, nwords, nchars);
//cout << Line << endl;
//cout << nwords << endl;
}
}


What I am trying to do is read each line of the file with getline() and storing it into string Line, then counting the white spaces in Line via find(), where the result should give me the amount of words(if I add 1 to the result), however I cant seem to get it to find any white spaces other than the first one when find() is called.

however I cant seem to get it to find any white spaces other than the first one when find() is called.

I think the reason why the code doesn't work
the way you want it to is that you have to skip
over the space just found. Try find(' ', pos + 1).

Code:

while(!poem.eof())
{
      getline(poem,Line,'\n');
      analyzeLine(Line, longest, nwords, nchars);
//cout << Line << endl;
//cout << nwords << endl;
}

getline(poem, Line, '\n') could reach the eof and
then analyzeLine would be given invalid data I think.

It's better to have your functions do one thing and void
main is bad.

What if you had many spaces after each other:
"This is me"