Thread: Issues printing out a string array

Hi there,

I have a small issue when I am printing out an array from a user inputted string. The programs function is to have the user input a string and convert all the lower case letters to uppercase without using the toupper function but a self created one. This all works fine but when i print out the array in sequence it always starts with the @ symbol. Is there something that I am doing incorrectly?

Code:

#include <stdio.h>
#include <string.h>
#include <stdlib.h>


int checkupper(char*s);

int main(void)
{

   // declare the user char arrays to store the strings
    char user_x1[256] ;
    char convert;
    int length;
    int i,e;
    int ar_size;
    
    printf("Enter a String: ");
    fgets( user_x1, 256, stdin );  //fgets (char string, int size, file *stream
    length = strlen(user_x1);
    printf("%d",length);
    ar_size=length-1;
    printf("\n\nlength of the string: %d\n",length);
    printf("\n\narray size of the string: %d\n",ar_size);

    system("cls");
      
      for(i=0;i<=ar_size;i++) //for loop to CALL FUNCTION TO CONVERT TO UPPER
         {
           convert = checkupper(user_x1);
         }
    printf("User converted string: %c",user_x1);
          for(i=0;i<=ar_size;i++) 
         {
           printf("%C ",user_x1[i]);
         }        
         
         
     

    system("pause");
    return 0;
}

int checkupper(char*con)
    {
       while(*con)*con>96&*con<123?*con-=32:0,con++;
    }

Thanks in advance.....

Thanks for that...I was just getting confused with my coding. I see where I went wrong and with your additional code i understand what happened...

rgds

Nasty...

Code:

       while(*con)*con>96&*con<123?*con-=32:0,con++;

Better...

Code:

       while(*con)
          { if ((*con > 96) & (*con < 123 )
                  *con-=32 ;
             con++; }

Checkupper is not returning any value.
Actually it should not be called checkupper at all isn't it?

Originally Posted by codeprada

i assume u meant && :P

Hmmmm.... Yep, looks like I did ....