last word of string is not processed

[safe holly]

My code is supposed to take in a string with multiple words and return a set with unique word

(function "clear token" returns string without any punctuations or empty if no letter in word)

However if the word is very long, it doesn't take in last word of a string. Does anyone know what I did wrong?

[subtle bramble]

When your question is answered use !solved to mark the question as resolved.

Remember to ask specific questions, provide necessary details, and reduce your question to its simplest form. For more information use !howto ask.

[static cargo]

You can extract whole words from a string far easier using istringstream

set<string> gatherTokens(const string& text) {
    set<string> tokens;
    string token;
    
    istringstream str(text);
    while (str >> token)
    {   
         tokens.insert(token);
    }
}

[safe holly]

thank you so much, is there any way I could simplify this 2 while loop?

    while(ispunct(s[end]) != 0)
    {
        s = s.substr(begining, end);
        end--;
    }

    while(ispunct(s[begining]) != 0)
    {
        begining++;
    }```

I tried taking out ```s = s.substr(begining, end);```

but it wouldn't work

[static cargo]

while (ispunct(s.back())
{
  s.pop_back();
}

[safe holly]

Thank you!

!solved

[subtle bramble]

Thank you and let us know if you have any more questions!

[SOLVED] last word of string is not processed

[static cargo]

I am not sure about removing punctuation from the front though.
There doesn't appear to be a string::pop_front() method.

Perhaps:

while (ispunct(s.front())
{
  s.erase(s.begin());
}

[subtle bramble]

@safe holly

This question thread is being automatically marked as solved.