CS50的Caesar Cipher

时间:2014-02-25 04:46:51

标签: c encryption cs50 caesar-cipher

我已经盯着这个问题好几个星期了,我一无所有!它不起作用,我知道的很多,但我不知道为什么或什么是错的。我知道开发人员吐出了关于我突出显示的那一行的“错误:预期表达”,但实际上这只是冰山一角。如果有人知道如何修复任何一小部分,我会非常感激!

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

int main(int argc, char *argv[]) 
{

    //Get the key
    if (argc != 2 || atoi(argv[1]) < 0)
    {
        printf("Usage: ./caesar k");
        return 1;
    }

    int key = atoi(argv[1]);
    string plaintex;
    string plaintext = GetString();

    for (int i = 0, n = strlen(plaintext); n < i; i++)
    {
        if (plaintext[i] > 'A' && plaintext[i] <= 'Z')
        {
            plaintext[i] = (plaintext[i] - 'A' + key) % 26 + 'A';
        }
    }  
    for (int i = 0, n = strlen(plaintext); n < i; i++)
    {           
        if (plaintext[i] >= 'A' && plaintext[i] >= 'Z')  // Highlighted line
        {
            plaintext[i] = (plaintext[i] - 'A' + key) % 26 + 'A';
        }
        else if (plaintext[i] >= 'a' && plaintext[i] < 'z')
        {
            plaintext[i] = (plaintext[i] - 'a' + key) % 26 + 'a';
        }
        else
        {
            printf("%c\n", plaintext[i]);
        }
    }
    return 0;
}

2 个答案:

答案 0 :(得分:2)

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

应该是

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

答案 1 :(得分:0)

A的符号表索引不如Z。最有可能A == 65和Z == 90。

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

你说的是“如果某物大于65且大于90”。逻辑没有任何意义,这与说“如果某事物大于90”是完全一样的。

你可能意味着

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