something's up with my substr() test

I was just testing out the substr() method to parse a string's words into a string vector. I was just re-creating what I saw in the book I'm reading so I would understand it, but why I'm not getting the desired effect alludes me. I also tried it with a pre-defined string but still no go.

Code:

---

#include <iostream>
#include <string>
#include <vector>

using std::cin;
using std::cout;
using std::endl;
using std::vector;
using std::string;


int main()
{
    string aStr;
    string temp = "";
    vector<string> parse;
    string::size_type i, j;
   
    cout << "Type a sentence." << endl;
    getline(cin, aStr);
    i = 0;
    while(i != aStr.size())
    {
        while(isspace(aStr[i]) && i != aStr.size())
            i++;
           
        j = i; 
           
        while(!isspace(aStr[j]) && j!= aStr.size())
            j++;
       
        temp = aStr.substr(i, j);
        if (i != j)
        {
            parse.push_back(temp);
            i = j;
        } 
    }
   
    cout << "You typed " << parse.size() << " words:" << endl;
   
    for (vector<string>::size_type i = 0; i < parse.size(); i++)
        cout << parse[i] << endl;   

   
    system("Pause");
    return 0;
}

---

Here's the output

Code:

---

Type a sentence.
The Quick Brown Fox Jumped Over The Fence!
You typed 8 words:
The
Quick Bro
Brown Fox Jumpe
Fox Jumped Over The
Jumped Over The Fence!
Over The Fence!
The Fence!
Fence!
Press any key to continue . . .

---

The second argument in substr is not an index, it's the number of characters in the substring. Don't ask me why that is, but it's messed me up a number of times.

So it should read:

Code:

---

temp = aStr.substr(i, j-i);

---

....I see, thanks. :p

I also noticed that I didn't include the cctype header, but the isspace function, which the book said is declared in the header, works.

Quote:

---

Originally Posted by indigo0086

I also noticed that I didn't include the cctype header, but the isspace function, which the book said is declared in the header, works.

---

Standard headers are allowed to include each another (However there's no requirement for this to happen) - the same is true for std::endl; which is declared in <ostream> - but most compilers #include <ostream> inside <iostream>