//Given the name of the song, this function will reformat //the first letter of each word to capitalized and the remaining letters are //lower case
// The code will first convert every nonspace character to lowercase.
// Then the first letter after each space will be reformatted to uppercase.
for (int i = 0; i < songname.length(); i ++)
if (songname [i] !=' ')
songname[i] = tolower(songname[i]);
if (songname[i] == ' ')
songname[i+1] = toupper(songname[i+1]);
And I got the following output instead:
Thank you for any help.
The error is basically that whenever you find a space you convert the following letter to uppercase, but in the next iteration of the loop that letter is examined, and since it's not a space it is converted to lowercase.
Also, you didn't consider that the very first letter should always be uppercase.
To check for spaces I'd use isspace() which also checks for newlines and tabs.
Thank you for the reply. I apologize that I forgot to mention the very first character in the input will always be a blankspace. Which mean songname will be a blankspace all the time, therefore I didn't convert the very first letter to be uppercase.
Since I put tolower(songname[i]) before toupper (songname[i]) in my loop. Wouldn't my loop convert every letter to lowercase first, and then it convert the first letter after each blankspace to uppercase?
Oh nevermind, I know what is wrong with my loop now. It converts a nonblank character to lower case. If the character is a
Blank then it converts the next character to upper case, but on the next time through the loop
The character you just capitalized is now converted to lower case
Wouldn't my loop convert every letter to lowercase first, and then it convert the first letter after each blankspace to uppercase?
Nope. A loop restarts when the flow of control reaches the end of the loop body or if it encounters a "continue" instruction. You loop checks if a character is not a space, then checks if it is a space and then restarts.
If you want to convert everything to lowercase and then examine it again you'll need to write two loops.
There is an off-by-one error in the loop; you need to fix that too.
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17
std::string convert_song( std::string songname )
if( !songname.empty() )
// convert the first char to upper case if it is lower case
songname = std::toupper( songname ) ;
// iterate till songname[i] is the second last char (songname[i+1] is the last char)
for( std::size_t i = 0 ; i < songname.size() - 1 ; ++i )
if( std::isspace( songname[i] ) ) songname[i+1] = std::toupper( songname[i+1] ) ;
else songname[i+1] = std::tolower( songname[i+1] ) ;
return songname ;