Thread: Remove upper case letters from a char string

Originally Posted by lavinpj1

It seems to only return a single character.

What am I doing wrong here?

What are you passing as input to the function?

Originally Posted by lavinpj1

It is supposed to loop through every character of the variable text and check to see if it is upper case, by comparing it with the uppercase equivialant of it.

Then a better name for the function would be helpfull, RemoveUpper seems to indicate a different purpose.

Originally Posted by lavinpj1

It loops through every character in the input and check if it is uppercase. If it isn't, then it sticks the character to the end of a new char variable, therefore removing upper case letters.

Is this what you are saying?

Code:

#include <stdio.h>
#include <ctype.h>

char *RemoveUpper(const char *text)
{
   static char new_str[4096];
   int i = 0;
   for ( ; *text; ++text )
   {
      if ( !isupper(*text) )
      {
         new_str[i++] = *text;
      }
   }
   new_str[i] = 0;
   return new_str;
}

int main(void)
{
   char text[] = "Hello World";
   puts(text);
   puts(RemoveUpper(text));
   return 0;
}

/* my output
Hello World
ello orld
*/

Ah, thanks very much Dave.

That worked great, but what I did want as islower, not isupper. The only problem I am having, is, that like all programming languages, it thinks characters are uppercase. This is what I want, although I do not want to remove spaces. How would I go about stopping it removing spaces?

EDIT: Tried...

Code:

if ( ( islower(*text) ) && ( *text != ' ' ) )
      {
         new_str[i++] = *text;
      }

with no luck.

Cheers

~Phil~

Originally Posted by lavinpj1

The only problem I am having, is, that like all programming languages, it thinks characters are uppercase.

Originally Posted by lavinpj1

This is what I want, although I do not want to remove spaces. How would I go about stopping it removing spaces?

You want to keep only lowercase or spaces?

Code:

#include <stdio.h>
#include <ctype.h>

char *RemoveUpper(const char *text)
{
   static char new_str[4096];
   int i = 0;
   for ( ; *text; ++text )
   {
      if ( islower(*text) || isspace(*text) )
      {
         new_str[i++] = *text;
      }
   }
   new_str[i] = 0;
   return new_str;
}

int main(void)
{
   char text[] = "The Quick Brown Fox Jumps Over The Lazy Dog.";
   puts(text);
   puts(RemoveUpper(text));
   return 0;
}

/* my output
The Quick Brown Fox Jumps Over The Lazy Dog.
he uick rown ox umps ver he azy og
*/

http://www.cppreference.com/stdstring/index.html
http://www-ccs.ucsd.edu/c/ctype.html
http://www.dinkumware.com/manuals/re.../&h=ctype.html