c - 如何修复Vigenere密码中从“z”回“a”的循环

我的Vigenere密码有问题。当我输入消息时，结果很好，但是如果字母超过“ z”，它不会循环回到“ a”，并且会打印出其他ascii字符。而且，当我输入消息时，有时得到的字符超出了我的需要。例如：关键是你好，消息是mmmmmm（我知道，消息不多，但这只是一个例子），输出是tqxx {{。请帮忙！！！！

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

int main( int argc, char *argv[])
{
    char *k;
    if (argc!=2)
    {
        printf("Give one argument, not 2, not 3, not 4 , and not any other amount other than one");
        printf("\n");
        return 1;
    }
    k = argv[1];
    printf("What is the message? ");
    string message = get_string();
    if (strlen(k)<strlen(message))
    {
        printf("Invalid response\n");
        return 0;
    }
    for (int i = 0, l = 0; i < strlen(message); i++, l++)
    {
        int x;
        if (message[i]>='A' && message[i] <='Z')
        {
            message[i] = message[i]-'A';
            message[i] = message[i] + (k[l] - 'A') % 26;
            message[i] = message[i] + 'A';
            message[i] = (char) x;
            if (x > 90)
            {
                x = x - 26;
                message[i] = (char) x;
            }
            printf("%c", message[i]);
        }
        if(message[i] >='a' && message[i] <= 'z')
        {
            message[i]= message[i]-'a';
            message[i] = message[i] + (k[l] - 'a') % 26;
            message[i] = message[i] + 'a';
            printf("%c", message[i]);
        }
      if ((message[i] < 'A') || (message[i] > 'z') || (message[i] > 'Z' && message[i] < 'a'))
        {
            printf("%c", message[i]);
        }
    }
    printf("\n");
    return 0;
}

最佳答案

您应该在message[i] + (k[l] - 'a')部分周围放置括号。模仅用于第二部分-(k[l] - 'a')-，而您希望将其应用于整个表达式。

此外，您假定k[l]与message[i]相同（大写/小写），但这不一定是正确的。您应该在每个内部分支中具有另一个条件，或者（最好）计算k字母的数字值并将它们存储为0-25范围内的数字。

第三，为什么x在那里？未初始化时使用它，通常我不知道它存在的原因。

最后一件事，您可能应该在l到达k末尾时再次将其设置为零，以防消息比键长。

关于c - 如何修复Vigenere密码中从“z”回“a”的循环，我们在Stack Overflow上找到一个类似的问题：https://stackoverflow.com/questions/44890032/