-1

我的目标是制作一个 Vigenere 密码。我试图通过从 argv 获取密钥，从用户获取字符串，然后通过我创建的一个函数传递消息和密钥，该函数将它们组合并在打印之前返回新值。出于某种原因，它只是在打印密钥。我认为这与新函数以及我如何尝试使用返回值有关。这是代码：

#include <stdio.h>
#include <stdlib.h>
#include <cs50.h>
#include <string.h>
#include <stdbool.h>
#include <ctype.h>
int new_crypt;
int encrypt(string , int );
int main(int argc, string argv[])
{
    if( argc != 2)
    {
        printf("Plese enter only one key");
        return 1;
    }

    string message = GetString();
    for(int i=0; i < strlen(message); i++)
    {
        int klen = strlen(argv[1]);
        //repeats the key until the message is over
        int key= argv[1][i%klen];
        bool kupper = isupper(key);
        bool klower = islower(key);
        bool kalpha = isalpha(key);
        if(kupper == true){
            //ASCII value of A is 65. 'A' = 0 shifts
            int k = key-65;
            int new_crypt = encrypt(message, k);
            printf("%c", new_crypt);
        }
        if(klower == true){
            //ASCII value of 'a' is 97. 'a' = 0 shifts
            int k = key- 97;
            int new_crypt = encrypt(message, k);
            printf("%c", new_crypt);
        }
        if(kalpha == false){
            int k = 0;
            int i = i-1;
            int new_crypt = encrypt(message, k);
            printf("%c", new_crypt);
        }

    }

    printf("\n");
    return 0;
}
int encrypt(string message, int k)
{
    for(int i=0; i < strlen(message); i++)
    {
        bool upper = isupper(message[i]);
        if(upper == true)
        {   //Makes sure the message doesnt go past 'Z'.. If it does it mod 90 it                                            /                   // and adds 65 ('A')
            int crypt = (message[i]+ k) % 90;
            if(crypt < 65)
            {
                int new_crypt = (crypt + 65);
                return new_crypt;
            }
            else{
                int new_crypt = crypt;
                return new_crypt;
            }
        }
        bool lower = islower(message[i]);
        if(lower == true)
        {
            int crypt = (message[i]+ k) % 123;
            if(crypt < 97)
            {
                int new_crypt = crypt + 97;
                return new_crypt;
            }
            else{
                int new_crypt = crypt;
                return new_crypt;
            }
        }
        bool alpha = isalpha(message[i]);
        if(alpha == false)
        {
            int new_crypt = message[i];
            return new_crypt;
        }
    }
    return 0;
}
4

1 回答 1

1

函数中的循环encrypt是完全没用的，因为没有一条return语句会导致循环终止并将控制权返回给encrypt. 这使得整个程序使用密钥中的连续元素重复加密消息的第一个字符。

解决此问题的最简单方法是进行以下更改

  • 从encrypt函数中删除循环

  • 将要加密的消息中的元素作为附加参数传递，从而进行签名

    int encrypt(string message, int k, int i)

一些杂项说明：

  • 全局变量new_crypt不在任何地方使用。您可以安全地删除它。（您应该尽可能合理地避免使用全局变量）。
  • 除了使用幻数65，您还可以使用字符文字'A'。这样做的好处是您不需要注释来解释数字 65，并且它始终是大写 A 的正确值，即使您最终不使用 ASCII。（但另见下一个项目符号）
  • 您的代码假定字母 A 到 Z（和 a 到 z）具有连续值（例如 Z == A+26）。ASCII 编码中的英文字母可能会发生这种情况，但不能保证其他语言字母或编码。
于 2012-11-30T09:14:04.797 回答