Что не так в этой реализации Caesar Cipher?

Question

Я пытаюсь реализовать шифр Цезаря, но я не получаю ожидаемого результата. Что не так с кодом?

Key: 3 
Input: Hello 
Output I'm getting: KNUUX  
Expected Output: KHOOR 

Код:

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

int main(int argc, string argv[]) 
{ 
    if(argc!=2) 
    { 
     return 1; 
    } 
    int k, i; 
    k = atoi(argv[1]); 
    printf("Enter the String to Encrypt: "); 
    string s=GetString(); 
    for(i=0; i<strlen(s); i++) 
    { 
     if('A'>=s[i]<='Z') 
     { 
      s[i]=((s[i] - 'A' + k)%26) +'A'; 
     } 
     else if('a'>=s[i]<='z') 
     { 
      s[i]=((s[i] - 'a' + k)%26) +'a'; 
     } 
    } 
    printf("The Encrypted Text is %s\n",s);   
} 

Answer 1

if('A'>=s[i]<='Z') 

, конечно, не делать то, что вы, кажется, ждет.

Вы, вероятно, хотите:

if ((s[i] >= 'A') && (s[i] <= 'Z')) 

Answer 2

for(i=0; s[i]; ++i) 
    if(isalpha(s[i])) 
     s[i]=(toupper(s[i]) - 'A' + k)%26 +'A';