Странный выход программы C с использованием функции toupper

Question

Вот моя программа для прочтения всех букв, введенных со стандартного ввода. Но некоторые результаты очень странные. Например, если ввод «lorem ipsum», а выход будет «LOREM IPSUMS?». Если ввод представляет собой один символ, такой как «m», выход будет «MZ # X?» , "S?" и "Z # X?" не должны быть здесь, но они присоединяются к выходу.

Почему это происходит?

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

int main(void){ 
     char input; 
     char upper[100]; 
     int count = 0; 

     while((input = getchar())){ 
       if(input == '\n') 
         break; 
       if(input >= 'a' && input <= 'z') 
         input = toupper(input); 
       *(upper + count) = input; 
       count++; 
     } 

     printf("%s\n", upper); 

     return 0; 
} 

Answer 1

Вы можете установить все элементы в массиве, чтобы быть 0 (ASCII NUL), потому что мы не имеем ни малейшего понятия, что находится в upper массиве.

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

int main(void){ 
    char input; 
    char upper[100]; 
    int count = 0; 

    memset(upper, 0, 100); 

    while((input = getchar())){ 
      if(input == '\n') 
        break; 
      if(input >= 'a' && input <= 'z') 
        input = toupper(input); 
      *(upper + count) = input; 
      count++; 
    } 

    printf("%s\n", upper); 

    return 0; 
} 

Если вы не знаете, что делает memset, вы можете сделать это с помощью цикла for.

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

int main(void){ 
    char input; 
    char upper[100]; 
    int count = 0; 

    for (int i = 0; i < 100; i++) { 
     upper[i] = 0; 
    } 

    while((input = getchar())){ 
      if(input == '\n') 
        break; 
      if(input >= 'a' && input <= 'z') 
        input = toupper(input); 
      *(upper + count) = input; 
      count++; 
    } 

    printf("%s\n", upper); 

    return 0; 
} 

Answer 2

Ваш код работает для меня. Единственное, но это ожидается, вы получите мусор после строки, потому что ваш массив символов имеет длину 100. Вы должны положить 0 в конце строки, чтобы сообщить printf, что ваша строка заканчивается там. положить

*(upper + count) = 0; 

перед тем принтf.

Answer 3

Ваш код имеет некоторые проблемы:

• Нулевая терминатор не добавил в конце строки.
• Нет никакой проверки индекса за пределами выделенного массива.
• Пользователь может прерывать ввод без новой строки, создавая цикл программы навсегда.
• getchar возвращает int.

Вы можете попробовать это исправить:

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

const int MAX_CHAR = 101; 

int main() { 
    int input; 
    char upper[MAX_CHAR]; 
    int count = 0; 

    while(count < MAX_CHAR - 1) { 
     input = getchar(); 
     if (input == EOF || input == '\n' || input == '\r') 
      break; 
     if (input >= 'a' && input <= 'z') 
      input = toupper(input); 
     upper[count] = input; 
     count++; 
    } 
    // add the null terminator 
    upper[count] = '\0'; 

    printf("%s\n", upper); 

    return 0; 
}