cs50 caesar - cypher 不打印出了什么问题？

Question

卡在这个问题上。 代码编译并接受输入。 但是，当尝试将密钥“1”与纯文本“a”一起使用时，我期待“b”但在我检查调试器时返回“\001”（未打印）。 有人可以帮我解释为什么会这样吗？ 我怀疑错误是当我为 cypher 测试分配 memory 时。或者在我的 for / if 语句中实际执行 cypher 时。

int main(int argc,string argv[]) 
{   
    check_commands(argc);

    //checks if key is alpha
    string key = argv[1];
    for (int i = 0; i < strlen(key); i++)
    {
        if (isalpha(key[i]))
        {
            printf("Usage: ./caesar key\n");
            return 1;
        }
    }
    //convert string argv[1] into an int
    int k = atoi(key);

    //ask user for pplaintect
    string plaintext = get_string("Plaintext:  ");
    //create cyphertext variable
    int s = strlen(plaintext);
    char *cyphertext = malloc(s + 1);
    printf("Cyphertext: ");

    for (int i = 0; i < s; i++)
    {
        if (isalpha(plaintext[i]))
        {
            if (isupper(plaintext[i]))
            {
                // cypher based of k but keep upper
                cyphertext[i] = ((plaintext[i] - 65) + k) % 26;
                //print uppercase cypher
                printf("%s", cyphertext);
            }

            if (islower(plaintext[i]))
            {
                //cypher based of k but keep lower
                cyphertext[i] = ((plaintext[i] - 97) + k) % 26;
                //print lowercase cypher
                printf("%s", cyphertext);
            }
        }
        else
        {
            printf("%c", plaintext[i]);
        }
    }
    free(cyphertext);
    printf("\n");
}

int check_commands(int argc)
{ //checks wether we have two command line arguments
    if (argc != 2)
    {
        printf("Usage: ./caesar key\n");
        return 1; //error
    }
    return 0;
}

Answer 1

您只是将模数放入cyphertext 。 您需要将其添加到字母表的第一个字符以获得相应的字母，就像在添加k之前减去该字符一样。

                cyphertext[i] = 'A' + ((plaintext[i] - 'A') + k) % 26;

此外，不要对65和97之类的 ASCII 代码进行硬编码。 使用字符文字。

Answer 2

您分配了一个未初始化的 memory

char *cyphertext = malloc(s + 1);

它不包含字符串。 所以你不能像这样在printf s调用中使用转换说明符

printf("%s", cyphertext);

也在这些陈述中

cyphertext[i] = ((plaintext[i] - 65) + k) % 26;

和

cyphertext[i] = ((plaintext[i] - 97) + k) % 26;

您正在存储不可打印的字母字符。

还有这个记录

int s = strlen(plaintext);

是多余的。

使用 do-while 循环而不是 for 循环。 例如

size_t i = 0;

do
{
    if ( isupper( ( unsigned char )plaintext[i] ) )
    {
        // cypher based of k but keep upper
        cyphertext[i] = 'A' + ( plaintext[i] - 'A' + k ) % 26;
    }
    else if ( islower( ( unsigned char )plaintext[i] ) )
    {
        //cypher based of k but keep lower
        cyphertext[i] = 'a' + ( plaintext[i] - 'a' + k ) % 26;
    }
    else
    {
        cyphertext[i] = plaintext[i];
    }
} while ( plaintext[i++] != '\0' );

puts( cyphertext );
free( cyphertext );