Question

我的目标是制作一个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;
}

Answer 1

encrypt函数中的循环完全没用，因为没有return语句的循环体没有路径导致循环终止并且控制权返回给{{{1}调用者1}}。这使得程序作为一个整体重复地使用密钥中的连续元素加密消息的第一个字符。

最简单的方法是进行以下更改

从encrypt功能
作为附加参数传递要加密的消息中的元素，进行签名
```
encrypt
```

一些杂项评论：

全局变量int encrypt(string message, int k, int i)未在任何地方使用。你可以安全地删除它。（你应该尽可能避免使用全局变量）。
您也可以使用字符文字new_crypt，而不是使用幻数65。这样做的好处是，即使您最终没有使用ASCII，也不需要注释来解释数字65，并且它总是正确的大写字母A的值。（但也见下一个子弹）
您的代码假定字母A到Z（和a到z）具有连续值（例如Z == A + 26）。这可能恰好是ASCII编码中的英文字母，但不保证其他语言字母或编码。

使用C中函数的返回值

1 个答案: