Thread: counting the white space seperated words?

"...counts the whitespace-separated words in that file."

This sounds like it wants you to count the words; in your program, it looks as if you are trying to count the white-space.

Anyways, a few things:

Initialize x to zero.
Add your return statement to the end of main.
In your x=x++ statement, just make it x++ (the result is the same).

I wouldn't start to read in anything until the while loop is started.
Start your loop and read in your strings one at a time. You could use Count_Space >> msg; which skips white space, so you wouldn't have to make any comparisons. Just increase x every time a string is read in (which would be a single word). When it fails (Count_Space.fail()) then break from the loop.

well. at first i realised it was count the words. after first trying to count spaces. but i figure right now itd be a start to count the whitespaces instead of the words. and switch it up when i can do that.

and i understood the first two sections of what you said. but i might need some trial and error for the last haha (Y)

thanks alot. this will definitely help

edit: 3 seconds and the program works o.O..... thank you god! (Cpro).

but im a bit confused about how to get Count_space.fail() worked in?

would i do something like

while (Count_space<<msg && Count_space != Count_space.fail()
{
x++
//return x sorry. not a function. mistake
};
once again excuse me. this is my first time using C++ in about 6-9 months

and also. i seen it easy to just ad cout<<x + 1;
instead of counting each individual word. i guess this is correct. but is there a better method?

unless i add string spc to string msg. as then it would still be counting all the words. incidently

"but im a bit confused about how to get Count_space.fail() worked in?"

Well, you don't have to use it. It just depends on how you make your while loop. This is how I did it:

Code:

while(1)
{
     Count_Space >> msg;
     if(Count_Space.fail()) break;
     x++;
}

So, the loop will break when Count_Space >> msg fails to read in a string (Count_Space.fail() will equal true). There are a few ways to make this loop work, and you don't have to use Count_Space.fail(). I just saw this method used in a book once, and I liked it. For example, you could also do:

Code:

while(Count_Space >> msg)
{
     x++;
}

"and also. i seen it easy to just ad cout<<x + 1;
instead of counting each individual word. i guess this is correct. but is there a better method?"

I'm not sure what you mean here. You must count each individual word.

"and also. i seen it easy to just ad cout<<x + 1;"

this is what i mean by that.

heres the last bit of code

Code:

    while(1)
{
     Count_Space >> msg;
     if(Count_Space.fail()) break;
     x++;
};

    cout<<"string has"<<x+1 <<"words in it";
    cin.get();
    return 0;
}

ive put the return in.
and i cheated by adding a 1 to whatever 'x' was. to make up for the extra word in there. but clearly this is cheating.

and while im here. and i think itd be a more then easy question to answer. why wont my 'cin.get();' wait for the keypress? it just closes.

Code:

#include <iostream>
#include <fstream>
#include <string>

using namespace std;

int main(){
    int x =0;
    string msg;
    string msg2;
    cout<<"please enter a string\n";
    cin>>msg;
    ofstream count_space ("count_space.txt");
    count_space<<msg;
    count_space.close();
    ifstream Count_Space ("count_space.txt");
    Count_Space>>msg2;
        while(1)
{
     Count_Space >> msg;
     if(Count_Space.fail()) break;
     x++;
};
    cout<<"string has"<<x<<"words in it";
    cin.get();
    return 0;
}

perhaps i need to close the file im working with?

Originally Posted by lilhawk2892

because this was an excercise i guess i didnt take care to respectfully name variables. i suppose i should have thought about that before i made a post about it.

If you do not practice to develop good habits when doing an exercise, then what is the point of the exercise?

Originally Posted by lilhawk2892

but the problems seemed to be all cleared up and understood except for the cin.get() problem. it just keeps closing too early

That is because a newline is left in the buffer, and it is consumed by that cin.get() call. You should ignore what is left in the buffer by cin.ignore(numeric_limits<streamsize>::max(), '\n') (or cin.ignore(1000, '\n') before using cin.get(). However, it would be simpler to just run your program from a suitable IDE or a command prompt.

Originally Posted by lilhawk2892

but i used cin.ignore(); without the parameters. is this okay? or dangerous?

It will result in your earlier "problem" if there is more than one character in the buffer.

alright i see now. im having the small problem of always coming up with 8. but that shouldnt take more then a few minutes to get past