I tried to write a function to do simple shifting of characters (either to the left or right, depending on the shift parameter in the following function). Capital letters remain to be capital letters. Here is my approach:
char encodeCaesarCipherChar(char ch, int shift)
{
char result;
if (!isalpha(ch)) return ch;
result = ch + shift;
if (islower(ch) && result < 'a') {
result += int('z') - 1;
result -= int('a');
} else if (islower(ch) && result > 'z') {
result -= int('z');
result += int('a') - 1;
} else if (isupper(ch) && result < 'A') {
result += int('Z') - 1;
result -= int('A');
} else if (isupper(ch) && result > 'Z') {
result -= int('Z');
result += int('A') - 1;
}
return result;
}
This function stops working properly when the input character is 's' and beyond. Could anyone please point out what's the problem to my approach?
Thanks in advance.
Best Answer
's' + 13 will overflow a signed
char
. Keep the result in anint
and cast tochar
after adjusting the number and before returning.