Thread: Character encoding: encoded_character = 10 + (plaintext_character - 'a') + 'A'

Originally Posted by christophergray

NULL is defined as the pointer type ((void *)0) in libio.h, but you are assigning it to a char type. And the assignment is redundant anyway because you scan in and compute other values for your chars before printing them. This code works fine if you just leave out the = NULL.

To make this code fine - need to check return value of scanf before using any variable it filled. And also there is library function for converting to upper case that works even if the encoding layout is different from the supposed in this code.

Code:

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

int main() {

    char encoded_character = 0;
    char plaintext_character = 0;
    int ret;
    //3.1 Get encoded_character and plaintext_character from user
    printf("Please input your plaintext character ");
    //3.2 Scan
    ret = scanf("%c", &plaintext_character);
    if(ret != 1){
        printf("Read failed\n");
        return -1;
    }

    //3.3 Charcater encoded
    encoded_character = toupper(plaintext_character);
    //3.4 printf encoded character
    printf("encoded_character is = %c\n",encoded_character);
    
    return 0;
}