这个Caesar Cipher的实现有什么问题?

时间:2014-02-13 03:24:52

标签: c cs50 caesar-cipher

我正在尝试实施Caesar密码,但我没有得到预期的输出。代码有什么问题?

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);        
}

2 个答案:

答案 0 :(得分:3)

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

当然没有做你似乎期待的事情。

你可能想要:

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

答案 1 :(得分:1)

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