Преобразование строчных букв в верхний регистр

Question

У меня есть программа для изменения строки и преобразования ее в верхний регистр. Если я пишу helloworld!, выход должен быть !DLROWOLLEH. Но если я напишу hello world!, то выход будет !DLRO. Не могли бы вы рассказать мне, где может возникнуть проблема?

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

char * reverse(const char * text) 
{ 

if (text==NULL) 
return NULL; 

int length = strlen(text); 
char * reversed_string = malloc(length+1); 

for(int i = 0; i < length/2; ++i) 
    { 
     reversed_string[i] = text[(length-1) - i]; 
     reversed_string[(length-1) - i] = text[i]; 
    } 

reversed_string[length] = '\0'; 

//upper(reversed_string); 

return reversed_string; 
} 




void upper(char *str1)           
{ 
    while(*str1!='\0')             
     { 
      if(*str1>96&&*str1<123)       
       *str1=*str1-32;          
     str1++;               
     } 
} 

int main(int argc, char * argv[]) 
{ 
    char p[256]; 
    fgets(p, sizeof(p), stdin); 

    char * rev_str = reverse(p); 
    upper(rev_str); 
    printf("%s\n", rev_str); 

    rev_str = 0; 
    return 0; 
} 

Answer 1

Проблема здесь

for(int i = 0; i < length/2; ++i) 

Это length является нечетным числом (например, 11 в вашем примере), это будет неявно округлять вниз, и, как следствие, вы никогда не писать средний элемент в строке. В вашем случае это было 0, но это не гарантировано, поэтому любой персонаж мог появиться там, вместо того, чтобы разорвать строку раньше.

Самое простое исправление будет изменено на (length+1)/2, но это приведет к тому, что вы будете писать средний элемент дважды.

На самом деле, я думаю, что это намного проще, если вы просто перевернули строку, просто перебирая ее в одном направлении, а не в обоих направлениях.

Answer 2

Я изменил ваш код и работает так, как ожидалось.

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

char * reverse(const char * text) 
{ 

    if (text==NULL) 
     return NULL; 

    unsigned long length = strlen(text); 
    char * reversed_string = malloc(length+1); 

    for(int i = 0; i < length; ++i) 
    { 
     reversed_string[i] = text[(length-1) - i]; 
     //reversed_string[(length-1) - i] = text[i]; 
    } 

    reversed_string[length] = '\0'; 

    //upper(reversed_string); 

    return reversed_string; 
} 




void upper(char *str1) 
{ 
    while(*str1!='\0') 
    { 
     if(*str1>96&&*str1<123) 
      *str1=*str1-32; 
     str1++; 
    } 
} 

int main(int argc, char * argv[]) 
{ 

    char p[256]; 
    fgets(p, sizeof(p), stdin); 

    char * rev_str = reverse(p); 
    printf("%s\n", rev_str); 
    upper(rev_str); 
    printf("%s\n", rev_str); 

    rev_str = 0; 

    return 0; 
}